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

Unified Diff: ui/gfx/image/image_family_unittest.cc

Issue 854713003: More old files deletion. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Fix tryjobs? Created 5 years, 11 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 | « ui/gfx/image/image_family.cc ('k') | ui/gfx/image/image_ios.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/gfx/image/image_family_unittest.cc
diff --git a/ui/gfx/image/image_family_unittest.cc b/ui/gfx/image/image_family_unittest.cc
deleted file mode 100644
index f9dd4017305bc4d168533cf2b9ddeef1970718f1..0000000000000000000000000000000000000000
--- a/ui/gfx/image/image_family_unittest.cc
+++ /dev/null
@@ -1,177 +0,0 @@
-// Copyright 2013 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 "testing/gtest/include/gtest/gtest.h"
-#include "ui/gfx/image/image.h"
-#include "ui/gfx/image/image_family.h"
-#include "ui/gfx/image/image_skia.h"
-#include "ui/gfx/image/image_unittest_util.h"
-
-namespace {
-
-namespace gt = gfx::test;
-
-// Tests that |image| != NULL, and has the given width and height.
-// This is a macro instead of a function, so that the correct line numbers are
-// reported when a test fails.
-#define EXPECT_IMAGE_NON_NULL_AND_SIZE(image, expected_width, expected_height) \
-do { \
- const gfx::Image* image_ = image; \
- EXPECT_TRUE(image_); \
- EXPECT_EQ(expected_width, image_->Width()); \
- EXPECT_EQ(expected_height, image_->Height()); \
-} while(0)
-
-class ImageFamilyTest : public testing::Test {
- public:
- // Construct an ImageFamily. Implicitly tests Add and Empty.
- virtual void SetUp() override {
- EXPECT_TRUE(image_family_.empty());
-
- // Aspect ratio 1:1.
- image_family_.Add(gt::CreateImageSkia(32, 32));
- EXPECT_FALSE(image_family_.empty());
- image_family_.Add(gt::CreateImageSkia(16, 16));
- image_family_.Add(gt::CreateImageSkia(64, 64));
- // Duplicate (should override previous one).
- // Insert an Image directly, instead of an ImageSkia.
- gfx::Image image(gt::CreateImageSkia(32, 32));
- image_family_.Add(image);
- // Aspect ratio 1:4.
- image_family_.Add(gt::CreateImageSkia(3, 12));
- image_family_.Add(gt::CreateImageSkia(12, 48));
- // Aspect ratio 4:1.
- image_family_.Add(gt::CreateImageSkia(512, 128));
- image_family_.Add(gt::CreateImageSkia(256, 64));
-
- EXPECT_FALSE(image_family_.empty());
- }
-
- gfx::ImageFamily image_family_;
-};
-
-TEST_F(ImageFamilyTest, Clear) {
- image_family_.clear();
- EXPECT_TRUE(image_family_.empty());
-}
-
-// Tests iteration over an ImageFamily.
-TEST_F(ImageFamilyTest, Iteration) {
- gfx::ImageFamily::const_iterator it = image_family_.begin();
- gfx::ImageFamily::const_iterator end = image_family_.end();
-
- // Expect iteration in order of aspect ratio (from thinnest to widest), then
- // size.
- EXPECT_TRUE(it != end);
- EXPECT_EQ(gfx::Size(3, 12), it->Size());
- ++it;
- EXPECT_TRUE(it != end);
- EXPECT_EQ(gfx::Size(12, 48), it->Size());
- it++; // Test post-increment.
- EXPECT_TRUE(it != end);
- EXPECT_EQ(gfx::Size(16, 16), it->Size());
- ++it;
- EXPECT_TRUE(it != end);
- EXPECT_EQ(gfx::Size(32, 32), it->Size());
- --it; // Test decrement
- EXPECT_TRUE(it != end);
- EXPECT_EQ(gfx::Size(16, 16), it->Size());
- ++it;
- ++it;
- EXPECT_TRUE(it != end);
- EXPECT_EQ(gfx::Size(64, 64), (*it).Size()); // Test operator*.
- ++it;
- EXPECT_TRUE(it != end);
- EXPECT_EQ(gfx::Size(256, 64), it->Size());
- ++it;
- EXPECT_TRUE(it != end);
- EXPECT_EQ(gfx::Size(512, 128), it->Size());
- ++it;
-
- EXPECT_TRUE(it == end);
-}
-
-TEST_F(ImageFamilyTest, Get) {
- // Get on an empty family.
- gfx::ImageFamily empty_family;
- EXPECT_TRUE(empty_family.empty());
- EXPECT_FALSE(empty_family.GetBest(32, 32));
- EXPECT_FALSE(empty_family.GetBest(0, 32));
- EXPECT_FALSE(empty_family.GetBest(32, 0));
-
- // Get various aspect ratios and sizes on the sample family.
-
- // 0x0 (expect the smallest square image).
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(0, 0), 16, 16);
- // GetBest(0, N) or GetBest(N, 0) should be treated the same as GetBest(0, 0).
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(0, 16), 16, 16);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(0, 64), 16, 16);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(16, 0), 16, 16);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(64, 0), 16, 16);
-
- // Thinner than thinnest image.
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(2, 12), 3, 12);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(2, 13), 12, 48);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(10, 60), 12, 48);
-
- // Between two images' aspect ratio.
- // Note: Testing the boundary around 1:2 and 2:1, half way to 1:4 and 4:1.
- // Ties are broken by favouring the thinner aspect ratio.
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(63, 32), 64, 64);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(64, 32), 64, 64);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(65, 32), 256, 64);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(32, 63), 64, 64);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(32, 64), 12, 48);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(32, 65), 12, 48);
-
- // Exact match aspect ratio.
- // Exact match size.
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(32, 32), 32, 32);
- // Slightly smaller.
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(31, 31), 32, 32);
- // Much smaller.
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(17, 17), 32, 32);
- // Exact match size.
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(16, 16), 16, 16);
- // Smaller than any image.
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(3, 3), 16, 16);
- // Larger than any image.
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(512, 512), 64, 64);
- // 1:4 aspect ratio.
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(16, 64), 12, 48);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(2, 8), 3, 12);
- // 4:1 aspect ratio.
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(64, 16), 256, 64);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(260, 65), 512, 128);
-
- // Wider than widest image.
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(255, 51), 256, 64);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(260, 52), 512, 128);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(654, 129), 512, 128);
-}
-
-// Test adding and looking up images with 0 width and height.
-TEST_F(ImageFamilyTest, ZeroWidthAndHeight) {
- // An empty Image. Should be considered to have 0 width and height.
- image_family_.Add(gfx::Image());
- // Images with 0 width OR height should be treated the same as an image with 0
- // width AND height (in fact, the ImageSkias should be indistinguishable).
- image_family_.Add(gt::CreateImageSkia(32, 0));
- image_family_.Add(gt::CreateImageSkia(0, 32));
-
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(0, 0), 0, 0);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(1, 1), 16, 16);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(32, 32), 32, 32);
-
- // GetBest(0, N) or GetBest(N, 0) should be treated the same as GetBest(0, 0).
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(0, 1), 0, 0);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(0, 32), 0, 0);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(1, 0), 0, 0);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(32, 0), 0, 0);
-
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(1, 32), 12, 48);
- EXPECT_IMAGE_NON_NULL_AND_SIZE(image_family_.GetBest(32, 1), 256, 64);
-}
-
-} // namespace
« no previous file with comments | « ui/gfx/image/image_family.cc ('k') | ui/gfx/image/image_ios.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698