Make NetLog take in callbacks that return Values

net/base/capturing_net_log.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 "net/base/capturing_net_log.h"
 
 #include "base/logging.h"
+#include "base/values.h"
 
 namespace net {
 
-CapturingNetLog::Entry::Entry(EventType type,
-                              const base::TimeTicks& time,
-                              Source source,
-                              EventPhase phase,
-                              EventParameters* extra_parameters)
-    : type(type), time(time), source(source), phase(phase),
-      extra_parameters(extra_parameters) {
+CapturingNetLog::CapturedEntry::CapturedEntry(
+    EventType type,
+    const base::TimeTicks& time,
+    Source source,
+    EventPhase phase)
+    : type(type), time(time), source(source), phase(phase) {
 }
 
-CapturingNetLog::Entry::~Entry() {}
+CapturingNetLog::CapturedEntry::~CapturedEntry() {}
 
 CapturingNetLog::CapturingNetLog(size_t max_num_entries)
     : last_id_(0),
       max_num_entries_(max_num_entries),
       log_level_(LOG_ALL_BUT_BYTES) {
 }
 
 CapturingNetLog::~CapturingNetLog() {}
 
-void CapturingNetLog::GetEntries(EntryList* entry_list) const {
+void CapturingNetLog::GetEntries(CapturedEntryList* entry_list) const {
   base::AutoLock lock(lock_);
-  *entry_list = entries_;
+  *entry_list = captured_entries_;
 }
 
 void CapturingNetLog::Clear() {
   base::AutoLock lock(lock_);
-  entries_.clear();
+  captured_entries_.clear();
 }
 
 void CapturingNetLog::SetLogLevel(NetLog::LogLevel log_level) {
   base::AutoLock lock(lock_);
   log_level_ = log_level;
 }
 
-void CapturingNetLog::AddEntry(
-    EventType type,
-    const Source& source,
-    EventPhase phase,
-    const scoped_refptr<EventParameters>& extra_parameters) {
-  DCHECK(source.is_valid());
+void CapturingNetLog::OnAddEntry(const net::NetLog::Entry& entry) {
+  DCHECK(entry.source().is_valid());
   base::AutoLock lock(lock_);
-  Entry entry(type, base::TimeTicks::Now(), source, phase, extra_parameters);
-  if (entries_.size() + 1 < max_num_entries_)
-    entries_.push_back(entry);
+  if (captured_entries_.size() >= max_num_entries_)
+    return;
+  captured_entries_.push_back(
+      CapturedEntry(entry.type(),
+                    base::TimeTicks::Now(),
+                    entry.source(),
+                    entry.phase()));
 }
 
 uint32 CapturingNetLog::NextID() {
   return base::subtle::NoBarrier_AtomicIncrement(&last_id_, 1);
 }
 
 NetLog::LogLevel CapturingNetLog::GetLogLevel() const {
   base::AutoLock lock(lock_);
   return log_level_;
 }
@@ skipping 16 matching lines @@
 
 CapturingBoundNetLog::CapturingBoundNetLog(size_t max_num_entries)
     : capturing_net_log_(max_num_entries),
       net_log_(BoundNetLog::Make(&capturing_net_log_,
                                  net::NetLog::SOURCE_NONE)) {
 }
 
 CapturingBoundNetLog::~CapturingBoundNetLog() {}
 
 void CapturingBoundNetLog::GetEntries(
-    CapturingNetLog::EntryList* entry_list) const {
+    CapturingNetLog::CapturedEntryList* entry_list) const {
   capturing_net_log_.GetEntries(entry_list);
 }
 
 void CapturingBoundNetLog::Clear() {
   capturing_net_log_.Clear();
 }
 
 void CapturingBoundNetLog::SetLogLevel(NetLog::LogLevel log_level) {
   capturing_net_log_.SetLogLevel(log_level);
 }
 
 }  // namespace net