Summary: | REGRESSION: html input doesn't honor change to value | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dennis Rowe <shr3kst3r> | ||||||||||
Component: | Forms | Assignee: | Darin Adler <darin> | ||||||||||
Status: | VERIFIED FIXED | ||||||||||||
Severity: | Critical | CC: | apple, joost, junado | ||||||||||
Priority: | P1 | ||||||||||||
Version: | 420+ | ||||||||||||
Hardware: | Mac | ||||||||||||
OS: | OS X 10.4 | ||||||||||||
Attachments: |
|
Description
Dennis Rowe
2006-01-23 08:23:06 PST
Created attachment 5876 [details]
Reduced example of bug
Reduced version of bug.
Confirmed, this is a critical issue. Created attachment 5877 [details] Don't go through QString I don't think an automated test is possible, much like in bug 6221. Created attachment 5878 [details]
Updated patch
Added bug # to the ChangeLog entry
Comment on attachment 5878 [details]
Updated patch
Good local fix, but I don't understand why an empty DOMString turns into a null DOMString when round tripping to QString.
That should be fixed too.
r=me
Created attachment 5879 [details]
Trying to fix the DOMString::qstring problem as well
I think it's OK to fix it under this bug. Not sure the fix is optimal.
Comment on attachment 5879 [details]
Trying to fix the DOMString::qstring problem as well
Nice. r=me
I'll land this one. I downloaded the source and compiled it and tested it against the reduced test case and also against the case that the reduced test came from. In both cases the new patch worked as it should. Thanks! *** Bug 6776 has been marked as a duplicate of this bug. *** Removing Regression keyword from bugs already fixed. *** Bug 7136 has been marked as a duplicate of this bug. *** |