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

Unified Diff: chrome/browser/autocomplete/in_memory_url_index.h

Issue 1184923006: Revert of Abstract //content-level dependencies out of InMemoryURLIndex (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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
« no previous file with comments | « no previous file | chrome/browser/autocomplete/in_memory_url_index.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/autocomplete/in_memory_url_index.h
diff --git a/chrome/browser/autocomplete/in_memory_url_index.h b/chrome/browser/autocomplete/in_memory_url_index.h
index c18c233c3648830936bcb4d2c9d9c6bd7e65b14b..c881a1ff132ca4566b059cb4a5dd5d8f539d6a79 100644
--- a/chrome/browser/autocomplete/in_memory_url_index.h
+++ b/chrome/browser/autocomplete/in_memory_url_index.h
@@ -18,8 +18,6 @@
#include "base/memory/weak_ptr.h"
#include "base/strings/string16.h"
#include "base/task/cancelable_task_tracker.h"
-#include "base/threading/sequenced_worker_pool.h"
-#include "base/threading/thread_checker.h"
#include "components/history/core/browser/history_db_task.h"
#include "components/history/core/browser/history_service_observer.h"
#include "components/history/core/browser/history_types.h"
@@ -47,8 +45,6 @@
}
class URLIndexPrivateData;
-
-typedef std::set<std::string> SchemeSet;
// The URL history source.
// Holds portions of the URL database in memory in an indexed form. Used to
@@ -105,10 +101,8 @@
// characters.
InMemoryURLIndex(bookmarks::BookmarkModel* bookmark_model,
history::HistoryService* history_service,
- base::SequencedWorkerPool* worker_pool,
const base::FilePath& history_dir,
- const std::string& languages,
- const SchemeSet& client_schemes_to_whitelist);
+ const std::string& languages);
~InMemoryURLIndex() override;
// Opens and prepares the index of historical URL visits. If the index private
@@ -158,7 +152,7 @@
explicit RebuildPrivateDataFromHistoryDBTask(
InMemoryURLIndex* index,
const std::string& languages,
- const SchemeSet& scheme_whitelist);
+ const std::set<std::string>& scheme_whitelist);
bool RunOnDBThread(history::HistoryBackend* backend,
history::HistoryDatabase* db) override;
@@ -169,7 +163,7 @@
InMemoryURLIndex* index_; // Call back to this index at completion.
std::string languages_; // Languages for word-breaking.
- SchemeSet scheme_whitelist_; // Schemes to be indexed.
+ std::set<std::string> scheme_whitelist_; // Schemes to be indexed.
bool succeeded_; // Indicates if the rebuild was successful.
scoped_refptr<URLIndexPrivateData> data_; // The rebuilt private data.
@@ -268,7 +262,7 @@
}
// Returns the set of whitelisted schemes. For unit testing only.
- const SchemeSet& scheme_whitelist() { return scheme_whitelist_; }
+ const std::set<std::string>& scheme_whitelist() { return scheme_whitelist_; }
// The BookmarkModel; may be null when testing.
bookmarks::BookmarkModel* bookmark_model_;
@@ -285,7 +279,7 @@
std::string languages_;
// Only URLs with a whitelisted scheme are indexed.
- SchemeSet scheme_whitelist_;
+ std::set<std::string> scheme_whitelist_;
// The index's durable private data.
scoped_refptr<URLIndexPrivateData> private_data_;
@@ -317,8 +311,6 @@
// HistoryServiceLoaded Notification.
bool listen_to_history_service_loaded_;
- base::ThreadChecker thread_checker_;
-
DISALLOW_COPY_AND_ASSIGN(InMemoryURLIndex);
};
« no previous file with comments | « no previous file | chrome/browser/autocomplete/in_memory_url_index.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698