Issue metadata
Sign in to add a comment
|
108.7%-168% regression in blink_perf.paint at 611922:611999 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Dec 4
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/1005b4e2140000
,
Dec 4
📍 Found a significant difference after 1 commit. https://pinpoint-dot-chromeperf.appspot.com/job/1005b4e2140000 gpu: Disable CCPR for OOP raster. by khushalsagar@chromium.org https://chromium.googlesource.com/chromium/src/+/193eda157bdb1c66d9afdc9c54a36e0ccc249293 transform-changes: 2408 → 6500 (+4093) Understanding performance regressions: http://g.co/ChromePerformanceRegressions Benchmark documentation link: https://bit.ly/blink-perf-benchmarks
,
Jan 9
This is interesting. I assumed changing the path renderer will be a purely gpu side change but it seems to be affecting paint timing!? +chrishtr/pdr, does this measure anything past the paint stage in blink. My change should have only affected the work happening in the GPU process's main thread.
,
Jan 9
Looking at the trace of transform-changes.html with and without your change, the blink side is roughly the same performance-wise. For example, there are 15 beginframes in both, and the total time spent in the renderer is roughly the same (748ms before, 763ms after). I think the cause of this regression is somewhere else which is causing blink to be updated much less frequently. Can you take a trace locally with more trace events enabled and compare with/without your change? I think this will show where the regression is. |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Dec 4