Issue metadata
Sign in to add a comment
|
6.3% regression in system_health.common_desktop at 532680:532779 |
||||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Feb 2 2018
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/145c90a6840000
,
Feb 3 2018
📍 Found a significant difference after 1 commit. https://pinpoint-dot-chromeperf.appspot.com/job/145c90a6840000 Surface synchronization: Reduce synchronization deadline for navigation by fsamuel@chromium.org chromium @ b46e0efc8062d290e0657c12f22d0453dfd94188 Understanding performance regressions: http://g.co/ChromePerformanceRegressions
,
Feb 3 2018
Reducing the synchronization deadline causes a regression in timeToFirstPaint while we were expecting the opposite to happen... so weird. Need to investigate.
,
Feb 5 2018
,
Feb 15 2018
+cblume, ericrk FYI: I think we now understand the underlying cause of this regression: allocating a new surface on every navigation added a round trip to from the renderer main thread to the browser UI thread and back to the renderer main thread which incurs queuing delay in both browser UI and renderer main (both busy threads during page loads). When we had higher synchronization deadlines, some of that cost was offset by doing less work in the browser UI thread (since UI frames were blocked). When the deadline was reduced, we got less jank in browser UI but we slowed down loading slightly. This problem will hopefully go away in M67 when we hope to introduce child allocated surface IDs. This will avoid the round trip to allocate a surface ID, and allow the renderer to make forward progress sooner. Thus, I'm moving this to M67.
,
Feb 15 2018
Issue 808649 has been merged into this issue.
,
Feb 15 2018
Issue 808616 has been merged into this issue.
,
Feb 15 2018
,
Feb 15 2018
,
Apr 2 2018
Since we will not address this immediately, I'm removing the M67 and Performance Sheriff lable.
,
Jul 16
This metric is more noisy than before, but it definitely has improved. Marking as FIXED now. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Feb 2 2018