Issue metadata
Sign in to add a comment
|
51.1%-15810.9% regression in blink_perf.css at 489052:489240 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Jul 28 2017
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/8972813341565078416
,
Jul 29 2017
=== BISECT JOB RESULTS === Bisect was unable to run to completion Error: INFRA_FAILURE The bisect was able to narrow the range, you can try running with: good_revision: 3d467dbf951242c8d10982c79362bdf7bebca1ad bad_revision : 620529aba9b0f9fa1ef7704b03af4ae2349e1216 If failures persist contact the team (see below) and report the error. Bisect Details Configuration: android_nexus7_perf_bisect Benchmark : blink_perf.css Metric : Document::rebuildLayoutTree/Document::rebuildLayoutTree Revision Result N chromium@489077 0.0407667 +- 0.00313647 6 good chromium@489123 0.0420228 +- 0.00248855 6 good chromium@489153 0.0419175 +- 0.00217716 6 good chromium@489171 0.0418474 +- 0.00193608 6 good chromium@489174 6.10318 +- 0.104728 6 bad chromium@489176 6.0818 +- 0.0791164 6 bad chromium@489180 6.0142 +- 0.102923 6 bad chromium@489189 5.77054 +- 0.0311886 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 blink_perf.css More information on addressing performance regressions: http://g.co/ChromePerformanceRegressions Debug information about this bisect: https://chromeperf.appspot.com/buildbucket_job_status/8972813341565078416 For feedback, file a bug with component Speed>Bisection
,
Aug 2 2017
The graphs for this bug improves quite a bit later on, but it's likely because [1] reduces the DOM tree sizes for the relevant tests. [1] https://chromium.googlesource.com/chromium/src/+/744f09725b57a1d3e0640c300045946756d0774d
,
Aug 2 2017
I'm not sure I read the graph data correctly, but it seems the regressed graphs actually is an accumulation of Document::RebuildLayoutTree times for all of blink_perf.css. If that's correct, the introduction of a new test[1] in the regression range is probably the "cause". Leaving to shend@ to mark as wontfix if appropriate. [1] https://chromium.googlesource.com/chromium/src/+/a65c9b380ae231c5b23ac16573e593260ca575ed
,
Aug 2 2017
Thanks rune, yeah looks like my patch. Marking as Wont-Fix since we're just adding more tests. nainar, would this also be the reason why your remove style sharing patch was being blamed for perf regressions?
,
Aug 9 2017
|
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Jul 28 2017