Componentize TopSites, TopSitesBackend, TopSitesDatabase

components/history/core/browser/top_sites_database.cc

 // Copyright (c) 2012 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 "chrome/browser/history/top_sites_database.h"
+#include "components/history/core/browser/top_sites_database.h"
 
 #include "base/files/file_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/metrics/histogram.h"
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
-#include "chrome/browser/history/top_sites.h"
 #include "components/history/core/browser/history_types.h"
+#include "components/history/core/browser/top_sites.h"
 #include "components/history/core/common/thumbnail_score.h"
 #include "sql/connection.h"
 #include "sql/recovery.h"
 #include "sql/statement.h"
 #include "sql/transaction.h"
 #include "third_party/sqlite/sqlite3.h"
 
 // Description of database table:
 //
 // thumbnails
@@ skipping 105 matching lines @@
   // Track invariants resolved by FixThumbnailsTable().
   RECOVERY_EVENT_INVARIANT_RANK,
   RECOVERY_EVENT_INVARIANT_REDIRECT,
   RECOVERY_EVENT_INVARIANT_CONTIGUOUS,
 
   // Always keep this at the end.
   RECOVERY_EVENT_MAX,
 };
 
 void RecordRecoveryEvent(RecoveryEventType recovery_event) {
-  UMA_HISTOGRAM_ENUMERATION("History.TopSitesRecovery",
-                            recovery_event, RECOVERY_EVENT_MAX);
+  UMA_HISTOGRAM_ENUMERATION("History.TopSitesRecovery", recovery_event,
+                            RECOVERY_EVENT_MAX);
 }
 
 // Most corruption comes down to atomic updates between pages being broken
 // somehow.  This can result in either missing data, or overlapping data,
 // depending on the operation broken.  This table has large rows, which will use
 // overflow pages, so it is possible (though unlikely) that a chain could fit
 // together and yield a row with errors.
 void FixThumbnailsTable(sql::Connection* db) {
   // Enforce invariant separating forced and non-forced thumbnails.
   const char kFixRankSql[] =
@@ skipping 137 matching lines @@
     // insight.
     RecordRecoveryEvent(RECOVERY_EVENT_FAILED_COMMIT);
     return;
   }
 
   // Track the size of the recovered database relative to the size of the input
   // database.  The size should almost always be smaller, unless the input
   // database was empty to start with.  If the percentage results are very low,
   // something is awry.
   int64 final_size = 0;
-  if (original_size > 0 &&
-      base::GetFileSize(db_path, &final_size) &&
+  if (original_size > 0 && base::GetFileSize(db_path, &final_size) &&
       final_size > 0) {
     UMA_HISTOGRAM_PERCENTAGE("History.TopSitesRecoveredPercentage",
                              final_size * 100 / original_size);
   }
 
   // Using 10,000 because these cases mostly care about "none recovered" and
   // "lots recovered".  More than 10,000 rows recovered probably means there's
   // something wrong with the profile.
   UMA_HISTOGRAM_COUNTS_10000("History.TopSitesRecoveredRowsThumbnails",
                              thumbnails_recovered);
@@ skipping 182 matching lines @@
   if (rank == kRankOfNonExistingURL) {
     AddPageThumbnail(url, new_rank, thumbnail);
   } else {
     UpdatePageRankNoTransaction(url, new_rank);
     UpdatePageThumbnail(url, thumbnail);
   }
 
   transaction.Commit();
 }
 
