Implemented new Translate API for purely Objective-C clients.

ios/web_view/internal/translate/web_view_translate_client.mm

Index: ios/web_view/internal/translate/web_view_translate_client.mm
diff --git a/ios/web_view/internal/translate/web_view_translate_client.mm b/ios/web_view/internal/translate/web_view_translate_client.mm
index 6b8ee429b9f5a4d9318d59ca9c9976062c7e9da7..e6d4a9f74da1fe2b8cb58b5ef54236122db65b82 100644
--- a/ios/web_view/internal/translate/web_view_translate_client.mm
+++ b/ios/web_view/internal/translate/web_view_translate_client.mm
@@ -20,11 +20,10 @@
 #include "ios/web/public/browser_state.h"
 #import "ios/web/public/web_state/web_state.h"
 #include "ios/web_view/internal/pref_names.h"
-#import "ios/web_view/internal/translate/cwv_translate_manager_impl.h"
+#import "ios/web_view/internal/translate/cwv_translation_controller_internal.h"
 #include "ios/web_view/internal/translate/web_view_translate_accept_languages_factory.h"
 #include "ios/web_view/internal/translate/web_view_translate_ranker_factory.h"
 #include "ios/web_view/internal/web_view_browser_state.h"
-#import "ios/web_view/public/cwv_translate_delegate.h"
 #include "url/gurl.h"
 
 #if !defined(__has_feature) || !__has_feature(objc_arc)
@@ -63,12 +62,6 @@ void WebViewTranslateClient::ShowTranslateUI(
     const std::string& target_language,
     translate::TranslateErrors::Type error_type,
     bool triggered_from_menu) {
-  if (!delegate_.get())
-    return;
-
-  if (error_type != translate::TranslateErrors::NONE)
-    step = translate::TRANSLATE_STEP_TRANSLATE_ERROR;
-
   translate_manager_->GetLanguageState().SetTranslateEnabled(true);
 
   if (step == translate::TRANSLATE_STEP_BEFORE_TRANSLATE &&
@@ -76,32 +69,15 @@ void WebViewTranslateClient::ShowTranslateUI(
     return;
   }
 
-  base::scoped_nsobject<CWVTranslateManagerImpl> criwv_manager(
-      [[CWVTranslateManagerImpl alloc]
-          initWithTranslateManager:translate_manager_.get()
-                    sourceLanguage:source_language
-                    targetLanguage:target_language]);
-
-  CRIWVTransateStep criwv_step;
-  switch (step) {
-    case translate::TRANSLATE_STEP_BEFORE_TRANSLATE:
-      criwv_step = CRIWVTransateStepBeforeTranslate;
-      break;
-    case translate::TRANSLATE_STEP_TRANSLATING:
-      criwv_step = CRIWVTransateStepTranslating;
-      break;
-    case translate::TRANSLATE_STEP_AFTER_TRANSLATE:
-      criwv_step = CRIWVTransateStepAfterTranslate;
-      break;
-    case translate::TRANSLATE_STEP_TRANSLATE_ERROR:
-      criwv_step = CRIWVTransateStepError;
-      break;
-    case translate::TRANSLATE_STEP_NEVER_TRANSLATE:
-      NOTREACHED() << "Never translate is not supported yet in web_view.";
-      criwv_step = CRIWVTransateStepError;
-      break;
+  if (!translation_controller_) {

[Eugene But (OOO till 7-30), 2017/04/28 07:18:07]

There is no need for this check calling method on nil is totally safe in
Objective-C 

[jzw1, 2017/04/28 09:55:33]

I see. I need to read more about these wrapper classes :P

+    return;
   }
-  [delegate_ translateStepChanged:criwv_step manager:criwv_manager.get()];
+
+  [translation_controller_ updateTranslateStep:step
+                                sourceLanguage:source_language
+                                targetLanguage:target_language
+                                     errorType:error_type
+                             triggeredFromMenu:triggered_from_menu];
 }
 
 translate::TranslateDriver* WebViewTranslateClient::GetTranslateDriver() {