Issue metadata
Sign in to add a comment
|
78.3% regression in rasterize_and_record_micro.top_25 at 579495:579548 |
||||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Aug 6
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/15aae407a40000
,
Aug 7
📍 Found a significant difference after 1 commit. https://pinpoint-dot-chromeperf.appspot.com/job/15aae407a40000 Enable use-zoom-for-dsf on Android on M70 by eirage@chromium.org https://chromium.googlesource.com/chromium/src/+/d0b8bbd2dbf9c06dcbbd01a4b1285cf7a1bc2ba0 15.71 → 27.45 (+11.74) Understanding performance regressions: http://g.co/ChromePerformanceRegressions
,
Aug 7
,
Aug 22
I think this is expected, since the test iterating the area with a fixed size. Before the flag enable, the pixel is in dip, but it's physical pixel after. so the rasterize took more iterations, result in the rasterize_time increase significantly.
,
Aug 22
Thanks for investigating this. +benchmark owners, FYI
,
Aug 24
Reopen because the size is actually causing problem and not only the test.
,
Aug 27
Confirmed with vmpstr@ that the regression is due to the test doesn't apply dsf, which is different from real world. So this regression is expected and would not cause real harm. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Aug 6