Attachment 371489 Details for Bug 198608 – Patch

[patch] Patch

bug-198608-20190606143556.patch (text/plain), 1.66 KB, created by Antoine Quint on 2019-06-06 05:35:57 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Antoine Quint

Created: 2019-06-06 05:35:57 PDT

Size: 1.66 KB

patch

obsolete

>Subversion Revision: 246143
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index eaf2258fe9012431a1835d547eabc7f0831df861..e11339b2ac87b77a85fc35036f817839736e4348 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,15 @@
>+2019-06-06  Antoine Quint  <graouts@apple.com>
>+
>+        Remove duplicated websiteDataStoreParameters code from WebsitePoliciesData::applyToDocumentLoader()
>+        https://bugs.webkit.org/show_bug.cgi?id=198608
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Due to a rebasing error, r244218 re-introduced code that was removed in r244197. This patch removes this code for good.
>+
>+        * Shared/WebsitePoliciesData.cpp:
>+        (WebKit::WebsitePoliciesData::applyToDocumentLoader):
>+
> 2019-06-05  Said Abou-Hallawa  <sabouhallawa@apple.com>
> 
>         [Cocoa] REGRESSION(r244182): Inspector thinks CA commits can be nested
>diff --git a/Source/WebKit/Shared/WebsitePoliciesData.cpp b/Source/WebKit/Shared/WebsitePoliciesData.cpp
>index 464b8940875b3281232a21724f8f5ebc0ef67d44..3d81c877a5efb310cec59919601c3ca38876cbb7 100644
>--- a/Source/WebKit/Shared/WebsitePoliciesData.cpp
>+++ b/Source/WebKit/Shared/WebsitePoliciesData.cpp
>@@ -256,13 +256,6 @@ void WebsitePoliciesData::applyToDocumentLoader(WebsitePoliciesData&& websitePol
>         break;
>     }
> 
>-    if (websitePolicies.websiteDataStoreParameters) {
>-        if (auto* frame = documentLoader.frame()) {
>-            if (auto* page = frame->page())
>-                page->setSessionID(websitePolicies.websiteDataStoreParameters->networkSessionParameters.sessionID);
>-        }
>-    }
>-
>     auto* frame = documentLoader.frame();
>     if (!frame)
>         return;

Actions: View | Formatted Diff | Diff

Attachments on bug 198608: 371489