New issue
Advanced search Search tips

Issue 702846 link

Starred by 0 users

Issue metadata

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



Sign in to add a comment

38.6%-65.9% regression in cpu_utilization in media.android.tough_video_cases at 457226:457387

Project Member Reported by chcunningham@google.com, Mar 18 2017

Issue description

Nexus 5, 7 and 5x bots. 
 
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Mar 18 2017

Cc: chaopeng@chromium.org
Owner: chaopeng@chromium.org

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

Hi chaopeng@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 : chaopeng
  Commit : c7a846538699afcb05ba89a8dd8abbf5bbc418ab
  Date   : Thu Mar 16 03:16:08 2017
  Subject: Revert of Show Overlay Scrollbar when GestureScrollUpdate (patchset #10 id:260001 of https://codereview.chromium.org/2611013002/ )

Bisect Details
  Configuration: android_nexus7_perf_bisect
  Benchmark    : media.android.tough_video_cases
  Metric       : cpu_utilization/video.html?src_crowd1080.ogv
  Change       : 56.34% | 2.54673287083 -> 3.98147995609

Revision             Result                   N
chromium@457328      2.54673 +- 0.528217      6      good
chromium@457331      2.48935 +- 0.572795      6      good
chromium@457333      2.42457 +- 0.359952      6      good
chromium@457334      3.88806 +- 0.539747      6      bad       <--
chromium@457340      3.92427 +- 0.538234      6      bad
chromium@457352      3.92425 +- 0.629166      6      bad
chromium@457376      3.98148 +- 0.863653      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.crowd1080.ogv media.android.tough_video_cases

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

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


| 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!
Project Member

Comment 5 by 42576172...@developer.gserviceaccount.com, Mar 19 2017


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

Suspected Commit
  Author : chaopeng
  Commit : c7a846538699afcb05ba89a8dd8abbf5bbc418ab
  Date   : Thu Mar 16 03:16:08 2017
  Subject: Revert of Show Overlay Scrollbar when GestureScrollUpdate (patchset #10 id:260001 of https://codereview.chromium.org/2611013002/ )

Bisect Details
  Configuration: android_nexus5_perf_bisect
  Benchmark    : media.android.tough_video_cases
  Metric       : cpu_utilization/crowd1080.ogv_gpu
  Change       : 36.58% | 2.82070277623 -> 3.85258871137

Revision             Result                   N
chromium@457225      2.8207 +- 0.244337       6      good
chromium@457286      2.897 +- 0.177551        6      good
chromium@457316      2.69147 +- 0.306189      6      good
chromium@457331      2.91864 +- 0.656707      6      good
chromium@457333      2.76122 +- 0.317353      6      good
chromium@457334      3.76853 +- 0.310162      5      bad       <--
chromium@457335      3.70449 +- 0.52814       6      bad
chromium@457339      3.77863 +- 0.412388      6      bad
chromium@457346      3.85259 +- 0.382132      5      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 media.android.tough_video_cases

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

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


| 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!
Status: Assigned (was: Untriaged)
Cc: bokan@chromium.org
Cc: -chcunnin...@chcunningham.com chcunningham@chromium.org
Status: WontFix (was: Assigned)
From the graphs it looks like the revert regressed a bunch of metrics that were improved by the initial CL. We can close this and just look forward to the improvements when you re-land. 

Sign in to add a comment