Expose CloseEvent and CustomEvent to workers
authoryusukesuzuki@slowstart.org <yusukesuzuki@slowstart.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 13 Aug 2018 13:31:50 +0000 (13:31 +0000)
committeryusukesuzuki@slowstart.org <yusukesuzuki@slowstart.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 13 Aug 2018 13:31:50 +0000 (13:31 +0000)
https://bugs.webkit.org/show_bug.cgi?id=188493

Reviewed by Sam Weinig.

LayoutTests/imported/w3c:

* web-platform-tests/workers/semantics/interface-objects/001.worker-expected.txt:

Source/WebCore:

This patch exposes CloseEvent and CustomEvent to workers.
It is OK since they do not depend on things in the main thread.
For example, we cannot expose FormData right now since it depends
on Page.

We do not expose PromiseRejectionEvent in this patch since it will
be done in [1].

[1]: https://bugs.webkit.org/show_bug.cgi?id=188265

* Modules/websockets/CloseEvent.idl:
* dom/CustomEvent.idl:

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

LayoutTests/imported/w3c/ChangeLog
LayoutTests/imported/w3c/web-platform-tests/workers/semantics/interface-objects/001.worker-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/Modules/websockets/CloseEvent.idl
Source/WebCore/dom/CustomEvent.idl

index 2686a74..d12365a 100644 (file)
@@ -1,3 +1,12 @@
+2018-08-13  Yusuke Suzuki  <yusukesuzuki@slowstart.org>
+
+        Expose CloseEvent and CustomEvent to workers
+        https://bugs.webkit.org/show_bug.cgi?id=188493
+
+        Reviewed by Sam Weinig.
+
+        * web-platform-tests/workers/semantics/interface-objects/001.worker-expected.txt:
+
 2018-08-13  Oriol Brufau  <obrufau@igalia.com>
 
         [css-logical] Implement flow-relative margin, padding, border and sizing properties
index 57e3852..a4bd1f8 100644 (file)
@@ -15,7 +15,7 @@ PASS The Path2D interface object should be exposed.
 FAIL The PromiseRejectionEvent interface object should be exposed. assert_own_property: expected property "PromiseRejectionEvent" missing
 PASS The EventSource interface object should be exposed. 
 PASS The WebSocket interface object should be exposed. 
-FAIL The CloseEvent interface object should be exposed. assert_own_property: expected property "CloseEvent" missing
+PASS The CloseEvent interface object should be exposed. 
 FAIL The BroadcastChannel interface object should be exposed. assert_own_property: expected property "BroadcastChannel" missing
 PASS The ArrayBuffer interface object should be exposed. 
 PASS The Int8Array interface object should be exposed. 
@@ -43,7 +43,7 @@ PASS The FileReaderSync interface object should be exposed.
 PASS The EventTarget interface object should be exposed. 
 PASS The ErrorEvent interface object should be exposed. 
 PASS The Event interface object should be exposed. 
-FAIL The CustomEvent interface object should be exposed. assert_own_property: expected property "CustomEvent" missing
+PASS The CustomEvent interface object should be exposed. 
 PASS The DOMException interface object should be exposed. 
 PASS The ReadableStream interface object should be exposed. 
 FAIL The WritableStream interface object should be exposed. assert_own_property: expected property "WritableStream" missing
index 6c2382b..1c8c95e 100644 (file)
@@ -1,3 +1,23 @@
+2018-08-13  Yusuke Suzuki  <yusukesuzuki@slowstart.org>
+
+        Expose CloseEvent and CustomEvent to workers
+        https://bugs.webkit.org/show_bug.cgi?id=188493
+
+        Reviewed by Sam Weinig.
+
+        This patch exposes CloseEvent and CustomEvent to workers.
+        It is OK since they do not depend on things in the main thread.
+        For example, we cannot expose FormData right now since it depends
+        on Page.
+
+        We do not expose PromiseRejectionEvent in this patch since it will
+        be done in [1].
+
+        [1]: https://bugs.webkit.org/show_bug.cgi?id=188265
+
+        * Modules/websockets/CloseEvent.idl:
+        * dom/CustomEvent.idl:
+
 2018-08-13  Oriol Brufau  <obrufau@igalia.com>
 
         [css-logical] Implement flow-relative margin, padding, border and sizing properties
index db536c2..eeccfeb 100644 (file)
@@ -28,9 +28,9 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-// FIXME: This should be exposed to workers as well.
 [
     Constructor(DOMString type, optional CloseEventInit eventInitDict),
+    Exposed=(Window,Worker),
 ] interface CloseEvent : Event {
     readonly attribute boolean wasClean;
     readonly attribute unsigned short code;
index 7d39bb1..78fcea2 100644 (file)
@@ -24,9 +24,9 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
  */
 
-// FIXME: This should be exposed to workers as well.
 [
     Constructor(DOMString type, optional CustomEventInit eventInitDict),
+    Exposed=(Window,Worker),
     JSCustomMarkFunction,
 ] interface CustomEvent : Event {
     [CustomGetter] readonly attribute any detail;