Issue metadata
Sign in to add a comment
|
58.7% regression in rendering.desktop at 587356:587380 |
||||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Sep 6
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/1189fcd5640000
,
Sep 6
📍 Found a significant difference after 1 commit. https://pinpoint-dot-chromeperf.appspot.com/job/1189fcd5640000 turn off use-zoom-for-dsf on Android by eirage@chromium.org https://chromium.googlesource.com/chromium/src/+/14d5c4330c08ac9bcdcda7960561e3e92c135da6 1.258 → 1.036 (-0.2224) Understanding performance regressions: http://g.co/ChromePerformanceRegressions Benchmark documentation link: https://bit.ly/rendering-benchmarks
,
Sep 6
My change is turning off a flag on Android, should have affect on mac. Un-assign myself
,
Sep 6
s/should have/should have no/
,
Sep 6
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/1421958b640000
,
Sep 7
📍 Found a significant difference after 1 commit. https://pinpoint-dot-chromeperf.appspot.com/job/1421958b640000 Managing the context was dropped. Adding back in. by herb@google.com https://skia.googlesource.com/skia/+/aca2b9eebdaf5ec45ba87a25c60510988fc54756 0.8655 → 1.266 (+0.4002) Understanding performance regressions: http://g.co/ChromePerformanceRegressions Benchmark documentation link: https://bit.ly/rendering-benchmarks
,
Sep 17
Hey Khushal, In some ways it looks like having the scaler context have a longer lifetime was a pretty good performance win for about 5-10% memory use. Is this something we want to explore?
,
Sep 17
Here is a corresponding memory increase bug 883755 .
,
Sep 17
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Sep 6