Attachment 347819 Details for Bug 188856 – Patch for landing

[patch] Patch for landing

bug-188856-20180822104552.patch (text/plain), 8.34 KB, created by John Wilander on 2018-08-22 10:45:53 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: John Wilander

Created: 2018-08-22 10:45:53 PDT

Size: 8.34 KB

patch

obsolete

>Subversion Revision: 235180
>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index 4fda7e94317d16ad2697559f6f8f1b5f41c37b00..c12d6d40b358c374389fd59844628a711dea3f8a 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,16 @@
>+2018-08-22  John Wilander  <wilander@apple.com>
>+
>+        Make ResourceLoadObserver::logWebSocketLoading() handle websockets in detached frames
>+        https://bugs.webkit.org/show_bug.cgi?id=188856
>+
>+        Unreviewed test gardening.
>+
>+        * http/tests/websocket/connection-refusal-in-frame-resource-load-statistics-expected.txt: Added.
>+        * http/tests/websocket/connection-refusal-in-frame-resource-load-statistics.html: Renamed from LayoutTests/http/tests/websocket/construct-in-detached-frame-resource-load-statistics.html.
>+        * http/tests/websocket/construct-in-detached-frame-resource-load-statistics-expected.txt: Removed.
>+        * http/tests/websocket/resources/construct-in-detached-frame-resource-load-statistics.html: Removed.
>+        * http/tests/websocket/resources/localhost-websocket-connect.html: Added.
>+
> 2018-08-22  Per Arne Vollan  <pvollan@apple.com>
> 
>         [Win] Some video tests under http/tests/security are crashing on EWS.
>diff --git a/LayoutTests/http/tests/websocket/connection-refusal-in-frame-resource-load-statistics-expected.txt b/LayoutTests/http/tests/websocket/connection-refusal-in-frame-resource-load-statistics-expected.txt
>new file mode 100644
>index 0000000000000000000000000000000000000000..4be267aaff85277313a54cbe7faf08ec688a86c5
>--- /dev/null
>+++ b/LayoutTests/http/tests/websocket/connection-refusal-in-frame-resource-load-statistics-expected.txt
>@@ -0,0 +1,12 @@
>+CONSOLE MESSAGE: WebSocket network error: The operation couldnât be completed. Connection refused
>+CONSOLE MESSAGE: WebSocket network error: The operation couldnât be completed. Connection refused
>+Construct a cross-site WebSocket in a frame with server-side refusal. The test passes if Resource Load Statistics logs it properly.
>+
>+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
>+
>+
>+PASS localhost registered as subresource under 127.0.0.1.
>+PASS successfullyParsed is true
>+
>+TEST COMPLETE
>+
>diff --git a/LayoutTests/http/tests/websocket/connection-refusal-in-frame-resource-load-statistics.html b/LayoutTests/http/tests/websocket/connection-refusal-in-frame-resource-load-statistics.html
>new file mode 100644
>index 0000000000000000000000000000000000000000..b3a8aa6e00bdd5c0ee223b42d473e7666c186300
>--- /dev/null
>+++ b/LayoutTests/http/tests/websocket/connection-refusal-in-frame-resource-load-statistics.html
>@@ -0,0 +1,36 @@
>+<!DOCTYPE html>
>+<head>
>+    <script src="../resources/js-test-pre.js"></script>
>+    <script src="/resourceLoadStatistics/resources/util.js"></script>
>+</head>
>+<body onload="runTest()">
>+<script>
>+    description('Construct a cross-site WebSocket in a frame with server-side refusal. The test passes if Resource Load Statistics logs it properly.');
>+
>+    window.jsTestIsAsync = true;
>+
>+    function completeTest() {
>+        if (testRunner.isStatisticsRegisteredAsSubresourceUnder("http://localhost", "http://127.0.0.1"))
>+            testPassed("localhost registered as subresource under 127.0.0.1.");
>+        else
>+            testFailed("localhost not registered as subresource under 127.0.0.1.");
>+        setEnableFeature(false, finishJSTest);
>+    }
>+
>+    function runTest() {
>+        setEnableFeature(true, function() {
>+            if (testRunner.isStatisticsRegisteredAsSubresourceUnder("http://localhost", "http://127.0.0.1"))
>+                testFailed("localhost already registered as subresource under 127.0.0.1.");
>+
>+            testRunner.setStatisticsNotifyPagesWhenDataRecordsWereScanned(true);
>+            testRunner.installStatisticsDidScanDataRecordsCallback(completeTest);
>+
>+            let iframeElement = document.createElement("iframe");
>+            iframeElement.src = "resources/localhost-websocket-connect.html";
>+            iframeElement.id = "testIframe";
>+            document.body.appendChild(iframeElement);
>+        });
>+    }
>+</script>
>+<script src="../resources/js-test-post.js"></script>
>+</body>
>diff --git a/LayoutTests/http/tests/websocket/construct-in-detached-frame-resource-load-statistics-expected.txt b/LayoutTests/http/tests/websocket/construct-in-detached-frame-resource-load-statistics-expected.txt
>deleted file mode 100644
>index 848cf2171ce96382ca68ba048043dba16644a09a..0000000000000000000000000000000000000000
>--- a/LayoutTests/http/tests/websocket/construct-in-detached-frame-resource-load-statistics-expected.txt
>+++ /dev/null
>@@ -1,5 +0,0 @@
>-PASS localhost registered as subresource under 127.0.0.1.
>-PASS successfullyParsed is true
>-
>-TEST COMPLETE
>-
>diff --git a/LayoutTests/http/tests/websocket/construct-in-detached-frame-resource-load-statistics.html b/LayoutTests/http/tests/websocket/construct-in-detached-frame-resource-load-statistics.html
>deleted file mode 100644
>index 1f094304e1a8f12399f80fc418f617b0b7677074..0000000000000000000000000000000000000000
>--- a/LayoutTests/http/tests/websocket/construct-in-detached-frame-resource-load-statistics.html
>+++ /dev/null
>@@ -1,55 +0,0 @@
>-<!DOCTYPE html>
>-<head>
>-    <script src="../resources/js-test-pre.js"></script>
>-    <script src="/resourceLoadStatistics/resources/util.js"></script>
>-</head>
>-<body onload="runTest()">
>-<script>
>-    description('Construct a WebSocket in a detached frame. The test passes if Resource Load Statistics logs it properly.');
>-
>-    window.jsTestIsAsync = true;
>-
>-    function detachIframe() {
>-        var testIframe = document.getElementById('testIframe');
>-        testIframe.parentNode.remove(testIframe);
>-    }
>-
>-    function completeTest() {
>-        if (testRunner.isStatisticsRegisteredAsSubresourceUnder("http://localhost", "http://127.0.0.1"))
>-            testPassed("localhost registered as subresource under 127.0.0.1.");
>-        else
>-            testFailed("localhost not registered as subresource under 127.0.0.1.");
>-        setEnableFeature(false, finishJSTest);
>-    }
>-
>-    let dataRecordsScanned = false;
>-    function didScanDataRecords() {
>-        dataRecordsScanned = true;
>-        if (createdWebSocket)
>-            completeTest();
>-    }
>-
>-    let createdWebSocket = false;
>-    function didCreateWebSocket() {
>-        createdWebSocket = true;
>-        if (dataRecordsScanned)
>-            completeTest();
>-    }
>-
>-    function runTest() {
>-        setEnableFeature(true, function() {
>-            if (testRunner.isStatisticsRegisteredAsSubresourceUnder("http://localhost", "http://127.0.0.1"))
>-                testFailed("localhost already registered as subresource under 127.0.0.1.");
>-
>-            testRunner.setStatisticsNotifyPagesWhenDataRecordsWereScanned(true);
>-            testRunner.installStatisticsDidScanDataRecordsCallback(didScanDataRecords);
>-
>-            let iframeElement = document.createElement("iframe");
>-            iframeElement.src = "resources/construct-in-detached-frame-resource-load-statistics.html";
>-            iframeElement.id = "testIframe";
>-            document.body.appendChild(iframeElement);
>-        });
>-    }
>-</script>
>-<script src="../resources/js-test-post.js"></script>
>-</body>
>diff --git a/LayoutTests/http/tests/websocket/resources/construct-in-detached-frame-resource-load-statistics.html b/LayoutTests/http/tests/websocket/resources/construct-in-detached-frame-resource-load-statistics.html
>deleted file mode 100644
>index ab7518b8447693d65441fb4dcb88559182a628c8..0000000000000000000000000000000000000000
>--- a/LayoutTests/http/tests/websocket/resources/construct-in-detached-frame-resource-load-statistics.html
>+++ /dev/null
>@@ -1,13 +0,0 @@
>-<!DOCTYPE html>
>-<script>
>-    var parentWindow = parent;
>-    var webSocketClass = WebSocket;
>-
>-    parentWindow.detachIframe();
>-    try {
>-        new webSocketClass('ws://localhost/');
>-    } catch (e) {
>-        parentWindow.console.log(e.message);
>-    }
>-    parentWindow.didCreateWebSocket();
>-</script>
>diff --git a/LayoutTests/http/tests/websocket/resources/localhost-websocket-connect.html b/LayoutTests/http/tests/websocket/resources/localhost-websocket-connect.html
>new file mode 100644
>index 0000000000000000000000000000000000000000..04170e467ed4268726bd941f789b0d2c8aabbc91
>--- /dev/null
>+++ b/LayoutTests/http/tests/websocket/resources/localhost-websocket-connect.html
>@@ -0,0 +1,4 @@
>+<!DOCTYPE html>
>+<script>
>+    new WebSocket('ws://localhost/');
>+</script>

Actions: View | Formatted Diff | Diff

Attachments on bug 188856: 347819 | 347822 | 347828 | 347854