Eliminate OmniboxMetricsProvider listening to notification

components/omnibox/browser/omnibox_client.h

 // Copyright 2013 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 COMPONENTS_OMNIBOX_BROWSER_OMNIBOX_CLIENT_H_
 #define COMPONENTS_OMNIBOX_BROWSER_OMNIBOX_CLIENT_H_
 
 #include "base/basictypes.h"
 #include "components/omnibox/browser/autocomplete_provider_client.h"
 #include "components/omnibox/browser/omnibox_navigation_observer.h"
@@ skipping 119 matching lines @@
                              bool has_focus) = 0;
 
   // Called when input has been accepted.
   virtual void OnInputAccepted(const AutocompleteMatch& match) = 0;
 
   // Called when the edit model is being reverted back to its unedited state.
   virtual void OnRevert() = 0;
 
   // Called to notify clients that a URL was opened from the omnibox.
   // TODO(blundell): Eliminate this method in favor of having all //chrome-
-  // level listeners of the OMNIBOX_OPENED_URL instead observe OmniboxEditModel.
-  // Note that this is not trivial because (a) most of those listeners listen
-  // for the notification from all Profiles and (b) the notification is also
-  // sent from AutocompleteControllerAndroid, and it's unclear which listeners
-  // are also listening from it being sent from there.
+  // level listeners of the OMNIBOX_OPENED_URL instead either (a) observe
+  // OmniboxEditModel if they were listening for the notification for a single
+  // Profile, or (b) observe OmniboxEventGlobalTracker if they were listening
+  // for the notification from all Profiles.  Note however that (a) in
+  // particular is not trivial: the notification is also sent from
+  // AutocompleteControllerAndroid, and it's unclear which listeners are also
+  // listening from it being sent from there. For (b) this is not a problem, as
+  // AutocompleteControllerAndroid calls out to OmniboxEventGlobalTracker at the
+  // same time that it sends the notification.
   virtual void OnURLOpenedFromOmnibox(OmniboxLog* log) = 0;
 
   // Called when a bookmark is launched from the omnibox.
   virtual void OnBookmarkLaunched() = 0;
 
   // Discards the state for all pending and transient navigations.
   virtual void DiscardNonCommittedNavigations() = 0;
 };
 
 #endif  // COMPONENTS_OMNIBOX_BROWSER_OMNIBOX_CLIENT_H_