Bug 133110

Summary: [WK2] Fixed layout size of UIProcess isn't sync with WebProcess while viewport meta tag is changed.
Product: WebKit Reporter: Gyuyoung Kim <gyuyoung.kim>
Component: WebKit2Assignee: Gyuyoung Kim <gyuyoung.kim>
Status: NEW    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
WIP none

Gyuyoung Kim
Reported 2014-05-20 00:41:29 PDT
When viewport attribute is updated, WebPage::sendViewportAttributesChanged() changes fixed layout size of WebCore(ScrollView). However, it doesn't update fixed layout size with new fixed layout size for UIProcess. It can cause that UIProcess API (e.g. WKPageSetFixedLayoutSize()) can't change fixed layout size because WebPageProxy::m_fixedLayoutSize wasn't updated. void WebPage::sendViewportAttributesChanged() { ... // This also takes care of the relayout. setFixedLayoutSize(roundedIntSize(attr.layoutSize)); ... } void WebPage::setFixedLayoutSize(const IntSize& size) { FrameView* view = mainFrameView(); ... view->setFixedLayoutSize(size); => Change WebCore's fixed layout size ! ... } If WebPageProxy::m_fixedLayoutSize wasn't updated, WKPageSetFixedLayoutSize() can't update fixed layout size for WebCore. void WebPageProxy::setFixedLayoutSize(const IntSize& size) { ... if (size == m_fixedLayoutSize) return; m_fixedLayoutSize = size; m_process->send(Messages::WebPage::SetFixedLayoutSize(size), m_pageID); }
Attachments
WIP (1.43 KB, patch)
2014-05-20 00:42 PDT, Gyuyoung Kim
no flags
Gyuyoung Kim
Comment 1 2014-05-20 00:42:52 PDT
Note You need to log in before you can comment on or make changes to this bug.