Upstream service worker message event tests to WPT

third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/ServiceWorkerGlobalScope/resources/extendable-message-event-utils.js

Index: third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/ServiceWorkerGlobalScope/resources/extendable-message-event-utils.js
diff --git a/third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/ServiceWorkerGlobalScope/resources/extendable-message-event-utils.js b/third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/ServiceWorkerGlobalScope/resources/extendable-message-event-utils.js
new file mode 100644
index 0000000000000000000000000000000000000000..1fa8c5a9c0981aa4fd96ef29bedb6a8003f483e7
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/ServiceWorkerGlobalScope/resources/extendable-message-event-utils.js
@@ -0,0 +1,78 @@
+var ExtendableMessageEventUtils = {};

[falken, 2017/03/21 14:42:56]

This file seems to be using a four space indent mostly, is there a reason for
that? In our style guide which most existing WPT SW tests are based on
<https://www.chromium.org/blink/serviceworker/testing> we use a two space
indent.

(BTW eventually I'd like the WPT test style to use a more common JS style, but I
don't want to do a big reformatting until we've finishing upstreaming the
existing tests.)

[mike3, 2017/03/21 17:15:29]

There's some inconsistency in the WPT tests, but I may have chosen the
lesser-used
version. Happy to reformat, though.

+
+// Create a representation of a given ExtendableMessageEvent that is suitable
+// for transmission via the `postMessage` API.
+ExtendableMessageEventUtils.serialize = function(event) {
+    var ports = event.ports.map(function(port) {
+            return { constructor: { name: port.constructor.name } };
+        });

[falken, 2017/03/21 14:42:56]

I think according to our style guide this is indented like so:

var ports = events.ports.map(function(port) {
      return {...};
  });

[mike3, 2017/03/21 17:15:29]

Acknowledged.

+    return {
+            constructor: {
+                name: event.constructor.name
+            },
+            origin: event.origin,
+            lastEventId: event.lastEventId,
+            source: {
+                constructor: {
+                    name: event.source.constructor.name
+                },
+                url: event.source.url,
+                frameType: event.source.frameType,
+                visibilityState: event.source.visibilityState,
+                focused: event.source.focused
+            },
+            ports: ports
+        };

[falken, 2017/03/21 14:42:56]

I think according to our style guide lines 10-24 should just be indented 2
spaces:

return {
  a: x,
  b: y
};

[mike3, 2017/03/21 17:15:29]

Acknowledged.

+};
+
+// Compare the actual and expected values of an ExtendableMessageEvent that has
+// been transformed using the `serialize` function defined in this file.
+ExtendableMessageEventUtils.assert_equals = function(actual, expected) {
+    assert_equals(
+        actual.constructor.name, expected.constructor.name, 'event constructor'
+    );
+    assert_equals(actual.origin, expected.origin, 'event `origin` property');
+    assert_equals(
+        actual.lastEventId,
+        expected.lastEventId,
+        'event `lastEventId` property'
+    );
+
+    assert_equals(
+        actual.source.constructor.name,
+        expected.source.constructor.name,
+        'event `source` property constructor'
+    );
+    assert_equals(
+        actual.source.url, expected.source.url, 'event `source` property `url`'
+    );
+    assert_equals(
+        actual.source.frameType,
+        expected.source.frameType,
+        'event `source` property `frameType`'
+    );
+    assert_equals(
+        actual.source.visibilityState,
+        expected.source.visibilityState,
+        'event `source` property `visibilityState`'
+    );
+    assert_equals(
+        actual.source.focused,
+        expected.source.focused,
+        'event `source` property `focused`'
+    );
+
+    assert_equals(
+        actual.ports.length,
+        expected.ports.length,
+        'event `ports` property length'
+    );
+
+    for (var idx = 0; idx < expected.ports.length; ++idx) {
+        assert_equals(
+            actual.ports[idx].constructor.name,
+            expected.ports[idx].constructor.name,
+            'MessagePort #' + idx + ' constructor'
+        );
+    }
+};