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

content/browser/devtools/protocol/tracing_handler.cc

Index: content/browser/devtools/protocol/tracing_handler.cc
diff --git a/content/browser/devtools/protocol/tracing_handler.cc b/content/browser/devtools/protocol/tracing_handler.cc
index 6e9fa55aa48cce3a0f1cd6cb22c0a734675ee6cd..b86d07b772879915085fb62134e79e72935b7be5 100644
--- a/content/browser/devtools/protocol/tracing_handler.cc
+++ b/content/browser/devtools/protocol/tracing_handler.cc
@@ -142,8 +142,9 @@ void TracingHandler::OnRecordingEnabled(
   client_->SendStartResponse(command, StartResponse::Create());
 }
 
-void TracingHandler::OnBufferUsage(float usage) {
-  client_->BufferUsage(BufferUsageParams::Create()->set_value(usage));
+void TracingHandler::OnBufferUsage(float percent_full,
+                                   size_t approximate_event_count) {
+  client_->BufferUsage(BufferUsageParams::Create()->set_value(percent_full));

[nduca, 2014/11/14 18:11:16]

i think you're changing the protocol response to be a dict instead of an objct
here? or are you adding a field? if a field name changed, we need to update
tracing ui and telemetry, I think...

[yurys, 2014/11/17 09:10:01]

I'm not changing the event here. My plan was to add a new field for the event
count and rename value field to percentFull. This should be done in several
steps to keep exiting code working. caseq told me that Telemetry doesn't depend
on this event and I thought that tracing uses tracing_ui.cc as backend. I will
double check but it seems that Telemetry in fact doesn't depend on this event.
It is used by chromedriver though and I see about_tracing.js references it. Is
the later needed when running tracing via remote debugger?

 }
 
 void TracingHandler::OnCategoriesReceived(
@@ -189,13 +190,11 @@ void TracingHandler::SetupTimer(double usage_reporting_interval) {
   base::TimeDelta interval = base::TimeDelta::FromMilliseconds(
       std::ceil(usage_reporting_interval));
   buffer_usage_poll_timer_.reset(new base::Timer(
-      FROM_HERE,
-      interval,
-      base::Bind(
-          base::IgnoreResult(&TracingController::GetTraceBufferPercentFull),
-          base::Unretained(TracingController::GetInstance()),
-          base::Bind(&TracingHandler::OnBufferUsage,
-                     weak_factory_.GetWeakPtr())),
+      FROM_HERE, interval,
+      base::Bind(base::IgnoreResult(&TracingController::GetTraceBufferUsage),
+                 base::Unretained(TracingController::GetInstance()),
+                 base::Bind(&TracingHandler::OnBufferUsage,
+                            weak_factory_.GetWeakPtr())),
       true));
   buffer_usage_poll_timer_->Reset();
 }