Bug 62077

Summary: Switch paintMask and paintObject to use IntPoint
Product: WebKit Reporter: Levi Weintraub <leviw>
Component: Layout and RenderingAssignee: Levi Weintraub <leviw>
Status: RESOLVED FIXED    
Severity: Normal CC: darin, eae, eric, joepeck, simon.fraser
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 60318    
Attachments:
Description Flags
Patch none

Levi Weintraub
Reported 2011-06-03 17:42:05 PDT
paintObject is still on tx/ty, but paintMask was converted to IntSize passed by value. Bringing it in-line with the agreed-upon convention of a const IntPoint&.
Attachments
Patch (31.23 KB, patch)
2011-06-03 18:01 PDT, Levi Weintraub 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Levi Weintraub
Comment 1 2011-06-03 18:01:44 PDT
Created attachment 96001 [details] Patch
Eric Seidel (no email)
Comment 2 2011-06-03 18:49:51 PDT
Comment on attachment 96001 [details] Patch OK.
Levi Weintraub
Comment 3 2011-06-03 19:25:59 PDT
Comment on attachment 96001 [details] Patch Clearing flags on attachment: 96001 Committed r88087: <http://trac.webkit.org/changeset/88087>
Levi Weintraub
Comment 4 2011-06-03 19:26:04 PDT
All reviewed patches have been landed. Closing bug.
Joseph Pecoraro
Comment 5 2011-06-03 21:50:48 PDT
This seemed to have caused a clang build warning (which is treated as an error). <http://webkit.org/b/62084> clan build error: hidden overloaded virtual function
Note You need to log in before you can comment on or make changes to this bug.