Bug 28180 - [WML] WMLPageState is not allowed to store the active card, it needs to be done per document
Summary: [WML] WMLPageState is not allowed to store the active card, it needs to be do...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nikolas Zimmermann
URL:
Keywords:
Depends on:
Blocks: 20393
  Show dependency treegraph
 
Reported: 2009-08-11 10:33 PDT by Nikolas Zimmermann
Modified: 2009-08-11 12:29 PDT (History)
1 user (show)

See Also:


Attachments
Initial patch (9.34 KB, patch)
2009-08-11 11:39 PDT, Nikolas Zimmermann
staikos: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Nikolas Zimmermann 2009-08-11 10:33:13 PDT
A WMLPageState may be reused for a new document, on the same Page object.
Thus the "WMLCardElement* activeCard" is not allowed to be stored there, leading to crashes under certain circumstances.

Going to past a patch soon, including a reproducable test for this.
Comment 1 Nikolas Zimmermann 2009-08-11 11:39:03 PDT
Created attachment 34580 [details]
Initial patch
Comment 2 Nikolas Zimmermann 2009-08-11 12:29:42 PDT
Sending        WebCore/ChangeLog
Adding         WebCore/manual-tests/wml/select-onpick-event-crash.wml
Sending        WebCore/wml/WMLCardElement.cpp
Sending        WebCore/wml/WMLDoElement.cpp
Sending        WebCore/wml/WMLDocument.cpp
Sending        WebCore/wml/WMLDocument.h
Sending        WebCore/wml/WMLGoElement.cpp
Sending        WebCore/wml/WMLPageState.cpp
Sending        WebCore/wml/WMLPageState.h
Sending        WebCore/wml/WMLPrevElement.cpp
Sending        WebCore/wml/WMLRefreshElement.cpp
Transmitting file data ...........
Committed revision 47048.
http://trac.webkit.org/changeset/47048