Closed
Bug 379975
Opened 17 years ago
Closed 17 years ago
[FIX]Crash [@ nsFrameManager::InsertFrames()] when getting documentElement that has a binding attached to it
Categories
(Core :: Layout, defect, P1)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla1.9alpha5
People
(Reporter: Gavin, Assigned: bzbarsky)
Details
(Keywords: crash, testcase)
Crash Data
Attachments
(2 files)
|
575 bytes,
text/html
|
Details | |
|
1019 bytes,
patch
|
roc
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
aParentFrame is null at http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/layout/base/nsFrameManager.cpp&rev=1.245&mark=675#667 .
|
Reporter | |
Updated•17 years ago
|
Attachment #264032 -
Attachment description: testcase → testcase (crashes on load)
|
|
Reporter | |
Updated•17 years ago
|
|
Assignee | |
Comment 1•17 years ago
|
||
When the XBL loads, it tries to reframe the bound node (in this case the root). The non-inline script means we haven't called InitialReflow yet, so we don't have a viewport frame and all that jazz. For non-root nodes this doesn't matter, since GetPrimaryFrameFor() on the parent returns null, and then we bail out of frame construction. But for the root we need to deal here.
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #264036 -
Flags: superreview?(roc)
Attachment #264036 -
Flags: review?(roc)
|
|
Assignee | |
Updated•17 years ago
|
Priority: -- → P1
Summary: Crash [@ nsFrameManager::InsertFrames()] when getting documentElement that has a binding attached to it → [FIX]Crash [@ nsFrameManager::InsertFrames()] when getting documentElement that has a binding attached to it
Target Milestone: --- → mozilla1.9alpha5
Attachment #264036 -
Flags: superreview?(roc)
Attachment #264036 -
Flags: superreview+
Attachment #264036 -
Flags: review?(roc)
Attachment #264036 -
Flags: review+
|
|
Assignee | |
Comment 2•17 years ago
|
||
Fixed.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
|
||
Updated•13 years ago
|
Crash Signature: [@ nsFrameManager::InsertFrames()]
|
||
Comment 3•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/a810a5e16f04
Flags: in-testsuite? → in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•