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

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

Issue 184663002: Omnibox: Make URLs of Bookmarks Searchable (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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.cc
diff --git a/chrome/browser/history/url_index_private_data.cc b/chrome/browser/history/url_index_private_data.cc
index 717d2825b146a472aff0eb6a0e7e60ac8dbfdc53..a07ec0642643d993b5f8e7235a41002c507ed70d 100644
--- a/chrome/browser/history/url_index_private_data.cc
+++ b/chrome/browser/history/url_index_private_data.cc
@@ -22,6 +22,7 @@
#include "chrome/browser/autocomplete/autocomplete_provider.h"
#include "chrome/browser/autocomplete/url_prefix.h"
#include "chrome/browser/bookmarks/bookmark_service.h"
+#include "chrome/browser/bookmarks/bookmark_utils.h"
#include "chrome/browser/history/history_database.h"
#include "chrome/browser/history/history_db_task.h"
#include "chrome/browser/history/history_service.h"
@@ -755,10 +756,12 @@ void URLIndexPrivateData::AddRowWordsToIndex(const URLRow& row,
HistoryID history_id = static_cast<HistoryID>(row.id());
// Split URL into individual, unique words then add in the title words.
const GURL& gurl(row.url());
- const base::string16& url = CleanUpUrlForMatching(gurl, languages);
+ const base::string16& url =
+ bookmark_utils::CleanUpUrlForMatching(gurl, languages);
String16Set url_words = String16SetFromString16(url,
word_starts ? &word_starts->url_word_starts_ : NULL);
- const base::string16& title = CleanUpTitleForMatching(row.title());
+ const base::string16& title =
+ bookmark_utils::CleanUpTitleForMatching(row.title());
String16Set title_words = String16SetFromString16(title,
word_starts ? &word_starts->title_word_starts_ : NULL);
String16Set words;
@@ -1236,9 +1239,11 @@ bool URLIndexPrivateData::RestoreWordStartsMap(
iter != history_info_map_.end(); ++iter) {
RowWordStarts word_starts;
const URLRow& row(iter->second.url_row);
- const base::string16& url = CleanUpUrlForMatching(row.url(), languages);
+ const base::string16& url =
+ bookmark_utils::CleanUpUrlForMatching(row.url(), languages);
String16VectorFromString16(url, false, &word_starts.url_word_starts_);
- const base::string16& title = CleanUpTitleForMatching(row.title());
+ const base::string16& title =
+ bookmark_utils::CleanUpTitleForMatching(row.title());
String16VectorFromString16(title, false, &word_starts.title_word_starts_);
word_starts_map_[iter->first] = word_starts;
}

Powered by Google App Engine
This is Rietveld 408576698