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..abc6a0402372b9caf7904027b7ec8614274e9ca7 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
+  // suggestions dropdown.
+  optional bool is_top_result_hidden_in_dropdown = 14 [deprecated = true];

[Mark P, 2015/06/10 04:26:18]

Given that changed the name of the field internally, you should change the name
of the field here too.  We want the internal and external files to remain in
sync.

[dschuyler, 2015/06/10 18:29:33]

Done.

 
   // 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.