Last Comment Bug 781722 - Crash from a scratchpad's sandbox: "Assertion Failure: rt->heapState == JSRuntime::Idle"
: Crash from a scratchpad's sandbox: "Assertion Failure: rt->heapState == JSRun...
: assertion, crash
Product: Core
Classification: Components
Component: XPConnect (show other bugs)
: Trunk
: x86_64 Linux
-- critical (vote)
: mozilla17
Assigned To: Nobody; OK to take it and work on it
: Andrew Overholt [:overholt]
Depends on:
  Show dependency treegraph
Reported: 2012-08-09 18:44 PDT by Thaddee Tyl [:espadrine]
Modified: 2012-08-14 06:00 PDT (History)
4 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Backtrace (13.22 KB, text/plain)
2012-08-09 20:15 PDT, Thaddee Tyl [:espadrine]
no flags Details
Loosen over-tight assertion in JS_GetGlobalForScopeChain. (840 bytes, patch)
2012-08-12 16:43 PDT, Nicholas Nethercote [:njn]
wmccloskey: review+
Details | Diff | Splinter Review

Description User image Thaddee Tyl [:espadrine] 2012-08-09 18:44:27 PDT

Situation: writing in the scratchpad that may have evaluated
something in the background.

I got the following message, too:
Assertion failure: rt->heapState == JSRuntime::Idle, at /home/tyl/files/cloud/fx-team/js/src/jsapi.cpp:241
Comment 1 User image Thaddee Tyl [:espadrine] 2012-08-09 20:15:14 PDT
Created attachment 650766 [details]
Comment 2 User image Bobby Holley (:bholley) (busy with Stylo) 2012-08-10 01:23:50 PDT
Bill, Nick, looks like we're running into an AssertHeapIsIdle assertion while collecting compartment stats. Can you have a look at the stack and say where we're going wrong?
Comment 3 User image Nicholas Nethercote [:njn] 2012-08-12 16:43:27 PDT
Created attachment 651240 [details] [diff] [review]
Loosen over-tight assertion in JS_GetGlobalForScopeChain.

I assume it's reasonable for JS_GetGlobalForScopeChain() to be called within
a call to nsXPConnect::GetNativeOfWrapper()?  If so, the assertion is too
strong, and should be changed to AssertHeapIsIdleOrIterating().

It's possible that we'll see this same overtight assertion in other
jsapi.cpp functions that can be called within GetNativeOfWrapper().
Comment 4 User image Nicholas Nethercote [:njn] 2012-08-13 16:52:57 PDT
Comment 5 User image Ed Morley [:emorley] 2012-08-14 06:00:39 PDT

Note You need to log in before you can comment on or make changes to this bug.