Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(449)

Unified Diff: content/browser/tracing/trace_message_filter.cc

Issue 717083003: Report trace buffer usage as number of events, not only percentage (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/tracing/trace_message_filter.cc
diff --git a/content/browser/tracing/trace_message_filter.cc b/content/browser/tracing/trace_message_filter.cc
index e74ca96caa65dc75aeaa79375748f320749e7046..895b3aab5e18d733fc2089e2116d82867cb71710 100644
--- a/content/browser/tracing/trace_message_filter.cc
+++ b/content/browser/tracing/trace_message_filter.cc
@@ -28,7 +28,7 @@ void TraceMessageFilter::OnChannelClosing() {
OnCaptureMonitoringSnapshotAcked();
if (is_awaiting_buffer_percent_full_ack_)
- OnTraceBufferPercentFullReply(0.0f);
+ OnTraceBufferPercentFullReply(0.0f, 0);
TracingControllerImpl::GetInstance()->RemoveTraceMessageFilter(this);
}
@@ -157,11 +157,12 @@ void TraceMessageFilter::OnWatchEventMatched() {
TracingControllerImpl::GetInstance()->OnWatchEventMatched();
}
-void TraceMessageFilter::OnTraceBufferPercentFullReply(float percent_full) {
+void TraceMessageFilter::OnTraceBufferPercentFullReply(float percent_full,
+ size_t buffer_usage) {
if (is_awaiting_buffer_percent_full_ack_) {
is_awaiting_buffer_percent_full_ack_ = false;
TracingControllerImpl::GetInstance()->OnTraceBufferPercentFullReply(
- this, percent_full);
+ this, percent_full, buffer_usage);
} else {
NOTREACHED();
}

Powered by Google App Engine
This is Rietveld 408576698