Summary: | [BlackBerry] Webpage was cut off after rotating to landscape and then rotating back to portrait on specific website | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Xiaobo Wang <xiaobwang> | ||||
Component: | WebKit BlackBerry | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Major | CC: | jkjiang, mifenton, rwlbuis, tonikitoo, webkit.review.bot, yong.li.webkit | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Other | ||||||
OS: | Other | ||||||
Attachments: |
|
Description
Xiaobo Wang
2013-01-29 18:38:16 PST
When rotating back to portrait, WebPagePrivate::overflowExceedsContentsSize() was called. It will call setViewMode(), in which we will get fixedLayoutSize() and compare with the fixedLayoutSize in the FrameView. But WebPagePrivate::fixedLaoutSize() returned wrong size because the width was reset to 0 and then fallback to defaultLayoutSize.width(). if (m_pendingOrientation != -1 && !m_nestedLayoutFinishedCount) width = 0; Can be fixed by adding a check of m_overflowExceedsContentsSize, only reset width if m_overflowExceedsContentsSize is not true. Created attachment 185378 [details]
patch
Comment on attachment 185378 [details]
patch
LGTM.
Comment on attachment 185378 [details] patch Clearing flags on attachment: 185378 Committed r141276: <http://trac.webkit.org/changeset/141276> All reviewed patches have been landed. Closing bug. |