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

Unified Diff: chrome/browser/history/history_backend.h

Issue 16951015: Remove TextDatabase from the history service. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@replace_fts
Patch Set: Created 7 years, 6 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: chrome/browser/history/history_backend.h
diff --git a/chrome/browser/history/history_backend.h b/chrome/browser/history/history_backend.h
index fb18d6f34b4bb6898c0c8550de95259362f5696f..7895f8e4edd1e4d64dfc8c4b0a0c771b59170d6f 100644
--- a/chrome/browser/history/history_backend.h
+++ b/chrome/browser/history/history_backend.h
@@ -19,7 +19,6 @@
#include "chrome/browser/history/history_database.h"
#include "chrome/browser/history/history_marshaling.h"
#include "chrome/browser/history/history_types.h"
-#include "chrome/browser/history/text_database_manager.h"
#include "chrome/browser/history/thumbnail_database.h"
#include "chrome/browser/history/visit_tracker.h"
#include "chrome/browser/search_engines/template_url_id.h"
@@ -161,11 +160,6 @@ class HistoryBackend : public base::RefCountedThreadSafe<HistoryBackend>,
const GURL& url,
base::Time end_ts);
-
- // Indexing ------------------------------------------------------------------
-
- void SetPageContents(const GURL& url, const string16& contents);
-
// Querying ------------------------------------------------------------------
// ScheduleAutocomplete() never frees |provider| (which is globally live).
@@ -641,9 +635,6 @@ class HistoryBackend : public base::RefCountedThreadSafe<HistoryBackend>,
const string16& text_query,
const QueryOptions& options,
QueryResults* result);
- void QueryHistoryFTS(const string16& text_query,
- const QueryOptions& options,
- QueryResults* result);
// Committing ----------------------------------------------------------------
@@ -826,6 +817,9 @@ class HistoryBackend : public base::RefCountedThreadSafe<HistoryBackend>,
// The IDs of the URLs may change.
bool ClearAllMainHistory(const URLRows& kept_urls);
+ // Deletes the FTS index database files, which are no longer used.
+ void DeleteFTSIndexDatabases();
+
// Returns the BookmarkService, blocking until it is loaded. This may return
// NULL during testing.
BookmarkService* GetBookmarkService();
@@ -857,10 +851,6 @@ class HistoryBackend : public base::RefCountedThreadSafe<HistoryBackend>,
// Stores old history in a larger, slower database.
scoped_ptr<ArchivedDatabase> archived_db_;
- // Full text database manager, possibly NULL if the database could not be
- // created.
- scoped_ptr<TextDatabaseManager> text_database_;
-
// Manages expiration between the various databases.
ExpireHistoryBackend expirer_;

Powered by Google App Engine
This is Rietveld 408576698