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

content/browser/tracing/tracing_ui.cc

Index: content/browser/tracing/tracing_ui.cc
diff --git a/content/browser/tracing/tracing_ui.cc b/content/browser/tracing/tracing_ui.cc
index 95e3ab92cdec6ce18a5c966dbbcfad86ce97e4e6..684f5663bdcaa465a78ee2c59b07d7e89853fc07 100644
--- a/content/browser/tracing/tracing_ui.cc
+++ b/content/browser/tracing/tracing_ui.cc
@@ -129,9 +129,10 @@ void OnRecordingEnabledAck(const WebUIDataSource::GotDataCallback& callback) {
   callback.Run(res);
 }
 
-void OnTraceBufferPercentFullResult(
-    const WebUIDataSource::GotDataCallback& callback, float result) {
-  std::string str = base::DoubleToString(result);
+void OnTraceBufferUsageResult(const WebUIDataSource::GotDataCallback& callback,
+                              float percent_full,
+                              size_t approximate_event_count) {
+  std::string str = base::DoubleToString(percent_full);

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

here i notice you're not updating the tracing ui. I think it'd be polite to
update the tracing ui as well.... or at least add a new xhr endpoint
json/get_buffer_status that returns an object and then filling a trace-viwer bug
to switch to that endpoint.

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

I'm not updating devtools handler in this patch either as it requires changing
the protocol and will be done in a few steps. 
Done. I've added json/get_buffer_status which can be used in trace-viewer. Will
file a bug and try to update it to use this new endpoint.

   callback.Run(base::RefCountedString::TakeString(&str));
 }
 
@@ -205,8 +206,8 @@ bool OnBeginJSONRequest(const std::string& path,
     return BeginRecording(data, callback);
   }
   if (path == "json/get_buffer_percent_full") {
-    return TracingController::GetInstance()->GetTraceBufferPercentFull(
-        base::Bind(OnTraceBufferPercentFullResult, callback));
+    return TracingController::GetInstance()->GetTraceBufferUsage(
+        base::Bind(OnTraceBufferUsageResult, callback));
   }
   if (path == "json/end_recording") {
     return TracingController::GetInstance()->DisableRecording(