Omnibox: Create DemoteByType Experiment

chrome/browser/autocomplete/autocomplete_result.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_AUTOCOMPLETE_AUTOCOMPLETE_RESULT_H_
 #define CHROME_BROWSER_AUTOCOMPLETE_AUTOCOMPLETE_RESULT_H_
 
 #include <stddef.h>
 
 #include <map>
@@ skipping 118 matching lines @@
 #else
   typedef ACMatches::iterator::difference_type matches_difference_type;
 #endif
 
   // operator=() by another name.
   void CopyFrom(const AutocompleteResult& rhs);
 
   // 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);
+  void AddMatch(const AutocompleteInput& input,

[Peter Kasting, 2013/08/06 19:33:50]

I think you should just pass the actual PageClassification here instead of the
whole Input (which you don't need).  (2 places)

[Mark P, 2013/08/06 22:00:15]

Done (both places).

+                const AutocompleteMatch& match);
 
   // 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,
+  void MergeMatchesByProvider(const AutocompleteInput& input,
+                              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_RESULT_H_