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

Unified Diff: chrome/browser/autocomplete/in_memory_url_index_types.cc

Issue 1178203004: Componentize InMemoryURLIndexTypes (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
Index: chrome/browser/autocomplete/in_memory_url_index_types.cc
diff --git a/chrome/browser/autocomplete/in_memory_url_index_types.cc b/chrome/browser/autocomplete/in_memory_url_index_types.cc
deleted file mode 100644
index e0f8b82ad279c278e82e2e03ea64218493ba012a..0000000000000000000000000000000000000000
--- a/chrome/browser/autocomplete/in_memory_url_index_types.cc
+++ /dev/null
@@ -1,165 +0,0 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "chrome/browser/autocomplete/in_memory_url_index_types.h"
-
-#include <algorithm>
-#include <functional>
-#include <iterator>
-#include <numeric>
-#include <set>
-
-#include "base/i18n/break_iterator.h"
-#include "base/i18n/case_conversion.h"
-#include "base/strings/string_util.h"
-#include "net/base/escape.h"
-#include "net/base/net_util.h"
-
-namespace {
-// The maximum number of characters to consider from an URL and page title
-// while matching user-typed terms.
-const size_t kMaxSignificantChars = 200;
-}
-
-// Matches within URL and Title Strings ----------------------------------------
-
-TermMatches MatchTermInString(const base::string16& term,
- const base::string16& cleaned_string,
- int term_num) {
- const size_t kMaxCompareLength = 2048;
- const base::string16& short_string =
- (cleaned_string.length() > kMaxCompareLength) ?
- cleaned_string.substr(0, kMaxCompareLength) : cleaned_string;
- TermMatches matches;
- for (size_t location = short_string.find(term);
- location != base::string16::npos;
- location = short_string.find(term, location + 1))
- matches.push_back(TermMatch(term_num, location, term.length()));
- return matches;
-}
-
-// Comparison function for sorting TermMatches by their offsets.
-bool MatchOffsetLess(const TermMatch& m1, const TermMatch& m2) {
- return m1.offset < m2.offset;
-}
-
-TermMatches SortAndDeoverlapMatches(const TermMatches& matches) {
- if (matches.empty())
- return matches;
- TermMatches sorted_matches = matches;
- std::sort(sorted_matches.begin(), sorted_matches.end(), MatchOffsetLess);
- TermMatches clean_matches;
- TermMatch last_match;
- for (TermMatches::const_iterator iter = sorted_matches.begin();
- iter != sorted_matches.end(); ++iter) {
- if (iter->offset >= last_match.offset + last_match.length) {
- last_match = *iter;
- clean_matches.push_back(last_match);
- }
- }
- return clean_matches;
-}
-
-std::vector<size_t> OffsetsFromTermMatches(const TermMatches& matches) {
- std::vector<size_t> offsets;
- for (TermMatches::const_iterator i = matches.begin(); i != matches.end();
- ++i) {
- offsets.push_back(i->offset);
- offsets.push_back(i->offset + i->length);
- }
- return offsets;
-}
-
-TermMatches ReplaceOffsetsInTermMatches(const TermMatches& matches,
- const std::vector<size_t>& offsets) {
- DCHECK_EQ(2 * matches.size(), offsets.size());
- TermMatches new_matches;
- std::vector<size_t>::const_iterator offset_iter = offsets.begin();
- for (TermMatches::const_iterator term_iter = matches.begin();
- term_iter != matches.end(); ++term_iter, ++offset_iter) {
- const size_t starting_offset = *offset_iter;
- ++offset_iter;
- const size_t ending_offset = *offset_iter;
- if ((starting_offset != base::string16::npos) &&
- (ending_offset != base::string16::npos) &&
- (starting_offset != ending_offset)) {
- TermMatch new_match(*term_iter);
- new_match.offset = starting_offset;
- new_match.length = ending_offset - starting_offset;
- new_matches.push_back(new_match);
- }
- }
- return new_matches;
-}
-
-// Utility Functions -----------------------------------------------------------
-
-String16Set String16SetFromString16(const base::string16& cleaned_uni_string,
- WordStarts* word_starts) {
- String16Vector words =
- String16VectorFromString16(cleaned_uni_string, false, word_starts);
- String16Set word_set;
- for (String16Vector::const_iterator iter = words.begin(); iter != words.end();
- ++iter)
- word_set.insert(base::i18n::ToLower(*iter).substr(0, kMaxSignificantChars));
- return word_set;
-}
-
-String16Vector String16VectorFromString16(
- const base::string16& cleaned_uni_string,
- bool break_on_space,
- WordStarts* word_starts) {
- if (word_starts)
- word_starts->clear();
- base::i18n::BreakIterator iter(cleaned_uni_string,
- break_on_space ? base::i18n::BreakIterator::BREAK_SPACE :
- base::i18n::BreakIterator::BREAK_WORD);
- String16Vector words;
- if (!iter.Init())
- return words;
- while (iter.Advance()) {
- if (break_on_space || iter.IsWord()) {
- base::string16 word(iter.GetString());
- size_t initial_whitespace = 0;
- if (break_on_space) {
- base::string16 trimmed_word;
- base::TrimWhitespace(word, base::TRIM_LEADING, &trimmed_word);
- initial_whitespace = word.length() - trimmed_word.length();
- base::TrimWhitespace(trimmed_word, base::TRIM_TRAILING, &word);
- }
- if (word.empty())
- continue;
- words.push_back(word);
- if (!word_starts)
- continue;
- size_t word_start = iter.prev() + initial_whitespace;
- if (word_start < kMaxSignificantChars)
- word_starts->push_back(word_start);
- }
- }
- return words;
-}
-
-Char16Set Char16SetFromString16(const base::string16& term) {
- Char16Set characters;
- for (base::string16::const_iterator iter = term.begin(); iter != term.end();
- ++iter)
- characters.insert(*iter);
- return characters;
-}
-
-// HistoryInfoMapValue ---------------------------------------------------------
-
-HistoryInfoMapValue::HistoryInfoMapValue() {}
-HistoryInfoMapValue::~HistoryInfoMapValue() {}
-
-// RowWordStarts ---------------------------------------------------------------
-
-RowWordStarts::RowWordStarts() {}
-RowWordStarts::~RowWordStarts() {}
-
-void RowWordStarts::Clear() {
- url_word_starts_.clear();
- title_word_starts_.clear();
-}

Powered by Google App Engine
This is Rietveld 408576698