-bool TopSitesDatabase::UpdatePageThumbnail(
-    const MostVisitedURL& url, const Images& thumbnail) {
+bool TopSitesDatabase::UpdatePageThumbnail(const MostVisitedURL& url,
+                                           const Images& thumbnail) {
   sql::Statement statement(db_->GetCachedStatement(
       SQL_FROM_HERE,
       "UPDATE thumbnails SET "
       "title = ?, thumbnail = ?, redirects = ?, "
       "boring_score = ?, good_clipping = ?, at_top = ?, last_updated = ?, "
       "load_completed = ?, last_forced = ?"
       "WHERE url = ? "));
   statement.BindString16(0, url.title);
   if (thumbnail.thumbnail.get() && thumbnail.thumbnail->front()) {
     statement.BindBlob(1, thumbnail.thumbnail->front(),
@@ skipping 41 matching lines @@
       << "opposite.";
   statement.BindInt64(10, last_forced);
   if (!statement.Run())
     return;
 
   // Update rank if this is not a forced thumbnail.
   if (new_rank != kRankOfForcedURL)
     UpdatePageRankNoTransaction(url, new_rank);
 }
 
-void TopSitesDatabase::UpdatePageRank(const MostVisitedURL& url,
-                                      int new_rank) {
+void TopSitesDatabase::UpdatePageRank(const MostVisitedURL& url, int new_rank) {
   DCHECK((url.last_forced_time.ToInternalValue() == 0) ==
          (new_rank != kRankOfForcedURL))
       << "Thumbnail without a forced time stamp has a forced rank, or the "
       << "opposite.";
   sql::Transaction transaction(db_.get());
   transaction.Begin();
   UpdatePageRankNoTransaction(url, new_rank);
   transaction.Commit();
 }
 
 // Caller should have a transaction open.
-void TopSitesDatabase::UpdatePageRankNoTransaction(
-    const MostVisitedURL& url, int new_rank) {
+void TopSitesDatabase::UpdatePageRankNoTransaction(const MostVisitedURL& url,
+                                                   int new_rank) {
   DCHECK_GT(db_->transaction_nesting(), 0);
   DCHECK((url.last_forced_time.is_null()) == (new_rank != kRankOfForcedURL))
       << "Thumbnail without a forced time stamp has a forced rank, or the "
       << "opposite.";
 
   int prev_rank = GetURLRank(url);
   if (prev_rank == kRankOfNonExistingURL) {
     LOG(WARNING) << "Updating rank of an unknown URL: " << url.url.spec();
     return;
   }
@@ skipping 57 matching lines @@
       SQL_FROM_HERE,
       "UPDATE thumbnails "
       "SET url_rank = ?, last_forced = ? "
       "WHERE url == ?"));
   set_statement.BindInt(0, new_rank);
   set_statement.BindInt64(1, url.last_forced_time.ToInternalValue());
   set_statement.BindString(2, url.url.spec());
   set_statement.Run();
 }
 
-bool TopSitesDatabase::GetPageThumbnail(const GURL& url,
-                                        Images* thumbnail) {
+bool TopSitesDatabase::GetPageThumbnail(const GURL& url, Images* thumbnail) {
   sql::Statement statement(db_->GetCachedStatement(
       SQL_FROM_HERE,
       "SELECT thumbnail, boring_score, good_clipping, at_top, last_updated "
       "FROM thumbnails WHERE url=?"));
   statement.BindString(0, url.spec());
   if (!statement.Step())
     return false;
 
   std::vector<unsigned char> data;
   statement.ColumnBlobAsVector(0, &data);
@@ skipping 47 matching lines @@
   if (!delete_statement.Run())
     return false;
 
   return transaction.Commit();
 }
 
 sql::Connection* TopSitesDatabase::CreateDB(const base::FilePath& db_name) {
   scoped_ptr<sql::Connection> db(new sql::Connection());
   // Settings copied from ThumbnailDatabase.
   db->set_histogram_tag("TopSites");
-  db->set_error_callback(base::Bind(&DatabaseErrorCallback,
-                                    db.get(), db_name));
+  db->set_error_callback(base::Bind(&DatabaseErrorCallback, db.get(), db_name));
   db->set_page_size(4096);
   db->set_cache_size(32);
 
   if (!db->Open(db_name))
     return NULL;
   return db.release();
 }
 
 }  // namespace history