Closed Bug 935786 Opened 6 years ago Closed 6 years ago
Assertion failure: !Is
Inner Window() || Is Current Inner Window() at ns Global Window .cpp:8909
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: nobody → peterv
Status: NEW → ASSIGNED
Does the desktop testcase in bug 937318 help?
Now we have a stack for it: https://tbpl.mozilla.org/php/getParsedLog.php?id=30446580&tree=Pine&full=1#error1
Hopefully people will look at the warning :-(.
Attachment #830757 - Flags: review?(bugs)
I pushed this patch to pine but it turned it orange: https://tbpl.mozilla.org/?tree=Pine
With the additional check to keep the old behaviour of allowing an outer in nsAutoWindowStateHelper.
Attachment #831449 - Flags: review+
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
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
You need to log in before you can comment on or make changes to this bug.