Index: components/history/core/browser/history_backend_unittest.cc |
diff --git a/components/history/core/browser/history_backend_unittest.cc b/components/history/core/browser/history_backend_unittest.cc |
index 63dfe4b73094472f69d092164756c5ebca68bf4b..8c5b37bb16440ba8d4d4e184e54534c6cd24ab34 100644 |
--- a/components/history/core/browser/history_backend_unittest.cc |
+++ b/components/history/core/browser/history_backend_unittest.cc |
@@ -593,18 +593,22 @@ TEST_F(HistoryBackendTest, DeleteAll) { |
std::vector<unsigned char> data; |
data.push_back('a'); |
- EXPECT_TRUE(backend_->thumbnail_db_->AddFaviconBitmap(favicon1, |
- new base::RefCountedBytes(data), base::Time::Now(), kSmallSize)); |
+ EXPECT_TRUE(backend_->thumbnail_db_->AddFaviconBitmap( |
+ favicon1, new base::RefCountedBytes(data), FaviconBitmapType::ON_VISIT, |
+ base::Time::Now(), kSmallSize)); |
data[0] = 'b'; |
- EXPECT_TRUE(backend_->thumbnail_db_->AddFaviconBitmap(favicon1, |
- new base::RefCountedBytes(data), base::Time::Now(), kLargeSize)); |
+ EXPECT_TRUE(backend_->thumbnail_db_->AddFaviconBitmap( |
+ favicon1, new base::RefCountedBytes(data), FaviconBitmapType::ON_VISIT, |
+ base::Time::Now(), kLargeSize)); |
data[0] = 'c'; |
- EXPECT_TRUE(backend_->thumbnail_db_->AddFaviconBitmap(favicon2, |
- new base::RefCountedBytes(data), base::Time::Now(), kSmallSize)); |
+ EXPECT_TRUE(backend_->thumbnail_db_->AddFaviconBitmap( |
+ favicon2, new base::RefCountedBytes(data), FaviconBitmapType::ON_VISIT, |
+ base::Time::Now(), kSmallSize)); |
data[0] = 'd'; |
- EXPECT_TRUE(backend_->thumbnail_db_->AddFaviconBitmap(favicon2, |
- new base::RefCountedBytes(data), base::Time::Now(), kLargeSize)); |
+ EXPECT_TRUE(backend_->thumbnail_db_->AddFaviconBitmap( |
+ favicon2, new base::RefCountedBytes(data), FaviconBitmapType::ON_VISIT, |
+ base::Time::Now(), kLargeSize)); |
// First visit two URLs. |
URLRow row1(GURL("http://www.google.com/")); |
@@ -2076,18 +2080,18 @@ TEST_F(HistoryBackendTest, SetFaviconsSameFaviconURLForTwoPages) { |
EXPECT_EQ(2u, favicon_bitmaps.size()); |
} |
-// Tests calling SetLastResortFavicons(). Neither |page_url| nor |icon_url| are |
+// Tests calling SetOnDemandFavicons(). Neither |page_url| nor |icon_url| are |
// known to the database. |
-TEST_F(HistoryBackendTest, SetLastResortFaviconsForEmptyDB) { |
+TEST_F(HistoryBackendTest, SetOnDemandFaviconsForEmptyDB) { |
GURL page_url("http://www.google.com"); |
GURL icon_url("http:/www.google.com/favicon.ico"); |
std::vector<SkBitmap> bitmaps; |
bitmaps.push_back(CreateBitmap(SK_ColorRED, kSmallEdgeSize)); |
- // Call SetLastResortFavicons() with a different icon URL and bitmap data. |
- EXPECT_TRUE(backend_->SetLastResortFavicons(page_url, favicon_base::FAVICON, |
- icon_url, bitmaps)); |
+ // Call SetOnDemandFavicons() with a different icon URL and bitmap data. |
+ EXPECT_TRUE(backend_->SetOnDemandFavicons(page_url, favicon_base::FAVICON, |
+ icon_url, bitmaps)); |
favicon_base::FaviconID favicon_id = |
backend_->thumbnail_db_->GetFaviconIDForFaviconURL(icon_url, |
@@ -2102,9 +2106,9 @@ TEST_F(HistoryBackendTest, SetLastResortFaviconsForEmptyDB) { |
EXPECT_EQ(base::Time(), favicon_bitmap.last_updated); |
} |
-// Tests calling SetLastResortFavicons(). |page_url| is known to the database |
+// Tests calling SetOnDemandFavicons(). |page_url| is known to the database |
// but |icon_url| is not (the second should be irrelevant though). |
-TEST_F(HistoryBackendTest, SetLastResortFaviconsForPageInDB) { |
+TEST_F(HistoryBackendTest, SetOnDemandFaviconsForPageInDB) { |
GURL page_url("http://www.google.com"); |
GURL icon_url1("http:/www.google.com/favicon1.ico"); |
GURL icon_url2("http:/www.google.com/favicon2.ico"); |
@@ -2118,10 +2122,10 @@ TEST_F(HistoryBackendTest, SetLastResortFaviconsForPageInDB) { |
favicon_base::FAVICON); |
ASSERT_NE(0, original_favicon_id); |
- // Call SetLastResortFavicons() with a different icon URL and bitmap data. |
+ // Call SetOnDemandFavicons() with a different icon URL and bitmap data. |
bitmaps[0] = CreateBitmap(SK_ColorWHITE, kSmallEdgeSize); |
- EXPECT_FALSE(backend_->SetLastResortFavicons(page_url, favicon_base::FAVICON, |
- icon_url2, bitmaps)); |
+ EXPECT_FALSE(backend_->SetOnDemandFavicons(page_url, favicon_base::FAVICON, |
+ icon_url2, bitmaps)); |
EXPECT_EQ(0, backend_->thumbnail_db_->GetFaviconIDForFaviconURL( |
icon_url2, favicon_base::FAVICON)); |
@@ -2133,9 +2137,9 @@ TEST_F(HistoryBackendTest, SetLastResortFaviconsForPageInDB) { |
EXPECT_NE(base::Time(), favicon_bitmap.last_updated); |
} |
-// Tests calling SetLastResortFavicons(). |page_url| is not known to the |
+// Tests calling SetOnDemandFavicons(). |page_url| is not known to the |
// database but |icon_url| is. |
-TEST_F(HistoryBackendTest, SetLastResortFaviconsForIconInDB) { |
+TEST_F(HistoryBackendTest, SetOnDemandFaviconsForIconInDB) { |
const GURL old_page_url("http://www.google.com/old"); |
const GURL page_url("http://www.google.com/"); |
const GURL icon_url("http://www.google.com/icon"); |
@@ -2149,10 +2153,10 @@ TEST_F(HistoryBackendTest, SetLastResortFaviconsForIconInDB) { |
favicon_base::FAVICON); |
ASSERT_NE(0, original_favicon_id); |
- // Call SetLastResortFavicons() with a different bitmap. |
+ // Call SetOnDemandFavicons() with a different bitmap. |
bitmaps[0] = CreateBitmap(SK_ColorWHITE, kSmallEdgeSize); |
- EXPECT_FALSE(backend_->SetLastResortFavicons(page_url, favicon_base::FAVICON, |
- icon_url, bitmaps)); |
+ EXPECT_FALSE(backend_->SetOnDemandFavicons(page_url, favicon_base::FAVICON, |
+ icon_url, bitmaps)); |
EXPECT_EQ(original_favicon_id, |
backend_->thumbnail_db_->GetFaviconIDForFaviconURL( |