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

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

Issue 959343004: Move InMemoryURLIndex outside of history namespace (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@in-memory-url-index
Patch Set: Rebase Created 5 years, 9 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/autocomplete/url_index_private_data.cc ('k') | chrome/browser/history/history_service.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/history/history_service.h
diff --git a/chrome/browser/history/history_service.h b/chrome/browser/history/history_service.h
index 036e0c56a407c060ffc67837b8b11bafe0b44d62..d52ca51553c883022f4a81d47cd43e8f087eb19e 100644
--- a/chrome/browser/history/history_service.h
+++ b/chrome/browser/history/history_service.h
@@ -34,6 +34,8 @@
#include "ui/base/page_transition_types.h"
class GURL;
+class InMemoryURLIndex;
+class InMemoryURLIndexTest;
class PageUsageRequest;
class SkBitmap;
@@ -55,8 +57,6 @@ class HistoryQueryTest;
class HistoryServiceObserver;
class HistoryTest;
class InMemoryHistoryBackend;
-class InMemoryURLIndex;
-class InMemoryURLIndexTest;
struct KeywordSearchTermVisit;
class PageUsageData;
class URLDatabase;
@@ -134,9 +134,7 @@ class HistoryService : public syncer::SyncableService, public KeyedService {
history::TypedUrlSyncableService* GetTypedUrlSyncableService() const;
// Return the quick history index.
- history::InMemoryURLIndex* InMemoryIndex() const {
- return in_memory_url_index_.get();
- }
+ InMemoryURLIndex* InMemoryIndex() const { return in_memory_url_index_.get(); }
// KeyedService:
void Shutdown() override;
@@ -522,7 +520,7 @@ class HistoryService : public syncer::SyncableService, public KeyedService {
friend class history::HistoryTest;
friend class HistoryURLProvider;
friend class HistoryURLProviderTest;
- friend class history::InMemoryURLIndexTest;
+ friend class ::InMemoryURLIndexTest;
template<typename Info, typename Callback> friend class DownloadRequest;
friend class PageUsageRequest;
friend class RedirectRequest;
@@ -806,7 +804,7 @@ class HistoryService : public syncer::SyncableService, public KeyedService {
// The index used for quick history lookups.
// TODO(mrossetti): Move in_memory_url_index out of history_service.
// See http://crbug.com/138321
- scoped_ptr<history::InMemoryURLIndex> in_memory_url_index_;
+ scoped_ptr<InMemoryURLIndex> in_memory_url_index_;
ObserverList<history::HistoryServiceObserver> observers_;
base::CallbackList<void(const std::set<GURL>&)>
« no previous file with comments | « chrome/browser/autocomplete/url_index_private_data.cc ('k') | chrome/browser/history/history_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698