Chromium Code Reviews| Index: chrome/browser/ui/android/infobars/translate_infobar.cc |
| diff --git a/chrome/browser/ui/android/infobars/translate_infobar.cc b/chrome/browser/ui/android/infobars/translate_infobar.cc |
| index b49efe28f6204e1e77e39d550bb96b02679ea77d..fc237c07333c9502b0601b5e0c0e606ffa3c0862 100644 |
| --- a/chrome/browser/ui/android/infobars/translate_infobar.cc |
| +++ b/chrome/browser/ui/android/infobars/translate_infobar.cc |
| @@ -17,6 +17,7 @@ |
| #include "chrome/browser/translate/android/translate_utils.h" |
| #include "chrome/browser/translate/chrome_translate_client.h" |
| #include "chrome/browser/ui/android/infobars/translate_compact_infobar.h" |
| +#include "components/translate/core/browser/translate_browser_metrics.h" |
| #include "components/translate/core/browser/translate_infobar_delegate.h" |
| #include "jni/TranslateInfoBar_jni.h" |
| @@ -28,12 +29,19 @@ using base::android::ScopedJavaLocalRef; |
| std::unique_ptr<infobars::InfoBar> ChromeTranslateClient::CreateInfoBar( |
| std::unique_ptr<translate::TranslateInfoBarDelegate> delegate) const { |
| - if (base::FeatureList::IsEnabled(translate::kTranslateCompactUI)) |
| + if (base::FeatureList::IsEnabled(translate::kTranslateCompactUI)) { |
| + translate::TranslateBrowserMetrics::ReportInitiationStatus( |
|
Leo
2017/07/06 06:36:42
We need to add same condition (TRANSLATE_STEP_BEFO
ramyasharma
2017/07/06 06:49:49
Thanks Leo. Good point. I have moved the logging a
|
| + translate::TranslateBrowserMetrics::INITIATION_STATUS_CREATE_INFOBAR); |
| return base::MakeUnique<TranslateCompactInfoBar>(std::move(delegate)); |
| - else |
| - return base::MakeUnique<TranslateInfoBar>(std::move(delegate)); |
| -} |
| + } |
| + if (delegate.get()->translate_step() == |
| + translate::TRANSLATE_STEP_BEFORE_TRANSLATE) { |
| + translate::TranslateBrowserMetrics::ReportInitiationStatus( |
| + translate::TranslateBrowserMetrics::INITIATION_STATUS_CREATE_INFOBAR); |
| + } |
| + return base::MakeUnique<TranslateInfoBar>(std::move(delegate)); |
| +} |
| // TranslateInfoBar ----------------------------------------------------------- |