Bug 1610621 : ClearKeptObjects() should return undefined. r=jonco

Differential Revision: https://phabricator.services.mozilla.com/D60633 --HG-- extra : moz-landing-system : lando
dothq · Jan 22, 2020 · 10b18ed · 10b18ed
1 parent c1c4d77
commit 10b18ed
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/js/src/jit-test/tests/gc/bug-1610621.js b/js/src/jit-test/tests/gc/bug-1610621.js
@@ -0,0 +1 @@
+assertEq(clearKeptObjects(), undefined);
diff --git a/js/src/shell/js.cpp b/js/src/shell/js.cpp
@@ -2762,7 +2762,9 @@ static bool CpuNow(JSContext* cx, unsigned argc, Value* vp) {
 }
 
 static bool ClearKeptObjects(JSContext* cx, unsigned argc, Value* vp) {
+  CallArgs args = CallArgsFromVp(argc, vp);
   JS::ClearKeptObjects(cx);
+  args.rval().setUndefined();
   return true;
 }