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)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla1.9alpha1
People
(Reporter: taralx, Assigned: bzbarsky)
References
Details
Attachments
(2 files)
331 bytes,
text/html
|
Details | |
1.31 KB,
patch
|
darin.moz
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
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:
Honestly, the tabspinner should stop as soon as the IFRAME is removed.
Assignee | ||
Comment 2•19 years ago
|
||
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?
Comment 3•19 years ago
|
||
*** Bug 305471 has been marked as a duplicate of this bug. ***
Updated•19 years ago
|
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
Assignee | ||
Comment 4•19 years ago
|
||
Attachment #201489 -
Flags: superreview?(jst)
Attachment #201489 -
Flags: review?(darin)
Comment 5•19 years ago
|
||
Comment on attachment 201489 [details] [diff] [review] Like so, say sr=jst
Attachment #201489 -
Flags: superreview?(jst) → superreview+
Assignee | ||
Updated•19 years ago
|
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
Updated•19 years ago
|
Attachment #201489 -
Flags: review?(darin) → review+
Assignee | ||
Comment 6•19 years ago
|
||
Fixed on trunk.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•