Bug 80910 - [chromium] vsync ticks are needed in invisible tabs when we are still updating resources
Summary: [chromium] vsync ticks are needed in invisible tabs when we are still updatin...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nat Duca
URL:
Keywords:
: 80995 (view as bug list)
Depends on:
Blocks:
 
Reported: 2012-03-12 17:23 PDT by Nat Duca
Modified: 2012-03-13 15:50 PDT (History)
4 users (show)

See Also:


Attachments
Patch (5.14 KB, patch)
2012-03-12 17:23 PDT, Nat Duca
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Nat Duca 2012-03-12 17:23:11 PDT
[chromium] vsync ticks are needed in invisible tabs when we are still updating resources
Comment 1 Nat Duca 2012-03-12 17:23:35 PDT
Created attachment 131464 [details]
Patch
Comment 2 Nat Duca 2012-03-12 17:26:55 PDT
When we go in the background to another tab just before we begin a frame and commit, and there are more updates than can fit in a frame, then we forget to tick the scheduler. This leaves us the main thread blocked in the commit.
Comment 3 Nat Duca 2012-03-13 10:03:52 PDT
*** Bug 80995 has been marked as a duplicate of this bug. ***
Comment 4 Iain Merrick 2012-03-13 10:28:39 PDT
Confirmed that this patch fixes the bug for me.
Comment 5 James Robinson 2012-03-13 14:45:50 PDT
Comment on attachment 131464 [details]
Patch

R=me

I think it would be useful to have a way to abort pending updates as well when going !visible or when dropping resources, so we don't upload until tiles that we wanna drop or have already dropped.
Comment 6 Nat Duca 2012-03-13 15:06:08 PDT
(In reply to comment #5)
> I think it would be useful to have a way to abort pending updates as well when going !visible or when dropping resources, so we don't upload until tiles that we wanna drop or have already dropped.

Or, maybe we can make the updates just skip --- e.g. we do the update step but early out the individual updates if they're discovered to be pointless. That'd allow us to speed through the update step if we were trying to drop command buffer resources, but still always drop the command buffer resources in a consistent place [COMMIT_IDLE].
Comment 7 WebKit Review Bot 2012-03-13 15:50:35 PDT
Comment on attachment 131464 [details]
Patch

Clearing flags on attachment: 131464

Committed r110627: <http://trac.webkit.org/changeset/110627>
Comment 8 WebKit Review Bot 2012-03-13 15:50:39 PDT
All reviewed patches have been landed.  Closing bug.