Omnibox: Make the Controller Reorder Matches for Inlining

chrome/browser/autocomplete/autocomplete_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_AUTOCOMPLETE_AUTOCOMPLETE_MATCH_H_
 #define CHROME_BROWSER_AUTOCOMPLETE_AUTOCOMPLETE_MATCH_H_
 
 #include <map>
 #include <string>
 #include <vector>
@@ skipping 233 matching lines @@
   // This string is loaded into the location bar when the item is selected
   // by pressing the arrow keys. This may be different than a URL, for example,
   // for search suggestions, this would just be the search terms.
   string16 fill_into_edit;
 
   // The position within fill_into_edit from which we'll display the inline
   // autocomplete string.  This will be string16::npos if this match should
   // not be inline autocompleted.
   size_t inline_autocomplete_offset;
 
+  // If false, the omnibox should prevent this match from being the default
+  // match.

[Peter Kasting, 2013/07/17 18:29:48]

Nit: I might add something like:

Providers should set this to true only if the user's input, plus any inline
autocompletion on this match, would lead the user to expect a navigation to this
match's destination URL.  For example, with input "foo", a search for "bar" or
navigation to "bar.com" should not set this flag; a navigation to "foo.com"
should only set this flag if ".com" will be inline autocompleted; and a
navigation to "foo/" (an intranet host) or search for "foo" could (and probably
should) set this flag.

[Mark P, 2013/07/17 20:39:18]

That's a good explanation.  Inserted with minor editing.

+  bool allowed_to_be_default_match;
+
   // The URL to actually load when the autocomplete item is selected. This URL
   // should be canonical so we can compare URLs with strcmp to avoid dupes.
   // It may be empty if there is no possible navigation.
   GURL destination_url;
 
   // The destination URL with "www." stripped off for better dupe finding.
   GURL stripped_destination_url;
 
   // The main text displayed in the address bar dropdown.
   string16 contents;
@@ skipping 61 matching lines @@
       const string16& text,
       const ACMatchClassifications& classifications) const;
 #endif
 };
 
 typedef AutocompleteMatch::ACMatchClassification ACMatchClassification;
 typedef std::vector<ACMatchClassification> ACMatchClassifications;
 typedef std::vector<AutocompleteMatch> ACMatches;
 
 #endif  // CHROME_BROWSER_AUTOCOMPLETE_AUTOCOMPLETE_MATCH_H_