Index: chrome/browser/history/top_sites_database.cc |
diff --git a/chrome/browser/history/top_sites_database.cc b/chrome/browser/history/top_sites_database.cc |
index d563fa3a0edd526dfd2597269ba03aec06f8632a..99f0bb44ec277662bed13040ccc4ad6eb52071f7 100644 |
--- a/chrome/browser/history/top_sites_database.cc |
+++ b/chrome/browser/history/top_sites_database.cc |
@@ -194,7 +194,7 @@ void TopSitesDatabaseImpl::UpdatePageRankNoTransaction( |
const MostVisitedURL& url, int new_rank) { |
int prev_rank = GetURLRank(url); |
if (prev_rank == -1) { |
- NOTREACHED() << "Updating rank of an unknown URL."; |
+ NOTREACHED() << "Updating rank of an unknown URL: " << url.url.spec(); |
return; |
} |
@@ -263,7 +263,7 @@ bool TopSitesDatabaseImpl::GetPageThumbnail(const GURL& url, |
} |
int TopSitesDatabaseImpl::GetRowCount() { |
- int result = -1; |
+ int result = 0; |
sql::Statement select_statement(db_.GetCachedStatement( |
SQL_FROM_HERE, |
"SELECT COUNT (url) FROM thumbnails")); |
@@ -299,6 +299,9 @@ int TopSitesDatabaseImpl::GetURLRank(const MostVisitedURL& url) { |
// Remove the record for this URL. Returns true iff removed successfully. |
bool TopSitesDatabaseImpl::RemoveURL(const MostVisitedURL& url) { |
int old_rank = GetURLRank(url); |
+ if (old_rank < 0) |
+ return false; |
+ |
sql::Transaction transaction(&db_); |
transaction.Begin(); |
// Decrement all following ranks. |