Bug 167196

Summary: [Threaded Compositor] Initialize the threaded compositor with the current size
Product: WebKit Reporter: Carlos Garcia Campos <cgarcia>
Component: WebKit2Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bugs-noreply
Priority: P2 Keywords: Gtk
Version: WebKit Local Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch zan: review+

Description Carlos Garcia Campos 2017-01-19 00:34:35 PST
We are always creating the threaded compositor with and empty size and then a sizeDidChange always happen when the backing store state changes. This is always happening because the threaded compositor is created before the first backing store state, but if we wanted to create it later, for example to enter/leave AC mode on demand, the threaded compositor will not have the viewport size unless the window is resized, or sizeDidChange is called manually when entering AC mode. Creating the threaded compositor is sync and changing the size too, so it's better to do both things at the same time using the same sync operation.
Comment 1 Carlos Garcia Campos 2017-01-19 00:43:33 PST
Created attachment 299239 [details]
Patch
Comment 2 Zan Dobersek 2017-01-19 09:08:25 PST
Comment on attachment 299239 [details]
Patch

LGTM.
Comment 3 Carlos Garcia Campos 2017-01-19 22:57:09 PST
Committed r210954: <http://trac.webkit.org/changeset/210954>