| Index: remoting/host/screen_resolution_unittest.cc
|
| diff --git a/remoting/host/screen_resolution_unittest.cc b/remoting/host/screen_resolution_unittest.cc
|
| index dce7bc0cd2c7defa0c032a2f8ec3efdfb7e7dfc0..01de134b9690f6a006afd7214e079f7ed253b8d6 100644
|
| --- a/remoting/host/screen_resolution_unittest.cc
|
| +++ b/remoting/host/screen_resolution_unittest.cc
|
| @@ -12,69 +12,38 @@
|
| namespace remoting {
|
|
|
| TEST(ScreenResolutionTest, Empty) {
|
| - ScreenResolution resolution;
|
| - EXPECT_TRUE(resolution.IsEmpty());
|
| + ScreenResolution resolution1(
|
| + webrtc::DesktopSize(100, 100), webrtc::DesktopVector(10, 10));
|
| + EXPECT_FALSE(resolution1.IsEmpty());
|
|
|
| - resolution.dimensions_.set(-1, 0);
|
| - EXPECT_TRUE(resolution.IsEmpty());
|
| + ScreenResolution resolution2(
|
| + webrtc::DesktopSize(), webrtc::DesktopVector(10, 10));
|
| + EXPECT_TRUE(resolution2.IsEmpty());
|
|
|
| - resolution.dimensions_.set(0, -1);
|
| - EXPECT_TRUE(resolution.IsEmpty());
|
| -
|
| - resolution.dimensions_.set(-1, -1);
|
| - EXPECT_TRUE(resolution.IsEmpty());
|
| -
|
| - resolution.dpi_.set(-1, 0);
|
| - EXPECT_TRUE(resolution.IsEmpty());
|
| -
|
| - resolution.dpi_.set(0, -1);
|
| - EXPECT_TRUE(resolution.IsEmpty());
|
| -
|
| - resolution.dpi_.set(-1, -1);
|
| - EXPECT_TRUE(resolution.IsEmpty());
|
| -}
|
| -
|
| -TEST(ScreenResolutionTest, Invalid) {
|
| - ScreenResolution resolution;
|
| - EXPECT_TRUE(resolution.IsValid());
|
| -
|
| - resolution.dimensions_.set(-1, 0);
|
| - EXPECT_FALSE(resolution.IsValid());
|
| -
|
| - resolution.dimensions_.set(0, -1);
|
| - EXPECT_FALSE(resolution.IsValid());
|
| -
|
| - resolution.dimensions_.set(-1, -1);
|
| - EXPECT_FALSE(resolution.IsValid());
|
| -
|
| - resolution.dpi_.set(-1, 0);
|
| - EXPECT_FALSE(resolution.IsValid());
|
| -
|
| - resolution.dpi_.set(0, -1);
|
| - EXPECT_FALSE(resolution.IsValid());
|
| -
|
| - resolution.dpi_.set(-1, -1);
|
| - EXPECT_FALSE(resolution.IsValid());
|
| + ScreenResolution resolution3(
|
| + webrtc::DesktopSize(1, 1), webrtc::DesktopVector(0, 0));
|
| + EXPECT_TRUE(resolution3.IsEmpty());
|
| }
|
|
|
| TEST(ScreenResolutionTest, Scaling) {
|
| ScreenResolution resolution(
|
| - SkISize::Make(100, 100), SkIPoint::Make(10, 10));
|
| + webrtc::DesktopSize(100, 100), webrtc::DesktopVector(10, 10));
|
|
|
| - EXPECT_EQ(resolution.ScaleDimensionsToDpi(SkIPoint::Make(5, 5)),
|
| - SkISize::Make(50, 50));
|
| + EXPECT_TRUE(webrtc::DesktopSize(50, 50).equals(
|
| + resolution.ScaleDimensionsToDpi(webrtc::DesktopVector(5, 5))));
|
|
|
| - EXPECT_EQ(resolution.ScaleDimensionsToDpi(SkIPoint::Make(20, 20)),
|
| - SkISize::Make(200, 200));
|
| + EXPECT_TRUE(webrtc::DesktopSize(200, 200).equals(
|
| + resolution.ScaleDimensionsToDpi(webrtc::DesktopVector(20, 20))));
|
| }
|
|
|
| TEST(ScreenResolutionTest, ScalingSaturation) {
|
| ScreenResolution resolution(
|
| - SkISize::Make(10000000, 1000000), SkIPoint::Make(1, 1));
|
| + webrtc::DesktopSize(10000000, 1000000), webrtc::DesktopVector(1, 1));
|
|
|
| - EXPECT_EQ(resolution.ScaleDimensionsToDpi(SkIPoint::Make(1000000, 1000000)),
|
| - SkISize::Make(std::numeric_limits<int32>::max(),
|
| - std::numeric_limits<int32>::max()));
|
| + int32 max_int = std::numeric_limits<int32>::max();
|
| + EXPECT_TRUE(webrtc::DesktopSize(max_int, max_int).equals(
|
| + resolution.ScaleDimensionsToDpi(
|
| + webrtc::DesktopVector(1000000, 1000000))));
|
| }
|
|
|
| } // namespace remoting
|
|
|