Closed
Bug 864580
Opened 11 years ago
Closed 10 years ago
[jsdbg2] Assertion failure: site->firstBreakpoint() == __null, at jsscript.cpp:2472
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
VERIFIED
FIXED
mozilla29
People
(Reporter: decoder, Assigned: jonco)
References
Details
(Keywords: assertion, testcase, Whiteboard: [jsbugmon:update])
The following testcase asserts on mozilla-central revision 4f364593f8bf (run with --ion-eager): eval("(function() { " + "\ var g = newGlobal('new-compartment');\ var dbg = new Debugger();\ var gw = dbg.addDebuggee(g);\ g.eval('' +\ function fib(n) {});\ gw.makeDebuggeeValue(g.fib).script.setBreakpoint(0, { $ : function (f) { } });\ dbg.removeDebuggee(g);\ " + " })();"); eval("(function() { " + "\ schedulegc(this);\ gc('compartment');\ " + " })();");
Reporter | ||
Updated•11 years ago
|
Whiteboard: [jsbugmon:update,bisect]
Reporter | ||
Updated•11 years ago
|
Whiteboard: [jsbugmon:update,bisect] → [jsbugmon:update]
Reporter | ||
Comment 1•11 years ago
|
||
JSBugMon: Bisection requested, result: autoBisect shows this is probably related to the following changeset: The first bad revision is: changeset: 91130:15a23c3923ff user: Bill McCloskey date: Tue Apr 03 14:07:38 2012 -0700 summary: Bug 742570 - Improve GC testing functions (r=igor) This iteration took 77.050 seconds to run.
Reporter | ||
Updated•10 years ago
|
Whiteboard: [jsbugmon:update] → [jsbugmon:update,ignore]
Reporter | ||
Comment 2•10 years ago
|
||
JSBugMon: The testcase found in this bug no longer reproduces (tried revision 73eefb421e2a).
Comment 3•10 years ago
|
||
autoBisect shows this is probably related to the following changeset: The first good revision is: changeset: http://hg.mozilla.org/mozilla-central/rev/b8a6bf4b4fa3 user: Jon Coppeard date: Sat Jan 11 13:52:35 2014 +0000 summary: Bug 956324 - Remove all breakpoints from a debugger when the last debuggee is removed r=billm Jon, is bug 956324 a likely fix?
Flags: needinfo?(jcoppeard)
Assignee | ||
Comment 4•10 years ago
|
||
Yes, definitely. Can you check in the testcase as a test though? I think Jim said we don't exercise removing debuggees in the browser (and obviously the jittests didn't cant this before) so this would be good to have.
Flags: needinfo?(jcoppeard)
Comment 5•10 years ago
|
||
Marking FIXED by bug 956324, and setting in-testsuite? flag.
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Whiteboard: [jsbugmon:update,ignore] → [jsbugmon:update]
Updated•10 years ago
|
Reporter | ||
Updated•10 years ago
|
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 6•10 years ago
|
||
JSBugMon: This bug has been automatically verified fixed.
You need to log in
before you can comment on or make changes to this bug.
Description
•