Summary: | [test case] dhtml causes form corruption when going back to previous page. | ||
---|---|---|---|
Product: | [Applications] konqueror | Reporter: | Jon Gabrielson <jonkde> |
Component: | khtml forms | Assignee: | Konqueror Developers <konq-bugs> |
Status: | RESOLVED LATER | ||
Severity: | grave | CC: | jens-bugs.kde.org, maksim, mueller, nick |
Priority: | NOR | ||
Version: | 3.5 | ||
Target Milestone: | --- | ||
Platform: | Mandrake RPMs | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Attachments: | test case |
Description
Jon Gabrielson
2003-03-20 21:48:32 UTC
Created attachment 8240 [details]
test case
excellent test case. I saw it before, but never could reproduce it reliable
As far as I can see, the restoration code simply can't handle stuff being added to document in order other than parse order. Now, the DOM proper doesn't trigger the problem because the parser isn't involved, but innerHTML uses the parser, of course. I think Apple actually has a solution for this --- http://www.cs.cornell.edu/~maksim/WC/changesets/1503.html (later amended to not restore passwords wrong) --- the core idea is nice, though I am not sure I like the implementation details. *** Bug 74343 has been marked as a duplicate of this bug. *** *** Bug 138905 has been marked as a duplicate of this bug. *** for what it's worth: since a few days my favourite site http://www.google.com/advanced_search is broken with konqueror (on different linux systems). I guess google has started using content konqueror can't deal with. Just enter any search term, press enter (all fields are ignored in my case but that might as well be another bug), press back button => some fields contain "...X", "X..." and other garbage. Ugh. I guess I can't defer this one any more.. SVN commit 789902 by orlovich: Be more careful not to make a mistake when restoring form entries. Still imperfect, but I am not sure what perfect means here.. CCBUG:56188 M +8 -5 html/html_formimpl.cpp M +2 -4 html/htmlparser.cpp M +40 -10 xml/dom_docimpl.cpp M +4 -1 xml/dom_docimpl.h WebSVN link: http://websvn.kde.org/?view=rev&revision=789902 SVN commit 789918 by orlovich: automatically merged revision 789902: Be more careful not to make a mistake when restoring form entries. Still imperfect, but I am not sure what perfect means here.. CCBUG:56188 M +8 -5 html/html_formimpl.cpp M +2 -4 html/htmlparser.cpp M +40 -10 xml/dom_docimpl.cpp M +4 -1 xml/dom_docimpl.h WebSVN link: http://websvn.kde.org/?view=rev&revision=789918 Works in trunk r794020. Still present in 3.5.9 Can't reproduce in trunk 799157 As this bug is fixed in recent releases it's being closed. Setting it to LATER because it does qualify for backporting to the 3.x branch. |