Omnibox: Make HistoryURL Highlight Titles like HistoryQuick Provider

chrome/browser/history/scored_history_match.h

 // 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.
 
 #ifndef CHROME_BROWSER_HISTORY_SCORED_HISTORY_MATCH_H_
 #define CHROME_BROWSER_HISTORY_SCORED_HISTORY_MATCH_H_
 
 #include <map>
 #include <set>
 #include <vector>
@@ skipping 45 matching lines @@
   // with url_info to make tie-breaking a bit smarter.
   static bool MatchScoreGreater(const ScoredHistoryMatch& m1,
                                 const ScoredHistoryMatch& m2);
 
   // Accessors:
   int raw_score() const { return raw_score_; }
   const TermMatches& url_matches() const { return url_matches_; }
   const TermMatches& title_matches() const { return title_matches_; }
   bool can_inline() const { return can_inline_; }
 
+  // Returns |term_matches| after removing all matches that are not at a

[Mark P, 2013/11/22 02:26:32]

This is a simple move of this function from private to public.

+  // word break that starts after position |start_pos|.  If |start_pos| is
+  // string::npos, does no filtering and simply returns |term_matches|.
+  static TermMatches FilterTermMatchesByWordStarts(
+      const TermMatches& term_matches,
+      const WordStarts& word_starts,
+      const size_t start_pos);
+
  private:
   friend class ScoredHistoryMatchTest;
 
   // The number of days of recency scores to precompute.
   static const int kDaysToPrecomputeRecencyScoresFor;
 
   // The number of raw term score buckets use; raw term scores
   // greater this are capped at the score of the largest bucket.
   static const int kMaxRawTermScore;
 
   // Return a topicality score based on how many matches appear in the
   // url and the page's title and where they are (e.g., at word
   // boundaries).  Revises |url_matches_| and |title_matches_| in the
   // process so they only reflect matches used for scoring.  (For
   // instance, some mid-word matches are not given credit in scoring.)
   float GetTopicalityScore(const int num_terms,
                            const string16& cleaned_up_url,
                            const RowWordStarts& word_starts);
 
-  // Helper function for GetTopicalityScore().
-  // Returns |term_matches| after removing all matches that are not at a
-  // word break that starts after position |start_pos|.  If |start_pos| is
-  // string::npos, does no filtering and simply returns |term_matches|.
-  static TermMatches FilterTermMatchesByWordStarts(
-      const TermMatches& term_matches,
-      const WordStarts& word_starts,
-      const size_t start_pos);
-
   // Precalculates raw_term_score_to_topicality_score, used in
   // GetTopicalityScore().
   static void FillInTermScoreToTopicalityScoreArray();
 
   // Returns a recency score based on |last_visit_days_ago|, which is
   // how many days ago the page was last visited.
   static float GetRecencyScore(int last_visit_days_ago);
 
   // Pre-calculates days_ago_to_recency_numerator_, used in
   // GetRecencyScore().
@@ skipping 68 matching lines @@
   // matches) because if a non-inlineable match comes first than all matches
   // will get demoted later in HistoryQuickProvider to non-inlineable scores.
   // Set to -1 to indicate no maximum score.
   static int max_assigned_score_for_non_inlineable_matches_;
 };
 typedef std::vector<ScoredHistoryMatch> ScoredHistoryMatches;
 
 }  // namespace history
 
 #endif  // CHROME_BROWSER_HISTORY_SCORED_HISTORY_MATCH_H_