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

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: Sync and rebase. Created 7 years, 5 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 c4010b7119d735b0260dbd556e7d8586edf76618..fd9287869db18f3a7c6ce03a2e62f941112e0e51 100644
--- a/chrome/browser/history/history_backend.h
+++ b/chrome/browser/history/history_backend.h
@@ -20,7 +20,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"
@@ -162,11 +161,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).
@@ -564,6 +558,7 @@ class HistoryBackend : public base::RefCountedThreadSafe<HistoryBackend>,
FRIEND_TEST_ALL_PREFIXES(HistoryBackendTest, QueryFilteredURLs);
FRIEND_TEST_ALL_PREFIXES(HistoryBackendTest, UpdateVisitDuration);
FRIEND_TEST_ALL_PREFIXES(HistoryBackendTest, ExpireHistoryForTimes);
+ FRIEND_TEST_ALL_PREFIXES(HistoryBackendTest, DeleteFTSIndexDatabases);
friend class ::TestingProfile;
@@ -658,9 +653,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 ----------------------------------------------------------------
@@ -843,6 +835,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();
@@ -875,10 +870,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_;
« no previous file with comments | « chrome/browser/history/expire_history_backend_unittest.cc ('k') | chrome/browser/history/history_backend.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698