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

Side by Side Diff: content/renderer/media/media_stream_video_renderer_sink.h

Issue 1849003002: Add video frame refresh to MediaStream and VideoCapture stacks. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed nick's PS3 comments (moving non-observer impl out of MSVideoSink interface). Created 4 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_RENDERER_MEDIA_RTC_VIDEO_RENDERER_H_ 5 #ifndef CONTENT_RENDERER_MEDIA_RTC_VIDEO_RENDERER_H_
6 #define CONTENT_RENDERER_MEDIA_RTC_VIDEO_RENDERER_H_ 6 #define CONTENT_RENDERER_MEDIA_RTC_VIDEO_RENDERER_H_
7 7
8 #include "base/callback.h" 8 #include "base/callback.h"
9 #include "base/macros.h" 9 #include "base/macros.h"
10 #include "base/memory/weak_ptr.h" 10 #include "base/memory/weak_ptr.h"
(...skipping 12 matching lines...) Expand all
23 23
24 namespace media { 24 namespace media {
25 class GpuVideoAcceleratorFactories; 25 class GpuVideoAcceleratorFactories;
26 } // namespace media 26 } // namespace media
27 27
28 namespace content { 28 namespace content {
29 29
30 // MediaStreamVideoRendererSink is a VideoFrameProvider designed for rendering 30 // MediaStreamVideoRendererSink is a VideoFrameProvider designed for rendering
31 // Video MediaStreamTracks [1], MediaStreamVideoRendererSink implements 31 // Video MediaStreamTracks [1], MediaStreamVideoRendererSink implements
32 // MediaStreamVideoSink in order to render video frames provided from a 32 // MediaStreamVideoSink in order to render video frames provided from a
33 // MediaStreamVideoTrack, to which it AddToVideoTrack()s itself when the 33 // MediaStreamVideoTrack, to which it connects itself when the
34 // VideoFrameProvider is Start()ed and RemoveFromVideoTrack()s itself when the 34 // VideoFrameProvider is Start()ed, and disconnects itself when the latter is
35 // latter is Stop()ed. 35 // Stop()ed.
36 // 36 //
37 // [1] http://dev.w3.org/2011/webrtc/editor/getusermedia.html#mediastreamtrack 37 // [1] http://dev.w3.org/2011/webrtc/editor/getusermedia.html#mediastreamtrack
38 // 38 //
39 // TODO(wuchengli): Add unit test. See the link below for reference. 39 // TODO(wuchengli): Add unit test. See the link below for reference.
40 // http://src.chromium.org/viewvc/chrome/trunk/src/content/renderer/media/rtc_vi 40 // http://src.chromium.org/viewvc/chrome/trunk/src/content/renderer/media/rtc_vi
41 // deo_decoder_unittest.cc?revision=180591&view=markup 41 // deo_decoder_unittest.cc?revision=180591&view=markup
42 class CONTENT_EXPORT MediaStreamVideoRendererSink 42 class CONTENT_EXPORT MediaStreamVideoRendererSink
43 : NON_EXPORTED_BASE(public VideoFrameProvider), 43 : NON_EXPORTED_BASE(public VideoFrameProvider),
44 NON_EXPORTED_BASE(public MediaStreamVideoSink) { 44 NON_EXPORTED_BASE(public MediaStreamVideoSink) {
45 public: 45 public:
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
93 const scoped_refptr<base::SingleThreadTaskRunner> media_task_runner_; 93 const scoped_refptr<base::SingleThreadTaskRunner> media_task_runner_;
94 94
95 base::WeakPtrFactory<MediaStreamVideoRendererSink> weak_factory_; 95 base::WeakPtrFactory<MediaStreamVideoRendererSink> weak_factory_;
96 96
97 DISALLOW_COPY_AND_ASSIGN(MediaStreamVideoRendererSink); 97 DISALLOW_COPY_AND_ASSIGN(MediaStreamVideoRendererSink);
98 }; 98 };
99 99
100 } // namespace content 100 } // namespace content
101 101
102 #endif // CONTENT_RENDERER_MEDIA_RTC_VIDEO_RENDERER_H_ 102 #endif // CONTENT_RENDERER_MEDIA_RTC_VIDEO_RENDERER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698