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

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

Issue 9316109: Move Ownership of IMUI to HistoryService. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Qualify globally scoped friend classes. Created 8 years, 10 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
« no previous file with comments | « chrome/browser/history/history_unittest.cc ('k') | chrome/browser/history/in_memory_history_backend.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/history/in_memory_history_backend.h
===================================================================
--- chrome/browser/history/in_memory_history_backend.h (revision 120346)
+++ chrome/browser/history/in_memory_history_backend.h (working copy)
@@ -41,16 +41,9 @@
virtual ~InMemoryHistoryBackend();
// Initializes the backend from the history database pointed to by the
- // full path in |history_filename|. |history_dir| is the path to the
- // directory containing the history database and is also used
- // as the directory where the InMemoryURLIndex's cache is kept. |db| is
- // used for building the InMemoryURLIndex. |languages| gives the
- // preferred user languages with which URLs and page titles are
- // interpreted while decomposing into words and characters during indexing.
- bool Init(const FilePath& history_filename,
- const FilePath& history_dir,
- URLDatabase* db,
- const std::string& languages);
+ // full path in |history_filename|. |db| is used for setting up the
+ // InMemoryDatabase.
+ bool Init(const FilePath& history_filename, URLDatabase* db);
// Does initialization work when this object is attached to the history
// system on the main thread. The argument is the profile with which the
@@ -69,9 +62,6 @@
const content::NotificationSource& source,
const content::NotificationDetails& details) OVERRIDE;
- // Return the quick history index.
- history::InMemoryURLIndex* InMemoryIndex() const { return index_.get(); }
-
private:
FRIEND_TEST_ALL_PREFIXES(HistoryBackendTest, DeleteAll);
@@ -95,9 +85,6 @@
// initialization.
Profile* profile_;
- // The index used for quick history lookups.
- scoped_ptr<history::InMemoryURLIndex> index_;
-
DISALLOW_COPY_AND_ASSIGN(InMemoryHistoryBackend);
};
« no previous file with comments | « chrome/browser/history/history_unittest.cc ('k') | chrome/browser/history/in_memory_history_backend.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698