Closed
Bug 308182
Opened 19 years ago
Closed 18 years ago
[FIX]Iframes are not drawn after being shown
Categories
(Core :: Layout: Images, Video, and HTML Frames, defect, P2)
Core
Layout: Images, Video, and HTML Frames
Tracking
()
RESOLVED
FIXED
mozilla1.9alpha1
People
(Reporter: erik, Assigned: bzbarsky)
Details
(Keywords: fixed1.8.1, testcase)
Attachments
(2 files)
442 bytes,
text/html
|
Details | |
2.50 KB,
patch
|
Biesinger
:
review+
jst
:
superreview+
jst
:
approval-branch-1.8.1+
|
Details | Diff | Splinter Review |
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...
Reporter | ||
Comment 1•19 years ago
|
||
Assignee | ||
Comment 3•18 years ago
|
||
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.
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #209311 -
Flags: superreview?(jst)
Attachment #209311 -
Flags: review?(cbiesinger)
Assignee | ||
Comment 4•18 years ago
|
||
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
Updated•18 years ago
|
Attachment #209311 -
Flags: review?(cbiesinger) → review+
Comment 5•18 years ago
|
||
Comment on attachment 209311 [details] [diff] [review] Patch to fix sr=jst
Attachment #209311 -
Flags: superreview?(jst) → superreview+
Assignee | ||
Comment 6•18 years ago
|
||
Fixed.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 7•18 years ago
|
||
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?
Assignee | ||
Updated•18 years ago
|
Flags: blocking1.8.1?
Updated•18 years ago
|
Attachment #209311 -
Flags: approval1.8.1? → branch-1.8.1?(jst)
Updated•18 years ago
|
Attachment #209311 -
Flags: branch-1.8.1?(jst) → branch-1.8.1+
Assignee | ||
Updated•18 years ago
|
Flags: blocking1.8.1?
Updated•6 years ago
|
Product: Core → Core Graveyard
Updated•6 years ago
|
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.
Description
•