Bug 78511

Summary: Update usage of LayoutUnits in RenderLayer
Product: WebKit Reporter: Levi Weintraub <leviw>
Component: Layout and RenderingAssignee: Levi Weintraub <leviw>
Status: RESOLVED FIXED    
Severity: Normal CC: eae, eric, jamesr, simon.fraser, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 60318    
Attachments:
Description Flags
Patch
none
Patch for landing none

Levi Weintraub
Reported 2012-02-13 11:44:17 PST
RenderLayer hosts native controls that must be communicated with via integers, but are also nested and need to carry subpixel positioning data.
Attachments
Patch (19.65 KB, patch)
2012-02-13 13:56 PST, Levi Weintraub 		no flags
DetailsFormatted DiffDiff
Patch for landing (19.04 KB, patch)
2012-02-14 11:24 PST, Levi Weintraub 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Levi Weintraub
Comment 1 2012-02-13 13:56:46 PST
Created attachment 126825 [details] Patch
Eric Seidel (no email)
Comment 2 2012-02-13 14:14:25 PST
Simon should see this.
Simon Fraser (smfr)
Comment 3 2012-02-13 19:03:17 PST
Comment on attachment 126825 [details] Patch I'll take your word for it.
Levi Weintraub
Comment 4 2012-02-14 11:24:21 PST
Created attachment 127004 [details] Patch for landing
Levi Weintraub
Comment 5 2012-02-14 11:27:19 PST
(In reply to comment #3) > (From update of attachment 126825 [details]) > I'll take your word for it. Thanks for the review!
WebKit Review Bot
Comment 6 2012-02-14 13:14:11 PST
The commit-queue encountered the following flaky tests while processing attachment 127004 [details]: perf/array-reverse.html bug 78593 (author: ojan@chromium.org) The commit-queue is continuing to process your patch.
WebKit Review Bot
Comment 7 2012-02-14 13:17:51 PST
Comment on attachment 127004 [details] Patch for landing Clearing flags on attachment: 127004 Committed r107731: <http://trac.webkit.org/changeset/107731>
WebKit Review Bot
Comment 8 2012-02-14 13:18:07 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.