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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: remoting/host/video_frame_capturer_helper_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
« remoting/capturer/video_frame_capturer_win.cc ('K') | « remoting/host/video_frame_capturer_helper.cc ('k') | remoting/host/video_frame_capturer_linux.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/video_frame_capturer_helper_unittest.cc
diff --git a/remoting/host/video_frame_capturer_helper_unittest.cc b/remoting/host/video_frame_capturer_helper_unittest.cc
deleted file mode 100644
index 89c1c4076d33a3fb4ba38448e80a62a59c1310fd..0000000000000000000000000000000000000000
--- a/remoting/host/video_frame_capturer_helper_unittest.cc
+++ /dev/null
@@ -1,215 +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_helper.h"
-
-#include "base/memory/scoped_ptr.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-namespace remoting {
-
-class VideoFrameCapturerHelperTest : public testing::Test {
- protected:
- VideoFrameCapturerHelper capturer_helper_;
-};
-
-bool Equals(const SkRegion& region1, const SkRegion& region2) {
- SkRegion::Iterator iter1(region1);
- SkRegion::Iterator iter2(region2);
- while (!iter1.done()) {
- SkIRect rect1 = iter1.rect();
- iter1.next();
- if (iter2.done()) {
- return false;
- }
- SkIRect rect2 = iter2.rect();
- iter2.next();
- if (rect1 != rect2) {
- return false;
- }
- }
- if (!iter2.done()) {
- return false;
- }
- return true;
-}
-
-TEST_F(VideoFrameCapturerHelperTest, ClearInvalidRegion) {
- SkRegion region;
- capturer_helper_.InvalidateRegion(SkRegion(SkIRect::MakeXYWH(1, 2, 3, 4)));
- capturer_helper_.ClearInvalidRegion();
- capturer_helper_.SwapInvalidRegion(&region);
- ASSERT_TRUE(region.isEmpty());
-}
-
-TEST_F(VideoFrameCapturerHelperTest, InvalidateRegion) {
- SkRegion region;
- capturer_helper_.SwapInvalidRegion(&region);
- ASSERT_TRUE(Equals(SkRegion(SkIRect::MakeEmpty()), region));
-
- capturer_helper_.InvalidateRegion(SkRegion(SkIRect::MakeXYWH(1, 2, 3, 4)));
- region.setEmpty();
- capturer_helper_.SwapInvalidRegion(&region);
- ASSERT_TRUE(Equals(SkRegion(SkIRect::MakeXYWH(1, 2, 3, 4)), region));
-
- capturer_helper_.InvalidateRegion(SkRegion(SkIRect::MakeXYWH(1, 2, 3, 4)));
- capturer_helper_.InvalidateRegion(SkRegion(SkIRect::MakeXYWH(4, 2, 3, 4)));
- region.setEmpty();
- capturer_helper_.SwapInvalidRegion(&region);
- ASSERT_TRUE(Equals(SkRegion(SkIRect::MakeXYWH(1, 2, 6, 4)), region));
-}
-
-TEST_F(VideoFrameCapturerHelperTest, InvalidateScreen) {
- SkRegion region;
- capturer_helper_.InvalidateScreen(SkISize::Make(12, 34));
- capturer_helper_.SwapInvalidRegion(&region);
- ASSERT_TRUE(Equals(SkRegion(SkIRect::MakeWH(12, 34)), region));
-}
-
-TEST_F(VideoFrameCapturerHelperTest, InvalidateFullScreen) {
- SkRegion region;
- capturer_helper_.set_size_most_recent(SkISize::Make(12, 34));
- capturer_helper_.InvalidateFullScreen();
- capturer_helper_.SwapInvalidRegion(&region);
- ASSERT_TRUE(Equals(SkRegion(SkIRect::MakeWH(12, 34)), region));
-}
-
-TEST_F(VideoFrameCapturerHelperTest, SizeMostRecent) {
- ASSERT_EQ(SkISize::Make(0, 0), capturer_helper_.size_most_recent());
- capturer_helper_.set_size_most_recent(SkISize::Make(12, 34));
- ASSERT_EQ(SkISize::Make(12, 34), capturer_helper_.size_most_recent());
-}
-
-TEST_F(VideoFrameCapturerHelperTest, SetLogGridSize) {
- capturer_helper_.set_size_most_recent(SkISize::Make(10, 10));
-
- SkRegion region;
- capturer_helper_.SwapInvalidRegion(&region);
- ASSERT_TRUE(Equals(SkRegion(SkIRect::MakeEmpty()), region));
-
- capturer_helper_.InvalidateRegion(SkRegion(SkIRect::MakeXYWH(7, 7, 1, 1)));
- region.setEmpty();
- capturer_helper_.SwapInvalidRegion(&region);
- ASSERT_TRUE(Equals(SkRegion(SkIRect::MakeXYWH(7, 7, 1, 1)), region));
-
- capturer_helper_.SetLogGridSize(-1);
- capturer_helper_.InvalidateRegion(SkRegion(SkIRect::MakeXYWH(7, 7, 1, 1)));
- region.setEmpty();
- capturer_helper_.SwapInvalidRegion(&region);
- ASSERT_TRUE(Equals(SkRegion(SkIRect::MakeXYWH(7, 7, 1, 1)), region));
-
- capturer_helper_.SetLogGridSize(0);
- capturer_helper_.InvalidateRegion(SkRegion(SkIRect::MakeXYWH(7, 7, 1, 1)));
- region.setEmpty();
- capturer_helper_.SwapInvalidRegion(&region);
- ASSERT_TRUE(Equals(SkRegion(SkIRect::MakeXYWH(7, 7, 1, 1)), region));
-
- capturer_helper_.SetLogGridSize(1);
- capturer_helper_.InvalidateRegion(SkRegion(SkIRect::MakeXYWH(7, 7, 1, 1)));
- region.setEmpty();
- capturer_helper_.SwapInvalidRegion(&region);
- ASSERT_TRUE(Equals(SkRegion(SkIRect::MakeXYWH(6, 6, 2, 2)), region));
-
- capturer_helper_.SetLogGridSize(2);
- capturer_helper_.InvalidateRegion(SkRegion(SkIRect::MakeXYWH(7, 7, 1, 1)));
- region.setEmpty();
- capturer_helper_.SwapInvalidRegion(&region);
- ASSERT_TRUE(Equals(SkRegion(SkIRect::MakeXYWH(4, 4, 4, 4)), region));
-
- capturer_helper_.SetLogGridSize(0);
- capturer_helper_.InvalidateRegion(SkRegion(SkIRect::MakeXYWH(7, 7, 1, 1)));
- region.setEmpty();
- capturer_helper_.SwapInvalidRegion(&region);
- ASSERT_TRUE(Equals(SkRegion(SkIRect::MakeXYWH(7, 7, 1, 1)), region));
-}
-
-void TestExpandRegionToGrid(const SkRegion& region, int log_grid_size,
- const SkRegion& expandedRegionExpected) {
- scoped_ptr<SkRegion> expandedRegion1(
- VideoFrameCapturerHelper::ExpandToGrid(region, log_grid_size));
- ASSERT_TRUE(Equals(expandedRegionExpected, *expandedRegion1));
- scoped_ptr<SkRegion> expandedRegion2(
- VideoFrameCapturerHelper::ExpandToGrid(*expandedRegion1, log_grid_size));
- ASSERT_TRUE(Equals(*expandedRegion1, *expandedRegion2));
-}
-
-void TestExpandRectToGrid(int l, int t, int r, int b, int log_grid_size,
- int lExpanded, int tExpanded,
- int rExpanded, int bExpanded) {
- TestExpandRegionToGrid(SkRegion(SkIRect::MakeLTRB(l, t, r, b)), log_grid_size,
- SkRegion(SkIRect::MakeLTRB(lExpanded, tExpanded,
- rExpanded, bExpanded)));
-}
-
-TEST_F(VideoFrameCapturerHelperTest, ExpandToGrid) {
- const int LOG_GRID_SIZE = 4;
- const int GRID_SIZE = 1 << LOG_GRID_SIZE;
- for (int i = -2; i <= 2; i++) {
- int x = i * GRID_SIZE;
- for (int j = -2; j <= 2; j++) {
- int y = j * GRID_SIZE;
- TestExpandRectToGrid(x + 0, y + 0, x + 1, y + 1, LOG_GRID_SIZE,
- x + 0, y + 0, x + GRID_SIZE, y + GRID_SIZE);
- TestExpandRectToGrid(x + 0, y + GRID_SIZE - 1, x + 1, y + GRID_SIZE,
- LOG_GRID_SIZE,
- x + 0, y + 0, x + GRID_SIZE, y + GRID_SIZE);
- TestExpandRectToGrid(x + GRID_SIZE - 1, y + GRID_SIZE - 1,
- x + GRID_SIZE, y + GRID_SIZE, LOG_GRID_SIZE,
- x + 0, y + 0, x + GRID_SIZE, y + GRID_SIZE);
- TestExpandRectToGrid(x + GRID_SIZE - 1, y + 0,
- x + GRID_SIZE, y + 1, LOG_GRID_SIZE,
- x + 0, y + 0, x + GRID_SIZE, y + GRID_SIZE);
- TestExpandRectToGrid(x - 1, y + 0, x + 1, y + 1, LOG_GRID_SIZE,
- x - GRID_SIZE, y + 0, x + GRID_SIZE, y + GRID_SIZE);
- TestExpandRectToGrid(x - 1, y - 1, x + 1, y + 0, LOG_GRID_SIZE,
- x - GRID_SIZE, y - GRID_SIZE, x + GRID_SIZE, y);
- TestExpandRectToGrid(x + 0, y - 1, x + 1, y + 1, LOG_GRID_SIZE,
- x, y - GRID_SIZE, x + GRID_SIZE, y + GRID_SIZE);
- TestExpandRectToGrid(x - 1, y - 1, x + 0, y + 1, LOG_GRID_SIZE,
- x - GRID_SIZE, y - GRID_SIZE, x, y + GRID_SIZE);
-
- SkRegion region(SkIRect::MakeLTRB(x - 1, y - 1, x + 1, y + 1));
- region.op(SkIRect::MakeLTRB(x - 1, y - 1, x + 0, y + 0),
- SkRegion::kDifference_Op);
- SkRegion expandedRegionExpected(SkIRect::MakeLTRB(
- x - GRID_SIZE, y - GRID_SIZE, x + GRID_SIZE, y + GRID_SIZE));
- expandedRegionExpected.op(
- SkIRect::MakeLTRB(x - GRID_SIZE, y - GRID_SIZE, x + 0, y + 0),
- SkRegion::kDifference_Op);
- TestExpandRegionToGrid(region, LOG_GRID_SIZE, expandedRegionExpected);
-
- region.setRect(SkIRect::MakeLTRB(x - 1, y - 1, x + 1, y + 1));
- region.op(SkIRect::MakeLTRB(x - 1, y + 0, x + 0, y + 1),
- SkRegion::kDifference_Op);
- expandedRegionExpected.setRect(SkIRect::MakeLTRB(
- x - GRID_SIZE, y - GRID_SIZE, x + GRID_SIZE, y + GRID_SIZE));
- expandedRegionExpected.op(
- SkIRect::MakeLTRB(x - GRID_SIZE, y + 0, x + 0, y + GRID_SIZE),
- SkRegion::kDifference_Op);
- TestExpandRegionToGrid(region, LOG_GRID_SIZE, expandedRegionExpected);
-
- region.setRect(SkIRect::MakeLTRB(x - 1, y - 1, x + 1, y + 1));
- region.op(SkIRect::MakeLTRB(x + 0, y + 0, x + 1, y + 1),
- SkRegion::kDifference_Op);
- expandedRegionExpected.setRect(SkIRect::MakeLTRB(
- x - GRID_SIZE, y - GRID_SIZE, x + GRID_SIZE, y + GRID_SIZE));
- expandedRegionExpected.op(
- SkIRect::MakeLTRB(x + 0, y + 0, x + GRID_SIZE, y + GRID_SIZE),
- SkRegion::kDifference_Op);
- TestExpandRegionToGrid(region, LOG_GRID_SIZE, expandedRegionExpected);
-
- region.setRect(SkIRect::MakeLTRB(x - 1, y - 1, x + 1, y + 1));
- region.op(SkIRect::MakeLTRB(x + 0, y - 1, x + 1, y + 0),
- SkRegion::kDifference_Op);
- expandedRegionExpected.setRect(SkIRect::MakeLTRB(
- x - GRID_SIZE, y - GRID_SIZE, x + GRID_SIZE, y + GRID_SIZE));
- expandedRegionExpected.op(
- SkIRect::MakeLTRB(x + 0, y - GRID_SIZE, x + GRID_SIZE, y + 0),
- SkRegion::kDifference_Op);
- TestExpandRegionToGrid(region, LOG_GRID_SIZE, expandedRegionExpected);
- }
- }
-}
-
-} // namespace remoting
« remoting/capturer/video_frame_capturer_win.cc ('K') | « remoting/host/video_frame_capturer_helper.cc ('k') | remoting/host/video_frame_capturer_linux.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698