Attachment 369890 Details for Bug 197856 – Patch

[patch] Patch

bug-197856-20190514141014.patch (text/plain), 4.21 KB, created by Sihui Liu on 2019-05-14 14:10:15 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Sihui Liu

Created: 2019-05-14 14:10:15 PDT

Size: 4.21 KB

patch

obsolete

>Subversion Revision: 245299
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index 03ba86189e5f52eb44e9115fe21002ed6bba3903..72448d292008e03dbd4bc34de72a5c94cfe352ac 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,19 @@
>+2019-05-14  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):
>+        * UIProcess/WebProcessLifetimeObserver.h:
>+        (WebKit::WebProcessLifetimeObserver::hasProcess const):
>+
> 2019-05-14  Youenn Fablet  <youenn@apple.com>
> 
>         A service worker process should app nap when all its clients app nap
>diff --git a/Source/WebKit/UIProcess/WebProcessCache.cpp b/Source/WebKit/UIProcess/WebProcessCache.cpp
>index 797d7e5cca8a75261068ba6dd7271aebc5480d6b..359916f6edf4de2d37c80b0f916f656dad408a46 100644
>--- a/Source/WebKit/UIProcess/WebProcessCache.cpp
>+++ b/Source/WebKit/UIProcess/WebProcessCache.cpp
>@@ -246,6 +246,8 @@ WebProcessCache::CachedProcess::CachedProcess(Ref<WebProcessProxy>&& process)
>     : m_process(WTFMove(process))
>     , m_evictionTimer(RunLoop::main(), this, &CachedProcess::evictionTimerFired)
> {
>+    RELEASE_ASSERT(!m_process->pageCount());
>+    RELEASE_ASSERT(!m_process->websiteDataStore().hasProcess(process.ptr()));
>     m_process->setIsInProcessCache(true);
>     m_evictionTimer.startOneShot(cachedProcessLifetime);
> }
>diff --git a/Source/WebKit/UIProcess/WebProcessLifetimeObserver.cpp b/Source/WebKit/UIProcess/WebProcessLifetimeObserver.cpp
>index 3d75abdcb27032c0e1f71db2939badd52e334d5b..c8023de41616475010f428239a7bed07b17f85d7 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,13 @@ WebProcessLifetimeObserver::~WebProcessLifetimeObserver()
> void WebProcessLifetimeObserver::addWebPage(WebPageProxy& webPageProxy, WebProcessProxy& process)
> {
>     ASSERT(process.state() == WebProcessProxy::State::Running);
>+    RELEASE_ASSERT(!process.isInProcessCache());
>+    RELEASE_ASSERT(!process.isPrewarmed());
> 
>-    if (m_processes.add(&process).isNewEntry)
>+    if (m_processes.add(&process).isNewEntry) {
>+        RELEASE_LOG(Loading, "%p - WebProcessLifetimeObserver::addWebPage: webPID = %i, pageID = %" PRIu64, this, process.processIdentifier(), webPageProxy.pageID());
>         webProcessWillOpenConnection(process, *process.connection());
>+    }
> 
>     webPageWillOpenConnection(webPageProxy, *process.connection());
> }
>@@ -57,8 +62,10 @@ void WebProcessLifetimeObserver::removeWebPage(WebPageProxy& webPageProxy, WebPr
> 
>     webPageDidCloseConnection(webPageProxy, *process.connection());
> 
>-    if (m_processes.remove(&process))
>+    if (m_processes.remove(&process)) {
>+        RELEASE_LOG(Loading, "%p - WebProcessLifetimeObserver::removeWebPage: webPID = %i, pageID = %" PRIu64, this, process.processIdentifier(), webPageProxy.pageID());
>         webProcessDidCloseConnection(process, *process.connection());
>+    }
> }
> 
> WTF::IteratorRange<HashCountedSet<WebProcessProxy*>::const_iterator::Keys> WebProcessLifetimeObserver::processes() const
>diff --git a/Source/WebKit/UIProcess/WebProcessLifetimeObserver.h b/Source/WebKit/UIProcess/WebProcessLifetimeObserver.h
>index 61ce22b6f6608b64a28323ff146cf467d51bf218..66939734561e4201a20f42a366ddd02a44704e95 100644
>--- a/Source/WebKit/UIProcess/WebProcessLifetimeObserver.h
>+++ b/Source/WebKit/UIProcess/WebProcessLifetimeObserver.h
>@@ -48,6 +48,8 @@ public:
> 
>     WTF::IteratorRange<HashCountedSet<WebProcessProxy*>::const_iterator::Keys> processes() const;
> 
>+    bool hasProcess(WebProcessProxy* process) const { return m_processes.contains(process); }
>+
> private:
>     friend class WebProcessLifetimeTracker;
>

Actions: View | Formatted Diff | Diff

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