fav icon -> favicon. Pass 5: fav_icon -> favicon

chrome/browser/history/expire_history_backend_unittest.cc

 // Copyright (c) 2010 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 <string>
 #include <utility>
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/file_path.h"
@@ skipping 183 matching lines @@
 
   // Four times for each visit.
   visit_times[3] = Time::Now();
   visit_times[2] = visit_times[3] - TimeDelta::FromDays(1);
   visit_times[1] = visit_times[3] - TimeDelta::FromDays(2);
   visit_times[0] = visit_times[3] - TimeDelta::FromDays(3);
 
   // Two favicons. The first two URLs will share the same one, while the last
   // one will have a unique favicon.
   FavIconID favicon1 = thumb_db_->AddFavIcon(GURL("http://favicon/url1"),
-                                             FAV_ICON);
+                                             FAVICON);
   FavIconID favicon2 = thumb_db_->AddFavIcon(GURL("http://favicon/url2"),
-                                             FAV_ICON);
+                                             FAVICON);
 
   // Three URLs.
   URLRow url_row1(GURL("http://www.google.com/1"));
   url_row1.set_last_visit(visit_times[0]);
   url_row1.set_visit_count(1);
   url_ids[0] = main_db_->AddURL(url_row1);
   thumb_db_->AddIconMapping(url_row1.url(), favicon1);
 
   URLRow url_row2(GURL("http://www.google.com/2"));
   url_row2.set_last_visit(visit_times[2]);
