Summary: | no-caret-repaint-in-non-content-editable-element.html does not need to disable CaretBrowsing | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Antonio Gomes <tonikitoo> | ||||
Component: | Tools / Tests | Assignee: | Antonio Gomes <tonikitoo> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | ||||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | PC | ||||||
OS: | OS X 10.5 | ||||||
Bug Depends on: | 51372 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
Antonio Gomes
2010-12-19 14:10:33 PST
Created attachment 76961 [details] patch v1 (r74349, r=xan) Comment on attachment 76961 [details] patch v1 (r74349, r=xan) Makes sense to me. Comment on attachment 76961 [details] patch v1 (r74349, r=xan) Clearing flags on attachment: 76961 Committed r74349: <http://trac.webkit.org/changeset/74349> After talking to Daniel Bates, the original author of this test, we both agreed that the statement has no affect as DRTs are today (since caret browsing it reset before each test execution). However, explicitly setting caret browsing to DISABLED ensures that the test will run under the circumstance it needs to catch the bug: caret browsing is DISABLED. We decided to put this code back in, and he will add a more descriptive comment saying why it should be there. See bug 51372 |