Issue metadata
Sign in to add a comment
|
2.1%-7.2% regression in system_health.memory_desktop at 600527:601000 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Oct 25
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/101eb9cee40000
,
Oct 25
📍 Found a significant difference after 1 commit. https://pinpoint-dot-chromeperf.appspot.com/job/101eb9cee40000 cc: Avoid acquiring the context lock if unnecessary. by khushalsagar@chromium.org https://chromium.googlesource.com/chromium/src/+/5d4e09620ff79ac668c72ad3677f993b75e617ce memory:chrome:all_processes:reported_by_os:system_memory:private_footprint_size: 1.477e+08 → 1.516e+08 (+3.911e+06) Understanding performance regressions: http://g.co/ChromePerformanceRegressions Benchmark documentation link: https://bit.ly/system-health-benchmarks
,
Oct 25
This change restores us back to the value we were at before https://chromium.googlesource.com/chromium/src/+/cd8fbb77cfa164eef66378204a588443af9bb875. Its meant to fix a bug that was introduced in the previous change. The change affected the timing of when a cleanup task runs in the renderer for skia/worker context resources for the worse in a lot of cases. Looks like it might have inadvertently triggered in some cases where it wasn't happening earlier? |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Oct 25