Closed
Bug 935786
Opened 11 years ago
Closed 11 years ago
Assertion failure: !IsInnerWindow() || IsCurrentInnerWindow() at nsGlobalWindow.cpp:8909
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
mozilla28
Tracking | Status | |
---|---|---|
firefox28 | --- | fixed |
People
(Reporter: gwagner, Assigned: peterv)
References
Details
(Keywords: assertion, Whiteboard: [qa-])
Attachments
(2 files)
(deleted),
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
peterv
:
review+
|
Details | Diff | Splinter Review |
I am seeing this assertion on mochitests on a debug emulator build.
These tests are currently only enabled on pine and cedar:
https://tbpl.mozilla.org/php/getParsedLog.php?id=30232634&tree=Pine&full=1#error2
It started with this merge: https://hg.mozilla.org/projects/pine/pushloghtml?startID=18&endID=19
Seems like bug 932309 caused it.
Smaug wrote a little debug patch for it. The results should show up here:https://tbpl.mozilla.org/?tree=Pine&showall=1
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → peterv
Status: NEW → ASSIGNED
Comment 1•11 years ago
|
||
Does the desktop testcase in bug 937318 help?
Reporter | ||
Comment 2•11 years ago
|
||
Now we have a stack for it:
https://tbpl.mozilla.org/php/getParsedLog.php?id=30446580&tree=Pine&full=1#error1
Assignee | ||
Comment 3•11 years ago
|
||
Hopefully people will look at the warning :-(.
Attachment #830757 -
Flags: review?(bugs)
Updated•11 years ago
|
Attachment #830757 -
Flags: review?(bugs) → review+
Reporter | ||
Comment 4•11 years ago
|
||
I pushed this patch to pine but it turned it orange:
https://tbpl.mozilla.org/?tree=Pine
Assignee | ||
Comment 5•11 years ago
|
||
With the additional check to keep the old behaviour of allowing an outer in nsAutoWindowStateHelper.
Attachment #831449 -
Flags: review+
Comment 6•11 years ago
|
||
I went ahead and pushed this to inbound to hopefully clear up bug 934769 while the OOM work continues.
https://hg.mozilla.org/integration/mozilla-inbound/rev/594b69bb0ac3
Comment 7•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•