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

Side by Side Diff: chrome/browser/manifest/manifest_icon_downloader_unittest.cc

Issue 1261143004: Implement manifest icon downloader (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix compile Created 5 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chrome/browser/manifest/manifest_icon_downloader.h"
6
7 #include <string>
8 #include <vector>
9
10 #include "testing/gtest/include/gtest/gtest.h"
11
12 class ManifestIconDownloaderTest : public testing::Test {
13 protected:
14 ManifestIconDownloaderTest() = default;
15 ~ManifestIconDownloaderTest() override = default;
16
17 int FindBitmap(const int ideal_icon_size_in_px,
18 const int lowest_icon_size_in_px,
19 const std::vector<SkBitmap>& bitmaps) {
20 return ManifestIconDownloader::FindClosestBitmapIndex(
21 ideal_icon_size_in_px, lowest_icon_size_in_px, bitmaps);
22 }
23
24 SkBitmap CreateDummyBitmap(int width, int height) {
25 SkBitmap bitmap;
26 bitmap.allocN32Pixels(width, height);
27 bitmap.setImmutable();
28 return bitmap;
29 }
30
31 DISALLOW_COPY_AND_ASSIGN(ManifestIconDownloaderTest);
32 };
33
34 TEST_F(ManifestIconDownloaderTest, NoIcons) {
35 ASSERT_EQ(-1, FindBitmap(0, 0, std::vector<SkBitmap>()));
36 }
37
38 TEST_F(ManifestIconDownloaderTest, ExactIsChosen) {
39 std::vector<SkBitmap> vector;
40 vector.push_back(CreateDummyBitmap(10, 10));
41
42 ASSERT_EQ(0, FindBitmap(10, 0, vector));
43 }
44
45 TEST_F(ManifestIconDownloaderTest, BiggerIsChosen) {
46 std::vector<SkBitmap> vector;
47 vector.push_back(CreateDummyBitmap(20, 20));
48
49 ASSERT_EQ(0, FindBitmap(10, 0, vector));
50 }
51
52 TEST_F(ManifestIconDownloaderTest, SmallerBelowCutoffIsIgnored) {
53 std::vector<SkBitmap> vector;
54 vector.push_back(CreateDummyBitmap(10, 10));
55
56 ASSERT_EQ(-1, FindBitmap(20, 15, vector));
57 }
58
59 TEST_F(ManifestIconDownloaderTest, SmallerAboveCutoffIsChosen) {
60 std::vector<SkBitmap> vector;
61 vector.push_back(CreateDummyBitmap(10, 10));
62
63 ASSERT_EQ(0, FindBitmap(20, 5, vector));
64 }
65
66 TEST_F(ManifestIconDownloaderTest, ExactIsPreferredOverBigger) {
67 std::vector<SkBitmap> vector;
68 vector.push_back(CreateDummyBitmap(20, 20));
69 vector.push_back(CreateDummyBitmap(10, 10));
70
71 ASSERT_EQ(1, FindBitmap(10, 0, vector));
72 }
73
74 TEST_F(ManifestIconDownloaderTest, ExactIsPreferredOverSmaller) {
75 std::vector<SkBitmap> vector;
76 vector.push_back(CreateDummyBitmap(10, 10));
77 vector.push_back(CreateDummyBitmap(20, 20));
78
79 ASSERT_EQ(1, FindBitmap(20, 0, vector));
80 }
81
82 TEST_F(ManifestIconDownloaderTest, BiggerIsPreferredOverCloserSmaller) {
83 std::vector<SkBitmap> vector;
84 vector.push_back(CreateDummyBitmap(10, 10));
85 vector.push_back(CreateDummyBitmap(20, 20));
86
87 ASSERT_EQ(1, FindBitmap(11, 0, vector));
88 }
89
90 TEST_F(ManifestIconDownloaderTest, ClosestToExactIsChosen) {
91 std::vector<SkBitmap> vector;
92 vector.push_back(CreateDummyBitmap(20, 20));
93 vector.push_back(CreateDummyBitmap(25, 25));
94
95 ASSERT_EQ(0, FindBitmap(10, 0, vector));
96 }
97
98 TEST_F(ManifestIconDownloaderTest, ClosestToExactIsChosenUnordered) {
99 std::vector<SkBitmap> vector;
100 vector.push_back(CreateDummyBitmap(25, 25));
101 vector.push_back(CreateDummyBitmap(20, 20));
102
103 ASSERT_EQ(1, FindBitmap(10, 0, vector));
104 }
105
106 TEST_F(ManifestIconDownloaderTest, MixedReturnsBiggestClosest) {
107 std::vector<SkBitmap> vector;
108 vector.push_back(CreateDummyBitmap(10, 10));
109 vector.push_back(CreateDummyBitmap(8, 8));
110 vector.push_back(CreateDummyBitmap(6, 6));
111
112 ASSERT_EQ(0, FindBitmap(9, 0, vector));
113 }
114
115 TEST_F(ManifestIconDownloaderTest, MixedCanReturnMiddle) {
116 std::vector<SkBitmap> vector;
117 vector.push_back(CreateDummyBitmap(10, 10));
118 vector.push_back(CreateDummyBitmap(8, 8));
119 vector.push_back(CreateDummyBitmap(6, 6));
120
121 ASSERT_EQ(1, FindBitmap(7, 0, vector));
122 }
123
124 TEST_F(ManifestIconDownloaderTest, NonSquareHeightIsIgnored) {
125 std::vector<SkBitmap> vector;
126 vector.push_back(CreateDummyBitmap(15, 10));
127
128 ASSERT_EQ(-1, FindBitmap(10, 0, vector));
129 }
130
131 TEST_F(ManifestIconDownloaderTest, NonSquareWidthIsIgnored) {
132 std::vector<SkBitmap> vector;
133 vector.push_back(CreateDummyBitmap(15, 10));
134
135 ASSERT_EQ(-1, FindBitmap(15, 0, vector));
136 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698