Changeset 206123 in webkit

Timestamp:

Sep 19, 2016 3:53:12 PM (8 years ago)

Author:

Keith Rollin

Message:

Reduce logging from WebResourceLoader::didReceiveData
​https://bugs.webkit.org/show_bug.cgi?id=162233

Reviewed by Antti Koivisto.

Only log once even if called multiple times for a single resource.

• WebProcess/Network/WebResourceLoader.cpp:

(WebKit::WebResourceLoader::didReceiveData):

• WebProcess/Network/WebResourceLoader.h:

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• WebProcess/Network/WebResourceLoader.cpp (modified) (1 diff)
• WebProcess/Network/WebResourceLoader.h (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2016-09-19  Keith Rollin  <krollin@apple.com>
+
+        Reduce logging from WebResourceLoader::didReceiveData
+        https://bugs.webkit.org/show_bug.cgi?id=162233
+
+        Reviewed by Antti Koivisto.
+
+        Only log once even if called multiple times for a single resource.
+
+        * WebProcess/Network/WebResourceLoader.cpp:
+        (WebKit::WebResourceLoader::didReceiveData):
+        * WebProcess/Network/WebResourceLoader.h:
+
 2016-09-19  Daniel Bates  <dabates@apple.com>
 

trunk/Source/WebKit2/WebProcess/Network/WebResourceLoader.cpp

@@ -134 +134 @@
 {
     LOG(Network, "(WebProcess) WebResourceLoader::didReceiveData of size %lu for '%s'", data.size(), m_coreLoader->url().string().latin1().data());
-    RELEASE_LOG_IF_ALLOWED("didReceiveData: (size = %lu)", data.size());
+
+    if (!m_hasReceivedData) {
+        RELEASE_LOG_IF_ALLOWED("didReceiveData: (size = %lu)", data.size());
+        m_hasReceivedData = true;
+    }
 
 #if USE(QUICK_LOOK)

trunk/Source/WebKit2/WebProcess/Network/WebResourceLoader.h

@@ -86 +86 @@
 
     RefPtr<WebCore::ResourceLoader> m_coreLoader;
+    bool m_hasReceivedData { false };
 };