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

Unified Diff: chromecast/media/base/media_resource_tracker.h

Issue 2348603002: [chromecast] Track MediaPipelineBackend used by CastRenderer. (Closed)
Patch Set: Created 4 years, 3 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: chromecast/media/base/media_resource_tracker.h
diff --git a/chromecast/media/base/media_resource_tracker.h b/chromecast/media/base/media_resource_tracker.h
index 52a3011fddd711ce67308fecb0dbc611a5ba554b..96b913e0815ea4c708711c6954511ad7cb726770 100644
--- a/chromecast/media/base/media_resource_tracker.h
+++ b/chromecast/media/base/media_resource_tracker.h
@@ -32,6 +32,20 @@ namespace media {
// thread.
class MediaResourceTracker {
public:
+ // Helper class to manage media resource usage count.
+ // Create an instance of this class when a media resource is created.
+ // Delete the instance *after* the media resource is deleted.
+ // This class is not thread-safe. It must be created and deleted on
+ // |MediaResourceTracker::media_task_runner_|.
+ class ScopedUsage {
+ public:
+ ScopedUsage(MediaResourceTracker* tracker);
+ ~ScopedUsage();
+
+ private:
+ MediaResourceTracker* tracker_;
halliwell 2016/09/15 20:54:07 nit: should DISALLOW_COPY_AND_ASSIGN
alokp 2016/09/15 21:01:48 Done.
+ };
+
MediaResourceTracker(
const scoped_refptr<base::SingleThreadTaskRunner>& ui_task_runner,
const scoped_refptr<base::SingleThreadTaskRunner>& media_task_runner);

Powered by Google App Engine
This is Rietveld 408576698