Closed Bug 308182 Opened 15 years ago Closed 15 years ago
[FIX]Iframes are not drawn after being shown
If a div with display none contains an iframe and then the display of the div is changed to block (or "") the content of the iframe is not shown (the borders of the iframe are shown). I think this one worked in the Aviary branch but it does not work in the 1.8 branch or on the trunk. I'll try to verify that it really worked in 1.7. Test case comming...
This is also broken with Firefox 1.0.x
This has been broken since about forever, actually... The problem is that we destroy the frame but don't null out the parentWidget on the docshell. Then when the document starts to load we create a presentation (see DocumentViewerImpl::InitInternal), even though the iframe is display:none by then. The patch just makes sure we unset the parentWidget when the nsSubDocumentFrame dies; the rest is cleanup that I thought made the code clearer.
The other question is whether this is worth fixing on the 1.8.1 branch. I think it may be...
OS: Windows XP → All
Priority: -- → P2
Hardware: PC → All
Summary: Iframes are not drawn after being shown → [FIX]Iframes are not drawn after being shown
Target Milestone: --- → mozilla1.9alpha
15 years ago
Attachment #209311 - Flags: review?(cbiesinger) → review+
Comment on attachment 209311 [details] [diff] [review] Patch to fix sr=jst
Attachment #209311 - Flags: superreview?(jst) → superreview+
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Comment on attachment 209311 [details] [diff] [review] Patch to fix I think it's worth it to take this on the 1.8 branch. This makes sure to not have a dangling widget pointer on the docshell, basically.
Attachment #209311 - Flags: approval1.8.1?
Attachment #209311 - Flags: approval1.8.1? → branch-1.8.1?(jst)
Attachment #209311 - Flags: branch-1.8.1?(jst) → branch-1.8.1+
Fixed on branch.
Component: Layout: HTML Frames → Layout: Images
Product: Core Graveyard → Core
You need to log in before you can comment on or make changes to this bug.