The default bug view has changed. See this FAQ.

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

RESOLVED FIXED in mozilla1.9alpha1

Status

()

Core
DOM
P2
normal
RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: Taral, Assigned: bz)

Tracking

Trunk
mozilla1.9alpha1
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

12 years ago
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:
(Reporter)

Comment 1

12 years ago
Created attachment 189367 [details]
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?

Comment 3

12 years ago
*** Bug 305471 has been marked as a duplicate of this bug. ***

Updated

12 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
Created attachment 201489 [details] [diff] [review]
Like so, say
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

Updated

12 years ago
Attachment #201489 - Flags: review?(darin) → review+
Fixed on trunk.
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.