Issue metadata
Sign in to add a comment
|
10.2%-880.1% regression in speedometer at 464689:464734 |
||||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Apr 17 2017
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/8982048508122817984
,
Apr 17 2017
,
Apr 17 2017
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/8982048417679513696
,
Apr 17 2017
=== Auto-CCing suspected CL author nednguyen@google.com === Hi nednguyen@google.com, 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 : nednguyen Commit : 9a1915060877f610870645a720d8d6eb177d0a0e Date : Fri Apr 14 11:50:07 2017 Subject: Revert of Convert android to use X509CertificateBytes instead of X509CertificateOpenSSL. (patchset #9 id:160001 of https://codereview.chromium.org/2786173003/ ) Bisect Details Configuration: android_nexus6_perf_bisect Benchmark : v8.runtimestats.browsing_mobile Metric : GC:duration_avg/browse_news/browse_news_nytimes Change : 783.14% | 91.4271666667 -> 807.4325 Revision Result N chromium@464713 91.4272 +- 73.8683 6 good chromium@464714 87.3807 +- 53.0432 6 good chromium@464715 816.896 +- 193.559 6 bad <-- chromium@464716 849.473 +- 93.3459 6 bad chromium@464719 776.402 +- 196.825 6 bad chromium@464724 807.432 +- 106.171 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=browse.news.nytimes v8.runtimestats.browsing_mobile Debug Info https://chromeperf.appspot.com/buildbucket_job_status/8982048508122817984 Is this bisect wrong? https://chromeperf.appspot.com/bad_bisect?try_job_id=5902372905156608 | 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 17 2017
That CL was fixing a breakage. Without it, loading the page would return 404.
,
Apr 17 2017
=== BISECT JOB RESULTS === Perf regression found with culprit Suspected Commit Author : nednguyen Commit : 9a1915060877f610870645a720d8d6eb177d0a0e Date : Fri Apr 14 11:50:07 2017 Subject: Revert of Convert android to use X509CertificateBytes instead of X509CertificateOpenSSL. (patchset #9 id:160001 of https://codereview.chromium.org/2786173003/ ) Bisect Details Configuration: android_nexus6_perf_bisect Benchmark : v8.runtimestats.browsing_mobile Metric : Compile:duration_avg/browse_media/browse_media_imgur Change : 85.82% | 514.147666667 -> 955.372 Revision Result N chromium@464713 514.148 +- 23.9553 6 good chromium@464714 520.515 +- 14.949 6 good chromium@464715 943.755 +- 19.4122 6 bad <-- chromium@464716 944.279 +- 26.858 6 bad chromium@464719 949.381 +- 32.0148 6 bad chromium@464724 955.372 +- 53.7364 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=browse.media.imgur v8.runtimestats.browsing_mobile Debug Info https://chromeperf.appspot.com/buildbucket_job_status/8982048417679513696 Is this bisect wrong? https://chromeperf.appspot.com/bad_bisect?try_job_id=5837420651610112 | 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 18 2017
Issue 712367 has been merged into this issue.
,
Apr 19 2017
Feel free to close it if appropriate.
,
Apr 19 2017
This is expected because that CL fixed certificate errors and make the pages loaded again.
,
Apr 20 2017
,
Apr 20 2017
Issue 713613 has been merged into this issue.
,
Apr 20 2017
Issue 713613 has been merged into this issue. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by hablich@chromium.org
, Apr 17 2017