Issue metadata
Sign in to add a comment
|
7.5%-15.6% regression in page_cycler_v2_site_isolation.basic_oopif at 454513:454985 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Mar 8 2017
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/8985649576984548656
,
Mar 9 2017
=== Auto-CCing suspected CL author csharrison@chromium.org === Hi csharrison@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 : csharrison Commit : 279edadbd6962b7afa97c166896ae69ceb797da4 Date : Mon Mar 06 17:05:49 2017 Subject: Turn off LazyParseCSS on trybots for now Bisect Details Configuration: mac_10_11_perf_bisect Benchmark : page_cycler_v2.basic_oopif Metric : timeToFirstContentfulPaint_avg/pcv1-warm/http___booking.com Change : 17.18% | 89.5266666661 -> 104.905666659 Revision Result N chromium@454739 89.5267 +- 1.33644 6 good chromium@454875 90.7652 +- 0.953932 6 good chromium@454877 91.3972 +- 1.42509 6 good chromium@454878 90.9803 +- 1.49672 6 good chromium@454879 106.316 +- 2.4592 6 bad <-- chromium@454882 108.049 +- 6.34273 6 bad chromium@454889 104.917 +- 1.973 6 bad chromium@454903 104.343 +- 1.98573 6 bad chromium@454930 105.948 +- 2.65241 6 bad chromium@454985 104.906 +- 2.01534 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=http...booking.com page_cycler_v2.basic_oopif Debug Info https://chromeperf.appspot.com/buildbucket_job_status/8985649576984548656 Is this bisect wrong? https://chromeperf.appspot.com/bad_bisect?try_job_id=6121859730374656 | 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!
,
Mar 9 2017
Yes, the experiment is cool and improves page load time, but we are pausing it due to memory concerns. See issue 692932 .
,
Mar 9 2017
Ha! Bisect doesn't tie things together and we must've missed doing this bisect at the time. Sorry for the noise and good luck! |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by benhenry@google.com
, Mar 8 2017