Implemented new Translate API for purely Objective-C clients.

ios/web_view/internal/translate/cwv_translation_controller_internal.h

+// Copyright 2017 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 IOS_WEB_VIEW_INTERNAL_TRANSLATE_CWV_TRANSLATION_CONTROLLER_INTERNAL_H
+#define IOS_WEB_VIEW_INTERNAL_TRANSLATE_CWV_TRANSLATION_CONTROLLER_INTERNAL_H
+
+#import "ios/web_view/public/cwv_translation_controller.h"
+
+#include "components/translate/core/browser/translate_step.h"

[michaeldo, 2017/05/01 18:25:37]

Please replace translate_step and web_state with "class" forward declarations.

[jzw1, 2017/05/08 03:36:28]

How do I forward declare the translate_step enum? I'm looking at
translate_client.h and it seems they also import it directly.

[michaeldo, 2017/05/08 15:15:48]

Sorry, didn't realize it was for the enum. You can't as far as I know, but if it
forces "lots" of full includes, you can move the enum to it's own header. No
need to do that now, but that is one possibility.

+#include "components/translate/core/common/translate_errors.h"
+#import "ios/web/public/web_state/web_state.h"
+#import "ios/web_view/internal/translate/web_view_translate_client.h"

[michaeldo, 2017/05/01 18:25:37]

Is web_view_translate_client.h import needed?

[jzw1, 2017/05/08 03:36:28]

Thanks for catch

+
+NS_ASSUME_NONNULL_BEGIN
+
+// Some internal methods needed to hide any C++ details.
+@interface CWVTranslationController ()
+
+// Set to create a new translate stack under the new webState.
+@property(nonatomic, assign) web::WebState* webState;
+
+// Called to keep this class informed of the current translate progress.
+// |step| the state of current translation.
+// |sourceLanguage| the source language associated with the current |step|.
+// |targetLanguage| the target langauge associated with the current |step|.
+// |errorType| the error, if any for the current |step|.
+// |triggeredFromMenu| should be true if this was a result from user action.
+- (void)updateTranslateStep:(translate::TranslateStep)step
+             sourceLanguage:(const std::string&)sourceLanguage
+             targetLanguage:(const std::string&)targetLanguage
+                  errorType:(translate::TranslateErrors::Type)errorType
+          triggeredFromMenu:(bool)triggeredFromMenu;
+
+@end
+
+NS_ASSUME_NONNULL_END
+
+#endif  // IOS_WEB_VIEW_INTERNAL_TRANSLATE_CWV_TRANSLATION_CONTROLLER_INTERNAL_H