removing ShouldHideTopMatch

components/omnibox/autocomplete_result.cc

Index: components/omnibox/autocomplete_result.cc
diff --git a/components/omnibox/autocomplete_result.cc b/components/omnibox/autocomplete_result.cc
index f2d171f71353ff9eecb46eccfab2ca3761119898..84d68009380151ff94aed7033e6884ed0788e262 100644
--- a/components/omnibox/autocomplete_result.cc
+++ b/components/omnibox/autocomplete_result.cc
@@ -294,25 +294,6 @@ AutocompleteMatch* AutocompleteResult::match_at(size_t index) {
   return &matches_[index];
 }
 
-bool AutocompleteResult::ShouldHideTopMatch() const {
-  return chrome::ShouldHideTopVerbatimMatch() &&
-      TopMatchIsStandaloneVerbatimMatch();
-}
-
-bool AutocompleteResult::TopMatchIsStandaloneVerbatimMatch() const {
-  if (empty() || !match_at(0).IsVerbatimType())

[Peter Kasting, 2015/05/28 23:06:24]

These are the only uses of IsVerbatimType(), so you can remove that function as
well.

[dschuyler, 2015/05/29 20:38:39]

Done.

-    return false;
-
-  // Skip any copied matches, under the assumption that they'll be expired and
-  // disappear.  We don't want this disappearance to cause the visibility of the
-  // top match to change.
-  for (const_iterator i(begin() + 1); i != end(); ++i) {
-    if (!i->from_previous)
-      return !i->IsVerbatimType();
-  }
-  return true;
-}
-
 void AutocompleteResult::Reset() {
   matches_.clear();
   default_match_ = end();