[typedarrays] Check detached buffer at start of typed array methods

test/mjsunit/regress/regress-353004.js

Index: test/mjsunit/regress/regress-353004.js
diff --git a/test/mjsunit/regress/regress-353004.js b/test/mjsunit/regress/regress-353004.js
index 7e1fb7e939e8b5c9cf59f01e84fe9d3316b2ac33..f035a30dbf3e0116713124dcb5decef7cc68bdd2 100644
--- a/test/mjsunit/regress/regress-353004.js
+++ b/test/mjsunit/regress/regress-353004.js
@@ -59,18 +59,9 @@ assertThrows(function() {
 
 var buffer9 = new ArrayBuffer(1024);
 var array9 = new Uint8Array(buffer9);
-var array10 = array9.subarray({valueOf : function() {
+assertThrows(() =>
+  array9.subarray({valueOf : function() {
     %ArrayBufferNeuter(buffer9);
     return 0;
-  }}, 1024);
+  }}, 1024), TypeError);
 assertEquals(0, array9.length);
-assertEquals(0, array10.length);
-
-var buffer11 = new ArrayBuffer(1024);
-var array11 = new Uint8Array(buffer11);
-var array12 = array11.subarray(0, {valueOf : function() {
-      %ArrayBufferNeuter(buffer11);
-      return 1024;
-    }});
-assertEquals(0, array11.length);
-assertEquals(0, array12.length);

[Dan Ehrenberg, 2017/03/28 19:39:28]

Rather than deleting the test, can you change it to expect what's the new
current behavior? In this case, it should throw a TypeError.

[Choongwoo Han, 2017/03/29 05:36:54]

Done.