Closed Bug 300849 Opened 19 years ago Closed 19 years ago

[FIX]IFRAME removed during its own onload event never turns off tabspinner

Categories

(Core :: DOM: Core & HTML, defect, P2)

defect

Tracking

()

RESOLVED FIXED
mozilla1.9alpha1

People

(Reporter: taralx, Assigned: bzbarsky)

References

Details

Attachments

(2 files)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2) Gecko/20050630 Firefox/1.0+
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2) Gecko/20050630 Firefox/1.0+

If an IFRAME is removed during its own load event, the tabspinner will run forever.

Reproducible: Always

Steps to Reproduce:
Attached file Testcase
Honestly, the tabspinner should stop as soon as the IFRAME is removed.
Sounds like we remove ourselves from the docloader tree before we get a chance
to notify our parent... Should we save the parent ptr before firing onload or
something?
*** Bug 305471 has been marked as a duplicate of this bug. ***
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: IFRAME removed during its own load event never turns off tabspinner → IFRAME removed during its own onload event never turns off tabspinner
Attached patch Like so, saySplinter Review
Attachment #201489 - Flags: superreview?(jst)
Attachment #201489 - Flags: review?(darin)
Comment on attachment 201489 [details] [diff] [review]
Like so, say

sr=jst
Attachment #201489 - Flags: superreview?(jst) → superreview+
Assignee: general → bzbarsky
OS: Windows XP → All
Priority: -- → P2
Hardware: PC → All
Summary: IFRAME removed during its own onload event never turns off tabspinner → [FIX]IFRAME removed during its own onload event never turns off tabspinner
Target Milestone: --- → mozilla1.9alpha
Attachment #201489 - Flags: review?(darin) → review+
Fixed on trunk.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.