InstantExtended(gtk): Hide top match if told to so.

chrome/browser/metrics/metrics_log.cc

Index: chrome/browser/metrics/metrics_log.cc
diff --git a/chrome/browser/metrics/metrics_log.cc b/chrome/browser/metrics/metrics_log.cc
index 7a0f21049fd8f365287ef6e9d556370a2fdd4b92..9b18ec53d10a914f363f7981c8175612958a2a7e 100644
--- a/chrome/browser/metrics/metrics_log.cc
+++ b/chrome/browser/metrics/metrics_log.cc
@@ -860,13 +860,8 @@ void MetricsLog::RecordOmniboxOpenedURL(const OmniboxLog& log) {
   omnibox_event->set_current_page_classification(
       AsOmniboxEventPageClassification(log.current_page_classification));
   omnibox_event->set_input_type(AsOmniboxEventInputType(log.input_type));
-
-  // The view code to hide the top result is currently only implemented on the
-  // Mac and for views.
-#if defined(OS_MACOSX) || defined(TOOLKIT_VIEWS)

[Mark P, 2013/09/14 21:58:45]

Is this necessarily true?  We've implemented it for Mac, Views, and GTK.  Does
this mean now it works on Android and iOS?  I don't know how their omnibox is
rendered, but I thought, for instance, that Android has special views code, so
recording ShouldHideTopMatch on Android (when you're not actually hiding the top
match) will be misleading.

   omnibox_event->set_is_top_result_hidden_in_dropdown(
       log.result.ShouldHideTopMatch());
-#endif  // defined(OS_MACOSX) || defined(TOOLKIT_VIEWS)
 
   for (AutocompleteResult::const_iterator i(log.result.begin());
        i != log.result.end(); ++i) {