Closed Bug 745681 Opened 12 years ago Closed 12 years ago

AfterOnloadScripts can use an old document element

Categories

(Testing :: Reftest, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla14

People

(Reporter: khuey, Assigned: khuey)

References

Details

Attachments

(1 file, 1 obsolete file)

Attached patch Patch (obsolete) — Splinter Review
e.g. in 483818-1.html, where the document is replaced by a document.write.
Attachment #615257 - Flags: review?(dbaron)
Attached patch PatchSplinter Review
There's another issue here, which is that we hold onto a <canvas> from the first document indefinitely.
Attachment #616378 - Flags: review?(dbaron)
Attachment #615257 - Attachment is obsolete: true
Attachment #615257 - Flags: review?(dbaron)
Comment on attachment 616378 [details] [diff] [review]
Patch

Maybe also null-out currentDoc after its last use (in OnDocumentLoad) so that the its inner functions can't later use it?  r=dbaron
Attachment #616378 - Flags: review?(dbaron) → review+
http://hg.mozilla.org/mozilla-central/rev/b9936b8bcccf
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla14
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: