Issue metadata
Sign in to add a comment
|
9.3% regression in rendering.mobile at 590823:590876 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Sep 13
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/14ac4875640000
,
Sep 18
📍 Found significant differences after each of 3 commits. https://pinpoint-dot-chromeperf.appspot.com/job/14ac4875640000 Roll src/third_party/skia b8e125281d50..4f94ef6cbd26 (4 commits) by chromium-autoroll@skia-public.iam.gserviceaccount.com https://chromium.googlesource.com/chromium/src/+/9e370c4631a403f35e0c5d80761cb43582649692 203.6 → 218.5 (+14.96) Remove a large amount of plumbing in UI compositor by mdjones@chromium.org https://chromium.googlesource.com/chromium/src/+/fffc2a9d17cb1af7edb6cc640b3171e25955f356 218.5 → 219.6 (+1.114) WorkerTaskQueue.postTask() doesn't handle all task results correctly. by japhet@chromium.org https://chromium.googlesource.com/chromium/src/+/991a06f7b39f6f1f1cee7a7d32e980a97fa0d875 218.3 → 220.2 (+1.888) Understanding performance regressions: http://g.co/ChromePerformanceRegressions Benchmark documentation link: https://bit.ly/rendering-benchmarks
,
Sep 18
The code in my CL is entirely behind a flag, so it's very unlikely to be responsible. mdjones: is there any chance this is from your CL, or is likely to be the skia roll?
,
Sep 18
My patch removed some unused plumbing from the tab switcher; I'd be very surprised if that was the issue. The skia roll appears to have had the largest impact (203.6 → 218.5 (+14.96)) so I'd guess that. Updating components to include skia. The list of patches included in that roll is here: https://skia.googlesource.com/skia.git/+log/b8e125281d50..4f94ef6cbd26 |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Sep 13