Issue metadata
Sign in to add a comment
|
2.4% regression in smoothness.sync_scroll.key_mobile_sites_smooth at 450873:450898 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Feb 17 2017
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/8987421903876900272
,
Feb 17 2017
=== Auto-CCing suspected CL author wangxianzhu@chromium.org === Hi wangxianzhu@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 : Xianzhu Wang Commit : af710fcae8e5585e8628cd46d6f18786cc7ccb26 Date : Thu Feb 16 07:03:11 2017 Subject: Avoid false-positives of paint offset change detection Bisect Details Configuration: android_webview_arm64_aosp_perf_bisect Benchmark : smoothness.sync_scroll.key_mobile_sites_smooth Metric : frame_times/http___mobile-news.sandbox.google.com_news_pt0 Change : 1.71% | 16.6331395976 -> 16.9182445487 Revision Result N chromium@450872 16.6331 +- 0.132097 6 good chromium@450879 16.5907 +- 0.225484 6 good chromium@450882 16.6358 +- 0.0738017 6 good chromium@450883 16.8955 +- 0.226885 6 bad <-- chromium@450884 16.9164 +- 0.0986263 6 bad chromium@450885 16.9244 +- 0.10277 6 bad chromium@450898 16.9182 +- 0.104777 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 --story-filter=http...mobile.news.sandbox.google.com.news.pt0 smoothness.sync_scroll.key_mobile_sites_smooth Debug Info https://chromeperf.appspot.com/buildbucket_job_status/8987421903876900272 Is this bisect wrong? https://chromeperf.appspot.com/bad_bisect?try_job_id=5885295423127552 | 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!
,
Feb 17 2017
I think we can accept this, given that the change have much bigger performance improvement to other test cases.
,
Mar 10 2017
Hey wangxianzhu - which other test cases do you notice an improvement? Can you point them out as I'm not able to find them now?
,
Mar 10 2017
I associated several improvement alerts to this bug. You can browser them here: https://chromeperf.appspot.com/group_report?bug_id=693456
,
Mar 11 2017
,
May 4 2017
|
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by alexclarke@chromium.org
, Feb 17 2017