Avoid using computed property literals in TypedArrays code

src/typedarray.js

 // Copyright 2013 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 (function(global, utils) {
 
 "use strict";
 
 %CheckIsBootstrapping();
 
@@ skipping 123 matching lines @@
 
 function NAMEConstructByIterable(obj, iterable, iteratorFn) {
   var list = new InternalArray();
   // Reading the Symbol.iterator property of iterable twice would be
   // observable with getters, so instead, we call the function which
   // was already looked up, and wrap it in another iterable. The
   // __proto__ of the new iterable is set to null to avoid any chance
   // of modifications to Object.prototype being observable here.
   var iterator = %_CallFunction(iterable, iteratorFn);
   var newIterable = {
-    __proto__: null,
-    [symbolIterator]() { return iterator; }
+    __proto__: null
   };
+  // TODO(littledan): Computed properties don't work yet in nosnap.
+  // Rephrase when they do.

[arv (Not doing code reviews), 2015/06/16 16:41:04]

I guess we cannot use shipping features until the flag is removed?

+  newIterable[symbolIterator] = function() { return iterator; }
   for (var value of newIterable) {
     list.push(value);
   }
   NAMEConstructByArrayLike(obj, list);
 }
 
 function NAMEConstructor(arg1, arg2, arg3) {
   if (%_IsConstructCall()) {
     if (IS_ARRAYBUFFER(arg1) || IS_SHAREDARRAYBUFFER(arg1)) {
       NAMEConstructByArrayBuffer(this, arg1, arg2, arg3);
@@ skipping 341 matching lines @@
   "setUint32", DataViewSetUint32JS,
 
   "getFloat32", DataViewGetFloat32JS,
   "setFloat32", DataViewSetFloat32JS,
 
   "getFloat64", DataViewGetFloat64JS,
   "setFloat64", DataViewSetFloat64JS
 ]);
 
 })