Update NetworkResourceLoader logging statement
authorkrollin@apple.com <krollin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 28 Mar 2017 00:16:09 +0000 (00:16 +0000)
committerkrollin@apple.com <krollin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 28 Mar 2017 00:16:09 +0000 (00:16 +0000)
https://bugs.webkit.org/show_bug.cgi?id=170126

Reviewed by Alex Christensen.

NetworkResourceLoader contains a logging statement that prints, in
part: "start: Retrieving resource from cache". This is misleading,
since it's not necessarily the case that the resource will be loaded
from the cache. What is actually happening at the point where that is
logged is that the cache is being checked for the resource. It still
remains to be seen if the resource will be found in it. To address
this, change the logging to say "start: Checking cache for resource".

* NetworkProcess/NetworkResourceLoader.cpp:
(WebKit::NetworkResourceLoader::start):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@214444 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit2/ChangeLog
Source/WebKit2/NetworkProcess/NetworkResourceLoader.cpp

index 218b208..b690439 100644 (file)
@@ -1,3 +1,21 @@
+2017-03-27  Keith Rollin  <krollin@apple.com>
+
+        Update NetworkResourceLoader logging statement
+        https://bugs.webkit.org/show_bug.cgi?id=170126
+
+        Reviewed by Alex Christensen.
+
+        NetworkResourceLoader contains a logging statement that prints, in
+        part: "start: Retrieving resource from cache". This is misleading,
+        since it's not necessarily the case that the resource will be loaded
+        from the cache. What is actually happening at the point where that is
+        logged is that the cache is being checked for the resource. It still
+        remains to be seen if the resource will be found in it. To address
+        this, change the logging to say "start: Checking cache for resource".
+
+        * NetworkProcess/NetworkResourceLoader.cpp:
+        (WebKit::NetworkResourceLoader::start):
+
 2017-03-27  Joseph Pecoraro  <pecoraro@apple.com>
 
         Web Inspector: Saving files should not suggest the top level directory
index 16e9fd0..f66849a 100644 (file)
@@ -163,7 +163,7 @@ void NetworkResourceLoader::start()
 
 #if ENABLE(NETWORK_CACHE)
     if (canUseCache(originalRequest())) {
-        RELEASE_LOG_IF_ALLOWED("start: Retrieving resource from cache (pageID = %" PRIu64 ", frameID = %" PRIu64 ", resourceID = %" PRIu64 ", isMainResource = %d, isSynchronous = %d)", m_parameters.webPageID, m_parameters.webFrameID, m_parameters.identifier, isMainResource(), isSynchronous());
+        RELEASE_LOG_IF_ALLOWED("start: Checking cache for resource (pageID = %" PRIu64 ", frameID = %" PRIu64 ", resourceID = %" PRIu64 ", isMainResource = %d, isSynchronous = %d)", m_parameters.webPageID, m_parameters.webFrameID, m_parameters.identifier, isMainResource(), isSynchronous());
         retrieveCacheEntry(originalRequest());
         return;
     }