Bug 107922

Summary: Call FrameView::contentsResized() when setting fixed layout size
Product: WebKit Reporter: Alexandre Elias <aelias>
Component: New BugsAssignee: Alexandre Elias <aelias>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, hclam, jamesr, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 108120    
Bug Blocks: 108446    
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch
none
Patch for landing
none
Patch for landing none

Description Alexandre Elias 2013-01-24 23:34:53 PST
Call FrameView::contentResized() when setting fixed layout size
Comment 1 Alexandre Elias 2013-01-24 23:40:33 PST
Created attachment 184680 [details]
Patch
Comment 2 Alexandre Elias 2013-01-24 23:45:22 PST
*** Bug 107206 has been marked as a duplicate of this bug. ***
Comment 3 Adam Barth 2013-01-25 09:23:05 PST
Who is the right person to review this change?
Comment 4 James Robinson 2013-01-25 10:29:29 PST
Comment on attachment 184680 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=184680&action=review

R=me. The WebCore changes seem quite reasonable.

> Source/WebCore/platform/ScrollView.cpp:280
> +    if (useFixedLayout())

nit: use m_useFixedLayout in ScrollView.cpp, like the rest of the file does
Comment 5 Alexandre Elias 2013-01-25 13:59:21 PST
Created attachment 184807 [details]
Patch

Fix nit
Comment 6 WebKit Review Bot 2013-01-25 15:03:56 PST
Comment on attachment 184807 [details]
Patch

Clearing flags on attachment: 184807

Committed r140869: <http://trac.webkit.org/changeset/140869>
Comment 7 WebKit Review Bot 2013-01-25 15:04:00 PST
All reviewed patches have been landed.  Closing bug.
Comment 8 WebKit Review Bot 2013-01-28 15:02:17 PST
Re-opened since this is blocked by bug 108120
Comment 9 Alexandre Elias 2013-01-31 01:53:57 PST
Created attachment 185718 [details]
Patch
Comment 10 Alexandre Elias 2013-01-31 02:19:17 PST
Created attachment 185724 [details]
Patch
Comment 11 Alexandre Elias 2013-01-31 02:46:41 PST
Relanding as WebCore changes are identical.
Comment 12 Alexandre Elias 2013-01-31 02:48:20 PST
Created attachment 185728 [details]
Patch for landing
Comment 13 WebKit Review Bot 2013-01-31 04:11:35 PST
Comment on attachment 185728 [details]
Patch for landing

Rejecting attachment 185728 [details] from commit-queue.

New failing tests:
media/track/track-cues-enter-exit.html
media/track/track-cues-cuechange.html
Full output: http://queues.webkit.org/results/16280133
Comment 14 Alexandre Elias 2013-01-31 10:37:18 PST
Created attachment 185808 [details]
Patch for landing
Comment 15 WebKit Review Bot 2013-01-31 11:27:15 PST
Comment on attachment 185808 [details]
Patch for landing

Clearing flags on attachment: 185808

Committed r141450: <http://trac.webkit.org/changeset/141450>
Comment 16 WebKit Review Bot 2013-01-31 11:27:19 PST
All reviewed patches have been landed.  Closing bug.