Skip to content
Permalink
Browse files
[WK2] Add some logging related to NetworkProcess suspension
https://bugs.webkit.org/show_bug.cgi?id=159178

Reviewed by Andreas Kling.

Add some logging related to NetworkProcess suspension to help debug
suspension issues.

* NetworkProcess/NetworkProcess.cpp:
(WebKit::NetworkProcess::prepareToSuspend):
(WebKit::NetworkProcess::cancelPrepareToSuspend):
(WebKit::NetworkProcess::processDidResume):


Canonical link: https://commits.webkit.org/177323@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@202576 268f45cc-cd09-0410-ab3c-d52691b4dbfc
  • Loading branch information
cdumez committed Jun 28, 2016
1 parent 9b06c67 commit a33b2abcba271296cbc67cb86cea0fcc1ff15d13
Showing with 20 additions and 0 deletions.
  1. +15 −0 Source/WebKit2/ChangeLog
  2. +5 −0 Source/WebKit2/NetworkProcess/NetworkProcess.cpp
@@ -1,3 +1,18 @@
2016-06-28 Chris Dumez <cdumez@apple.com>

[WK2] Add some logging related to NetworkProcess suspension
https://bugs.webkit.org/show_bug.cgi?id=159178

Reviewed by Andreas Kling.

Add some logging related to NetworkProcess suspension to help debug
suspension issues.

* NetworkProcess/NetworkProcess.cpp:
(WebKit::NetworkProcess::prepareToSuspend):
(WebKit::NetworkProcess::cancelPrepareToSuspend):
(WebKit::NetworkProcess::processDidResume):

2016-06-28 Chris Dumez <cdumez@apple.com>

Add some more UIProcess-side logging related to process assertions
@@ -595,7 +595,10 @@ void NetworkProcess::processWillSuspendImminently(bool& handled)

void NetworkProcess::prepareToSuspend()
{
LOG_ALWAYS(true, "%p - NetworkProcess::prepareToSuspend()", this);
lowMemoryHandler(Critical::Yes);

LOG_ALWAYS(true, "%p - NetworkProcess::prepareToSuspend() Sending ProcessReadyToSuspend IPC message", this);
parentProcessConnection()->send(Messages::NetworkProcessProxy::ProcessReadyToSuspend(), 0);
}

@@ -605,10 +608,12 @@ void NetworkProcess::cancelPrepareToSuspend()
// we do not because prepareToSuspend() already replied with a NetworkProcessProxy::ProcessReadyToSuspend
// message. And NetworkProcessProxy expects to receive either a NetworkProcessProxy::ProcessReadyToSuspend-
// or NetworkProcessProxy::DidCancelProcessSuspension- message, but not both.
LOG_ALWAYS(true, "%p - NetworkProcess::cancelPrepareToSuspend()", this);
}

void NetworkProcess::processDidResume()
{
LOG_ALWAYS(true, "%p - NetworkProcess::processDidResume()", this);
}

void NetworkProcess::prefetchDNS(const String& hostname)

0 comments on commit a33b2ab

Please sign in to comment.