@@ skipping 189 matching lines @@
           found_typed_changed_notification = true;
     }
   }
   EXPECT_TRUE(found_delete_notification);
   EXPECT_EQ(row.typed_count() > 0, found_typed_changed_notification);
 }
 
 TEST_F(ExpireHistoryTest, DeleteFaviconsIfPossible) {
   // Add a favicon record.
   const GURL favicon_url("http://www.google.com/favicon.ico");
-  FavIconID icon_id = thumb_db_->AddFavIcon(favicon_url, FAV_ICON);
+  FavIconID icon_id = thumb_db_->AddFavIcon(favicon_url, FAVICON);
   EXPECT_TRUE(icon_id);
   EXPECT_TRUE(HasFavIcon(icon_id));
 
   // The favicon should be deletable with no users.
   std::set<FavIconID> favicon_set;
   favicon_set.insert(icon_id);
   expirer_.DeleteFaviconsIfPossible(favicon_set);
   EXPECT_FALSE(HasFavIcon(icon_id));
 
   // Add back the favicon.
@@ skipping 26 matching lines @@
 // should also get deleted.
 // Fails near end of month. http://crbug.com/43586
 TEST_F(ExpireHistoryTest, FLAKY_DeleteURLAndFavicon) {
   URLID url_ids[3];
   Time visit_times[4];
   AddExampleData(url_ids, visit_times);
 
   // Verify things are the way we expect with a URL row, favicon, thumbnail.
   URLRow last_row;
   ASSERT_TRUE(main_db_->GetURLRow(url_ids[2], &last_row));
-  FavIconID fav_icon_id = GetFavicon(last_row.url(), FAV_ICON);
-  EXPECT_TRUE(HasFavIcon(fav_icon_id));
+  FavIconID favicon_id = GetFavicon(last_row.url(), FAVICON);
+  EXPECT_TRUE(HasFavIcon(favicon_id));
   // TODO(sky): fix this, see comment in HasThumbnail.
   // EXPECT_TRUE(HasThumbnail(url_ids[2]));
 
   VisitVector visits;
   main_db_->GetVisitsForURL(url_ids[2], &visits);
   ASSERT_EQ(1U, visits.size());
   EXPECT_EQ(1, CountTextMatchesForURL(last_row.url()));
 
   // In this test we also make sure that any pending entries in the text
   // database manager are removed.
@@ skipping 28 matching lines @@
                         text_db_.get());
 
   // Run the text database expirer. This will flush any pending entries so we
   // can check that nothing was committed. We use a time far in the future so
   // that anything added recently will get flushed.
   TimeTicks expiration_time = TimeTicks::Now() + TimeDelta::FromDays(1);
   text_db_->FlushOldChangesForTime(expiration_time);
 
   // All the normal data + the favicon should be gone.
   EnsureURLInfoGone(last_row);
-  EXPECT_FALSE(GetFavicon(last_row.url(), FAV_ICON));
-  EXPECT_FALSE(HasFavIcon(fav_icon_id));
+  EXPECT_FALSE(GetFavicon(last_row.url(), FAVICON));
+  EXPECT_FALSE(HasFavIcon(favicon_id));
 }
 
 // Deletes a URL with a favicon that other URLs reference, so that the favicon
 // should not get deleted. This also tests deleting more than one visit.
 TEST_F(ExpireHistoryTest, DeleteURLWithoutFavicon) {
   URLID url_ids[3];
   Time visit_times[4];
   AddExampleData(url_ids, visit_times);
 
   // Verify things are the way we expect with a URL row, favicon, thumbnail.
   URLRow last_row;
   ASSERT_TRUE(main_db_->GetURLRow(url_ids[1], &last_row));
-  FavIconID fav_icon_id = GetFavicon(last_row.url(), FAV_ICON);
-  EXPECT_TRUE(HasFavIcon(fav_icon_id));
+  FavIconID favicon_id = GetFavicon(last_row.url(), FAVICON);
+  EXPECT_TRUE(HasFavIcon(favicon_id));
   // TODO(sky): fix this, see comment in HasThumbnail.
   // EXPECT_TRUE(HasThumbnail(url_ids[1]));
 
   VisitVector visits;
   main_db_->GetVisitsForURL(url_ids[1], &visits);
   EXPECT_EQ(2U, visits.size());
   EXPECT_EQ(1, CountTextMatchesForURL(last_row.url()));
 
   // Delete the URL and its dependencies.
   expirer_.DeleteURL(last_row.url());
 
   // All the normal data + the favicon should be gone.
   EnsureURLInfoGone(last_row);
-  EXPECT_TRUE(HasFavIcon(fav_icon_id));
+  EXPECT_TRUE(HasFavIcon(favicon_id));
 }
 
 // DeleteURL should not delete starred urls.
 TEST_F(ExpireHistoryTest, DontDeleteStarredURL) {
   URLID url_ids[3];
   Time visit_times[4];
   AddExampleData(url_ids, visit_times);
 
   URLRow url_row;
   ASSERT_TRUE(main_db_->GetURLRow(url_ids[2], &url_row));
 
   // Star the last URL.
   StarURL(url_row.url());
 
   // Attempt to delete the url.
   expirer_.DeleteURL(url_row.url());
 
   // Because the url is starred, it shouldn't be deleted.
   GURL url = url_row.url();
   ASSERT_TRUE(main_db_->GetRowForURL(url, &url_row));
 
   // And the favicon should exist.
-  FavIconID fav_icon_id = GetFavicon(url_row.url(), FAV_ICON);
-  EXPECT_TRUE(HasFavIcon(fav_icon_id));
+  FavIconID favicon_id = GetFavicon(url_row.url(), FAVICON);
+  EXPECT_TRUE(HasFavIcon(favicon_id));
 
   // But there should be no fts.
   ASSERT_EQ(0, CountTextMatchesForURL(url_row.url()));
 
   // And no visits.
   VisitVector visits;
   main_db_->GetVisitsForURL(url_row.id(), &visits);
   ASSERT_EQ(0U, visits.size());
 
   // Should still have the thumbnail.
@@ skipping 48 matching lines @@
   URLRow temp_row;
   ASSERT_TRUE(main_db_->GetURLRow(url_ids[1], &temp_row));
   EXPECT_TRUE(visit_times[2] == url_row1.last_visit());  // Previous value.
   EXPECT_TRUE(visit_times[1] == temp_row.last_visit());  // New value.
   EXPECT_EQ(2, url_row1.visit_count());
   EXPECT_EQ(1, temp_row.visit_count());
   EXPECT_EQ(1, url_row1.typed_count());
   EXPECT_EQ(0, temp_row.typed_count());
 
   // Verify that the middle URL's favicon and thumbnail is still there.
-  FavIconID fav_icon_id = GetFavicon(url_row1.url(), FAV_ICON);
-  EXPECT_TRUE(HasFavIcon(fav_icon_id));
+  FavIconID favicon_id = GetFavicon(url_row1.url(), FAVICON);
+  EXPECT_TRUE(HasFavIcon(favicon_id));
   // TODO(sky): fix this, see comment in HasThumbnail.
   // EXPECT_TRUE(HasThumbnail(url_row1.id()));
 
   // Verify that the last URL was deleted.
-  FavIconID fav_icon_id2 = GetFavicon(url_row2.url(), FAV_ICON);
+  FavIconID favicon_id2 = GetFavicon(url_row2.url(), FAVICON);
   EnsureURLInfoGone(url_row2);
-  EXPECT_FALSE(HasFavIcon(fav_icon_id2));
+  EXPECT_FALSE(HasFavIcon(favicon_id2));
 }
 
 // Expires only a specific URLs more recent than a given time, with no starred
 // items.  Our time threshold is such that the URL should be updated (we delete
 // one of the two visits).
 TEST_F(ExpireHistoryTest, FlushRecentURLsUnstarredRestricted) {
   URLID url_ids[3];
   Time visit_times[4];
   AddExampleData(url_ids, visit_times);
 
@@ skipping 30 matching lines @@
   URLRow temp_row;
   ASSERT_TRUE(main_db_->GetURLRow(url_ids[1], &temp_row));
   EXPECT_TRUE(visit_times[2] == url_row1.last_visit());  // Previous value.
   EXPECT_TRUE(visit_times[1] == temp_row.last_visit());  // New value.
   EXPECT_EQ(2, url_row1.visit_count());
   EXPECT_EQ(1, temp_row.visit_count());
   EXPECT_EQ(1, url_row1.typed_count());
   EXPECT_EQ(0, temp_row.typed_count());
 
   // Verify that the middle URL's favicon and thumbnail is still there.
-  FavIconID fav_icon_id = GetFavicon(url_row1.url(), FAV_ICON);
-  EXPECT_TRUE(HasFavIcon(fav_icon_id));
+  FavIconID favicon_id = GetFavicon(url_row1.url(), FAVICON);
+  EXPECT_TRUE(HasFavIcon(favicon_id));
   // TODO(sky): fix this, see comment in HasThumbnail.
   // EXPECT_TRUE(HasThumbnail(url_row1.id()));
 
   // Verify that the last URL was not touched.
   EXPECT_TRUE(main_db_->GetURLRow(url_ids[2], &temp_row));
-  EXPECT_TRUE(HasFavIcon(fav_icon_id));
+  EXPECT_TRUE(HasFavIcon(favicon_id));
   // TODO(sky): fix this, see comment in HasThumbnail.
   // EXPECT_TRUE(HasThumbnail(url_row2.id()));
 }
 
 // Expire a starred URL, it shouldn't get deleted
 TEST_F(ExpireHistoryTest, FlushRecentURLsStarred) {
   URLID url_ids[3];
   Time visit_times[4];
   AddExampleData(url_ids, visit_times);
 
@@ skipping 21 matching lines @@
   // Visit/typed count should not be updated for bookmarks.
   EXPECT_EQ(0, new_url_row1.typed_count());
   EXPECT_EQ(1, new_url_row1.visit_count());
   EXPECT_EQ(0, new_url_row2.typed_count());
   EXPECT_EQ(0, new_url_row2.visit_count());
 
   // Thumbnails and favicons should still exist. Note that we keep thumbnails
   // that may have been updated since the time threshold. Since the URL still
   // exists in history, this should not be a privacy problem, we only update
   // the visit counts in this case for consistency anyway.
-  FavIconID fav_icon_id = GetFavicon(url_row1.url(), FAV_ICON);
-  EXPECT_TRUE(HasFavIcon(fav_icon_id));
+  FavIconID favicon_id = GetFavicon(url_row1.url(), FAVICON);
+  EXPECT_TRUE(HasFavIcon(favicon_id));
   // TODO(sky): fix this, see comment in HasThumbnail.
   // EXPECT_TRUE(HasThumbnail(new_url_row1.id()));
-  fav_icon_id = GetFavicon(url_row1.url(), FAV_ICON);
-  EXPECT_TRUE(HasFavIcon(fav_icon_id));
+  favicon_id = GetFavicon(url_row1.url(), FAVICON);
+  EXPECT_TRUE(HasFavIcon(favicon_id));
   // TODO(sky): fix this, see comment in HasThumbnail.
   // EXPECT_TRUE(HasThumbnail(new_url_row2.id()));
 }
 
 TEST_F(ExpireHistoryTest, ArchiveHistoryBeforeUnstarred) {
   URLID url_ids[3];
   Time visit_times[4];
   AddExampleData(url_ids, visit_times);
 
   URLRow url_row1, url_row2;
@@ skipping 169 matching lines @@
   main_db_->GetVisitsForURL(url_id, &archived_visits);
   EXPECT_EQ(0U, archived_visits.size());
 }
 
 // TODO(brettw) add some visits with no URL to make sure everything is updated
 // properly. Have the visits also refer to nonexistent FTS rows.
 //
 // Maybe also refer to invalid favicons.
 
 }  // namespace history