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

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

Issue 10699032: autocomplete: Extract AutocompleteResult from autocomplete.*. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: move comments Created 8 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
« no previous file with comments | « no previous file | chrome/browser/autocomplete/autocomplete.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/autocomplete/autocomplete.h
diff --git a/chrome/browser/autocomplete/autocomplete.h b/chrome/browser/autocomplete/autocomplete.h
index 7912e006b36da1b5100eabe3a7e498aebbb270a6..99c95f9c1add76a27de9e07a2b8fed3cdd5ea6a1 100644
--- a/chrome/browser/autocomplete/autocomplete.h
+++ b/chrome/browser/autocomplete/autocomplete.h
@@ -9,184 +9,10 @@
#include <string>
#include "base/basictypes.h"
-#include "base/logging.h"
#include "base/string16.h"
-#include "base/time.h"
-#include "base/timer.h"
-#include "chrome/browser/autocomplete/autocomplete_types.h"
#include "googleurl/src/gurl.h"
#include "googleurl/src/url_parse.h"
-// The AutocompleteController is the center of the autocomplete system. A
-// class creates an instance of the controller, which in turn creates a set of
-// AutocompleteProviders to serve it. The owning class can ask the controller
-// to Start() a query; the controller in turn passes this call down to the
-// providers, each of which keeps track of its own matches and whether it has
-// finished processing the query. When a provider gets more matches or finishes
-// processing, it notifies the controller, which merges the combined matches
-// together and makes the result available to interested observers.
-//
-// The owner may also cancel the current query by calling Stop(), which the
-// controller will in turn communicate to all the providers. No callbacks will
-// happen after a request has been stopped.
-//
-// IMPORTANT: There is NO THREAD SAFETY built into this portion of the
-// autocomplete system. All calls to and from the AutocompleteController should
-// happen on the same thread. AutocompleteProviders are responsible for doing
-// their own thread management when they need to return matches asynchronously.
-//
-// The AutocompleteProviders each return different kinds of matches,
-// such as history or search matches. These matches are given
-// "relevance" scores. Higher scores are better matches than lower
-// scores. The relevance scores and classes providing the respective
-// matches are as listed below.
-//
-// IMPORTANT CAVEAT: The tables below are NOT COMPLETE. Developers
-// often forget to keep these tables in sync with the code when they
-// change scoring algorithms or add new providers. For example,
-// neither the HistoryQuickProvider (which is a provider that appears
-// often) nor the ShortcutsProvider are listed here. For the best
-// idea of how scoring works and what providers are affecting which
-// queries, play with chrome://omnibox/ for a while. While the tables
-// below may have some utility, nothing compares with first-hand
-// investigation and experience.
-//
-// UNKNOWN input type:
-// --------------------------------------------------------------------|-----
-// Keyword (non-substituting or in keyword UI mode, exact match) | 1500
-// Extension App (exact match) | 1425
-// HistoryURL (good exact or inline autocomplete matches, some inexact)| 1410++
-// HistoryURL (intranet url never visited match, some inexact matches) | 1400++
-// Search Primary Provider (past query in history within 2 days) | 1399**
-// Search Primary Provider (what you typed) | 1300
-// HistoryURL (what you typed, some inexact matches) | 1200++
-// Extension App (inexact match) | 1175*~
-// Keyword (substituting, exact match) | 1100
-// Search Primary Provider (past query in history older than 2 days) | 1050--
-// HistoryContents (any match in title of starred page) | 1000++
-// HistoryURL (some inexact matches) | 900++
-// Search Primary Provider (navigational suggestion) | 800++
-// HistoryContents (any match in title of nonstarred page) | 700++
-// Search Primary Provider (suggestion) | 600++
-// Built-in | 575++
-// HistoryContents (any match in body of starred page) | 550++
-// HistoryContents (any match in body of nonstarred page) | 500++
-// Keyword (inexact match) | 450
-// Search Secondary Provider (what you typed) | 250
-// Search Secondary Provider (past query in history) | 200--
-// Search Secondary Provider (navigational suggestion) | 150++
-// Search Secondary Provider (suggestion) | 100++
-//
-// REQUESTED_URL input type:
-// --------------------------------------------------------------------|-----
-// Keyword (non-substituting or in keyword UI mode, exact match) | 1500
-// Extension App (exact match) | 1425
-// HistoryURL (good exact or inline autocomplete matches, some inexact)| 1410++
-// HistoryURL (intranet url never visited match, some inexact matches) | 1400++
-// Search Primary Provider (past query in history within 2 days) | 1399**
-// HistoryURL (what you typed, some inexact matches) | 1200++
-// Extension App (inexact match) | 1175*~
-// Search Primary Provider (what you typed) | 1150
-// Keyword (substituting, exact match) | 1100
-// Search Primary Provider (past query in history older than 2 days) | 1050--
-// HistoryContents (any match in title of starred page) | 1000++
-// HistoryURL (some inexact matches) | 900++
-// Search Primary Provider (navigational suggestion) | 800++
-// HistoryContents (any match in title of nonstarred page) | 700++
-// Search Primary Provider (suggestion) | 600++
-// Built-in | 575++
-// HistoryContents (any match in body of starred page) | 550++
-// HistoryContents (any match in body of nonstarred page) | 500++
-// Keyword (inexact match) | 450
-// Search Secondary Provider (what you typed) | 250
-// Search Secondary Provider (past query in history) | 200--
-// Search Secondary Provider (navigational suggestion) | 150++
-// Search Secondary Provider (suggestion) | 100++
-//
-// URL input type:
-// --------------------------------------------------------------------|-----
-// Keyword (non-substituting or in keyword UI mode, exact match) | 1500
-// Extension App (exact match) | 1425
-// HistoryURL (good exact or inline autocomplete matches, some inexact)| 1410++
-// HistoryURL (intranet url never visited match, some inexact matches) | 1400++
-// HistoryURL (what you typed, some inexact matches) | 1200++
-// Extension App (inexact match) | 1175*~
-// Keyword (substituting, exact match) | 1100
-// HistoryURL (some inexact matches) | 900++
-// Search Primary Provider (what you typed) | 850
-// Search Primary Provider (navigational suggestion) | 800++
-// Search Primary Provider (past query in history) | 750--
-// Keyword (inexact match) | 700
-// Built-in | 575++
-// Search Primary Provider (suggestion) | 300++
-// Search Secondary Provider (what you typed) | 250
-// Search Secondary Provider (past query in history) | 200--
-// Search Secondary Provider (navigational suggestion) | 150++
-// Search Secondary Provider (suggestion) | 100++
-//
-// QUERY input type:
-// --------------------------------------------------------------------|-----
-// Search Primary or Secondary (past query in history within 2 days) | 1599**
-// Keyword (non-substituting or in keyword UI mode, exact match) | 1500
-// Keyword (substituting, exact match) | 1450
-// Extension App (exact match) | 1425
-// Search Primary Provider (past query in history within 2 days) | 1399**
-// Search Primary Provider (what you typed) | 1300
-// Extension App (inexact match) | 1175*~
-// Search Primary Provider (past query in history older than 2 days) | 1050--
-// HistoryContents (any match in title of starred page) | 1000++
-// HistoryURL (inexact match) | 900++
-// Search Primary Provider (navigational suggestion) | 800++
-// HistoryContents (any match in title of nonstarred page) | 700++
-// Search Primary Provider (suggestion) | 600++
-// HistoryContents (any match in body of starred page) | 550++
-// HistoryContents (any match in body of nonstarred page) | 500++
-// Keyword (inexact match) | 450
-// Search Secondary Provider (what you typed) | 250
-// Search Secondary Provider (past query in history) | 200--
-// Search Secondary Provider (navigational suggestion) | 150++
-// Search Secondary Provider (suggestion) | 100++
-//
-// FORCED_QUERY input type:
-// --------------------------------------------------------------------|-----
-// Extension App (exact match on title only, not url) | 1425
-// Search Primary Provider (past query in history within 2 days) | 1399**
-// Search Primary Provider (what you typed) | 1300
-// Extension App (inexact match on title only, not url) | 1175*~
-// Search Primary Provider (past query in history older than 2 days) | 1050--
-// HistoryContents (any match in title of starred page) | 1000++
-// Search Primary Provider (navigational suggestion) | 800++
-// HistoryContents (any match in title of nonstarred page) | 700++
-// Search Primary Provider (suggestion) | 600++
-// HistoryContents (any match in body of starred page) | 550++
-// HistoryContents (any match in body of nonstarred page) | 500++
-//
-// (A search keyword is a keyword with a replacement string; a bookmark keyword
-// is a keyword with no replacement string, that is, a shortcut for a URL.)
-//
-// There are two possible providers for search suggestions. If the user has
-// typed a keyword, then the primary provider is the keyword provider and the
-// secondary provider is the default provider. If the user has not typed a
-// keyword, then the primary provider corresponds to the default provider.
-//
-// Search providers may supply relevance values along with their results to be
-// used in place of client-side calculated values.
-//
-// The value column gives the ranking returned from the various providers.
-// ++: a series of matches with relevance from n up to (n + max_matches).
-// --: relevance score falls off over time (discounted 50 points @ 15 minutes,
-// 450 points @ two weeks)
-// **: relevance score falls off over two days (discounted 99 points after two
-// days).
-// *~: Partial matches get a score on a sliding scale from about 575-1125 based
-// on how many times the URL for the Extension App has been typed and how
-// many of the letters match.
-
-struct AutocompleteMatch;
-class AutocompleteProvider;
-
-// AutocompleteInput ----------------------------------------------------------
-
// The user input for an autocomplete query. Allows copying.
class AutocompleteInput {
public:
@@ -332,147 +158,4 @@ class AutocompleteInput {
MatchesRequested matches_requested_;
};
-// AutocompleteResult ---------------------------------------------------------
-
-// All matches from all providers for a particular query. This also tracks
-// what the default match should be if the user doesn't manually select another
-// match.
-class AutocompleteResult {
- public:
- typedef ACMatches::const_iterator const_iterator;
- typedef ACMatches::iterator iterator;
-
- // The "Selection" struct is the information we need to select the same match
- // in one result set that was selected in another.
- struct Selection {
- Selection()
- : provider_affinity(NULL),
- is_history_what_you_typed_match(false) {
- }
-
- // Clear the selection entirely.
- void Clear();
-
- // True when the selection is empty.
- bool empty() const {
- return destination_url.is_empty() && !provider_affinity &&
- !is_history_what_you_typed_match;
- }
-
- // The desired destination URL.
- GURL destination_url;
-
- // The desired provider. If we can't find a match with the specified
- // |destination_url|, we'll use the best match from this provider.
- const AutocompleteProvider* provider_affinity;
-
- // True when this is the HistoryURLProvider's "what you typed" match. This
- // can't be tracked using |destination_url| because its URL changes on every
- // keystroke, so if this is set, we'll preserve the selection by simply
- // choosing the new "what you typed" entry and ignoring |destination_url|.
- bool is_history_what_you_typed_match;
- };
-
- // Max number of matches we'll show from the various providers.
- static const size_t kMaxMatches;
-
- // The lowest score a match can have and still potentially become the default
- // match for the result set.
- static const int kLowestDefaultScore;
-
- AutocompleteResult();
- ~AutocompleteResult();
-
- // operator=() by another name.
- void CopyFrom(const AutocompleteResult& rhs);
-
- // Copies matches from |old_matches| to provide a consistant result set. See
- // comments in code for specifics.
- void CopyOldMatches(const AutocompleteInput& input,
- const AutocompleteResult& old_matches);
-
- // Adds a single match. The match is inserted at the appropriate position
- // based on relevancy and display order. This is ONLY for use after
- // SortAndCull() has been invoked, and preserves default_match_.
- void AddMatch(const AutocompleteMatch& match);
-
- // Adds a new set of matches to the result set. Does not re-sort.
- void AppendMatches(const ACMatches& matches);
-
- // Removes duplicates, puts the list in sorted order and culls to leave only
- // the best kMaxMatches matches. Sets the default match to the best match
- // and updates the alternate nav URL.
- void SortAndCull(const AutocompleteInput& input);
-
- // Returns true if at least one match was copied from the last result.
- bool HasCopiedMatches() const;
-
- // Vector-style accessors/operators.
- size_t size() const;
- bool empty() const;
- const_iterator begin() const;
- iterator begin();
- const_iterator end() const;
- iterator end();
-
- // Returns the match at the given index.
- const AutocompleteMatch& match_at(size_t index) const;
- AutocompleteMatch* match_at(size_t index);
-
- // Get the default match for the query (not necessarily the first). Returns
- // end() if there is no default match.
- const_iterator default_match() const { return default_match_; }
-
- const GURL& alternate_nav_url() const { return alternate_nav_url_; }
-
- // Clears the matches for this result set.
- void Reset();
-
- void Swap(AutocompleteResult* other);
-
-#ifndef NDEBUG
- // Does a data integrity check on this result.
- void Validate() const;
-#endif
-
- private:
- typedef std::map<AutocompleteProvider*, ACMatches> ProviderToMatches;
-
-#if defined(OS_ANDROID)
- // iterator::difference_type is not defined in the STL that we compile with on
- // Android.
- typedef int matches_difference_type;
-#else
- typedef ACMatches::iterator::difference_type matches_difference_type;
-#endif
-
- // Populates |provider_to_matches| from |matches_|.
- void BuildProviderToMatches(ProviderToMatches* provider_to_matches) const;
-
- // Returns true if |matches| contains a match with the same destination as
- // |match|.
- static bool HasMatchByDestination(const AutocompleteMatch& match,
- const ACMatches& matches);
-
- // Copies matches into this result. |old_matches| gives the matches from the
- // last result, and |new_matches| the results from this result.
- void MergeMatchesByProvider(const ACMatches& old_matches,
- const ACMatches& new_matches);
-
- ACMatches matches_;
-
- const_iterator default_match_;
-
- // The "alternate navigation URL", if any, for this result set. This is a URL
- // to try offering as a navigational option in case the user navigated to the
- // URL of the default match but intended something else. For example, if the
- // user's local intranet contains site "foo", and the user types "foo", we
- // default to searching for "foo" when the user may have meant to navigate
- // there. In cases like this, the default match will point to the "search for
- // 'foo'" result, and this will contain "http://foo/".
- GURL alternate_nav_url_;
-
- DISALLOW_COPY_AND_ASSIGN(AutocompleteResult);
-};
-
#endif // CHROME_BROWSER_AUTOCOMPLETE_AUTOCOMPLETE_H_
« no previous file with comments | « no previous file | chrome/browser/autocomplete/autocomplete.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698