New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 599762 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 608888
Owner:
Closed: May 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android
Pri: 2
Type: Feature

Blocked on:
issue 597816

Blocking:
issue 597790
issue 605189



Sign in to add a comment

[Compositor] Investigate commit throttling on the engine during initial page load.

Project Member Reported by khushals...@chromium.org, Apr 1 2016

Issue description

The initial page load can trigger multiple commits as the page is invalidated during layout.

We could possibly throttle these requests to batch up the invalidations and avoid repeated data transfers in multiple commit messages, but this would increase the time before the first frame is pushed to the client. Also, how much this would help depends on the granularity of deltas sent in the compositor messages.
 
Blocking: -597789 597790
Labels: M-52
If we throttle, not stop, it should not affect the first frame, right?

Imagine we push no more than one frame per 3-second unless the load is complete, this can ensure the fast first frame, and less unnecessary update.
I guess I should've worded that better, I meant the first useful frame. Its hard to tell in the compositor about when we have rendered something useful. So on a good network, if we get a commit request really quickly and we push a frame which hadn't been completely rendered, and then block sending more updates until the next frame interval (say 3 seconds), we would delay the user perceived page load time.

We could possibly use some signals from blink to decide when to send the first frame, like VisuallyNonEmpty here https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit/public/web/WebMeaningfulLayout.h. Not sure how good these are to depend on though.
Labels: Blimp-M52-Proj-Scope
Blocking: 605189
Mergedinto: 608888
Status: Duplicate (was: Assigned)
The bug talks about investigating commit throttling on the engine. Since the investigation for commit serialization data usage has pointed out that this throttling is necessary and crbug/608888 is already tracking work for the approaches to perform this, I am closing this bug in favour of the existing bug.
Labels: Archive-Blimp

Sign in to add a comment