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

Unified Diff: chrome/browser/history/expire_history_backend_unittest.cc

Issue 6698031: fav icon -> favicon. Pass 7: FavIconID -> FaviconID (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/history/expire_history_backend.cc ('k') | chrome/browser/history/history_backend.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/history/expire_history_backend_unittest.cc
diff --git a/chrome/browser/history/expire_history_backend_unittest.cc b/chrome/browser/history/expire_history_backend_unittest.cc
index f08ed8b900c790845731e4fca8493afb5cacffbf..96602c1abfb7c36acb90b1ef11abd94cadf18256 100644
--- a/chrome/browser/history/expire_history_backend_unittest.cc
+++ b/chrome/browser/history/expire_history_backend_unittest.cc
@@ -65,10 +65,10 @@ class ExpireHistoryTest : public testing::Test,
void AddExampleSourceData(const GURL& url, URLID* id);
// Returns true if the given favicon/thumanil has an entry in the DB.
- bool HasFavIcon(FavIconID favicon_id);
+ bool HasFavIcon(FaviconID favicon_id);
bool HasThumbnail(URLID url_id);
- FavIconID GetFavicon(const GURL& page_url, IconType icon_type);
+ FaviconID GetFavicon(const GURL& page_url, IconType icon_type);
// Returns the number of text matches for the given URL in the example data
// added by AddExampleData.
@@ -200,9 +200,9 @@ void ExpireHistoryTest::AddExampleData(URLID url_ids[3], Time visit_times[4]) {
// 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"),
+ FaviconID favicon1 = thumb_db_->AddFavIcon(GURL("http://favicon/url1"),
FAV_ICON);
- FavIconID favicon2 = thumb_db_->AddFavIcon(GURL("http://favicon/url2"),
+ FaviconID favicon2 = thumb_db_->AddFavIcon(GURL("http://favicon/url2"),
FAV_ICON);
// Three URLs.
@@ -310,7 +310,7 @@ void ExpireHistoryTest::AddExampleSourceData(const GURL& url, URLID* id) {
main_db_->AddVisit(&visit_row4, SOURCE_FIREFOX_IMPORTED);
}
-bool ExpireHistoryTest::HasFavIcon(FavIconID favicon_id) {
+bool ExpireHistoryTest::HasFavIcon(FaviconID favicon_id) {
if (!thumb_db_.get() || favicon_id == 0)
return false;
Time last_updated;
@@ -320,7 +320,7 @@ bool ExpireHistoryTest::HasFavIcon(FavIconID favicon_id) {
&icon_url);
}
-FavIconID ExpireHistoryTest::GetFavicon(const GURL& page_url,
+FaviconID ExpireHistoryTest::GetFavicon(const GURL& page_url,
IconType icon_type) {
IconMapping icon_mapping;
thumb_db_->GetIconMappingForPageURL(page_url, icon_type, &icon_mapping);
@@ -413,12 +413,12 @@ void ExpireHistoryTest::EnsureURLInfoGone(const URLRow& row) {
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, FAV_ICON);
EXPECT_TRUE(icon_id);
EXPECT_TRUE(HasFavIcon(icon_id));
// The favicon should be deletable with no users.
- std::set<FavIconID> favicon_set;
+ std::set<FaviconID> favicon_set;
favicon_set.insert(icon_id);
expirer_.DeleteFaviconsIfPossible(favicon_set);
EXPECT_FALSE(HasFavIcon(icon_id));
@@ -460,7 +460,7 @@ TEST_F(ExpireHistoryTest, FLAKY_DeleteURLAndFavicon) {
// 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);
+ FaviconID fav_icon_id = GetFavicon(last_row.url(), FAV_ICON);
EXPECT_TRUE(HasFavIcon(fav_icon_id));
// TODO(sky): fix this, see comment in HasThumbnail.
// EXPECT_TRUE(HasThumbnail(url_ids[2]));
@@ -524,7 +524,7 @@ TEST_F(ExpireHistoryTest, DeleteURLWithoutFavicon) {
// 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);
+ FaviconID fav_icon_id = GetFavicon(last_row.url(), FAV_ICON);
EXPECT_TRUE(HasFavIcon(fav_icon_id));
// TODO(sky): fix this, see comment in HasThumbnail.
// EXPECT_TRUE(HasThumbnail(url_ids[1]));
@@ -562,7 +562,7 @@ TEST_F(ExpireHistoryTest, DontDeleteStarredURL) {
ASSERT_TRUE(main_db_->GetRowForURL(url, &url_row));
// And the favicon should exist.
- FavIconID fav_icon_id = GetFavicon(url_row.url(), FAV_ICON);
+ FaviconID fav_icon_id = GetFavicon(url_row.url(), FAV_ICON);
EXPECT_TRUE(HasFavIcon(fav_icon_id));
// But there should be no fts.
@@ -632,13 +632,13 @@ TEST_F(ExpireHistoryTest, FlushRecentURLsUnstarred) {
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);
+ FaviconID fav_icon_id = GetFavicon(url_row1.url(), FAV_ICON);
EXPECT_TRUE(HasFavIcon(fav_icon_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 fav_icon_id2 = GetFavicon(url_row2.url(), FAV_ICON);
EnsureURLInfoGone(url_row2);
EXPECT_FALSE(HasFavIcon(fav_icon_id2));
}
@@ -691,7 +691,7 @@ TEST_F(ExpireHistoryTest, FlushRecentURLsUnstarredRestricted) {
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);
+ FaviconID fav_icon_id = GetFavicon(url_row1.url(), FAV_ICON);
EXPECT_TRUE(HasFavIcon(fav_icon_id));
// TODO(sky): fix this, see comment in HasThumbnail.
// EXPECT_TRUE(HasThumbnail(url_row1.id()));
@@ -740,7 +740,7 @@ TEST_F(ExpireHistoryTest, FlushRecentURLsStarred) {
// 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);
+ FaviconID fav_icon_id = GetFavicon(url_row1.url(), FAV_ICON);
EXPECT_TRUE(HasFavIcon(fav_icon_id));
// TODO(sky): fix this, see comment in HasThumbnail.
// EXPECT_TRUE(HasThumbnail(new_url_row1.id()));
« no previous file with comments | « chrome/browser/history/expire_history_backend.cc ('k') | chrome/browser/history/history_backend.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698