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

Unified Diff: remoting/host/video_frame_capturer_unittest.cc

Issue 11470028: Move screen capturers to remoting/capturer. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years 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/video_frame_capturer_mac_unittest.cc ('k') | remoting/host/video_frame_capturer_win.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/video_frame_capturer_unittest.cc
diff --git a/remoting/host/video_frame_capturer_unittest.cc b/remoting/host/video_frame_capturer_unittest.cc
deleted file mode 100644
index bb69c6caa80e55c7de0701184266a63fedce2f7d..0000000000000000000000000000000000000000
--- a/remoting/host/video_frame_capturer_unittest.cc
+++ /dev/null
@@ -1,101 +0,0 @@
-// 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.
-
-#include "remoting/host/video_frame_capturer.h"
-
-#include "base/bind.h"
-#if defined(OS_MACOSX)
-#include "base/mac/mac_util.h"
-#endif // defined(OS_MACOSX)
-#include "remoting/base/capture_data.h"
-#include "remoting/base/shared_buffer_factory.h"
-#include "remoting/host/host_mock_objects.h"
-#include "remoting/protocol/protocol_mock_objects.h"
-#include "testing/gmock/include/gmock/gmock.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-using ::testing::_;
-using ::testing::AnyNumber;
-
-namespace remoting {
-
-class MockSharedBufferFactory : public SharedBufferFactory {
- public:
- MockSharedBufferFactory() {}
- virtual ~MockSharedBufferFactory() {}
-
- MOCK_METHOD1(CreateSharedBuffer, scoped_refptr<SharedBuffer>(uint32));
- MOCK_METHOD1(ReleaseSharedBuffer, void(scoped_refptr<SharedBuffer>));
-
- private:
- DISALLOW_COPY_AND_ASSIGN(MockSharedBufferFactory);
-};
-
-MATCHER(DirtyRegionIsNonEmptyRect, "") {
- const SkRegion& dirty_region = arg->dirty_region();
- const SkIRect& dirty_region_bounds = dirty_region.getBounds();
- if (dirty_region_bounds.isEmpty()) {
- return false;
- }
- return dirty_region == SkRegion(dirty_region_bounds);
-}
-
-class VideoFrameCapturerTest : public testing::Test {
- public:
- scoped_refptr<SharedBuffer> CreateSharedBuffer(uint32 size);
-
- protected:
- scoped_ptr<VideoFrameCapturer> capturer_;
- MockSharedBufferFactory shared_buffer_factory_;
- MockVideoFrameCapturerDelegate delegate_;
-};
-
-scoped_refptr<SharedBuffer> VideoFrameCapturerTest::CreateSharedBuffer(
- uint32 size) {
- return scoped_refptr<SharedBuffer>(new SharedBuffer(size));
-}
-
-TEST_F(VideoFrameCapturerTest, StartCapturer) {
- capturer_ = VideoFrameCapturer::Create();
- capturer_->Start(&delegate_);
- capturer_->Stop();
-}
-
-TEST_F(VideoFrameCapturerTest, Capture) {
- // Assume that Start() treats the screen as invalid initially.
- EXPECT_CALL(delegate_,
- OnCaptureCompleted(DirtyRegionIsNonEmptyRect()));
- EXPECT_CALL(delegate_, OnCursorShapeChangedPtr(_))
- .Times(AnyNumber());
-
- capturer_ = VideoFrameCapturer::Create();
- capturer_->Start(&delegate_);
- capturer_->CaptureFrame();
- capturer_->Stop();
-}
-
-#if defined(OS_WIN)
-
-TEST_F(VideoFrameCapturerTest, UseSharedBuffers) {
- EXPECT_CALL(delegate_,
- OnCaptureCompleted(DirtyRegionIsNonEmptyRect()));
- EXPECT_CALL(delegate_, OnCursorShapeChangedPtr(_))
- .Times(AnyNumber());
-
- EXPECT_CALL(shared_buffer_factory_, CreateSharedBuffer(_))
- .Times(1)
- .WillOnce(Invoke(this, &VideoFrameCapturerTest::CreateSharedBuffer));
- EXPECT_CALL(shared_buffer_factory_, ReleaseSharedBuffer(_))
- .Times(1);
-
- capturer_ = VideoFrameCapturer::CreateWithFactory(&shared_buffer_factory_);
- capturer_->Start(&delegate_);
- capturer_->CaptureFrame();
- capturer_->Stop();
- capturer_.reset();
-}
-
-#endif // defined(OS_WIN)
-
-} // namespace remoting
« no previous file with comments | « remoting/host/video_frame_capturer_mac_unittest.cc ('k') | remoting/host/video_frame_capturer_win.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698