Issue metadata
Sign in to add a comment
|
8.2%-108.5% regression in rendering.mobile at 587530:587660 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Sep 6
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/145d0f15640000
,
Sep 13
📍 Found a significant difference after 1 commit. https://pinpoint-dot-chromeperf.appspot.com/job/145d0f15640000 Fix the error catching logic of should().throw() in audit.js by hongchan@chromium.org https://chromium.googlesource.com/chromium/src/+/02a673df348e86553b50467a7dde1afce427b713 217.2 → 137.6 (-79.59) Understanding performance regressions: http://g.co/ChromePerformanceRegressions Benchmark documentation link: https://bit.ly/rendering-benchmarks
,
Sep 14
The commit is nothing to do with the canvas rendering. It only touches layout tests for WebAudio. Are we sure this about the cause of this regression? WDYT rtoy@?
,
Sep 14
Agree, the change for audit.js shouldn't be affecting the canvas test. Perhaps the skia roll in https://chromium.googlesource.com/chromium/src/+/53e63388501b107ea5c7a2f60518300d91358541 is relevant? It's in the regression range.
,
Sep 14
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/119a6487640000
,
Sep 14
Something went wrong with the bisect, it was supposed to detect a regression, instead it says your CL improved performance. I agree that neither is likely when it only touches layout tests. Re-running the bisect.
,
Sep 15
📍 Couldn't reproduce a difference. https://pinpoint-dot-chromeperf.appspot.com/job/119a6487640000
,
Sep 17
Looks like this benchmark is just noisy. |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Sep 6