Bug 51633

Summary: Add comment to test case fast/repaint/no-caret-repaint-in-non-content-editable-element.html to clarify caret browsing disabled
Product: WebKit Reporter: Daniel Bates <dbates>
Component: Tools / TestsAssignee: Daniel Bates <dbates>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, eric, tonikitoo, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

Daniel Bates
Reported 2010-12-26 23:42:24 PST
Following up from a conversation with Antonio Gomes, we should add a comment to the test case LayoutTests/fast/repaint/no-caret-repaint-in-non-content-editable-element.html that to clarify that this test is only applicable when caret browsing is disabled.
Attachments
Patch (1.72 KB, patch)
2010-12-26 23:43 PST, Daniel Bates 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Daniel Bates
Comment 1 2010-12-26 23:43:27 PST
Created attachment 77482 [details] Patch
Daniel Bates
Comment 2 2010-12-28 16:56:49 PST
Comment on attachment 77482 [details] Patch Clearing flags on attachment: 77482 Committed r74732: <http://trac.webkit.org/changeset/74732>
Daniel Bates
Comment 3 2010-12-28 16:56:55 PST
All reviewed patches have been landed. Closing bug.
WebKit Review Bot
Comment 4 2010-12-28 17:43:42 PST
http://trac.webkit.org/changeset/74732 might have broken Leopard Intel Debug (Tests)
Note You need to log in before you can comment on or make changes to this bug.