Attachment 360505 Details for Bug 193977 – Patch

[patch] Patch

bug-193977-20190129150941.patch (text/plain), 2.93 KB, created by Chris Dumez on 2019-01-29 15:09:45 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Chris Dumez

Created: 2019-01-29 15:09:45 PST

Size: 2.93 KB

patch

obsolete

>Subversion Revision: 240676
>diff --git a/Tools/ChangeLog b/Tools/ChangeLog
>index 84bed67074218e52d8b734a744f3ff453f748f6e..6b8529061c11e4c1d2e5bfe2ef78088a0e06778a 100644
>--- a/Tools/ChangeLog
>+++ b/Tools/ChangeLog
>@@ -1,3 +1,17 @@
>+2019-01-29  Chris Dumez  <cdumez@apple.com>
>+
>+        REGRESSION: ProcessSwap.DoSameSiteNavigationAfterCrossSiteProvisionalLoadStarted Failing on iOS
>+        https://bugs.webkit.org/show_bug.cgi?id=193977
>+        <rdar://problem/47643716>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Update API test to address flakiness on iOS. Instead of doing the last navigation after receiving the
>+        didStartProvisionalLoad, do we right after we answer the navigation policy request. Flakiness was due
>+        to the previous provisional committing *before* we make the last load. 
>+
>+        * TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:
>+
> 2019-01-29  Chris Dumez  <cdumez@apple.com>
> 
>         Regression (r240046): [PSON] Spurious changes to [WKWebView url] and [WKWebView loading] after [WKWebView loadRequest]
>diff --git a/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm b/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm
>index db7c056a14983bec16c1b8cef4a5deead478a960..4d8d682c2c9c5a5468ad73d55263e4a0d14cde44 100644
>--- a/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm
>+++ b/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm
>@@ -2448,6 +2448,8 @@ TEST(ProcessSwap, DoSameSiteNavigationAfterCrossSiteProvisionalLoadStarted)
>     auto delegate = adoptNS([[PSONNavigationDelegate alloc] init]);
>     [webView setNavigationDelegate:delegate.get()];
> 
>+    [webView configuration].preferences.safeBrowsingEnabled = NO;
>+
>     NSURLRequest *request = [NSURLRequest requestWithURL:[NSURL URLWithString:@"pson://www.webkit.org/main1.html"]];
>     [webView loadRequest:request];
> 
>@@ -2456,14 +2458,15 @@ TEST(ProcessSwap, DoSameSiteNavigationAfterCrossSiteProvisionalLoadStarted)
> 
>     auto webkitPID = [webView _webProcessIdentifier];
> 
>-    didStartProvisionalLoad = false;
>-    request = [NSURLRequest requestWithURL:[NSURL URLWithString:@"pson://www.apple.com/main.html"]];
>-    [webView loadRequest:request];
>+    delegate->decidePolicyForNavigationAction = ^(WKNavigationAction *, void (^decisionHandler)(WKNavigationActionPolicy)) {
>+        decisionHandler(WKNavigationActionPolicyAllow);
> 
>-    TestWebKitAPI::Util::run(&didStartProvisionalLoad);
>-    didStartProvisionalLoad = false;
>+        dispatch_async(dispatch_get_main_queue(), ^{
>+            [webView loadRequest:[NSURLRequest requestWithURL:[NSURL URLWithString:@"pson://www.webkit.org/main2.html"]]];
>+        });
>+    };
> 
>-    request = [NSURLRequest requestWithURL:[NSURL URLWithString:@"pson://www.webkit.org/main2.html"]];
>+    request = [NSURLRequest requestWithURL:[NSURL URLWithString:@"pson://www.apple.com/main.html"]];
>     [webView loadRequest:request];
> 
>     TestWebKitAPI::Util::run(&done);

Actions: View | Formatted Diff | Diff

Attachments on bug 193977: 360505 | 360516