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

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

Issue 1191953004: Revert of Revert of Componentize HistoryURLProvider/ScoredHistoryMatch. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@prepare_history_url_provider_for_c14n
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/scored_history_match.h
diff --git a/chrome/browser/autocomplete/scored_history_match.h b/chrome/browser/autocomplete/scored_history_match.h
deleted file mode 100644
index d9b5ae4579c720c7af0b2a6f4e1e159c2554d203..0000000000000000000000000000000000000000
--- a/chrome/browser/autocomplete/scored_history_match.h
+++ /dev/null
@@ -1,196 +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.
-
-#ifndef CHROME_BROWSER_AUTOCOMPLETE_SCORED_HISTORY_MATCH_H_
-#define CHROME_BROWSER_AUTOCOMPLETE_SCORED_HISTORY_MATCH_H_
-
-#include <string>
-#include <vector>
-
-#include "base/strings/string16.h"
-#include "base/time/time.h"
-#include "components/history/core/browser/history_match.h"
-#include "components/history/core/browser/history_types.h"
-#include "components/omnibox/in_memory_url_index_types.h"
-#include "testing/gtest/include/gtest/gtest_prod.h"
-
-class ScoredHistoryMatchTest;
-
-// An HistoryMatch that has a score as well as metrics defining where in the
-// history item's URL and/or page title matches have occurred.
-struct ScoredHistoryMatch : public history::HistoryMatch {
- // ScoreMaxRelevance maps from an intermediate-score to the maximum
- // final-relevance score given to a URL for this intermediate score.
- // This is used to store the score ranges of HQP relevance buckets.
- // Please see GetFinalRelevancyScore() for details.
- typedef std::pair<double, int> ScoreMaxRelevance;
-
- // Required for STL, we don't use this directly.
- ScoredHistoryMatch();
-
- // Initializes the ScoredHistoryMatch with a raw score calculated for the
- // history item given in |row| with recent visits as indicated in |visits|. It
- // first determines if the row qualifies by seeing if all of the terms in
- // |terms_vector| occur in |row|. If so, calculates a raw score. This raw
- // score is in part determined by whether the matches occur at word
- // boundaries, the locations of which are stored in |word_starts|. For some
- // terms, it's appropriate to look for the word boundary within the term. For
- // instance, the term ".net" should look for a word boundary at the "n". These
- // offsets (".net" should have an offset of 1) come from
- // |terms_to_word_starts_offsets|. |is_url_bookmarked| indicates whether the
- // match's URL is referenced by any bookmarks, which can also affect the raw
- // score. The raw score allows the matches to be ordered and can be used to
- // influence the final score calculated by the client of this index. If the
- // row does not qualify the raw score will be 0. |languages| is used to help
- // parse/format the URL before looking for the terms.
- ScoredHistoryMatch(const history::URLRow& row,
- const VisitInfoVector& visits,
- const std::string& languages,
- const base::string16& lower_string,
- const String16Vector& terms_vector,
- const WordStarts& terms_to_word_starts_offsets,
- const RowWordStarts& word_starts,
- bool is_url_bookmarked,
- base::Time now);
-
- ~ScoredHistoryMatch();
-
- // Compares two matches by score. Functor supporting URLIndexPrivateData's
- // HistoryItemsForTerms function. Looks at particular fields within
- // with url_info to make tie-breaking a bit smarter.
- static bool MatchScoreGreater(const ScoredHistoryMatch& m1,
- const ScoredHistoryMatch& m2);
-
- // Returns |term_matches| after removing all matches that are not at a
- // word break that are in the range [|start_pos|, |end_pos|).
- // start_pos == string::npos is treated as start_pos = length of string.
- // (In other words, no matches will be filtered.)
- // end_pos == string::npos is treated as end_pos = length of string.
- static TermMatches FilterTermMatchesByWordStarts(
- const TermMatches& term_matches,
- const WordStarts& terms_to_word_starts_offsets,
- const WordStarts& word_starts,
- size_t start_pos,
- size_t end_pos);
-
- // The maximum number of recent visits to examine in GetFrequency().
- // Public so url_index_private_data.cc knows how many visits it is
- // expected to deliver (at minimum) to this class.
- static const size_t kMaxVisitsToScore;
-
- // An interim score taking into consideration location and completeness
- // of the match.
- int raw_score;
-
- // Both these TermMatches contain the set of matches that are considered
- // important. At this time, that means they exclude mid-word matches
- // except in the hostname of the URL. (Technically, during early
- // construction of ScoredHistoryMatch, they may contain all matches, but
- // unimportant matches are eliminated by GetTopicalityScore(), called
- // during construction.)
-
- // Term matches within the URL.
- TermMatches url_matches;
- // Term matches within the page title.
- TermMatches title_matches;
-
- // True if this is a candidate for in-line autocompletion.
- bool can_inline;
-
- private:
- friend class ScoredHistoryMatchTest;
- FRIEND_TEST_ALL_PREFIXES(ScoredHistoryMatchTest, GetFinalRelevancyScore);
- FRIEND_TEST_ALL_PREFIXES(ScoredHistoryMatchTest, GetHQPBucketsFromString);
- FRIEND_TEST_ALL_PREFIXES(ScoredHistoryMatchTest, ScoringBookmarks);
- FRIEND_TEST_ALL_PREFIXES(ScoredHistoryMatchTest, ScoringScheme);
- FRIEND_TEST_ALL_PREFIXES(ScoredHistoryMatchTest, ScoringTLD);
-
- // Initialize ScoredHistoryMatch statics. Must be called before any other
- // method of ScoredHistoryMatch and before creating any instances.
- static void Init();
-
- // 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 base::string16& cleaned_up_url,
- const WordStarts& terms_to_word_starts_offsets,
- const RowWordStarts& word_starts);
-
- // Returns a recency score based on |last_visit_days_ago|, which is
- // how many days ago the page was last visited.
- float GetRecencyScore(int last_visit_days_ago) const;
-
- // Examines the first kMaxVisitsToScore and return a score (higher is
- // better) based the rate of visits, whether the page is bookmarked, and
- // how often those visits are typed navigations (i.e., explicitly
- // invoked by the user). |now| is passed in to avoid unnecessarily
- // recomputing it frequently.
- float GetFrequency(const base::Time& now,
- const bool bookmarked,
- const VisitInfoVector& visits) const;
-
- // Combines the two component scores into a final score that's
- // an appropriate value to use as a relevancy score. Scoring buckets are
- // specified through |hqp_relevance_buckets|. Please see the function
- // implementation for more details.
- static float GetFinalRelevancyScore(
- float topicality_score,
- float frequency_score,
- const std::vector<ScoreMaxRelevance>& hqp_relevance_buckets);
-
- // Initializes the HQP experimental params: |hqp_relevance_buckets_|
- // to default buckets. If hqp experimental scoring is enabled, it
- // fetches the |hqp_experimental_scoring_enabled_|, |topicality_threshold_|
- // and |hqp_relevance_buckets_| from omnibox field trials.
- static void InitHQPExperimentalParams();
-
- // Helper function to parse the string containing the scoring buckets.
- // For example,
- // String: "0.0:400,1.5:600,12.0:1300,20.0:1399"
- // Buckets: vector[(0.0, 400),(1.5,600),(12.0,1300),(20.0,1399)]
- // Returns false, in case if it fail to parse the string.
- static bool GetHQPBucketsFromString(
- const std::string& buckets_str,
- std::vector<ScoreMaxRelevance>* hqp_buckets);
-
- // If true, assign raw scores to be max(whatever it normally would be, a
- // score that's similar to the score HistoryURL provider would assign).
- static bool also_do_hup_like_scoring_;
-
- // Untyped visits to bookmarked pages score this, compared to 1 for
- // untyped visits to non-bookmarked pages and 20 for typed visits.
- static int bookmark_value_;
-
- // True if we should fix certain bugs in frequency scoring.
- static bool fix_frequency_bugs_;
-
- // If true, we allow input terms to match in the TLD (e.g., ".com").
- static bool allow_tld_matches_;
-
- // If true, we allow input terms to match in the scheme (e.g., "http://").
- static bool allow_scheme_matches_;
-
- // The number of title words examined when computing topicality scores.
- // Words beyond this number are ignored.
- static size_t num_title_words_to_allow_;
-
- // True, if hqp experimental scoring is enabled.
- static bool hqp_experimental_scoring_enabled_;
-
- // |topicality_threshold_| is used to control the topicality scoring.
- // If |topicality_threshold_| > 0, then URLs with topicality-score < threshold
- // are given topicality score of 0. By default it is initalized to -1.
- static float topicality_threshold_;
-
- // |hqp_relevance_buckets_| gives mapping from (topicality*frequency)
- // to the final relevance scoring. Please see GetFinalRelevancyScore()
- // for more details and scoring method.
- static std::vector<ScoreMaxRelevance>* hqp_relevance_buckets_;
-};
-typedef std::vector<ScoredHistoryMatch> ScoredHistoryMatches;
-
-#endif // CHROME_BROWSER_AUTOCOMPLETE_SCORED_HISTORY_MATCH_H_
« no previous file with comments | « chrome/browser/autocomplete/in_memory_url_index.h ('k') | chrome/browser/autocomplete/scored_history_match.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698