Issue metadata
Sign in to add a comment
|
2.6%-48.4% regression in v8.runtimestats.browsing_desktop at 482844:482966 |
||||||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Jun 30 2017
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/8975390166360944672
,
Jun 30 2017
=== Auto-CCing suspected CL author kochi@chromium.org === Hi kochi@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 : Takayoshi Kochi Commit : fa75e2abb23205caa6d7f61edc89e672e83f6089 Date : Wed Jun 28 08:22:13 2017 Subject: Revert of Relanding [Android] Adding Smart GO/NEXT feature in Chrome (patchset #4 id:80001 of https://codereview.chromium.org/2948593002/ ) Bisect Details Configuration: mac_air_perf_bisect Benchmark : v8.runtimestats.browsing_desktop Metric : V8 C++:duration_avg/browse_social/browse_social_facebook_infinite_scroll Change : 60.58% | 1652.65583333 -> 2653.91133333 Revision Result N chromium@482843 1652.66 +- 65.404 6 good chromium@482882 1656.97 +- 109.139 6 good chromium@482901 1652.99 +- 107.125 6 good chromium@482911 1676.77 +- 78.7367 6 good chromium@482916 1684.6 +- 197.46 6 good chromium@482917 1668.42 +- 184.035 6 good chromium@482918 2705.67 +- 97.8649 6 bad <-- chromium@482920 2653.91 +- 110.348 6 bad To Run This Test src/tools/perf/run_benchmark -v --browser=release --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=browse.social.facebook.infinite.scroll v8.runtimestats.browsing_desktop More information on addressing performance regressions: http://g.co/ChromePerformanceRegressions Debug information about this bisect: https://chromeperf.appspot.com/buildbucket_job_status/8975390166360944672 For feedback, file a bug with component Speed>Bisection
,
Jun 30 2017
,
Jun 30 2017
,
Jul 10 2017
The original CL, by ajith.v landed at 481868 https://codereview.chromium.org/2948593002 and then reverted by me. The original CL caused some regression ( issue 737388 ) but also got some *progression* - and reverting it was detected as regression, it seems (see the graph in comment #1). My revert change just reverted to the original performance, thus no action can be taken for this issue. Closing. |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by bmeu...@chromium.org
, Jun 30 2017