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

Unified Diff: components/history/core/browser/history_backend.cc

Issue 1311403011: Removed archived history file deletion on browser start (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added obsolete tag to archived history histograms Created 5 years, 3 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
Index: components/history/core/browser/history_backend.cc
diff --git a/components/history/core/browser/history_backend.cc b/components/history/core/browser/history_backend.cc
index e5b0d1af9c9cd5edbaffd67714873b01341e1be2..ad61094c5702e56dddce0ef178a7d0071ac5969f 100644
--- a/components/history/core/browser/history_backend.cc
+++ b/components/history/core/browser/history_backend.cc
@@ -289,10 +289,6 @@ base::FilePath HistoryBackend::GetFaviconsFileName() const {
return history_dir_.Append(kFaviconsFilename);
}
-base::FilePath HistoryBackend::GetArchivedFileName() const {
- return history_dir_.Append(kArchivedHistoryFilename);
-}
-
SegmentID HistoryBackend::GetLastSegmentID(VisitID from_visit) {
// Set is used to detect referrer loops. Should not happen, but can
// if the database is corrupt.
@@ -618,7 +614,6 @@ void HistoryBackend::InitImpl(
history_dir_ = history_database_params.history_dir;
base::FilePath history_name = history_dir_.Append(kHistoryFilename);
base::FilePath thumbnail_name = GetFaviconsFileName();
- base::FilePath archived_name = GetArchivedFileName();
// Delete the old index database files which are no longer used.
DeleteFTSIndexDatabases();
@@ -682,12 +677,6 @@ void HistoryBackend::InitImpl(
thumbnail_db_.reset();
}
- // Nuke any files corresponding to the legacy Archived History Database, which
- // previously retained expired (> 3 months old) history entries, but, in the
- // end, was not used for much, and consequently has been removed as of M37.
- // TODO(engedy): Remove this code after the end of 2014.
- sql::Connection::Delete(archived_name);
-
// Generate the history and thumbnail database metrics only after performing
// any migration work.
if (base::RandInt(1, 100) == 50) {
« no previous file with comments | « components/history/core/browser/history_backend.h ('k') | components/history/core/browser/history_backend_db_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698