Attachment 372848 Details for Bug 199195 – Patch

[patch] Patch

bug-199195-20190625112133.patch (text/plain), 1.99 KB, created by Per Arne Vollan on 2019-06-25 11:21:34 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Per Arne Vollan

Created: 2019-06-25 11:21:34 PDT

Size: 1.99 KB

patch

obsolete

>Index: Source/WebKit/ChangeLog
>===================================================================
>--- Source/WebKit/ChangeLog	(revision 246797)
>+++ Source/WebKit/ChangeLog	(working copy)
>@@ -1,3 +1,18 @@
>+2019-06-25  Per Arne Vollan  <pvollan@apple.com>
>+
>+        Perform less work when a pre-warmed WebProcess is suspended or resumed.
>+        https://bugs.webkit.org/show_bug.cgi?id=199195
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Return early from WebProcess::actualPrepareToSuspend and WebProcess::processDidResume
>+        if this is a pre-warmed process. This is a confirmed improvement in page load time.
>+
>+        * WebProcess/WebProcess.cpp:
>+        (WebKit::WebProcess::actualPrepareToSuspend):
>+        (WebKit::WebProcess::cancelPrepareToSuspend):
>+        (WebKit::WebProcess::processDidResume):
>+
> 2019-06-25  Youenn Fablet  <youenn@apple.com>
> 
>         Close sockets with too high file descriptor
>Index: Source/WebKit/WebProcess/WebProcess.cpp
>===================================================================
>--- Source/WebKit/WebProcess/WebProcess.cpp	(revision 246794)
>+++ Source/WebKit/WebProcess/WebProcess.cpp	(working copy)
>@@ -1471,6 +1471,9 @@ void WebProcess::actualPrepareToSuspend(
>     SetForScope<bool> suspensionScope(m_isSuspending, true);
>     m_processIsSuspended = true;
> 
>+    if (m_processType == ProcessType::PrewarmedWebContent)
>+        return;
>+
> #if ENABLE(VIDEO)
>     suspendAllMediaBuffering();
>     if (auto* platformMediaSessionManager = PlatformMediaSessionManager::sharedManagerIfExists())
>@@ -1527,6 +1530,8 @@ void WebProcess::cancelPrepareToSuspend(
> 
>     m_processIsSuspended = false;
> 
>+    RELEASE_ASSERT(m_processType != ProcessType::PrewarmedWebContent);
>+
>     unfreezeAllLayerTrees();
> 
> #if PLATFORM(IOS_FAMILY)
>@@ -1603,6 +1608,9 @@ void WebProcess::processDidResume()
> 
>     m_processIsSuspended = false;
> 
>+    if (m_processType == ProcessType::PrewarmedWebContent)
>+        return;
>+
>     cancelMarkAllLayersVolatile();
>     unfreezeAllLayerTrees();
>

Actions: View | Formatted Diff | Diff

Attachments on bug 199195: 372848 | 372853 | 373132 | 373134 | 373186 | 373234