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

Unified Diff: content/renderer/media/rtc_video_render.h

Issue 10918052: create a separate WebMediaPlayer for URL derived from media stream (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: fix win build Created 8 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: content/renderer/media/rtc_video_render.h
===================================================================
--- content/renderer/media/rtc_video_render.h (revision 0)
+++ content/renderer/media/rtc_video_render.h (revision 0)
@@ -0,0 +1,69 @@
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CONTENT_RENDERER_MEDIA_LOCAL_VIDEO_RENDER_H_
+#define CONTENT_RENDERER_MEDIA_LOCAL_VIDEO_RENDER_H_
+
+#include "base/callback.h"
+#include "content/common/content_export.h"
+#include "third_party/libjingle/source/talk/app/webrtc/mediastreaminterface.h"
+#include "ui/gfx/size.h"
+#include "webkit/media/video_frame_provider.h"
+
+namespace base {
+class MessageLoopProxy;
+}
+
+// RTCVideoRender is a webkit_media::VideoFrameProvider designed for rendering
+// Video MediaStreamTracks,
+// http://dev.w3.org/2011/webrtc/editor/getusermedia.html#mediastreamtrack
+// RTCVideoRender implements webrtc::VideoRendererInterface in order to render
+// video frames provided from a webrtc::VideoTrackInteface.
+// RTCVideoRender register itself to the Video Track when the
+// VideoFrameProvider is started and deregisters itself when it is stopped.
+// Calls to webrtc::VideoTrackInterface must occur in the main thread.
+class CONTENT_EXPORT RTCVideoRender
scherkus (not reviewing) 2012/09/07 11:44:03 Shouldn't this be RTCVideoRenderer (extra "er" on
wjia(left Chromium) 2012/09/13 01:22:07 Done.
+ : NON_EXPORTED_BASE(public webkit_media::VideoFrameProvider),
+ NON_EXPORTED_BASE(public webrtc::VideoRendererInterface) {
+ public:
+ RTCVideoRender(
+ webrtc::VideoTrackInterface* video_track,
+ const base::Closure& error_cb,
+ const webkit_media::RepaintCB& repaint_cb);
+
+ // webkit_media::VideoFrameProvider implementation. Called in the main thread.
scherkus (not reviewing) 2012/09/07 11:44:03 s/in/on/
wjia(left Chromium) 2012/09/13 01:22:07 Done.
+ virtual void Start() OVERRIDE;
+ virtual void Stop() OVERRIDE;
+ virtual void Play() OVERRIDE;
+ virtual void Pause() OVERRIDE;
+
+ // webrtc::VideoRendererInterface implementation. May be called in
scherkus (not reviewing) 2012/09/07 11:44:03 s/in/on/
wjia(left Chromium) 2012/09/13 01:22:07 Done.
+ // a different thread.
+ virtual void SetSize(int width, int height) OVERRIDE;
+ virtual void RenderFrame(const cricket::VideoFrame* frame) OVERRIDE;
+
+ protected:
+ virtual ~RTCVideoRender();
+
+ private:
+ enum ProviderState {
scherkus (not reviewing) 2012/09/07 11:44:03 s/ProviderState/State/?
wjia(left Chromium) 2012/09/13 01:22:07 Done.
+ kStarted,
+ kPaused,
+ kStopped,
+ };
+
+ void DoRenderFrameOnMainThread(scoped_refptr<media::VideoFrame> video_frame);
+
+ base::Closure error_cb_;
+ webkit_media::RepaintCB repaint_cb_;
+ scoped_refptr<base::MessageLoopProxy> message_loop_proxy_;
+ ProviderState state_;
+
+ // The video track the renderer is connected to.
+ scoped_refptr<webrtc::VideoTrackInterface> video_track_;
+
+ DISALLOW_COPY_AND_ASSIGN(RTCVideoRender);
+};
+
+#endif // CONTENT_RENDERER_MEDIA_LOCAL_VIDEO_RENDER_H_
Property changes on: content/renderer/media/rtc_video_render.h
___________________________________________________________________
Added: svn:eol-style
+ LF

Powered by Google App Engine
This is Rietveld 408576698