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

Unified Diff: content/browser/media/capture/content_video_capture_device_core.cc

Issue 1031813002: Use proper tracing category in image/tab capture trace logs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Reuploaded patchset! Created 5 years, 9 months 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/media/capture/content_video_capture_device_core.cc
diff --git a/content/browser/media/capture/content_video_capture_device_core.cc b/content/browser/media/capture/content_video_capture_device_core.cc
index d730a9e7bb35e8894826c05a6bb52063f421f4c0..5030f6d633005666cc4961f9d52ee99a1d698b20 100644
--- a/content/browser/media/capture/content_video_capture_device_core.cc
+++ b/content/browser/media/capture/content_video_capture_device_core.cc
@@ -90,7 +90,7 @@ bool ThreadSafeCaptureOracle::ObserveEventAndDecideCapture(
// Consider the various reasons not to initiate a capture.
if (should_capture && !output_buffer.get()) {
- TRACE_EVENT_INSTANT1("mirroring",
+ TRACE_EVENT_INSTANT1("gpu.capture",
"PipelineLimited",
TRACE_EVENT_SCOPE_THREAD,
"trigger",
@@ -101,7 +101,7 @@ bool ThreadSafeCaptureOracle::ObserveEventAndDecideCapture(
// This is a normal and acceptable way to drop a frame. We've hit our
// capture rate limit: for example, the content is animating at 60fps but
// we're capturing at 30fps.
- TRACE_EVENT_INSTANT1("mirroring", "FpsRateLimited",
+ TRACE_EVENT_INSTANT1("gpu.capture", "FpsRateLimited",
TRACE_EVENT_SCOPE_THREAD,
"trigger", event_name);
}
@@ -109,13 +109,13 @@ bool ThreadSafeCaptureOracle::ObserveEventAndDecideCapture(
} else if (!should_capture && !output_buffer.get()) {
// We decided not to capture, but we wouldn't have been able to if we wanted
// to because no output buffer was available.
- TRACE_EVENT_INSTANT1("mirroring", "NearlyPipelineLimited",
+ TRACE_EVENT_INSTANT1("gpu.capture", "NearlyPipelineLimited",
TRACE_EVENT_SCOPE_THREAD,
"trigger", event_name);
return false;
}
int frame_number = oracle_.RecordCapture();
- TRACE_EVENT_ASYNC_BEGIN2("mirroring", "Capture", output_buffer.get(),
+ TRACE_EVENT_ASYNC_BEGIN2("gpu.capture", "Capture", output_buffer.get(),
"frame_number", frame_number,
"trigger", event_name);
// NATIVE_TEXTURE frames wrap a texture mailbox, which we don't have at the
@@ -189,7 +189,7 @@ void ThreadSafeCaptureOracle::DidCaptureFrame(
base::TimeTicks timestamp,
bool success) {
base::AutoLock guard(lock_);
- TRACE_EVENT_ASYNC_END2("mirroring", "Capture", buffer.get(),
+ TRACE_EVENT_ASYNC_END2("gpu.capture", "Capture", buffer.get(),
"success", success,
"timestamp", timestamp.ToInternalValue());

Powered by Google App Engine
This is Rietveld 408576698