Attachment 342304 Details for Bug 186438 – Patch

[patch] Patch

bug-186438-20180608124628.patch (text/plain), 3.10 KB, created by Chris Dumez on 2018-06-08 12:46:28 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Chris Dumez

Created: 2018-06-08 12:46:28 PDT

Size: 3.10 KB

patch

obsolete

>Subversion Revision: 232619
>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index b4d409afd9ba15e441b629ca07423cd74245262f..b99248c8932c452658db911db395059d6c679994 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,18 @@
>+2018-06-08  Chris Dumez  <cdumez@apple.com>
>+
>+        http/tests/cache/partitioned-cache.html fails with PSON enabled
>+        https://bugs.webkit.org/show_bug.cgi?id=186438
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        http/tests/cache/partitioned-cache.html was overriding a setting then navigating
>+        cross-origin, expecting the setting override to persist. However, with process
>+        swap on navigation enabled, the setting does not persist. To address the issue,
>+        we override the setting again in the document we navigate to.
>+
>+        * http/tests/cache/partitioned-cache-expected.txt:
>+        * http/tests/cache/resources/partitioned-cache-loader.html:
>+
> 2018-06-07  Mark Lam  <mark.lam@apple.com>
> 
>         Enhance run-jsc-stress-tests to allow a test to specify test specific options required for it to run.
>diff --git a/LayoutTests/http/tests/cache/partitioned-cache-expected.txt b/LayoutTests/http/tests/cache/partitioned-cache-expected.txt
>index 701137f1570c476307b386df6dd021220e5201e0..509c8dfd88a7c3ea5337f1c4f1fa399876e105b9 100644
>--- a/LayoutTests/http/tests/cache/partitioned-cache-expected.txt
>+++ b/LayoutTests/http/tests/cache/partitioned-cache-expected.txt
>@@ -1,8 +1,8 @@
>-CONSOLE MESSAGE: line 12: Document domain is now localhost
>-CONSOLE MESSAGE: line 13: Setting server-side persistent state to "localhost"
>-CONSOLE MESSAGE: line 22: Server state retrieved via a 1st party resource: "localhost"
>-CONSOLE MESSAGE: line 12: Document domain is now 127.0.0.1
>-CONSOLE MESSAGE: line 13: Setting server-side persistent state to "127.0.0.1"
>-CONSOLE MESSAGE: line 25: Server state retrieved via a 3rd party resource (possibly a cached result, but it shouldn't be): "127.0.0.1"
>+CONSOLE MESSAGE: line 15: Document domain is now localhost
>+CONSOLE MESSAGE: line 16: Setting server-side persistent state to "localhost"
>+CONSOLE MESSAGE: line 25: Server state retrieved via a 1st party resource: "localhost"
>+CONSOLE MESSAGE: line 15: Document domain is now 127.0.0.1
>+CONSOLE MESSAGE: line 16: Setting server-side persistent state to "127.0.0.1"
>+CONSOLE MESSAGE: line 28: Server state retrieved via a 3rd party resource (possibly a cached result, but it shouldn't be): "127.0.0.1"
> PASS response is "127.0.0.1"
> 
>diff --git a/LayoutTests/http/tests/cache/resources/partitioned-cache-loader.html b/LayoutTests/http/tests/cache/resources/partitioned-cache-loader.html
>index 6da5e41b144b8dc80940aff73316bf71e544b851..53bb3d9265430fc6a2f26d78646048da41abb37e 100644
>--- a/LayoutTests/http/tests/cache/resources/partitioned-cache-loader.html
>+++ b/LayoutTests/http/tests/cache/resources/partitioned-cache-loader.html
>@@ -2,6 +2,9 @@
> <head>
> <script src="../../resources/js-test-pre.js"></script>
> <script>
>+if (window.internals)
>+    internals.settings.setStorageBlockingPolicy('BlockThirdParty');
>+
> function setServerState(string)
> {
>     var xhr = new XMLHttpRequest;

Actions: View | Formatted Diff | Diff

Attachments on bug 186438: 342304