Closed
Bug 560447
Opened 15 years ago
Closed 15 years ago
"ASSERTION: setting NEEDS_FRAME on a node that already has a frame?" with image map
Categories
(Core :: Layout, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: jruderman, Assigned: tnikkel)
References
Details
(Keywords: assertion, testcase)
Attachments
(2 files)
|
(deleted),
text/html
|
Details | |
|
(deleted),
patch
|
bzbarsky
:
review+
|
Details | Diff | Splinter Review |
###!!! ASSERTION: setting NEEDS_FRAME on a node that already has a frame?: '!child->GetPrimaryFrame()', file /Users/jruderman/mozilla-central/layout/base/nsCSSFrameConstructor.cpp, line 6213
###!!! ASSERTION: NEEDS_FRAME set on a node that already has a frame?: '!child->GetPrimaryFrame()', file /Users/jruderman/mozilla-central/layout/base/nsCSSFrameConstructor.cpp, line 6271
|
Assignee | |
Comment 1•15 years ago
|
||
The mutation observer for the image map will set the primary frame to the image before the frame constructor gets the mutation notifications. So we need to add the those ugly area checks to some assertions. Even though the area's will have a primary frame, we want them to go through frame construction like they did before.
Assignee: nobody → tnikkel
Attachment #440678 -
Flags: review?(bzbarsky)
|
||
Comment 2•15 years ago
|
||
Comment on attachment 440678 [details] [diff] [review]
patch
r=bzbarsky. I hate <area>
Attachment #440678 -
Flags: review?(bzbarsky) → review+
|
|
Assignee | |
Comment 3•15 years ago
|
||
Status: NEW → RESOLVED
Closed: 15 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•