Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(974)

Unified Diff: test/mjsunit/harmony/futex.js

Issue 2642293002: Add Isolate parameter to disallow Atomics.wait (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/runtime/runtime-futex.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/harmony/futex.js
diff --git a/test/mjsunit/harmony/futex.js b/test/mjsunit/harmony/futex.js
index f90b773aa390aa4031b090fddf6bcc0d9050f609..77a89cc3000e49311f5098df529ec8d9d2735c45 100644
--- a/test/mjsunit/harmony/futex.js
+++ b/test/mjsunit/harmony/futex.js
@@ -95,6 +95,15 @@
assertEquals("timed-out", Atomics.wait(i32a, 0, 0, -Infinity));
})();
+(function TestWaitNotAllowed() {
+ % SetAllowAtomicsWait(false);
+ var i32a = new Int32Array(new SharedArrayBuffer(16));
+ assertThrows(function() {
+ Atomics.wait(i32a, 0, 0, -1);
+ });
+ % SetAllowAtomicsWait(true);
jochen (gone - plz use gerrit) 2017/01/23 08:33:07 no space between % and the intrinsic name please
binji 2017/01/23 18:26:30 Hm, looks like "git cl format" did this. I'll just
+})();
+
//// WORKER ONLY TESTS
if (this.Worker) {
« no previous file with comments | « src/runtime/runtime-futex.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698