OLD | NEW |
1 // Copyright 2013 The Chromium Authors. All rights reserved. | 1 // Copyright 2013 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #include "chrome/browser/extensions/favicon_downloader.h" | 5 #include "chrome/browser/extensions/favicon_downloader.h" |
6 | 6 |
7 #include <stddef.h> | 7 #include <stddef.h> |
8 | 8 |
9 #include "base/bind.h" | 9 #include "base/bind.h" |
10 #include "base/files/scoped_temp_dir.h" | 10 #include "base/files/scoped_temp_dir.h" |
(...skipping 85 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
96 FaviconDownloader::FaviconMap favicon_map_; | 96 FaviconDownloader::FaviconMap favicon_map_; |
97 int id_counter_; | 97 int id_counter_; |
98 DISALLOW_COPY_AND_ASSIGN(TestFaviconDownloader); | 98 DISALLOW_COPY_AND_ASSIGN(TestFaviconDownloader); |
99 }; | 99 }; |
100 | 100 |
101 TEST_F(FaviconDownloaderTest, SimpleDownload) { | 101 TEST_F(FaviconDownloaderTest, SimpleDownload) { |
102 const GURL favicon_url("http://www.google.com/favicon.ico"); | 102 const GURL favicon_url("http://www.google.com/favicon.ico"); |
103 TestFaviconDownloader downloader(web_contents(), std::vector<GURL>()); | 103 TestFaviconDownloader downloader(web_contents(), std::vector<GURL>()); |
104 | 104 |
105 std::vector<content::FaviconURL> favicon_urls; | 105 std::vector<content::FaviconURL> favicon_urls; |
106 favicon_urls.push_back(content::FaviconURL( | 106 favicon_urls.push_back( |
107 favicon_url, content::FaviconURL::FAVICON, std::vector<gfx::Size>())); | 107 content::FaviconURL(favicon_url, content::FaviconURL::IconType::kFavicon, |
| 108 std::vector<gfx::Size>())); |
108 downloader.set_initial_favicon_urls(favicon_urls); | 109 downloader.set_initial_favicon_urls(favicon_urls); |
109 EXPECT_EQ(0u, downloader.pending_requests()); | 110 EXPECT_EQ(0u, downloader.pending_requests()); |
110 | 111 |
111 downloader.Start(); | 112 downloader.Start(); |
112 EXPECT_EQ(1u, downloader.pending_requests()); | 113 EXPECT_EQ(1u, downloader.pending_requests()); |
113 | 114 |
114 std::vector<gfx::Size> sizes(1, gfx::Size(32, 32)); | 115 std::vector<gfx::Size> sizes(1, gfx::Size(32, 32)); |
115 downloader.CompleteImageDownload(0, favicon_urls[0].icon_url, sizes); | 116 downloader.CompleteImageDownload(0, favicon_urls[0].icon_url, sizes); |
116 EXPECT_EQ(0u, downloader.pending_requests()); | 117 EXPECT_EQ(0u, downloader.pending_requests()); |
117 | 118 |
118 EXPECT_EQ(1u, downloader.favicon_map().size()); | 119 EXPECT_EQ(1u, downloader.favicon_map().size()); |
119 EXPECT_EQ(1u, downloader.favicon_map()[favicon_url].size()); | 120 EXPECT_EQ(1u, downloader.favicon_map()[favicon_url].size()); |
120 } | 121 } |
121 | 122 |
122 TEST_F(FaviconDownloaderTest, DownloadWithUrlsFromWebContentsNotification) { | 123 TEST_F(FaviconDownloaderTest, DownloadWithUrlsFromWebContentsNotification) { |
123 const GURL favicon_url("http://www.google.com/favicon.ico"); | 124 const GURL favicon_url("http://www.google.com/favicon.ico"); |
124 TestFaviconDownloader downloader(web_contents(), std::vector<GURL>()); | 125 TestFaviconDownloader downloader(web_contents(), std::vector<GURL>()); |
125 | 126 |
126 std::vector<content::FaviconURL> favicon_urls; | 127 std::vector<content::FaviconURL> favicon_urls; |
127 favicon_urls.push_back(content::FaviconURL( | 128 favicon_urls.push_back( |
128 favicon_url, content::FaviconURL::FAVICON, std::vector<gfx::Size>())); | 129 content::FaviconURL(favicon_url, content::FaviconURL::IconType::kFavicon, |
| 130 std::vector<gfx::Size>())); |
129 EXPECT_EQ(0u, downloader.pending_requests()); | 131 EXPECT_EQ(0u, downloader.pending_requests()); |
130 | 132 |
131 // Start downloader before favicon URLs are loaded. | 133 // Start downloader before favicon URLs are loaded. |
132 downloader.Start(); | 134 downloader.Start(); |
133 EXPECT_EQ(0u, downloader.pending_requests()); | 135 EXPECT_EQ(0u, downloader.pending_requests()); |
134 | 136 |
135 downloader.UpdateFaviconURLs(favicon_urls); | 137 downloader.UpdateFaviconURLs(favicon_urls); |
136 EXPECT_EQ(1u, downloader.pending_requests()); | 138 EXPECT_EQ(1u, downloader.pending_requests()); |
137 | 139 |
138 std::vector<gfx::Size> sizes(1, gfx::Size(32, 32)); | 140 std::vector<gfx::Size> sizes(1, gfx::Size(32, 32)); |
(...skipping 11 matching lines...) Expand all Loading... |
150 | 152 |
151 std::vector<GURL> extra_urls; | 153 std::vector<GURL> extra_urls; |
152 // This should get downloaded. | 154 // This should get downloaded. |
153 extra_urls.push_back(favicon_url_2); | 155 extra_urls.push_back(favicon_url_2); |
154 // This is duplicated in the favicon urls and should only be downloaded once. | 156 // This is duplicated in the favicon urls and should only be downloaded once. |
155 extra_urls.push_back(empty_favicon); | 157 extra_urls.push_back(empty_favicon); |
156 | 158 |
157 TestFaviconDownloader downloader(web_contents(), extra_urls); | 159 TestFaviconDownloader downloader(web_contents(), extra_urls); |
158 std::vector<content::FaviconURL> favicon_urls; | 160 std::vector<content::FaviconURL> favicon_urls; |
159 favicon_urls.push_back(content::FaviconURL( | 161 favicon_urls.push_back(content::FaviconURL( |
160 favicon_url_1, content::FaviconURL::FAVICON, std::vector<gfx::Size>())); | 162 favicon_url_1, content::FaviconURL::IconType::kFavicon, |
| 163 std::vector<gfx::Size>())); |
161 // This is duplicated in the favicon urls and should only be downloaded once. | 164 // This is duplicated in the favicon urls and should only be downloaded once. |
162 favicon_urls.push_back(content::FaviconURL( | 165 favicon_urls.push_back(content::FaviconURL( |
163 empty_favicon, content::FaviconURL::FAVICON, std::vector<gfx::Size>())); | 166 empty_favicon, content::FaviconURL::IconType::kFavicon, |
| 167 std::vector<gfx::Size>())); |
164 // Invalid icons shouldn't get put into the download queue. | 168 // Invalid icons shouldn't get put into the download queue. |
165 favicon_urls.push_back( | 169 favicon_urls.push_back(content::FaviconURL( |
166 content::FaviconURL(GURL("http://www.google.com/invalid.ico"), | 170 GURL("http://www.google.com/invalid.ico"), |
167 content::FaviconURL::INVALID_ICON, | 171 content::FaviconURL::IconType::kInvalid, std::vector<gfx::Size>())); |
168 std::vector<gfx::Size>())); | |
169 downloader.set_initial_favicon_urls(favicon_urls); | 172 downloader.set_initial_favicon_urls(favicon_urls); |
170 downloader.Start(); | 173 downloader.Start(); |
171 EXPECT_EQ(3u, downloader.pending_requests()); | 174 EXPECT_EQ(3u, downloader.pending_requests()); |
172 | 175 |
173 std::vector<gfx::Size> sizes_1(1, gfx::Size(16, 16)); | 176 std::vector<gfx::Size> sizes_1(1, gfx::Size(16, 16)); |
174 downloader.CompleteImageDownload(0, favicon_url_1, sizes_1); | 177 downloader.CompleteImageDownload(0, favicon_url_1, sizes_1); |
175 | 178 |
176 std::vector<gfx::Size> sizes_2; | 179 std::vector<gfx::Size> sizes_2; |
177 sizes_2.push_back(gfx::Size(32, 32)); | 180 sizes_2.push_back(gfx::Size(32, 32)); |
178 sizes_2.push_back(gfx::Size(64, 64)); | 181 sizes_2.push_back(gfx::Size(64, 64)); |
(...skipping 15 matching lines...) Expand all Loading... |
194 const GURL favicon_url_2("http://www.google.com/favicon2.ico"); | 197 const GURL favicon_url_2("http://www.google.com/favicon2.ico"); |
195 | 198 |
196 std::vector<GURL> extra_urls; | 199 std::vector<GURL> extra_urls; |
197 extra_urls.push_back(favicon_url_1); | 200 extra_urls.push_back(favicon_url_1); |
198 | 201 |
199 TestFaviconDownloader downloader(web_contents(), extra_urls); | 202 TestFaviconDownloader downloader(web_contents(), extra_urls); |
200 | 203 |
201 // This favicon URL should be ignored. | 204 // This favicon URL should be ignored. |
202 std::vector<content::FaviconURL> favicon_urls; | 205 std::vector<content::FaviconURL> favicon_urls; |
203 favicon_urls.push_back(content::FaviconURL( | 206 favicon_urls.push_back(content::FaviconURL( |
204 favicon_url_2, content::FaviconURL::FAVICON, std::vector<gfx::Size>())); | 207 favicon_url_2, content::FaviconURL::IconType::kFavicon, |
| 208 std::vector<gfx::Size>())); |
205 downloader.set_initial_favicon_urls(favicon_urls); | 209 downloader.set_initial_favicon_urls(favicon_urls); |
206 downloader.SkipPageFavicons(); | 210 downloader.SkipPageFavicons(); |
207 downloader.Start(); | 211 downloader.Start(); |
208 EXPECT_EQ(1u, downloader.pending_requests()); | 212 EXPECT_EQ(1u, downloader.pending_requests()); |
209 | 213 |
210 std::vector<gfx::Size> sizes_1(1, gfx::Size(16, 16)); | 214 std::vector<gfx::Size> sizes_1(1, gfx::Size(16, 16)); |
211 downloader.CompleteImageDownload(0, favicon_url_1, sizes_1); | 215 downloader.CompleteImageDownload(0, favicon_url_1, sizes_1); |
212 | 216 |
213 // This download should not be finished and inserted into the map. | 217 // This download should not be finished and inserted into the map. |
214 std::vector<gfx::Size> sizes_2; | 218 std::vector<gfx::Size> sizes_2; |
215 sizes_2.push_back(gfx::Size(32, 32)); | 219 sizes_2.push_back(gfx::Size(32, 32)); |
216 sizes_2.push_back(gfx::Size(64, 64)); | 220 sizes_2.push_back(gfx::Size(64, 64)); |
217 downloader.CompleteImageDownload(1, favicon_url_2, sizes_2); | 221 downloader.CompleteImageDownload(1, favicon_url_2, sizes_2); |
218 EXPECT_EQ(0u, downloader.pending_requests()); | 222 EXPECT_EQ(0u, downloader.pending_requests()); |
219 | 223 |
220 EXPECT_EQ(1u, downloader.favicon_map().size()); | 224 EXPECT_EQ(1u, downloader.favicon_map().size()); |
221 EXPECT_EQ(1u, downloader.favicon_map()[favicon_url_1].size()); | 225 EXPECT_EQ(1u, downloader.favicon_map()[favicon_url_1].size()); |
222 EXPECT_EQ(0u, downloader.favicon_map()[favicon_url_2].size()); | 226 EXPECT_EQ(0u, downloader.favicon_map()[favicon_url_2].size()); |
223 } | 227 } |
OLD | NEW |