Attachment 356700 Details for Bug 192290 – Patch for landing

[patch] Patch for landing

bug-192290-20181205193231.patch (text/plain), 1.45 KB, created by Ryosuke Niwa on 2018-12-05 19:32:32 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Ryosuke Niwa

Created: 2018-12-05 19:32:32 PST

Size: 1.45 KB

patch

obsolete

>Index: Source/WebKit/ChangeLog
>===================================================================
>--- Source/WebKit/ChangeLog	(revision 238917)
>+++ Source/WebKit/ChangeLog	(working copy)
>@@ -1,3 +1,18 @@
>+2018-12-05  Ryosuke Niwa  <rniwa@webkit.org>
>+
>+        REGRESSION(PSON): Process swapping code doesn't set DisplayID in WebContent process
>+        https://bugs.webkit.org/show_bug.cgi?id=192290
>+
>+        Reviewed by Simon Fraser.
>+
>+        Fixed the bug by updating the DisplayID upon attaching to a WebContent process
>+        after a process swap as well as after the previous WebContent process had crashed.
>+
>+        The behavior change is tested by existing tests by the newly added assertion in WebCore.
>+
>+        * UIProcess/Cocoa/WebViewImpl.mm:
>+        (WebKit::WebViewImpl::didRelaunchProcess):
>+
> 2018-12-05  Chris Dumez  <cdumez@apple.com>
> 
>         Simplify logic inside WebPageProxy::continueNavigationInNewProcess()
>Index: Source/WebKit/UIProcess/Cocoa/WebViewImpl.mm
>===================================================================
>--- Source/WebKit/UIProcess/Cocoa/WebViewImpl.mm	(revision 238917)
>+++ Source/WebKit/UIProcess/Cocoa/WebViewImpl.mm	(working copy)
>@@ -1498,6 +1498,7 @@ void WebViewImpl::didRelaunchProcess()
>         m_gestureController->connectToProcess();
> 
>     accessibilityRegisterUIProcessTokens();
>+    windowDidChangeScreen(); // Make sure DisplayID is set.
> }
> 
> void WebViewImpl::setDrawsBackground(bool drawsBackground)

Actions: View | Formatted Diff | Diff

Attachments on bug 192290: 356674 | 356681 | 356682 | 356700