Report trace buffer usage as number of events, not only percentage

content/browser/tracing/tracing_controller_impl.cc

Index: content/browser/tracing/tracing_controller_impl.cc
diff --git a/content/browser/tracing/tracing_controller_impl.cc b/content/browser/tracing/tracing_controller_impl.cc
index 3e8e1ac7cd1c0fc2d5aa5df1fbf84e88b5f626d9..44d1e86c8afdaf6632a99ba0b9819f88fe526c6b 100644
--- a/content/browser/tracing/tracing_controller_impl.cc
+++ b/content/browser/tracing/tracing_controller_impl.cc
@@ -162,18 +162,19 @@ TracingController* TracingController::GetInstance() {
   return TracingControllerImpl::GetInstance();
 }
 
-TracingControllerImpl::TracingControllerImpl() :
-    pending_disable_recording_ack_count_(0),
-    pending_capture_monitoring_snapshot_ack_count_(0),
-    pending_trace_buffer_percent_full_ack_count_(0),
-    maximum_trace_buffer_percent_full_(0),
+TracingControllerImpl::TracingControllerImpl()
+    : pending_disable_recording_ack_count_(0),
+      pending_capture_monitoring_snapshot_ack_count_(0),
+      pending_trace_buffer_percent_full_ack_count_(0),
+      maximum_trace_buffer_percent_full_(0),
+      total_trace_buffer_usage_(0),
     // Tracing may have been enabled by ContentMainRunner if kTraceStartup
     // is specified in command line.
 #if defined(OS_CHROMEOS) || defined(OS_WIN)
-    is_system_tracing_(false),
+      is_system_tracing_(false),
 #endif
-    is_recording_(TraceLog::GetInstance()->IsEnabled()),
-    is_monitoring_(false) {
+      is_recording_(TraceLog::GetInstance()->IsEnabled()),
+      is_monitoring_(false) {
 }
 
 TracingControllerImpl::~TracingControllerImpl() {
@@ -513,15 +514,17 @@ bool TracingControllerImpl::GetTraceBufferPercentFull(
       trace_message_filters_.size() + 1;
   pending_trace_buffer_percent_full_filters_ = trace_message_filters_;
   maximum_trace_buffer_percent_full_ = 0;
+  total_trace_buffer_usage_ = 0;
 
   // Call OnTraceBufferPercentFullReply unconditionally for the browser process.
   // This will result in immediate execution of the callback if there are no
   // child processes.
-  BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
+  BrowserThread::PostTask(
+      BrowserThread::UI, FROM_HERE,
       base::Bind(&TracingControllerImpl::OnTraceBufferPercentFullReply,
-                  base::Unretained(this),
-                  scoped_refptr<TraceMessageFilter>(),
-                  TraceLog::GetInstance()->GetBufferPercentFull()));
+                 base::Unretained(this), scoped_refptr<TraceMessageFilter>(),
+                 TraceLog::GetInstance()->GetBufferPercentFull(),
+                 TraceLog::GetInstance()->GetBufferUsage()));
 
   // Notify all child processes.
   for (TraceMessageFilterSet::iterator it = trace_message_filters_.begin();
@@ -636,11 +639,11 @@ void TracingControllerImpl::RemoveTraceMessageFilter(
     TraceMessageFilterSet::const_iterator it =
         pending_trace_buffer_percent_full_filters_.find(trace_message_filter);
     if (it != pending_trace_buffer_percent_full_filters_.end()) {
-      BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
+      BrowserThread::PostTask(
+          BrowserThread::UI, FROM_HERE,
           base::Bind(&TracingControllerImpl::OnTraceBufferPercentFullReply,
                      base::Unretained(this),
-                     make_scoped_refptr(trace_message_filter),
-                     0));
+                     make_scoped_refptr(trace_message_filter), 0, 0));
     }
   }
 
@@ -817,13 +820,15 @@ void TracingControllerImpl::OnLocalMonitoringTraceDataCollected(
 
 void TracingControllerImpl::OnTraceBufferPercentFullReply(
     TraceMessageFilter* trace_message_filter,
-    float percent_full) {
+    float percent_full,
+    size_t buffer_usage) {
   if (!BrowserThread::CurrentlyOn(BrowserThread::UI)) {
-    BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
+    BrowserThread::PostTask(
+        BrowserThread::UI, FROM_HERE,
         base::Bind(&TracingControllerImpl::OnTraceBufferPercentFullReply,
                    base::Unretained(this),
-                   make_scoped_refptr(trace_message_filter),
-                   percent_full));
+                   make_scoped_refptr(trace_message_filter), percent_full,
+                   buffer_usage));
     return;
   }
 
@@ -838,11 +843,12 @@ void TracingControllerImpl::OnTraceBufferPercentFullReply(
 
   maximum_trace_buffer_percent_full_ =
       std::max(maximum_trace_buffer_percent_full_, percent_full);
+  total_trace_buffer_usage_ += buffer_usage;
 
   if (--pending_trace_buffer_percent_full_ack_count_ == 0) {
     // Trigger callback if one is set.
     pending_trace_buffer_percent_full_callback_.Run(
-        maximum_trace_buffer_percent_full_);
+        maximum_trace_buffer_percent_full_, total_trace_buffer_usage_);
     pending_trace_buffer_percent_full_callback_.Reset();
   }
 }