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

Unified Diff: media/video/capture/screen/screen_capture_frame_queue.cc

Issue 13983010: Use webrtc::DesktopCapturer for screen capturer implementation. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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 side-by-side diff with in-line comments
Download patch
Index: media/video/capture/screen/screen_capture_frame_queue.cc
diff --git a/media/video/capture/screen/screen_capture_frame_queue.cc b/media/video/capture/screen/screen_capture_frame_queue.cc
index 610a0628cfb753bf618d1779d003609c1ca2a79c..5402acf584031cbcf4dc8aab9b70eee54394adfa 100644
--- a/media/video/capture/screen/screen_capture_frame_queue.cc
+++ b/media/video/capture/screen/screen_capture_frame_queue.cc
@@ -7,33 +7,33 @@
#include <algorithm>
#include "base/basictypes.h"
-#include "media/video/capture/screen/screen_capture_frame.h"
+#include "base/logging.h"
+#include "base/threading/non_thread_safe.h"
+#include "media/video/capture/screen/shared_desktop_frame.h"
+#include "third_party/webrtc/modules/desktop_capture/desktop_frame.h"
namespace media {
-ScreenCaptureFrameQueue::ScreenCaptureFrameQueue()
- : current_(0),
- previous_(NULL) {
- SetAllFramesNeedUpdate();
-}
+ScreenCaptureFrameQueue::ScreenCaptureFrameQueue() : current_(0) {}
-ScreenCaptureFrameQueue::~ScreenCaptureFrameQueue() {
-}
+ScreenCaptureFrameQueue::~ScreenCaptureFrameQueue() {}
-void ScreenCaptureFrameQueue::DoneWithCurrentFrame() {
- previous_ = current_frame();
+void ScreenCaptureFrameQueue::MoveToNextFrame() {
current_ = (current_ + 1) % kQueueLength;
+
+ // Verify that the frame is not shared, i.e. that consumer has released it
+ // before attempting to capture again.
+ DCHECK(!frames_[current_] || !frames_[current_]->IsShared());
}
void ScreenCaptureFrameQueue::ReplaceCurrentFrame(
- scoped_ptr<ScreenCaptureFrame> frame) {
- frames_[current_] = frame.Pass();
- needs_update_[current_] = false;
+ scoped_ptr<webrtc::DesktopFrame> frame) {
+ frames_[current_].reset(SharedDesktopFrame::Wrap(frame.release()));
}
-void ScreenCaptureFrameQueue::SetAllFramesNeedUpdate() {
- std::fill(needs_update_, needs_update_ + arraysize(needs_update_), true);
- previous_ = NULL;
+void ScreenCaptureFrameQueue::Reset() {
+ for (int i = 0; i < kQueueLength; ++i)
+ frames_[i].reset();
}
} // namespace media
« no previous file with comments | « media/video/capture/screen/screen_capture_frame_queue.h ('k') | media/video/capture/screen/screen_capturer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698