Summary: | REGRESSION(r94274): setting input.value erroneously triggers focus event | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> | ||||||
Component: | Forms | Assignee: | Ryosuke Niwa <rniwa> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | ap, darin, enrica, ojan, sullivan, tkent | ||||||
Priority: | P1 | Keywords: | GoogleBug | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 69395 | ||||||||
Attachments: |
|
Description
Ryosuke Niwa
2011-10-03 19:10:59 PDT
Created attachment 109577 [details]
manual test
I've spent last 1.5 hours trying to create a reproduction without a success.
It seems like I'd have to create a manual test here :(
Created attachment 109578 [details]
Patch
Comment on attachment 109578 [details] Patch The change looks good, but why did r94274 make the problem? (In reply to comment #3) > (From update of attachment 109578 [details]) > The change looks good, but why did r94274 make the problem? r94274 added a call to updateInnerTextValue in setValue. We used to update the inner text value in updateFromElement so this had never been an issue (although we may had a use-after-free back then). Comment on attachment 109578 [details]
Patch
ok, I understand.
Committed r96628: <http://trac.webkit.org/changeset/96628> |