Issue metadata
Sign in to add a comment
|
16.7%-31.3% regression in rendering.desktop at 577911:577957 |
||||||||||||||||||||
Issue descriptionNoticeable regression.
,
Jul 26
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/14ad18d3a40000
,
Jul 26
📍 Found a significant difference after 1 commit. https://pinpoint-dot-chromeperf.appspot.com/job/14ad18d3a40000 Free more resources in gpu/oop raster when idle by enne@chromium.org https://chromium.googlesource.com/chromium/src/+/da08e1808cb0835dd647407869b84e6f30a745d0 97.44 → 127.7 (+30.26) Understanding performance regressions: http://g.co/ChromePerformanceRegressions
,
Jul 26
On mac, this patch at r577956 reverted ericrk's at r572391. What's curious is that frame times on this test appear to be entirely unchanged after ericrk's patch and changed after my revert. The patch revert is reverting two oneliners, so I feel pretty confident that it's doing the right thing. regressions from clearing fewer gpu raster resources (non-android): https://chromeperf.appspot.com/group_report?rev=572391 regressions from clearing fewer oop raster resources (android): https://chromeperf.appspot.com/group_report?rev=576180 regressions from reverting both: https://chromeperf.appspot.com/group_report?rev=577956 There's only one test that appears to have been affected negatively by this patch. The original bug at the beginning of all of this was trying to fix a 5% input latency regression. I think the memory savings are worth that original regression and this should just be closed WontFix. |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Jul 26