Attachment 369795 Details for Bug 197856 – Patch

[patch] Patch

bug-197856-20190513154842.patch (text/plain), 3.42 KB, created by Sihui Liu on 2019-05-13 15:48:42 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Sihui Liu

Created: 2019-05-13 15:48:42 PDT

Size: 3.42 KB

patch

obsolete

>Subversion Revision: 245233
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index 244ae3d3f5f8d3b0641a8a3e0286a7752305a161..ae9a3214c868cf52b0cd3ac66b7bfcbea814a66e 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,17 @@
>+2019-05-13  Sihui Liu  <sihui_liu@apple.com>
>+
>+        Add assertions to help diagnose crash at WebProcessProxy::processPool()
>+        https://bugs.webkit.org/show_bug.cgi?id=197856
>+        <rdar://problem/49341366>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * UIProcess/WebProcessCache.cpp:
>+        (WebKit::WebProcessCache::CachedProcess::CachedProcess):
>+        * UIProcess/WebProcessLifetimeObserver.cpp:
>+        (WebKit::WebProcessLifetimeObserver::addWebPage):
>+        (WebKit::WebProcessLifetimeObserver::removeWebPage):
>+
> 2019-05-12  Takashi Komori  <Takashi.Komori@sony.com>
> 
>         [Curl] Suppress extra didReceiveAuthenticationChallenge call when accessing a server which checks basic auth.
>diff --git a/Source/WebKit/UIProcess/WebProcessCache.cpp b/Source/WebKit/UIProcess/WebProcessCache.cpp
>index 797d7e5cca8a75261068ba6dd7271aebc5480d6b..9ab557eede6900c5dfd31b3776e23de209b9698b 100644
>--- a/Source/WebKit/UIProcess/WebProcessCache.cpp
>+++ b/Source/WebKit/UIProcess/WebProcessCache.cpp
>@@ -246,6 +246,7 @@ WebProcessCache::CachedProcess::CachedProcess(Ref<WebProcessProxy>&& process)
>     : m_process(WTFMove(process))
>     , m_evictionTimer(RunLoop::main(), this, &CachedProcess::evictionTimerFired)
> {
>+    RELEASE_ASSERT(!m_process->pageCount());
>     m_process->setIsInProcessCache(true);
>     m_evictionTimer.startOneShot(cachedProcessLifetime);
> }
>diff --git a/Source/WebKit/UIProcess/WebProcessLifetimeObserver.cpp b/Source/WebKit/UIProcess/WebProcessLifetimeObserver.cpp
>index 3d75abdcb27032c0e1f71db2939badd52e334d5b..8719f2cb6e72abe09ccc5fa9ada27873bacfc4ea 100644
>--- a/Source/WebKit/UIProcess/WebProcessLifetimeObserver.cpp
>+++ b/Source/WebKit/UIProcess/WebProcessLifetimeObserver.cpp
>@@ -26,6 +26,7 @@
> #include "config.h"
> #include "WebProcessLifetimeObserver.h"
> 
>+#include "Logging.h"
> #include "WebPageProxy.h"
> #include "WebProcessProxy.h"
> 
>@@ -42,9 +43,12 @@ WebProcessLifetimeObserver::~WebProcessLifetimeObserver()
> void WebProcessLifetimeObserver::addWebPage(WebPageProxy& webPageProxy, WebProcessProxy& process)
> {
>     ASSERT(process.state() == WebProcessProxy::State::Running);
>+    RELEASE_ASSERT(!process.isInProcessCache() && !process.isPrewarmed());
> 
>-    if (m_processes.add(&process).isNewEntry)
>+    if (m_processes.add(&process).isNewEntry) {
>+        RELEASE_LOG(Loading, "WebProcessLifetimeObserver::addWebPage: webPID = %i, pageID = %" PRIu64, process.processIdentifier(), webPageProxy.pageID());
>         webProcessWillOpenConnection(process, *process.connection());
>+    }
> 
>     webPageWillOpenConnection(webPageProxy, *process.connection());
> }
>@@ -57,8 +61,10 @@ void WebProcessLifetimeObserver::removeWebPage(WebPageProxy& webPageProxy, WebPr
> 
>     webPageDidCloseConnection(webPageProxy, *process.connection());
> 
>-    if (m_processes.remove(&process))
>+    if (m_processes.remove(&process)) {
>+        RELEASE_LOG(Loading, "WebProcessLifetimeObserver::removeWebPage: webPID = %i, pageID = %" PRIu64, process.processIdentifier(), webPageProxy.pageID());
>         webProcessDidCloseConnection(process, *process.connection());
>+    }
> }
> 
> WTF::IteratorRange<HashCountedSet<WebProcessProxy*>::const_iterator::Keys> WebProcessLifetimeObserver::processes() const

Actions: View | Formatted Diff | Diff

Attachments on bug 197856: 369795 | 369890 | 369900 | 369980