Bug 1066733: Fix JS Debugger test to add debuggees as intended. f=tromey r=sfink
authorJim Blandy <jimb@mozilla.com>
Mon, 15 Sep 2014 11:40:48 -0700
changeset 205350 cf7af109d888add6c381b30d974e3f252d34a6cf
parent 205349 8ed5d27728dc43cd122ca3bc43428fbab8566d84
child 205351 1d6858829094c8819371062f4e5e01c0b1d8051d
push id49155
push userjblandy@mozilla.com
push dateMon, 15 Sep 2014 18:41:34 +0000
treeherdermozilla-inbound@cf7af109d888 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssfink
bugs1066733
milestone35.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1066733: Fix JS Debugger test to add debuggees as intended. f=tromey r=sfink
js/src/jit-test/tests/debug/Debugger-debuggees-20.js
--- a/js/src/jit-test/tests/debug/Debugger-debuggees-20.js
+++ b/js/src/jit-test/tests/debug/Debugger-debuggees-20.js
@@ -4,24 +4,27 @@ var g1 = newGlobal();           // Creat
 var g2 = newGlobal();           // Created before the Debugger; not debuggee.
 
 var dbg = new Debugger;
 
 var g3 = newGlobal();           // Created after the Debugger; debuggee.
 var g4 = newGlobal();           // Created after the Debugger; not debuggee.
 
 var g1w = dbg.addDebuggee(g1);
+var g3w = dbg.addDebuggee(g3);
 assertEq(dbg.addAllGlobalsAsDebuggees(), undefined);
 
 // Get Debugger.Objects viewing the globals from their own compartments;
 // this is the sort that findAllGlobals and addDebuggee return.
-var g1w = g1w.makeDebuggeeValue(g1).unwrap();
+assertEq(g1w, g3w.makeDebuggeeValue(g1).unwrap());
+assertEq(g3w, g1w.makeDebuggeeValue(g3).unwrap());
+
 var g2w = g1w.makeDebuggeeValue(g2).unwrap();
-var g3w = g1w.makeDebuggeeValue(g3).unwrap();
 var g4w = g1w.makeDebuggeeValue(g4).unwrap();
+
 var thisw = g1w.makeDebuggeeValue(this).unwrap();
 
 // Check that they're all there.
 assertEq(dbg.hasDebuggee(g1w), true);
 assertEq(dbg.hasDebuggee(g2w), true);
 assertEq(dbg.hasDebuggee(g3w), true);
 assertEq(dbg.hasDebuggee(g4w), true);
 // The debugger's global is not a debuggee.