Remove JavaScriptDialogTabHelper's use of DidStartNavigationToPendingEntry.

BUG=817024

Change-Id: I60d8e6b269c0c8f565e689508789713f73703e2a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1729470Reviewed-by: Leonard Grey <lgrey@chromium.org>
Commit-Queue: Avi Drissman <avi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#682878}

chrome/browser/ui/javascript_dialogs/javascript_dialog_tab_helper.cc

@@ -419,16 +419,6 @@ void JavaScriptDialogTabHelper::DidStartNavigation(
   CloseDialog(DismissalCause::kTabNavigated, false, base::string16());
 }
 
-// TODO(avi): Remove this function since it is deprecated at the
-// WebContentsObserver level.
-void JavaScriptDialogTabHelper::DidStartNavigationToPendingEntry(
-    const GURL& url,
-    content::ReloadType reload_type) {
-  // Close the dialog if the user started a new navigation. This allows reloads
-  // and history navigations to proceed.
-  CloseDialog(DismissalCause::kTabNavigated, false, base::string16());
-}
-
 #if !defined(OS_ANDROID)
 void JavaScriptDialogTabHelper::OnBrowserSetLastActive(Browser* browser) {
   if (IsWebContentsForemost(web_contents())) {

chrome/browser/ui/javascript_dialogs/javascript_dialog_tab_helper.h

@@ -126,9 +126,6 @@ class JavaScriptDialogTabHelper
   void OnVisibilityChanged(content::Visibility visibility) override;
   void DidStartNavigation(
       content::NavigationHandle* navigation_handle) override;
-  void DidStartNavigationToPendingEntry(
-      const GURL& url,
-      content::ReloadType reload_type) override;
 
 #if !defined(OS_ANDROID)
   // BrowserListObserver: