Make ToPrimitive throw on symbol wrappers

test/mjsunit/es6/symbols.js

 // Copyright 2013 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 84 matching lines @@
   assertSame(Symbol, Object(Symbol()).__proto__.constructor)
   for (var i in symbols) {
     assertSame(Symbol, symbols[i].__proto__.constructor)
   }
 }
 TestConstructor()
 
 
 function TestValueOf() {
   for (var i in symbols) {
+    assertTrue(symbols[i] === Object(symbols[i]).valueOf())
     assertTrue(symbols[i] === symbols[i].valueOf())
+    assertTrue(Symbol.prototype.valueOf.call(Object(symbols[i])) === symbols[i])
     assertTrue(Symbol.prototype.valueOf.call(symbols[i]) === symbols[i])
   }
 }
 TestValueOf()
 
 
 function TestToString() {
   for (var i in symbols) {
     assertThrows(function() { String(symbols[i]) }, TypeError)
     assertThrows(function() { symbols[i] + "" }, TypeError)
-    assertTrue(isValidSymbolString(String(Object(symbols[i]))))
+    assertThrows(function() { String(Object(symbols[i])) }, TypeError)
     assertTrue(isValidSymbolString(symbols[i].toString()))
     assertTrue(isValidSymbolString(Object(symbols[i]).toString()))
     assertTrue(
       isValidSymbolString(Symbol.prototype.toString.call(symbols[i])))
     assertEquals(
       "[object Symbol]", Object.prototype.toString.call(symbols[i]))
   }
 }
 TestToString()
 
 
 function TestToBoolean() {
   for (var i in symbols) {
+    assertTrue(Boolean(Object(symbols[i])))
+    assertFalse(!Object(symbols[i]))
     assertTrue(Boolean(symbols[i]).valueOf())
     assertFalse(!symbols[i])
     assertTrue(!!symbols[i])
     assertTrue(symbols[i] && true)
     assertFalse(!symbols[i] && false)
     assertTrue(!symbols[i] || true)
     assertEquals(1, symbols[i] ? 1 : 2)
     assertEquals(2, !symbols[i] ? 1 : 2)
     if (!symbols[i]) assertUnreachable();
     if (symbols[i]) {} else assertUnreachable();
   }
 }
 TestToBoolean()
 
 
 function TestToNumber() {
   for (var i in symbols) {
+    assertThrows(function() { Number(Object(symbols[i])) }, TypeError)
+    assertThrows(function() { +Object(symbols[i]) }, TypeError)
     assertSame(NaN, Number(symbols[i]).valueOf())
     assertSame(NaN, symbols[i] + 0)
   }
 }
 TestToNumber()
 
 
 function TestEquality() {
   // Every symbol should equal itself, and non-strictly equal its wrapper.
   for (var i in symbols) {
@@ skipping 330 matching lines @@
   Realm.shared = symbol1
   assertSame("x1", Realm.eval(realm, "Symbol.keyFor(Realm.shared)"))
 
   var symbol3 = Realm.eval(realm, "Symbol.for('x3')")
   assertFalse(symbol1 === symbol3)
   assertFalse(symbol2 === symbol3)
   assertSame(symbol3, Symbol.for("x3"))
   assertSame("x3", Symbol.keyFor(symbol3))
 }
 TestRegistry()