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

Unified Diff: webrtc/api/remotevideocapturer.cc

Issue 1765423005: Change VideoRtpReceiver to create remote VideoTrack and VideoTrackSource. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Fixed up the comments. Created 4 years, 9 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
« no previous file with comments | « webrtc/api/remotevideocapturer.h ('k') | webrtc/api/remotevideocapturer_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/api/remotevideocapturer.cc
diff --git a/webrtc/api/remotevideocapturer.cc b/webrtc/api/remotevideocapturer.cc
index 8c1ffccb47c56184ac5f0246615e222ca9f70191..177ba5464114613534f0051068b72bb7bdb8e5bc 100644
--- a/webrtc/api/remotevideocapturer.cc
+++ b/webrtc/api/remotevideocapturer.cc
@@ -10,69 +10,4 @@
#include "webrtc/api/remotevideocapturer.h"
-#include "webrtc/base/logging.h"
-#include "webrtc/media/base/videoframe.h"
-
-namespace webrtc {
-
-RemoteVideoCapturer::RemoteVideoCapturer() {}
-
-RemoteVideoCapturer::~RemoteVideoCapturer() {}
-
-cricket::CaptureState RemoteVideoCapturer::Start(
- const cricket::VideoFormat& capture_format) {
- if (capture_state() == cricket::CS_RUNNING) {
- LOG(LS_WARNING)
- << "RemoteVideoCapturer::Start called when it's already started.";
- return capture_state();
- }
-
- LOG(LS_INFO) << "RemoteVideoCapturer::Start";
- SetCaptureFormat(&capture_format);
- return cricket::CS_RUNNING;
-}
-
-void RemoteVideoCapturer::Stop() {
- if (capture_state() == cricket::CS_STOPPED) {
- LOG(LS_WARNING)
- << "RemoteVideoCapturer::Stop called when it's already stopped.";
- return;
- }
-
- LOG(LS_INFO) << "RemoteVideoCapturer::Stop";
- SetCaptureFormat(NULL);
- SetCaptureState(cricket::CS_STOPPED);
-}
-
-bool RemoteVideoCapturer::IsRunning() {
- return capture_state() == cricket::CS_RUNNING;
-}
-
-bool RemoteVideoCapturer::GetPreferredFourccs(std::vector<uint32_t>* fourccs) {
- if (!fourccs)
- return false;
- fourccs->push_back(cricket::FOURCC_I420);
- return true;
-}
-
-bool RemoteVideoCapturer::GetBestCaptureFormat(
- const cricket::VideoFormat& desired, cricket::VideoFormat* best_format) {
- if (!best_format) {
- return false;
- }
-
- // RemoteVideoCapturer does not support capability enumeration.
- // Use the desired format as the best format.
- best_format->width = desired.width;
- best_format->height = desired.height;
- best_format->fourcc = cricket::FOURCC_I420;
- best_format->interval = desired.interval;
- return true;
-}
-
-bool RemoteVideoCapturer::IsScreencast() const {
- // TODO(ronghuawu): what about remote screencast stream.
- return false;
-}
-
-} // namespace webrtc
+// TODO(perkj): Remove this file once Chrome gyp file doesn't depend on it.
« no previous file with comments | « webrtc/api/remotevideocapturer.h ('k') | webrtc/api/remotevideocapturer_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698