Bug 124207

Summary: Make the placed floats tree use LayoutUnit instead of int
Product: WebKit Reporter: Bem Jones-Bey <bjonesbe>
Component: Layout and RenderingAssignee: Bem Jones-Bey <bjonesbe>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, esprehn+autocc, glenn, kondapallykalyan, WebkitBugTracker
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
New Patch none

Description Bem Jones-Bey 2013-11-12 08:22:31 PST
Floats use LayoutUnits for all dimensions, it doesn't really make sense to be converting to ints for the placed floats tree.
Comment 1 Bem Jones-Bey 2013-11-12 09:18:15 PST
Created attachment 216685 [details]
Patch
Comment 2 Bem Jones-Bey 2013-11-12 16:32:27 PST
Created attachment 216740 [details]
New Patch
Comment 3 Alexandru Chiculita 2013-11-12 16:36:23 PST
Comment on attachment 216740 [details]
New Patch

Looks good, r=me.
Comment 4 Bem Jones-Bey 2013-11-12 16:39:25 PST
Comment on attachment 216740 [details]
New Patch

Note that there is a small but non-zero chance that this could cause small precision issues with tests when subpixel layout is enabled. I have run all of the tests locally, and have not seen any issues, so I don't expect there to be any new issues from this change.
Comment 5 WebKit Commit Bot 2013-11-12 17:30:28 PST
Comment on attachment 216740 [details]
New Patch

Clearing flags on attachment: 216740

Committed r159166: <http://trac.webkit.org/changeset/159166>
Comment 6 WebKit Commit Bot 2013-11-12 17:30:30 PST
All reviewed patches have been landed.  Closing bug.