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

Side by Side Diff: content/renderer/media/webrtc/webrtc_video_capturer_adapter.h

Issue 282523003: Deliver video frames on libjingle worker thread to WebRtcVideoCapturerAdapter. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed the last round. Created 6 years, 7 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 | Annotate | Revision Log
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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_WEBRTC_WEBRTC_VIDEO_CAPTURER_ADAPTER_H_ 5 #ifndef CONTENT_RENDERER_MEDIA_WEBRTC_WEBRTC_VIDEO_CAPTURER_ADAPTER_H_
6 #define CONTENT_RENDERER_MEDIA_WEBRTC_WEBRTC_VIDEO_CAPTURER_ADAPTER_H_ 6 #define CONTENT_RENDERER_MEDIA_WEBRTC_WEBRTC_VIDEO_CAPTURER_ADAPTER_H_
7 7
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
11 #include "base/message_loop/message_loop_proxy.h"
12 #include "base/threading/thread_checker.h"
11 #include "content/common/content_export.h" 13 #include "content/common/content_export.h"
12 #include "media/base/video_frame.h" 14 #include "media/base/video_frame.h"
13 #include "media/video/capture/video_capture_types.h" 15 #include "media/video/capture/video_capture_types.h"
14 #include "third_party/libjingle/source/talk/media/base/videocapturer.h" 16 #include "third_party/libjingle/source/talk/media/base/videocapturer.h"
15 17
16 namespace content { 18 namespace content {
17 19
18 // WebRtcVideoCapturerAdapter implements a simple cricket::VideoCapturer that is 20 // WebRtcVideoCapturerAdapter implements a simple cricket::VideoCapturer that is
19 // used for VideoCapturing in libJingle and especially in PeerConnections. 21 // used for VideoCapturing in libJingle and especially in PeerConnections.
20 // The class is created and destroyed on the main render thread. 22 // The class is created and destroyed on the main render thread.
21 // PeerConnection access cricket::VideoCapturer from a libJingle worker thread. 23 // PeerConnection access cricket::VideoCapturer from a libJingle worker thread.
24 // An instance of WebRtcVideoCapturerAdapter is owned by an instance of
25 // webrtc::VideoSourceInterface in libJingle. The implementation of
26 // webrtc::VideoSourceInterface guarantees that this object is not deleted
27 // while it is still used in libJingle.
22 class CONTENT_EXPORT WebRtcVideoCapturerAdapter 28 class CONTENT_EXPORT WebRtcVideoCapturerAdapter
23 : NON_EXPORTED_BASE(public cricket::VideoCapturer) { 29 : NON_EXPORTED_BASE(public cricket::VideoCapturer) {
24 public: 30 public:
25 explicit WebRtcVideoCapturerAdapter(bool is_screencast); 31 explicit WebRtcVideoCapturerAdapter(bool is_screencast);
26 virtual ~WebRtcVideoCapturerAdapter(); 32 virtual ~WebRtcVideoCapturerAdapter();
27 33
34 // OnFrameCaptured delivers video frames to libjingle. It must be called on
35 // libjingles worker thread.
28 // This method is virtual for testing purposes. 36 // This method is virtual for testing purposes.
29 virtual void OnFrameCaptured(const scoped_refptr<media::VideoFrame>& frame); 37 virtual void OnFrameCaptured(const scoped_refptr<media::VideoFrame>& frame);
30 38
31 private: 39 private:
32 // cricket::VideoCapturer implementation. 40 // cricket::VideoCapturer implementation.
33 // These methods are accessed from a libJingle worker thread. 41 // These methods are accessed from a libJingle worker thread.
34 virtual cricket::CaptureState Start( 42 virtual cricket::CaptureState Start(
35 const cricket::VideoFormat& capture_format) OVERRIDE; 43 const cricket::VideoFormat& capture_format) OVERRIDE;
36 virtual void Stop() OVERRIDE; 44 virtual void Stop() OVERRIDE;
37 virtual bool IsRunning() OVERRIDE; 45 virtual bool IsRunning() OVERRIDE;
38 virtual bool GetPreferredFourccs(std::vector<uint32>* fourccs) OVERRIDE; 46 virtual bool GetPreferredFourccs(std::vector<uint32>* fourccs) OVERRIDE;
39 virtual bool GetBestCaptureFormat(const cricket::VideoFormat& desired, 47 virtual bool GetBestCaptureFormat(const cricket::VideoFormat& desired,
40 cricket::VideoFormat* best_format) OVERRIDE; 48 cricket::VideoFormat* best_format) OVERRIDE;
41 virtual bool IsScreencast() const OVERRIDE; 49 virtual bool IsScreencast() const OVERRIDE;
42 50
43 void UpdateI420Buffer(const scoped_refptr<media::VideoFrame>& src); 51 void UpdateI420Buffer(const scoped_refptr<media::VideoFrame>& src);
44 52
45 private: 53 // |thread_checker_| is bound to the libjingle worker thread.
54 base::ThreadChecker thread_checker_;
55
46 const bool is_screencast_; 56 const bool is_screencast_;
47 bool running_; 57 bool running_;
48 base::TimeDelta first_frame_timestamp_; 58 base::TimeDelta first_frame_timestamp_;
49 // |buffer_| used if cropping is needed. It is created only if needed and 59 // |buffer_| used if cropping is needed. It is created only if needed and
50 // owned by WebRtcVideoCapturerAdapter. If its created, it exists until 60 // owned by WebRtcVideoCapturerAdapter. If its created, it exists until
51 // WebRtcVideoCapturerAdapter is destroyed. 61 // WebRtcVideoCapturerAdapter is destroyed.
52 uint8* buffer_; 62 uint8* buffer_;
53 size_t buffer_size_; 63 size_t buffer_size_;
54 scoped_ptr<cricket::CapturedFrame> captured_frame_; 64 scoped_ptr<cricket::CapturedFrame> captured_frame_;
55 65
56 DISALLOW_COPY_AND_ASSIGN(WebRtcVideoCapturerAdapter); 66 DISALLOW_COPY_AND_ASSIGN(WebRtcVideoCapturerAdapter);
57 }; 67 };
58 68
59 } // namespace content 69 } // namespace content
60 70
61 #endif // CONTENT_RENDERER_MEDIA_WEBRTC_WEBRTC_VIDEO_CAPTURER_ADAPTER_H_ 71 #endif // CONTENT_RENDERER_MEDIA_WEBRTC_WEBRTC_VIDEO_CAPTURER_ADAPTER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698