Issue metadata
Sign in to add a comment
|
3.8%-5.2% regression in rendering.mobile/tasks_per_frame_total_all at 616405:616555 |
||||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Dec 20
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/179f788e140000
,
Dec 20
📍 Found a significant difference after 1 commit. https://pinpoint-dot-chromeperf.appspot.com/job/179f788e140000 [PE] Avoid cache conflict when painting LocalFrameView outside of lifecycle by wangxianzhu@chromium.org https://chromium.googlesource.com/chromium/src/+/81fc6554b2da741da30913dee5008cc83b20091d tasks_per_frame_total_all: 234.2 → 245.2 (+11.01) Understanding performance regressions: http://g.co/ChromePerformanceRegressions Benchmark documentation link: https://bit.ly/rendering-benchmarks
,
Dec 20
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/10349386140000
,
Dec 20
📍 Job complete. See results below. https://pinpoint-dot-chromeperf.appspot.com/job/10349386140000
,
Dec 21
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a465e8196b0d3b34cdc2e3acb7b8e79a1d0e4591 commit a465e8196b0d3b34cdc2e3acb7b8e79a1d0e4591 Author: Xianzhu Wang <wangxianzhu@chromium.org> Date: Fri Dec 21 01:47:55 2018 [PE] Fix SVGImage performance regression This partly revert the change in svg_image.cc in https://chromium-review.googlesource.com/c/chromium/src/+/1374779. For pre-CompositeAfterPaint, still update to pre paint instead of full cycle before calling PaintOutsideOfLifecycle(). Bug: 916860 Change-Id: Ic05f56ca17afbb2df9973ee4d32f742b1f4796d8 Reviewed-on: https://chromium-review.googlesource.com/c/1387465 Reviewed-by: Chris Harrelson <chrishtr@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#618428} [modify] https://crrev.com/a465e8196b0d3b34cdc2e3acb7b8e79a1d0e4591/third_party/blink/renderer/core/svg/graphics/svg_image.cc
,
Dec 21
Issue 917247 has been merged into this issue.
,
Dec 21
Issue 917236 has been merged into this issue.
,
Dec 21
Issue 917246 has been merged into this issue.
,
Dec 21
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Dec 20