Closed
Bug 303006
Opened 19 years ago
Closed 19 years ago
[FIX]ASSERTION: Placeholder relationship should have been torn down
Categories
(Core :: Layout, defect, P2)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla1.8beta4
People
(Reporter: bc, Assigned: bzbarsky)
References
()
Details
Attachments
(1 file)
2.21 KB,
patch
|
dbaron
:
review+
dbaron
:
superreview+
|
Details | Diff | Splinter Review |
ASSERTION: Placeholder relationship should have been torn down: '!shell->FrameManager()->GetPlaceholderFrameFor(mOutOfFlowFrame)', file c:/work/mozilla/builds/ff/trunk/mozilla/layout/generic/nsPlaceholderFrame.cpp, line 96 Found using a CVS debug build from 7/31 on WinXP during top site testing of 5469 pages. It appears to be new although I don't have good history of logs for full top site testing. I doubt this is related to the splitwindow bug, but ccing jst just in case.
Assignee | ||
Comment 1•19 years ago
|
||
Any idea which URL this might have been on?
Reporter | ||
Comment 2•19 years ago
|
||
Sorry, I thought I had included them. These were found in the full top site scan. (* - reproducible loading individual pages) <http://www.amazon.com/exec/obidos/tg/browse/-/172282/ref=sd_allcatpop_el/103-8516287-7640622> *<http://shop.lycos.com/> *<http://msn.careerbuilder.com/PLI/R/FindJobs.htm?siteid=CBMSNBC017> (lots) *<http://www.nytimes.com/adx/bin/adx_click.html?type=goto&page=homepage.nytimes.com/indexnew.html&pos=TopRight&camp=nwmutual12-nyt1&ad=184x901.gif&goto=http://www.nmfn.com>
Assignee | ||
Comment 3•19 years ago
|
||
Assignee: nobody → bzbarsky
Status: UNCONFIRMED → ASSIGNED
Attachment #193228 -
Flags: superreview?(dbaron)
Attachment #193228 -
Flags: review?(dbaron)
Assignee | ||
Updated•19 years ago
|
OS: Windows XP → All
Priority: -- → P2
Hardware: PC → All
Summary: ASSERTION: Placeholder relationship should have been torn down → [FIX]ASSERTION: Placeholder relationship should have been torn down
Target Milestone: --- → mozilla1.8beta4
Comment on attachment 193228 [details] [diff] [review] Fix >+ aFrameIn = nsPlaceholderFrame::GetRealFrameFor(placeholder); Use GetRealFrameForPlaceholder since you already did the GetType check, and r+sr=dbaron.
Attachment #193228 -
Flags: superreview?(dbaron)
Attachment #193228 -
Flags: superreview+
Attachment #193228 -
Flags: review?(dbaron)
Attachment #193228 -
Flags: review+
Assignee | ||
Comment 5•19 years ago
|
||
Fixed, with that change.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•