Bug 115442

Summary: Page::pageCount() is inaccurate when we allow the first paint in App Mode
Product: WebKit Reporter: Beth Dakin <bdakin>
Component: Layout and RenderingAssignee: Beth Dakin <bdakin>
Status: RESOLVED FIXED    
Severity: Normal CC: bdakin, commit-queue, esprehn+autocc, hyatt
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch hyatt: review+

Beth Dakin
Reported 2013-04-30 12:25:46 PDT
Page::pageCount() is inaccurate when we allow the first paint in App Mode, which happens when Document::setVisualUpdatesAllowed(true) is called. It should be accurate. After layout is updated, the page count should be accurate, but it is not for paginated content that contains images because auto-height images are given a layout height of 0 after the first pass. They require a second pass to get the right height. They shouldn't. <rdar://problem/13758445>
Attachments
Patch (1.58 KB, patch)
2013-04-30 12:30 PDT, Beth Dakin 		hyatt: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Beth Dakin
Comment 1 2013-04-30 12:30:36 PDT
Created attachment 200141 [details] Patch
Dave Hyatt
Comment 2 2013-04-30 12:52:23 PDT
Comment on attachment 200141 [details] Patch r=me
Beth Dakin
Comment 3 2013-04-30 12:53:42 PDT
Thanks, Dave! http://trac.webkit.org/changeset/149393
Note You need to log in before you can comment on or make changes to this bug.