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

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

Issue 19637009: Revert 212459 "Remove TextDatabase from the history service." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: 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: trunk/src/chrome/browser/history/history_backend.h
===================================================================
--- trunk/src/chrome/browser/history/history_backend.h (revision 212472)
+++ trunk/src/chrome/browser/history/history_backend.h (working copy)
@@ -20,6 +20,7 @@
#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,6 +162,11 @@
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).
@@ -558,7 +564,6 @@
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;
@@ -653,6 +658,9 @@
const string16& text_query,
const QueryOptions& options,
QueryResults* result);
+ void QueryHistoryFTS(const string16& text_query,
+ const QueryOptions& options,
+ QueryResults* result);
// Committing ----------------------------------------------------------------
@@ -835,9 +843,6 @@
// 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();
@@ -870,6 +875,10 @@
// 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