Modified GetMatchToPrefetch() to return a valid match for prefetching even when hide_verbatim flag i

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 105 matching lines @@
   // suggested matches.  This makes it more likely that users will see and
   // select useful non-verbatim matches.  (Note that hiding the verbatim match
   // this way is similar to how most other browsers' address bars behave.)
   //
   // We avoid hiding when the top two matches are both verbatim in order to
   // avoid potential confusion if a user were to see the second match just below
   // their typing and assume it would be the default action.
   //
   // Note that if the top match should be hidden and it is the only match,
   // the dropdown should be closed.
   bool ShouldHideTopMatch() const;

[Mark P, 2013/12/12 23:13:07]

I wouldn't be surprised if Peter asks you to revise this comment somewhat.  I'm
tempted to make the first sentence more focused on hiding and less on when to
hide.  And then you can probably make reference to the function below and not
worry about re-explaining it.

But I'd actually prefer you not revise anything now and simply wait for Peter's
comments and fix them in a second changelist after this one is submitted.

[kmadhusu, 2013/12/13 01:00:29]

Agreed. I will update the function comments in a follow up CL.

 
+  // Returns true if the top match is a verbatim search or URL match (see
+  // IsVerbatimType() in autocomplete_match.h), and the next match is not also
+  // some kind of verbatim match.
+  bool TopMatchIsVerbatimAndHasNoConsecutiveVerbatimMatches() const;
+
   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;
@@ skipping 52 matching lines @@
   // 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_