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

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

Issue 903493002: Componentize ScoredHistoryMatch (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 5 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
Index: chrome/browser/history/url_index_private_data.h
diff --git a/chrome/browser/history/url_index_private_data.h b/chrome/browser/history/url_index_private_data.h
index a2b4a6bd8c3766fb3801373a4b7e7d69f4a2ab54..b87447cbd84f36fe80a7a476a4fe22620e5089af 100644
--- a/chrome/browser/history/url_index_private_data.h
+++ b/chrome/browser/history/url_index_private_data.h
@@ -12,9 +12,9 @@
#include "base/gtest_prod_util.h"
#include "base/memory/ref_counted.h"
#include "chrome/browser/history/history_service.h"
-#include "chrome/browser/history/scored_history_match.h"
#include "components/history/core/browser/in_memory_url_index_cache.pb.h"
#include "components/history/core/browser/in_memory_url_index_types.h"
+#include "components/history/core/browser/scored_history_match.h"
class HistoryQuickProviderTest;
@@ -26,10 +26,10 @@ namespace history {
namespace imui = in_memory_url_index;
-class HistoryClient;
class HistoryDatabase;
class InMemoryURLIndex;
class RefCountedBool;
+class ScoredHistoryMatchClient;
// Current version of the cache file.
static const int kCurrentCacheFileVersion = 5;
@@ -67,11 +67,12 @@ class URLIndexPrivateData
// is used to help parse/format the URLs in the history index. In total,
// |max_matches| of items will be returned in the |ScoredHistoryMatches|
// vector.
- ScoredHistoryMatches HistoryItemsForTerms(base::string16 term_string,
- size_t cursor_position,
- size_t max_matches,
- const std::string& languages,
- HistoryClient* history_client);
+ ScoredHistoryMatches HistoryItemsForTerms(
+ base::string16 term_string,
+ size_t cursor_position,
+ size_t max_matches,
+ const std::string& languages,
+ const ScoredHistoryMatchClient* scored_history_match_client);
// Adds the history item in |row| to the index if it does not already already
// exist and it meets the minimum 'quick' criteria. If the row already exists
@@ -196,7 +197,7 @@ class URLIndexPrivateData
public:
AddHistoryMatch(const URLIndexPrivateData& private_data,
const std::string& languages,
- HistoryClient* history_client,
+ const ScoredHistoryMatchClient* scored_history_match_client,
const base::string16& lower_string,
const String16Vector& lower_terms,
const base::Time now);
@@ -209,7 +210,7 @@ class URLIndexPrivateData
private:
const URLIndexPrivateData& private_data_;
const std::string& languages_;
- HistoryClient* history_client_;
+ const ScoredHistoryMatchClient* scored_history_match_client_;
ScoredHistoryMatches scored_matches_;
const base::string16& lower_string_;
const String16Vector& lower_terms_;
« no previous file with comments | « chrome/browser/history/scored_history_match_unittest.cc ('k') | chrome/browser/history/url_index_private_data.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698