Use new WebStateObserver method in IOSTranslateDriver.

ReplaceState is treated as a navigation, but the old method
"navigationItemCommited" was not called in that case and would mess up
translate_manager's language_state, causing a incorrect translation offer.

Bug: 780578
Change-Id: Ie4a5f55c7020a4d76168fcb8e2388a7d47fc815f
Reviewed-on: https://chromium-review.googlesource.com/767043Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Eugene But <eugenebut@chromium.org>
Commit-Queue: John Wu <jzw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#516380}

components/translate/ios/browser/ios_translate_driver.h

@@ -50,9 +50,8 @@ class IOSTranslateDriver : public TranslateDriver,
   CreateLanguageDetectionCallback();
 
   // web::WebStateObserver methods.
-  void NavigationItemCommitted(
-      web::WebState* web_state,
-      const web::LoadCommittedDetails& load_details) override;
+  void DidFinishNavigation(web::WebState* web_state,
+                           web::NavigationContext* navigation_context) override;
   void WebStateDestroyed(web::WebState* web_state) override;
 
   // TranslateDriver methods.

components/translate/ios/browser/ios_translate_driver.mm

@@ -20,11 +20,11 @@
 #import "components/translate/ios/browser/language_detection_controller.h"
 #import "components/translate/ios/browser/translate_controller.h"
 #include "ios/web/public/browser_state.h"
-#include "ios/web/public/load_committed_details.h"
 #include "ios/web/public/navigation_item.h"
 #include "ios/web/public/navigation_manager.h"
 #include "ios/web/public/referrer.h"
 #include "ios/web/public/web_state/js/crw_js_injection_receiver.h"
+#include "ios/web/public/web_state/navigation_context.h"
 #include "ios/web/public/web_state/web_state.h"
 #include "ui/base/page_transition_types.h"
 #include "ui/base/window_open_disposition.h"
@@ -110,24 +110,24 @@ void IOSTranslateDriver::OnLanguageDetermined(
 
 // web::WebStateObserver methods
 
-void IOSTranslateDriver::NavigationItemCommitted(
+void IOSTranslateDriver::DidFinishNavigation(
     web::WebState* web_state,
-    const web::LoadCommittedDetails& load_details) {
+    web::NavigationContext* navigation_context) {
   DCHECK_EQ(web_state_, web_state);
 
   // Interrupt pending translations and reset various data when a navigation
   // happens. Desktop does it by tracking changes in the page ID, and
   // through WebContentObserver, but these concepts do not exist on iOS.
-  if (!load_details.is_in_page) {
+  if (!navigation_context->IsSameDocument()) {
     ++page_seq_no_;
     translate_manager_->set_current_seq_no(page_seq_no_);
   }
 
   // TODO(droger): support navigation types, like content/ does.
   const bool reload = ui::PageTransitionCoreTypeIs(
-      load_details.item->GetTransitionType(), ui::PAGE_TRANSITION_RELOAD);
-  translate_manager_->GetLanguageState().DidNavigate(load_details.is_in_page,
-                                                     true, reload);
+      navigation_context->GetPageTransition(), ui::PAGE_TRANSITION_RELOAD);
+  translate_manager_->GetLanguageState().DidNavigate(
+      navigation_context->IsSameDocument(), true, reload);
 }
 
 void IOSTranslateDriver::WebStateDestroyed(web::WebState* web_state) {