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

Unified Diff: remoting/host/fake_desktop_capturer.cc

Issue 455073004: Switch DesktopEnvironment to return a DesktopCapturer. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove ShapedDesktopCapturer::Create() Created 6 years, 4 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 | « remoting/host/fake_desktop_capturer.h ('k') | remoting/host/fake_desktop_environment.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/fake_desktop_capturer.cc
diff --git a/remoting/host/fake_screen_capturer.cc b/remoting/host/fake_desktop_capturer.cc
similarity index 80%
rename from remoting/host/fake_screen_capturer.cc
rename to remoting/host/fake_desktop_capturer.cc
index 355b7963570fcc0899fd43f010da2689025d62c8..e568b571c3b02c561d80f5eaa47b786aac2516aa 100644
--- a/remoting/host/fake_screen_capturer.cc
+++ b/remoting/host/fake_desktop_capturer.cc
@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "remoting/host/fake_screen_capturer.h"
+#include "remoting/host/fake_desktop_capturer.h"
#include "base/bind.h"
#include "base/logging.h"
@@ -12,11 +12,11 @@
namespace remoting {
-// FakeScreenCapturer generates a white picture of size kWidth x kHeight
+// FakeDesktopCapturer generates a white picture of size kWidth x kHeight
// with a rectangle of size kBoxWidth x kBoxHeight. The rectangle moves kSpeed
// pixels per frame along both axes, and bounces off the sides of the screen.
-static const int kWidth = FakeScreenCapturer::kWidth;
-static const int kHeight = FakeScreenCapturer::kHeight;
+static const int kWidth = FakeDesktopCapturer::kWidth;
+static const int kHeight = FakeDesktopCapturer::kHeight;
static const int kBoxWidth = 140;
static const int kBoxHeight = 140;
static const int kSpeed = 20;
@@ -40,7 +40,7 @@ class DefaultFrameGenerator
first_frame_(true) {}
scoped_ptr<webrtc::DesktopFrame> GenerateFrame(
- webrtc::ScreenCapturer::Callback* callback);
+ webrtc::DesktopCapturer::Callback* callback);
private:
friend class base::RefCountedThreadSafe<DefaultFrameGenerator>;
@@ -58,7 +58,7 @@ class DefaultFrameGenerator
};
scoped_ptr<webrtc::DesktopFrame> DefaultFrameGenerator::GenerateFrame(
- webrtc::ScreenCapturer::Callback* callback) {
+ webrtc::DesktopCapturer::Callback* callback) {
const int kBytesPerPixel = webrtc::DesktopFrame::kBytesPerPixel;
int buffer_size = kWidth * kHeight * kBytesPerPixel;
webrtc::SharedMemory* shared_memory =
@@ -120,28 +120,27 @@ scoped_ptr<webrtc::DesktopFrame> DefaultFrameGenerator::GenerateFrame(
} // namespace
-FakeScreenCapturer::FakeScreenCapturer()
- : callback_(NULL),
- mouse_shape_observer_(NULL) {
+FakeDesktopCapturer::FakeDesktopCapturer()
+ : callback_(NULL) {
frame_generator_ = base::Bind(&DefaultFrameGenerator::GenerateFrame,
new DefaultFrameGenerator());
}
-FakeScreenCapturer::~FakeScreenCapturer() {}
+FakeDesktopCapturer::~FakeDesktopCapturer() {}
-void FakeScreenCapturer::set_frame_generator(
+void FakeDesktopCapturer::set_frame_generator(
const FrameGenerator& frame_generator) {
DCHECK(!callback_);
frame_generator_ = frame_generator;
}
-void FakeScreenCapturer::Start(Callback* callback) {
+void FakeDesktopCapturer::Start(Callback* callback) {
DCHECK(!callback_);
DCHECK(callback);
callback_ = callback;
}
-void FakeScreenCapturer::Capture(const webrtc::DesktopRegion& region) {
+void FakeDesktopCapturer::Capture(const webrtc::DesktopRegion& region) {
base::Time capture_start_time = base::Time::Now();
scoped_ptr<webrtc::DesktopFrame> frame = frame_generator_.Run(callback_);
frame->set_capture_time_ms(
@@ -149,21 +148,4 @@ void FakeScreenCapturer::Capture(const webrtc::DesktopRegion& region) {
callback_->OnCaptureCompleted(frame.release());
}
-void FakeScreenCapturer::SetMouseShapeObserver(
- MouseShapeObserver* mouse_shape_observer) {
- DCHECK(!mouse_shape_observer_);
- DCHECK(mouse_shape_observer);
- mouse_shape_observer_ = mouse_shape_observer;
-}
-
-bool FakeScreenCapturer::GetScreenList(ScreenList* screens) {
- NOTIMPLEMENTED();
- return false;
-}
-
-bool FakeScreenCapturer::SelectScreen(webrtc::ScreenId id) {
- NOTIMPLEMENTED();
- return false;
-}
-
} // namespace remoting
« no previous file with comments | « remoting/host/fake_desktop_capturer.h ('k') | remoting/host/fake_desktop_environment.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698