Summary: | Selected option is not restored correctly | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Kent Tamura <tkent> | ||||||||
Component: | Forms | Assignee: | Kent Tamura <tkent> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | e.ryabtsev, morrita | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | 89628 | ||||||||||
Bug Blocks: | 23346 | ||||||||||
Attachments: |
|
Description
Kent Tamura
2012-06-20 17:49:21 PDT
Created attachment 148972 [details]
Patch
Comment on attachment 148972 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=148972&action=review > Source/WebCore/ChangeLog:36 > + Test: fast/forms/select/select-state-restore.html I hope we have another test which covers option element removal. > LayoutTests/fast/forms/select/select-state-restore.html:8 > +<p>Test to NOT save state for non-edited controls</p> Is this message intended? Comment on attachment 148972 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=148972&action=review >> Source/WebCore/ChangeLog:36 >> + Test: fast/forms/select/select-state-restore.html > > I hope we have another test which covers option element removal. ok, I'll update the test so that it covers removal. >> LayoutTests/fast/forms/select/select-state-restore.html:8 >> +<p>Test to NOT save state for non-edited controls</p> > > Is this message intended? Oh, it's a copypasta. I'll update it. Created attachment 149218 [details]
Patch 2
Created attachment 149219 [details]
Patch 3
Remove TABs
Committed r121128: <http://trac.webkit.org/changeset/121128> |