[es6] ship @@toStringTag

test/mjsunit/es6/prototype-ordinary-objects.js

 // Copyright 2015 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.
 
 
 var funcs = [
 
   // https://code.google.com/p/v8/issues/detail?id=4002
   // Error,
   // EvalError,
   // RangeError,
   // ReferenceError,
   // SyntaxError,
   // TypeError,
   // URIError,
 
   // https://code.google.com/p/v8/issues/detail?id=4003
   // RegExp,
 
   // https://code.google.com/p/v8/issues/detail?id=4004
   // Date,
 
   // https://code.google.com/p/v8/issues/detail?id=4006
   // String,
 
-  ArrayBuffer,
   Boolean,
-  DataView,
   Float32Array,
   Float64Array,
   Int16Array,
   Int32Array,
   Int8Array,
-  Map,
   Number,
   Object,
-  Promise,
-  // Proxy,
-  Set,
-  Symbol,
   Uint16Array,
   Uint32Array,
   Uint8Array,
-  Uint8ClampedArray,
-  WeakMap,
-  WeakSet,
+  Uint8ClampedArray
 ];
 
 for (var fun of funcs) {
   var p = fun.prototype;
   assertEquals('[object Object]', Object.prototype.toString.call(p));
 }
 
 
 // These still have special prototypes for legacy reason.
 var funcs = [
   Array,
   Function,
 ];
 
 for (var fun of funcs) {
   var p = fun.prototype;
   assertEquals('[object ' + fun.name + ']', Object.prototype.toString.call(p));
 }
+
+
+// @@toStringTag alters the result of O.p.toString()
+// %TypedArray% [ @@toStringTag ] is an accessor which returns undefined if

[arv (Not doing code reviews), 2015/04/09 13:41:22]

Do we have a bug then? Float32Array etc above should have a @@toStringTag.

Maybe the above test should first have deleted the @@toStringTag?

[caitp (gmail), 2015/04/09 14:10:22]

They have @@toStringTag --- it's a getter which is spec'd to return undefined if
`this` is not a %TypedArray% (eg. when getting it via the prototype, which is an
ordinary object).

It's doing the right thing here
Hmm --- since @@toStringTag is tested separately, I guess it's simpler to just
do that

+// receiver is not a %TypedArray%, and thus does not appear in this list.
+var funcs = [
+  ArrayBuffer,
+  DataView,
+  Map,
+  Promise,
+  Set,
+  Symbol,
+  WeakMap,
+  WeakSet
+];
+
+for (var fun of funcs) {
+  var p = fun.prototype;
+  var name = p[Symbol.toStringTag];
+  assertEquals(name, fun.name);
+  assertEquals(`[object ${name}]`, Object.prototype.toString.call(p));

[arv (Not doing code reviews), 2015/04/09 13:41:22]

update line 54 for consistency?

[caitp (gmail), 2015/04/09 14:10:22]

Acknowledged.

+}