Change FaviconData to be able to return data for multiple bitmaps for same icon URL

chrome/browser/history/history_backend_unittest.cc

Index: chrome/browser/history/history_backend_unittest.cc
diff --git a/chrome/browser/history/history_backend_unittest.cc b/chrome/browser/history/history_backend_unittest.cc
index 35b4fd9965432a8ca93f1f210edb64b9ee01e71a..5c4b86a0573135f1e60530ad8f87aa2ae8536204 100644
--- a/chrome/browser/history/history_backend_unittest.cc
+++ b/chrome/browser/history/history_backend_unittest.cc
@@ -1231,24 +1231,30 @@ TEST_F(HistoryBackendTest, GetFaviconForURL) {
       url, TOUCH_ICON, NULL));
 
   // Test the Fav icon for this URL.
-  FaviconData favicon;
-  ASSERT_TRUE(backend_->GetFaviconFromDB(url, FAVICON, &favicon));
+  std::vector<FaviconBitmapResult> favicon_bitmap_results;
+  ASSERT_TRUE(backend_->GetFaviconFromDB(url, FAVICON,
+                                         &favicon_bitmap_results));
+  EXPECT_EQ(1u, favicon_bitmap_results.size());
+  FaviconBitmapResult bitmap_result = favicon_bitmap_results[0];
   std::string favicon_data(
-      favicon.image_data->front(),
-      favicon.image_data->front() + favicon.image_data->size());
+      bitmap_result.bitmap_data->front(),
+      bitmap_result.bitmap_data->front() + bitmap_result.bitmap_data->size());
 
-  EXPECT_EQ(FAVICON, favicon.icon_type);
-  EXPECT_EQ(icon_url, favicon.icon_url);
+  EXPECT_EQ(FAVICON, bitmap_result.icon_type);
+  EXPECT_EQ(icon_url, bitmap_result.icon_url);
   EXPECT_EQ(blob_data, favicon_data);
 
   // Test the touch icon for this URL.
-  ASSERT_TRUE(backend_->GetFaviconFromDB(url, TOUCH_ICON, &favicon));
+  ASSERT_TRUE(backend_->GetFaviconFromDB(url, TOUCH_ICON,
+                                         &favicon_bitmap_results));
+  EXPECT_EQ(1u, favicon_bitmap_results.size());
+  bitmap_result = favicon_bitmap_results[0];
   std::string touchicon_data(
-      favicon.image_data->front(),
-      favicon.image_data->front() + favicon.image_data->size());
+      bitmap_result.bitmap_data->front(),
+      bitmap_result.bitmap_data->front() + bitmap_result.bitmap_data->size());
 
-  EXPECT_EQ(TOUCH_ICON, favicon.icon_type);
-  EXPECT_EQ(icon_url, favicon.icon_url);
+  EXPECT_EQ(TOUCH_ICON, bitmap_result.icon_type);
+  EXPECT_EQ(icon_url, bitmap_result.icon_url);
   EXPECT_EQ(blob_data, touchicon_data);
 }
 
@@ -1267,20 +1273,23 @@ TEST_F(HistoryBackendTest, CloneFaviconIsRestrictedToSameDomain) {
       url, FAVICON, NULL));
 
   // Validate starting state.
-  FaviconData favicon;
-  EXPECT_TRUE(backend_->GetFaviconFromDB(url, FAVICON, &favicon));
-  EXPECT_FALSE(backend_->GetFaviconFromDB(same_domain_url, FAVICON, &favicon));
-  EXPECT_FALSE(backend_->GetFaviconFromDB(foreign_domain_url,
-                                          FAVICON, &favicon));
+  std::vector<FaviconBitmapResult> favicon_bitmap_results;
+  EXPECT_TRUE(backend_->GetFaviconFromDB(url, FAVICON,
+                                         &favicon_bitmap_results));
+  EXPECT_FALSE(backend_->GetFaviconFromDB(same_domain_url, FAVICON,
+                                          &favicon_bitmap_results));
+  EXPECT_FALSE(backend_->GetFaviconFromDB(foreign_domain_url, FAVICON,
+                                          &favicon_bitmap_results));
 
   // Same-domain cloning should work.
   backend_->CloneFavicon(url, same_domain_url);
-  EXPECT_TRUE(backend_->GetFaviconFromDB(same_domain_url, FAVICON, &favicon));
+  EXPECT_TRUE(backend_->GetFaviconFromDB(same_domain_url, FAVICON,
+                                         &favicon_bitmap_results));
 
   // Foreign-domain cloning is forbidden.
   backend_->CloneFavicon(url, foreign_domain_url);
-  EXPECT_FALSE(backend_->GetFaviconFromDB(foreign_domain_url,
-                                          FAVICON, &favicon));
+  EXPECT_FALSE(backend_->GetFaviconFromDB(foreign_domain_url, FAVICON,
+                                          &favicon_bitmap_results));
 }
 
 TEST_F(HistoryBackendTest, QueryFilteredURLs) {