Added net logging to BaseFile.

chrome/browser/net/passive_log_collector.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/net/passive_log_collector.h"
 
 #include <algorithm>
 
 #include "base/compiler_specific.h"
 #include "base/string_util.h"
@@ skipping 71 matching lines @@
   trackers_[net::NetLog::SOURCE_DNS_TRANSACTION] = &dns_transaction_tracker_;
   trackers_[net::NetLog::SOURCE_ASYNC_HOST_RESOLVER_REQUEST] =
       &async_host_resolver_request_tracker_;
   trackers_[net::NetLog::SOURCE_UDP_SOCKET] = &udp_socket_tracker_;
   trackers_[net::NetLog::SOURCE_CERT_VERIFIER_JOB] =
       &cert_verifier_job_tracker_;
   trackers_[net::NetLog::SOURCE_HTTP_PIPELINED_CONNECTION] =
       &http_pipelined_connection_tracker_;
   trackers_[net::NetLog::SOURCE_FILESTREAM] =
       &file_stream_tracker_;
+  trackers_[net::NetLog::SOURCE_DOWNLOAD] =
+      &download_tracker_;
   // Make sure our mapping is up-to-date.
   for (size_t i = 0; i < arraysize(trackers_); ++i)
     DCHECK(trackers_[i]) << "Unhandled SourceType: " << i;
 }
 
 PassiveLogCollector::~PassiveLogCollector() {
 }
 
 void PassiveLogCollector::OnAddEntry(
     net::NetLog::EventType type,
@@ skipping 739 matching lines @@
     SourceInfo* out_info) {
   AddEntryToSourceInfo(entry, out_info);
   if (entry.type == net::NetLog::TYPE_HTTP_PIPELINED_CONNECTION &&
       entry.phase == net::NetLog::PHASE_END) {
     return ACTION_MOVE_TO_GRAVEYARD;
   }
   return ACTION_NONE;
 }
 
 //----------------------------------------------------------------------------
+// DowloadTracker
+//----------------------------------------------------------------------------
+
+const size_t
+PassiveLogCollector::DowloadTracker::kMaxNumSources = 200;
+
+const size_t
+PassiveLogCollector::DowloadTracker::kMaxGraveyardSize = 50;

[mmenke1, 2012/02/03 18:32:32]

nit:  Think this might be able to all fit on a single line (Same goes for
FileStreamTracker, coem to tihnk of it)

[ahendrickson, 2012/02/04 05:27:14]

Done.

+
+PassiveLogCollector::
+    DowloadTracker::DowloadTracker()
+        : SourceTracker(kMaxNumSources, kMaxGraveyardSize, NULL) {
+}
+
+PassiveLogCollector::SourceTracker::Action
+PassiveLogCollector::DowloadTracker::DoAddEntry(
+    const ChromeNetLog::Entry& entry,
+    SourceInfo* out_info) {

[mmenke1, 2012/02/03 18:32:32]

Should add the following here to prevent spammy messages from sitting in the
PassiveLogCollector for ages:

if (entry.type == net::NetLog::TYPE_DOWNLOAD_FILE_WRITTEN)
  return ACTION_NONE;

[ahendrickson, 2012/02/04 05:27:14]

Done.

+  AddEntryToSourceInfo(entry, out_info);
+  if (entry.type == net::NetLog::TYPE_DOWNLOAD_FILE_OPENED &&
+      entry.phase == net::NetLog::PHASE_END) {
+    return ACTION_MOVE_TO_GRAVEYARD;
+  }
+  return ACTION_NONE;
+}
+
+//----------------------------------------------------------------------------
 // FileStreamTracker
 //----------------------------------------------------------------------------
 
 const size_t
 PassiveLogCollector::FileStreamTracker::kMaxNumSources = 100;
 
 const size_t
 PassiveLogCollector::FileStreamTracker::kMaxGraveyardSize = 25;
 
 PassiveLogCollector::
     FileStreamTracker::FileStreamTracker()
         : SourceTracker(kMaxNumSources, kMaxGraveyardSize, NULL) {
 }
 
 PassiveLogCollector::SourceTracker::Action
 PassiveLogCollector::FileStreamTracker::DoAddEntry(
     const ChromeNetLog::Entry& entry,
     SourceInfo* out_info) {
   AddEntryToSourceInfo(entry, out_info);
   if (entry.type == net::NetLog::TYPE_FILE_STREAM_ALIVE &&
       entry.phase == net::NetLog::PHASE_END) {
     return ACTION_MOVE_TO_GRAVEYARD;
   }
   return ACTION_NONE;
 }