Summary: | WebKit allows selection that crosses the shadow boundary of a readonly input element | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> | ||||||||||
Component: | HTML Editing | Assignee: | Ryosuke Niwa <rniwa> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Major | CC: | adele, ap, commit-queue, darin, dglazkov, enrica, rolandsteiner, sam, tkent | ||||||||||
Priority: | P2 | ||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||
Hardware: | All | ||||||||||||
OS: | All | ||||||||||||
Bug Depends on: | |||||||||||||
Bug Blocks: | 59435 | ||||||||||||
Attachments: |
|
Description
Ryosuke Niwa
2011-05-02 18:31:18 PDT
Created attachment 92029 [details]
demo
This is not a regression from Safari 5. I actually wonder if it's a bug or intended behavior. (In reply to comment #2) > This is not a regression from Safari 5. I actually wonder if it's a bug or intended behavior. I don't think this can be an intended behavior. SelectionController has a selection that crosses the shadow boundary. For what's worth, window.getSelection().toString() returns "" and baseNode and extentExtent of DOMSelection report incorrect nodes. See also: bug 7275 (not very closely related). Created attachment 92046 [details]
DRT test
Here's a half-baked DRT test. I'm going to abandon this test and write a bunch of pixel tests because it can only indirectly check the behavior and it's not suitable as a regression test.
Created attachment 92049 [details]
fixes the bug
Created attachment 92050 [details]
fixed select-across-readonly-input-5.html
The commit-queue encountered the following flaky tests while processing attachment 92050 [details]: http/tests/xmlhttprequest/cross-origin-authorization.html bug 52398 (author: ap@webkit.org) The commit-queue is continuing to process your patch. Comment on attachment 92050 [details] fixed select-across-readonly-input-5.html Clearing flags on attachment: 92050 Committed r85669: <http://trac.webkit.org/changeset/85669> All reviewed patches have been landed. Closing bug. |