Bug 1239666 - part 3, devtools test case changes. r=sphink
authorLars T Hansen <lhansen@mozilla.com>
Mon, 18 Jan 2016 09:49:17 +0100
changeset 280659 cbb0ad1e2118
parent 280658 85f60fdd98fa
child 280660 56c7becfd682
push id70552
push userlhansen@mozilla.com
push dateWed, 20 Jan 2016 14:06:18 +0000
treeherdermozilla-inbound@cbb0ad1e2118 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssphink
bugs1239666
milestone46.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 1239666 - part 3, devtools test case changes. r=sphink
devtools/shared/heapsnapshot/tests/gtest/DoesCrossZoneBoundaries.cpp
devtools/shared/heapsnapshot/tests/gtest/DoesntCrossZoneBoundaries.cpp
--- a/devtools/shared/heapsnapshot/tests/gtest/DoesCrossZoneBoundaries.cpp
+++ b/devtools/shared/heapsnapshot/tests/gtest/DoesCrossZoneBoundaries.cpp
@@ -4,20 +4,22 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 // Test that heap snapshots cross zone boundaries when expected.
 
 #include "DevTools.h"
 
 DEF_TEST(DoesCrossZoneBoundaries, {
     // Create a new global to get a new zone.
+    JS::CompartmentOptions options;
     JS::RootedObject newGlobal(cx, JS_NewGlobalObject(cx,
                                                       getGlobalClass(),
                                                       nullptr,
-                                                      JS::FireOnNewGlobalHook));
+                                                      JS::FireOnNewGlobalHook,
+                                                      options));
     ASSERT_TRUE(newGlobal);
     JS::Zone* newZone = nullptr;
     {
       JSAutoCompartment ac(cx, newGlobal);
       ASSERT_TRUE(JS_InitStandardClasses(cx, newGlobal));
       newZone = js::GetContextZone(cx);
     }
     ASSERT_TRUE(newZone);
--- a/devtools/shared/heapsnapshot/tests/gtest/DoesntCrossZoneBoundaries.cpp
+++ b/devtools/shared/heapsnapshot/tests/gtest/DoesntCrossZoneBoundaries.cpp
@@ -4,20 +4,22 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 // Test that heap snapshots walk the zone boundaries correctly.
 
 #include "DevTools.h"
 
 DEF_TEST(DoesntCrossZoneBoundaries, {
     // Create a new global to get a new zone.
+    JS::CompartmentOptions options;
     JS::RootedObject newGlobal(cx, JS_NewGlobalObject(cx,
                                                       getGlobalClass(),
                                                       nullptr,
-                                                      JS::FireOnNewGlobalHook));
+                                                      JS::FireOnNewGlobalHook,
+                                                      options));
     ASSERT_TRUE(newGlobal);
     JS::Zone* newZone = nullptr;
     {
       JSAutoCompartment ac(cx, newGlobal);
       ASSERT_TRUE(JS_InitStandardClasses(cx, newGlobal));
       newZone = js::GetContextZone(cx);
     }
     ASSERT_TRUE(newZone);