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

Unified Diff: test/mjsunit/es6/typedarray.js

Issue 2090353003: Amend DataView, ArrayBuffer, and TypedArray methods to use ToIndex. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: enable test Created 4 years, 6 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
Index: test/mjsunit/es6/typedarray.js
diff --git a/test/mjsunit/es6/typedarray.js b/test/mjsunit/es6/typedarray.js
index 4bdf8226a8551c0fa250ffcdeb2c3f99b5ce08a1..a36efd9fe68981bdeaa973adb23b329a8e003b4c 100644
--- a/test/mjsunit/es6/typedarray.js
+++ b/test/mjsunit/es6/typedarray.js
@@ -44,11 +44,9 @@ function TestArrayBufferCreation() {
assertThrows(function() { new ArrayBuffer(-10); }, RangeError);
assertThrows(function() { new ArrayBuffer(-2.567); }, RangeError);
-/* TODO[dslomov]: Reenable the test
assertThrows(function() {
var ab1 = new ArrayBuffer(0xFFFFFFFFFFFF)
}, RangeError);
Dan Ehrenberg 2016/06/23 21:45:32 Eh, can you run this with msan or something before
bakkot 2016/06/28 23:41:07 Yes indeed; fails with "MemorySanitizer failed to
-*/
var ab = new ArrayBuffer();
assertSame(0, ab.byteLength);

Powered by Google App Engine
This is Rietveld 408576698