InstantController cleanup: don't send Instant info in DidFinishNavigation

This info is already sent in NavigationEntryCommitted, which covers all
relevant cases. No need to re-send almost immediately.

Bug: none
Change-Id: I125b8dd53a2c487c129cee98eb2d09527394263a
Reviewed-on: https://chromium-review.googlesource.com/827071Reviewed-by: Chris Pickel <sfiera@chromium.org>
Commit-Queue: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#524071}

chrome/browser/ui/search/instant_controller.cc

@@ -34,17 +34,6 @@ class InstantController::TabObserver : public content::WebContentsObserver {
     }
   }
 
-  void DidFinishNavigation(
-      content::NavigationHandle* navigation_handle) override {
-    // TODO(treib): Verify if this is necessary - NavigationEntryCommitted
-    // should already cover all cases.
-    if (navigation_handle->HasCommitted() &&
-        navigation_handle->IsInMainFrame() &&
-        search::IsInstantNTP(web_contents())) {
-      callback_.Run();
-    }
-  }
-
   base::Closure callback_;
 
   DISALLOW_COPY_AND_ASSIGN(TabObserver);