Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(90)

Side by Side Diff: components/translate/core/browser/translate_infobar_delegate.cc

Issue 2919343007: Check |errorCode| of translate.js and notify to Browser (Closed)
Patch Set: Addressed review comment. Proper order of enum variables in switch case. Created 3 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/translate/core/browser/translate_infobar_delegate.h" 5 #include "components/translate/core/browser/translate_infobar_delegate.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/i18n/string_compare.h" 10 #include "base/i18n/string_compare.h"
(...skipping 210 matching lines...) Expand 10 before | Expand all | Expand 10 after
221 UMA_HISTOGRAM_ENUMERATION("Translate.ShowErrorInfobar", 221 UMA_HISTOGRAM_ENUMERATION("Translate.ShowErrorInfobar",
222 error_type_, 222 error_type_,
223 TranslateErrors::TRANSLATE_ERROR_MAX); 223 TranslateErrors::TRANSLATE_ERROR_MAX);
224 ui_delegate_.OnErrorShown(error_type_); 224 ui_delegate_.OnErrorShown(error_type_);
225 switch (error_type_) { 225 switch (error_type_) {
226 case TranslateErrors::NETWORK: 226 case TranslateErrors::NETWORK:
227 return l10n_util::GetStringUTF16( 227 return l10n_util::GetStringUTF16(
228 IDS_TRANSLATE_INFOBAR_ERROR_CANT_CONNECT); 228 IDS_TRANSLATE_INFOBAR_ERROR_CANT_CONNECT);
229 case TranslateErrors::INITIALIZATION_ERROR: 229 case TranslateErrors::INITIALIZATION_ERROR:
230 case TranslateErrors::TRANSLATION_ERROR: 230 case TranslateErrors::TRANSLATION_ERROR:
231 case TranslateErrors::TRANSLATION_TIMEOUT:
232 case TranslateErrors::UNEXPECTED_SCRIPT_ERROR:
233 case TranslateErrors::BAD_ORIGIN:
234 case TranslateErrors::SCRIPT_LOAD_ERROR:
231 return l10n_util::GetStringUTF16( 235 return l10n_util::GetStringUTF16(
232 IDS_TRANSLATE_INFOBAR_ERROR_CANT_TRANSLATE); 236 IDS_TRANSLATE_INFOBAR_ERROR_CANT_TRANSLATE);
233 case TranslateErrors::UNKNOWN_LANGUAGE: 237 case TranslateErrors::UNKNOWN_LANGUAGE:
234 return l10n_util::GetStringUTF16( 238 return l10n_util::GetStringUTF16(
235 IDS_TRANSLATE_INFOBAR_UNKNOWN_PAGE_LANGUAGE); 239 IDS_TRANSLATE_INFOBAR_UNKNOWN_PAGE_LANGUAGE);
236 case TranslateErrors::UNSUPPORTED_LANGUAGE: 240 case TranslateErrors::UNSUPPORTED_LANGUAGE:
237 return l10n_util::GetStringFUTF16( 241 return l10n_util::GetStringFUTF16(
238 IDS_TRANSLATE_INFOBAR_UNSUPPORTED_PAGE_LANGUAGE, 242 IDS_TRANSLATE_INFOBAR_UNSUPPORTED_PAGE_LANGUAGE,
239 target_language_name()); 243 target_language_name());
240 case TranslateErrors::IDENTICAL_LANGUAGES: 244 case TranslateErrors::IDENTICAL_LANGUAGES:
(...skipping 179 matching lines...) Expand 10 before | Expand all | Expand 10 after
420 UMA_HISTOGRAM_BOOLEAN("Translate.DeclineTranslateCloseInfobar", true); 424 UMA_HISTOGRAM_BOOLEAN("Translate.DeclineTranslateCloseInfobar", true);
421 } 425 }
422 } 426 }
423 427
424 TranslateInfoBarDelegate* 428 TranslateInfoBarDelegate*
425 TranslateInfoBarDelegate::AsTranslateInfoBarDelegate() { 429 TranslateInfoBarDelegate::AsTranslateInfoBarDelegate() {
426 return this; 430 return this;
427 } 431 }
428 432
429 } // namespace translate 433 } // namespace translate
OLDNEW
« no previous file with comments | « components/translate/content/renderer/translate_helper.cc ('k') | components/translate/core/common/translate_errors.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698