Check |errorCode| of translate.js and notify to Browser

components/translate/core/browser/translate_infobar_delegate.cc

Index: components/translate/core/browser/translate_infobar_delegate.cc
diff --git a/components/translate/core/browser/translate_infobar_delegate.cc b/components/translate/core/browser/translate_infobar_delegate.cc
index cb32c684ddd1bc393344f6b502f3b3e7c22abfd9..f2df04181d9f8b5b755d621799ad4459f6af37ed 100644
--- a/components/translate/core/browser/translate_infobar_delegate.cc
+++ b/components/translate/core/browser/translate_infobar_delegate.cc
@@ -227,6 +227,10 @@ base::string16 TranslateInfoBarDelegate::GetMessageInfoBarText() {
       return l10n_util::GetStringUTF16(
           IDS_TRANSLATE_INFOBAR_ERROR_CANT_CONNECT);

[Takashi Toyoshima, 2017/07/05 05:37:17]

coding style nits:
Basically the "case" is placed in the order of enum declaration though
TRANSLATE_ERROR was placed together with INITIALIZATION_ERROR to return the same
value.

So, could you re-oreder SCRIPT_LOAD_ERROR through TRANSLATION_ERROR  below to
follow the enum declaration order?

It should be

INITIALIZATION_ERROR
TRANSLATION_ERROR
TRANSLATION_TIMEOUT
UNEXPECTED_SCRIPT_ERROR
BAD_ORIGIN
SCRIPT_LOAD_ERROR

[Gaja, 2017/07/05 05:48:42]

Done.

     case TranslateErrors::INITIALIZATION_ERROR:
+    case TranslateErrors::SCRIPT_LOAD_ERROR:
+    case TranslateErrors::UNEXPECTED_SCRIPT_ERROR:
+    case TranslateErrors::TRANSLATION_TIMEOUT:
+    case TranslateErrors::BAD_ORIGIN:
     case TranslateErrors::TRANSLATION_ERROR:
       return l10n_util::GetStringUTF16(
           IDS_TRANSLATE_INFOBAR_ERROR_CANT_TRANSLATE);