Issue metadata
Sign in to add a comment
|
8.1%-9.9% regression in thread_times.tough_scrolling_cases at 461196:461229 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Apr 5 2017
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/8983135190724199312
,
Apr 7 2017
=== Auto-CCing suspected CL author chongz@chromium.org === Hi chongz@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 : chongz Commit : 232fbc4bc87a4c35ea8ab60e1f7ff8e4e8e14de5 Date : Fri Mar 31 20:14:35 2017 Subject: Revert of Enable VsyncAlignedInput to Field Trial (patchset #1 id:40001 of https://codereview.chromium.org/2752363002/ ) Bisect Details Configuration: android_webview_arm64_aosp_perf_bisect Benchmark : thread_times.tough_scrolling_cases Metric : thread_raster_cpu_time_per_frame/thread_raster_cpu_time_per_frame Change : 6.01% | 9.21181407445 -> 9.76542163038 Revision Result N chromium@461195 9.21181 +- 0.498341 6 good chromium@461200 9.09005 +- 0.473304 9 good chromium@461201 9.1525 +- 0.559862 9 good chromium@461202 9.81334 +- 1.01058 9 bad <-- chromium@461204 9.71405 +- 0.805916 9 bad chromium@461212 9.75844 +- 0.37514 6 bad chromium@461229 9.76542 +- 0.423301 6 bad To Run This Test src/tools/perf/run_benchmark -v --browser=android-webview --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests thread_times.tough_scrolling_cases Debug Info https://chromeperf.appspot.com/buildbucket_job_status/8983135190724199312 Is this bisect wrong? https://chromeperf.appspot.com/bad_bisect?try_job_id=6449010358878208 | 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!
,
Apr 18 2017
,
Apr 20 2017
Issue 712392 has been merged into this issue.
,
Jul 27 2017
Explictly assigning. A CL you landed tripped one of the speed metrics we measure in the lab. If this is the first time this has happened to one of your CLs, or if it's been a while, please read: https://chromium.googlesource.com/chromium/src/+/master/docs/speed/addressing_performance_regressions.md We're looking for one of the following: 1. Justification via explanation 2. Plan to revert or fix 3. Angry rage throwing of equipment at my head Just be aware that I'm trained in trumpet playing and First Aid and am not afraid to use it. Note: This was a bulk edit message and not very personal.
,
Jul 27 2017
This is actually a revert of my previous Field Trial which caused some regressions. Mark as WontFix since the code has went back to the original state. |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by pmeenan@chromium.org
, Apr 5 2017