media/blink/video_frame_compositor.h - Issue 2159323002: Add tracing AutoOpenCloseEvent. - Code Review

Chromium Code Reviews

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

(429)

My Issues | Starred Open | Closed | All

Unified Diff: media/blink/video_frame_compositor.h

Issue 2159323002: Add tracing AutoOpenCloseEvent. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: fix formatting Created 4 years, 4 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

« no previous file with comments | « base/trace_event/video_playback_pae.cc ('k') | media/blink/video_frame_compositor.cc » ('j') | media/blink/video_frame_compositor.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: media/blink/video_frame_compositor.h

diff --git a/media/blink/video_frame_compositor.h b/media/blink/video_frame_compositor.h

index 9bb79fd056a9651221652435bc2ca29103973661..4baa92c23457ee0ba170b38933c84e3b47a7b6a4 100644

--- a/media/blink/video_frame_compositor.h

+++ b/media/blink/video_frame_compositor.h

@@ -14,6 +14,7 @@

#include "base/synchronization/lock.h"

#include "base/time/tick_clock.h"

#include "base/timer/timer.h"

+#include "base/trace_event/video_playback_pae.h"

#include "cc/layers/video_frame_provider.h"

#include "media/base/video_renderer_sink.h"

#include "media/blink/media_blink_export.h"

@@ -135,6 +136,10 @@ class MEDIA_BLINK_EXPORT VideoFrameCompositor

scoped_refptr<base::SingleThreadTaskRunner> compositor_task_runner_;

std::unique_ptr<base::TickClock> tick_clock_;

+ // Persistent async event

+ base::trace_event::VideoPlaybackPersistentAsyncEvent* persistent_async_ =

DaleCurtis 2016/08/29 19:08:48 No header initialization since the rest of these a

alexandermont 2016/08/29 21:38:20 Done

+ (nullptr);

+

// Allows tests to disable the background rendering task.

bool background_rendering_enabled_;

« no previous file with comments | « base/trace_event/video_playback_pae.cc ('k') | media/blink/video_frame_compositor.cc » ('j') | media/blink/video_frame_compositor.cc » ('J')

Powered by Google App Engine

This is Rietveld 408576698