Attachment 361993 Details for Bug 194640 – Patch

[patch] Patch

bug-194640-20190213213458.patch (text/plain), 1.80 KB, created by Jiewen Tan on 2019-02-13 21:34:59 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Jiewen Tan

Created: 2019-02-13 21:34:59 PST

Size: 1.80 KB

patch

obsolete

>Subversion Revision: 240992
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index 27536c72d5a786909818bffb0ddc711abcb8d5f4..756263ae67c2687c2403dc05bf78e59373ac9d59 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,14 @@
>+2019-02-13  Jiewen Tan  <jiewen_tan@apple.com>
>+
>+        Handle _WKNavigationActionPolicyAllowWithoutTryingAppLink for LoadOptimizer
>+        https://bugs.webkit.org/show_bug.cgi?id=194640
>+        <rdar://problem/48064140>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * UIProcess/Cocoa/NavigationState.mm:
>+        (WebKit::NavigationState::NavigationClient::decidePolicyForNavigationAction):
>+
> 2019-02-05  Brady Eidson  <beidson@apple.com>
> 
>         Add a new DownloadMap type that manages taking an assertion automatically.
>diff --git a/Source/WebKit/UIProcess/Cocoa/NavigationState.mm b/Source/WebKit/UIProcess/Cocoa/NavigationState.mm
>index b050b6d07efc08c0b20ca9de135ca9aa3410db44..44032e8f65fa81af7235ba0808c7192be2779b63 100644
>--- a/Source/WebKit/UIProcess/Cocoa/NavigationState.mm
>+++ b/Source/WebKit/UIProcess/Cocoa/NavigationState.mm
>@@ -589,7 +589,14 @@ void NavigationState::NavigationClient::decidePolicyForNavigationAction(WebPageP
>             localListener->download();
>             break;
>         case _WKNavigationActionPolicyAllowWithoutTryingAppLink:
>-            localListener->use(apiWebsitePolicies.get());
>+            tryOptimizingLoad(navigationAction->request(), webPageProxy, [localListener = WTFMove(localListener), websitePolicies = WTFMove(apiWebsitePolicies)] (bool optimizedLoad) {
>+                if (optimizedLoad) {
>+                    localListener->ignore();
>+                    return;
>+                }
>+
>+                localListener->use(websitePolicies.get());
>+            });
>             break;
>         }
>     };

Actions: View | Formatted Diff | Diff

Attachments on bug 194640: 361993