Bug 131808

Summary: [New Multicolumn] fast/multicol/multicol-with-child-renderLayer-for-input.html puts the textfield in the wrong place
Product: WebKit Reporter: Dave Hyatt <hyatt>
Component: Layout and RenderingAssignee: Dave Hyatt <hyatt>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, esprehn+autocc, glenn, hyatt, jonlee, kondapallykalyan, mstensho, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 131825    
Attachments:
Description Flags
Test case
none
Patch
hyatt: review-
Patch beidson: review+

Description Dave Hyatt 2014-04-17 13:37:34 PDT
The layer is in the wrong place. Forcing a new layout snaps it to the right place, so it's some problem where the layer position is initially incorrect (or only the widget position is incorrect and it syncs to the layer later).
Comment 1 Radar WebKit Bug Importer 2014-04-17 23:12:43 PDT
<rdar://problem/16656286>
Comment 2 Dave Hyatt 2014-04-21 13:44:03 PDT
This is a blocker for turning the new columns on.
Comment 3 Morten Stenshorne 2014-04-22 07:25:48 PDT
*** Bug 131984 has been marked as a duplicate of this bug. ***
Comment 4 Morten Stenshorne 2014-04-22 07:26:43 PDT
Created attachment 229879 [details]
Test case
Comment 5 Morten Stenshorne 2014-04-22 07:28:39 PDT
Created attachment 229880 [details]
Patch
Comment 6 Dave Hyatt 2014-04-22 08:52:46 PDT
Comment on attachment 229880 [details]
Patch

Your fix is basically just working around the fact that the flags were wrong. This is because some code is missing from FrameView. It's patched for the old columns but not for the new ones. I'll post a revised patch with your test case in a bit.
Comment 7 Dave Hyatt 2014-04-22 09:02:01 PDT
Created attachment 229884 [details]
Patch
Comment 8 Dave Hyatt 2014-04-22 09:08:50 PDT
Fixed in r167663.