Bug 61580

Summary: Switch paintFocusRing to IntPoint
Product: WebKit Reporter: Levi Weintraub <leviw>
Component: Layout and RenderingAssignee: Levi Weintraub <leviw>
Status: RESOLVED FIXED    
Severity: Normal CC: eae, eric, leviw, 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-05-26 16:35:40 PDT
Continuing to hate on tx/ty.
Attachments
Patch (3.60 KB, patch)
2011-05-26 16:42 PDT, Levi Weintraub 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Levi Weintraub
Comment 1 2011-05-26 16:42:09 PDT
Created attachment 95073 [details] Patch
Eric Seidel (no email)
Comment 2 2011-05-26 17:39:34 PDT
Comment on attachment 95073 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=95073&action=review > Source/WebCore/rendering/RenderObject.cpp:1000 > +void RenderObject::paintFocusRing(GraphicsContext* context, const IntPoint& paintOffset, RenderStyle* style) Seems like this is the origin the renderObject in question, with relation to its paint container? I guess that's a paintOffset?
Levi Weintraub
Comment 3 2011-05-31 10:12:43 PDT
Comment on attachment 95073 [details] Patch Clearing flags on attachment: 95073 Committed r87738: <http://trac.webkit.org/changeset/87738>
Levi Weintraub
Comment 4 2011-05-31 10:12:48 PDT
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.