Bug 66520 - Synchronize UIProcess's fixedLayout variables with WebProcess's ones
Summary: Synchronize UIProcess's fixedLayout variables with WebProcess's ones
Status: RESOLVED WORKSFORME
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-18 19:28 PDT by Eunmi Lee
Modified: 2012-01-30 15:52 PST (History)
6 users (show)

See Also:


Attachments
Patch for FixedLayout (6.02 KB, patch)
2011-08-18 20:53 PDT, Eunmi Lee
no flags Details | Formatted Diff | Diff
Patch for FixedLayout (5.97 KB, patch)
2011-09-19 06:04 PDT, Eunmi Lee
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eunmi Lee 2011-08-18 19:28:54 PDT
There are some problems to use fixedLayout in the WebKit2, so I made patch for that.

When we change the fixedLayout UIProcess's variables are set firstly, and then requests to apply changed fixedLayout are sended to the WebProcess.
In that case, UIProcess keeps changed values even though WebProcess fails to set fixedLayout.
So I changed codes to set UIProcess's fixedLayout variables when didChange message is received from WebProcess instead of setting directly.

The fixedLayout variables are reset whenever creating new FrameView.
It breaks consistency of UIProcess's fixedLayout variables and WebProcess's ones.
So I modified codes to maintain old FrameView's fixedLayout variables.
Comment 1 Eunmi Lee 2011-08-18 20:53:54 PDT
Created attachment 104449 [details]
Patch for FixedLayout
Comment 2 Eunmi Lee 2011-09-19 06:04:25 PDT
Created attachment 107842 [details]
Patch for FixedLayout

Rebase patch.
Comment 3 Eric Seidel (no email) 2011-12-19 12:04:26 PST
One of the WK2 reviewers should be able to tell you if this is right.
Comment 4 Eunmi Lee 2012-01-10 00:43:51 PST
WebPage.cpp's setUseFixedLayout() and setFixedLayoutSize() were changed to check mainFrameView by https://bugs.webkit.org/show_bug.cgi?id=73212.
So, WebPage's fixed layout functions will not fail and I don't have to return the fixedLayout variables back to the UIProcess.

The codes to maintain old fixedLayout variables were merged by https://bugs.webkit.org/show_bug.cgi?id=72511. 

So, this bug is not needed anymore and I want to change state to RESOLVED.
Comment 5 Eric Seidel (no email) 2012-01-30 15:52:56 PST
Comment on attachment 107842 [details]
Patch for FixedLayout

Cleared review? from attachment 107842 [details] so that this bug does not appear in http://webkit.org/pending-review.  If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review again).