New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 711864 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Closed: Apr 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression

Blocking:
issue 708844



Sign in to add a comment

A zero-to-nonzero to 4% regression in media.android.tough_video_cases at 464546:464658

Project Member Reported by crouleau@chromium.org, Apr 15 2017

Issue description

I think cpu usage just started working.
 
All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=711864

Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?keys=agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwuqXsAkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggsPSogsM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggv7W6ggM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggrW56AoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwpjj9QoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDg_L-dmAoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggq7rpgsM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggo7omwgM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggvuT_QkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggrGSgggM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggv_QxwgM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggrGS_AsM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwqil_AoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwtibpgoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwuitrwkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwtzY4QoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggpXKhAoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwqq_uwoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggu7SvwoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDg_NveqAkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwpyz7gkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwuzipAsM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggpXKhAgM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggpOMugoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDg_NveqAoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggpup6QsM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwq6YrgoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggqPV8AkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggvHvqAkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwqystgkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwpyz7goM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggtWb9goM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggvSnwwgM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggrH2kwkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwtyWqQoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggvHzsQoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgguPdowoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggrXkvgkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggorhlQgM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggrGnhgkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDg_LnMlAkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggrH2kwoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggtb-ywsM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggrDl7QkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggoemuwkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggrH2kwgM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggt_s_goM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggpaE4AkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggrGS_AoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggof8qgkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggpGDtwkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwoyckQsM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggvGBvgoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwoyckQkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwqj05QgM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggpmNyQkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggr2b4wsM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwvKP_AkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggo78-goM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgguj_8gkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwuzdgwgM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwtjRqgkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDg_IbnlAkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggvmn3QsM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggvbgxAkM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggr3ohAoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDg_NeWrQgM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggsu_yQgM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggv_QhwsM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggqXTiwsM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwtjRqgoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggvbgxAoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggrGnhgsM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDggv_vsAoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwqqGvQoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwtjGswoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgwvKP_AoM


Bot(s) for this bug's original alert(s):

android-nexus5
android-nexus5X
android-nexus7v2
android-webview-nexus5X
android-webview-nexus6
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Apr 15 2017

Cc: dfalcant...@chromium.org
Owner: dfalcant...@chromium.org

=== Auto-CCing suspected CL author dfalcantara@chromium.org ===

Hi dfalcantara@chromium.org, the bisect results pointed to your CL, please take a look at the
results.


=== BISECT JOB RESULTS ===
Perf regression found with culprit

Suspected Commit
  Author : dfalcantara
  Commit : 8ef9faffba78738c37df77a160fea80ee5f604af
  Date   : Thu Apr 13 23:30:05 2017
  Subject: 🔍 Update how shared preferences are handled by the widget

Bisect Details
  Configuration: android_nexus7_perf_bisect
  Benchmark    : media.android.tough_video_cases
  Metric       : cpu_utilization/video.html?src_crowd720_vp9.webm

Revision             Result                   N
chromium@464562      0.0 +- 0.0               6      good
chromium@464604      0.0 +- 0.0               6      good
chromium@464605      11.0563 +- 0.158416      6      bad       <--
chromium@464606      11.4641 +- 0.670077      6      bad
chromium@464607      11.6482 +- 0.447814      6      bad
chromium@464610      11.2928 +- 0.299865      6      bad
chromium@464615      11.1649 +- 0.253884      6      bad
chromium@464625      11.3032 +- 0.372701      6      bad
chromium@464645      11.428 +- 0.139247       6      bad

To Run This Test
  src/tools/perf/run_benchmark -v --browser=android-chromium --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=video.html.src.crowd720.vp9.webm media.android.tough_video_cases

Debug Info
  https://chromeperf.appspot.com/buildbucket_job_status/8982283122637189680

Is this bisect wrong?
  https://chromeperf.appspot.com/bad_bisect?try_job_id=5880966385500160


| O O | Visit http://www.chromium.org/developers/speed-infra/perf-bug-faq
|  X  | for more information addressing perf regression bugs. For feedback,
| / \ | file a bug with component Speed>Bisection.  Thank you!
Blocking: 708840
Cc: yus...@chromium.org
Wouldn't be surprised if this was related to the other regression that was introduced earlier for 711040.  Best thing to do for both is probably just move the initialization of the widget to ChromeActivity, or something; there's too many things that ChromeBrowserInitializer is used by.
Actually: crouleau: what's going on with that graph?  The whole thing looks hosed.
Cc: -crouleau@chromium.org
Owner: crouleau@chromium.org
Assigning back to crouleau temporarily for an answer to Comment 5; please assign back afterward if there's something I should be doing here.  The graph looks like it just fixed itself and returned to where it was before a gigantic gap appeared.
Blocking: -708840 708844
Project Member

Comment 8 by 42576172...@developer.gserviceaccount.com, Apr 18 2017

Cc: guidou@chromium.org
 Issue 712619  has been merged into this issue.
Status: WontFix (was: Untriaged)
Agreed with dfalcatara@'s Comment 6: The graph was simply fixing itself. Not sure how that could have been caused by your changelist.
Project Member

Comment 10 by 42576172...@developer.gserviceaccount.com, Apr 18 2017

 Issue 712620  has been merged into this issue.
Project Member

Comment 11 by 42576172...@developer.gserviceaccount.com, Apr 18 2017

 Issue 712616  has been merged into this issue.
Project Member

Comment 12 by 42576172...@developer.gserviceaccount.com, Apr 18 2017

 Issue 712618  has been merged into this issue.

Sign in to add a comment