Issue metadata
Sign in to add a comment
|
5.2%-47.1% regression in thread_times.key_mobile_sites_smooth at 537298:537429 |
||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Feb 21 2018
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/11e5dd27840000
,
Mar 6 2018
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/12943902440000
,
Mar 6 2018
boliu: The bisect in #2 didn't complete, but it points to your CL, "aw: Make DrawFunctor event toplevel". Looks like most of the graphs recovered, is this already fixed? Kicked off a second bisect just in case.
,
Mar 6 2018
the DrawFunctor CL regressing webview is expected and reported many times already: crbug.com/813972 essentially webview was missing a toplevel trace event which that CL fixed, so a bunch of cpu time was unaccounted for is now counted as webview cpu usage that CL is in webview-only code though, so I don't see how it can possibly affect chrome though, so not sure about the non-webview regressions
,
Mar 6 2018
📍 Found significant differences after each of 2 commits. https://pinpoint-dot-chromeperf.appspot.com/job/12943902440000 Renderer observer of frame submission by jonross@chromium.org https://chromium.googlesource.com/chromium/src/+/a2ff4f82109df55045dee9f54985a98054f86dc4 Omit the parent LayoutView's scroll offset from FrameRect by szager@chromium.org https://chromium.googlesource.com/chromium/src/+/cf91f7965c29e2bc93e81f79cef5d77481b00e6e Understanding performance regressions: http://g.co/ChromePerformanceRegressions
,
Mar 6 2018
For my change listed in #6 I have also since landed https://chromium-review.googlesource.com/c/chromium/src/+/934626 which should remove this impact in prod |
|||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Feb 21 2018