[JSC] Add @@toStringTag to WebAssembly.Global
authorysuzuki@apple.com <ysuzuki@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 4 May 2020 23:37:44 +0000 (23:37 +0000)
committerysuzuki@apple.com <ysuzuki@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 4 May 2020 23:37:44 +0000 (23:37 +0000)
https://bugs.webkit.org/show_bug.cgi?id=211372

Reviewed by Sam Weinig.

LayoutTests/imported/w3c:

* web-platform-tests/wasm/jsapi/global/toString.any-expected.txt:
* web-platform-tests/wasm/jsapi/global/toString.any.worker-expected.txt:

Source/JavaScriptCore:

As r260992 did for the other wasm prototypes, we should put @@toStringTag to WebAssembly.Global's prototype too.

* wasm/js/WebAssemblyGlobalPrototype.cpp:
(JSC::WebAssemblyGlobalPrototype::finishCreation):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@261125 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/imported/w3c/ChangeLog
LayoutTests/imported/w3c/web-platform-tests/wasm/jsapi/global/toString.any-expected.txt
LayoutTests/imported/w3c/web-platform-tests/wasm/jsapi/global/toString.any.worker-expected.txt
Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/wasm/js/WebAssemblyGlobalPrototype.cpp

index 0a75e59..604c0e3 100644 (file)
@@ -1,3 +1,13 @@
+2020-05-04  Yusuke Suzuki  <ysuzuki@apple.com>
+
+        [JSC] Add @@toStringTag to WebAssembly.Global
+        https://bugs.webkit.org/show_bug.cgi?id=211372
+
+        Reviewed by Sam Weinig.
+
+        * web-platform-tests/wasm/jsapi/global/toString.any-expected.txt:
+        * web-platform-tests/wasm/jsapi/global/toString.any.worker-expected.txt:
+
 2020-05-04  Emilio Cobos Ãlvarez  <emilio@crisal.io>
 
         Put lh / rlh units behind a flag until bug 211351 is sorted out.
index 312a2b1..b3ea983 100644 (file)
@@ -1,4 +1,4 @@
 
 PASS Object.prototype.toString on an Global 
-FAIL @@toStringTag exists on the prototype with the appropriate descriptor assert_own_property: expected property symbol "Symbol(Symbol.toStringTag)" missing
+PASS @@toStringTag exists on the prototype with the appropriate descriptor 
 
index 312a2b1..b3ea983 100644 (file)
@@ -1,4 +1,4 @@
 
 PASS Object.prototype.toString on an Global 
-FAIL @@toStringTag exists on the prototype with the appropriate descriptor assert_own_property: expected property symbol "Symbol(Symbol.toStringTag)" missing
+PASS @@toStringTag exists on the prototype with the appropriate descriptor 
 
index ebfb56a..85278ff 100644 (file)
@@ -1,3 +1,15 @@
+2020-05-04  Yusuke Suzuki  <ysuzuki@apple.com>
+
+        [JSC] Add @@toStringTag to WebAssembly.Global
+        https://bugs.webkit.org/show_bug.cgi?id=211372
+
+        Reviewed by Sam Weinig.
+
+        As r260992 did for the other wasm prototypes, we should put @@toStringTag to WebAssembly.Global's prototype too.
+
+        * wasm/js/WebAssemblyGlobalPrototype.cpp:
+        (JSC::WebAssemblyGlobalPrototype::finishCreation):
+
 2020-05-04  Devin Rousso  <drousso@apple.com>
 
         Web Inspector: Worker: should use the name of the worker if it exists
index 8a0e997..0993402 100644 (file)
@@ -143,6 +143,7 @@ void WebAssemblyGlobalPrototype::finishCreation(VM& vm, JSGlobalObject* globalOb
     JSFunction* valueSetterFunction = JSFunction::create(vm, globalObject, 1, "set value"_s, webAssemblyGlobalProtoSetterFuncValue, NoIntrinsic);
     GetterSetter* valueAccessor = GetterSetter::create(vm, globalObject, valueGetterFunction, valueSetterFunction);
     putDirectNonIndexAccessorWithoutTransition(vm, Identifier::fromString(vm, "value"_s), valueAccessor, static_cast<unsigned>(PropertyAttribute::Accessor));
+    JSC_TO_STRING_TAG_WITHOUT_TRANSITION();
 }
 
 WebAssemblyGlobalPrototype::WebAssemblyGlobalPrototype(VM& vm, Structure* structure)