removing ShouldHideTopMatch

components/metrics/proto/omnibox_event.proto

Index: components/metrics/proto/omnibox_event.proto
diff --git a/components/metrics/proto/omnibox_event.proto b/components/metrics/proto/omnibox_event.proto
index b3e6656344a1e180e94ee92726a9f79fba17123a..6980355ced14a3245e10b8bfc25b6d60b29136bd 100644
--- a/components/metrics/proto/omnibox_event.proto
+++ b/components/metrics/proto/omnibox_event.proto
@@ -39,9 +39,9 @@ message OmniboxEventProto {
   // This corresponds the index of the |suggestion| below.
   optional int32 selected_index = 5;
 
-  // Whether or not the top match was hidden in the omnibox suggestions
-  // dropdown.
-  optional bool is_top_result_hidden_in_dropdown = 14;
+  // DEPRECATED. Whether or not the top match was hidden in the omnibox

[Mark P, 2015/06/01 19:23:03]

This proto change looks good to me.  Please make this change internally first,
submit it, and then I'll approve this external changelist.

[dschuyler, 2015/06/09 23:01:45]

I'm hoping I did the right internal change here:
https://critique.corp.google.com/#review/94695891

Done.

+  // suggestions dropdown.
+  optional bool is_top_result_hidden_in_dropdown = 14 [deprecated=true];
 
   // Whether the omnibox popup is open.  It can be closed if, for instance,
   // the user clicks in the omnibox and hits return to reload the same page.