Implement the 2016Q2 Translate UI designe spec out in

chrome/browser/translate/chrome_translate_client.cc

 // Copyright 2014 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.
 
 #include "chrome/browser/translate/chrome_translate_client.h"
 
 #include <vector>
 
 #include "base/logging.h"
 #include "base/path_service.h"
@@ skipping 172 matching lines @@
   if (error_type != translate::TranslateErrors::NONE)
     step = translate::TRANSLATE_STEP_TRANSLATE_ERROR;
 
 #if !defined(USE_AURA)
   if (!TranslateService::IsTranslateBubbleEnabled()) {
     // Infobar UI.
     translate::TranslateInfoBarDelegate::Create(
         step != translate::TRANSLATE_STEP_BEFORE_TRANSLATE,
         translate_manager_->GetWeakPtr(),
         InfoBarService::FromWebContents(web_contents()),
-        web_contents()->GetBrowserContext()->IsOffTheRecord(),
-        step,
-        source_language,
-        target_language,
-        error_type,
-        triggered_from_menu);
+        web_contents()->GetBrowserContext()->IsOffTheRecord(), step,
+        source_language, target_language, error_type, triggered_from_menu);
     return;
   }
 #endif
 
   // Bubble UI.
   if (step == translate::TRANSLATE_STEP_BEFORE_TRANSLATE) {
     // TODO(droger): Move this logic out of UI code.
     GetLanguageState().SetTranslateEnabled(true);
-    if (!GetLanguageState().HasLanguageChanged())
+    // In the new UI, keep offer the translation after user navigate to

[msw, 2016/05/04 00:19:20]

grammar nit: "continue offering translation after the user navigates to another
page"... I'm not sure why this makes sense for the new UI and not the old UI,
but I presume someone more familiar with translation will review this CL.

[ftang, 2016/05/04 00:34:01]

Done. from my point of view, the old UI does not make sense but different
people have different opinion and there will be just endless arguments base
on what the people believe. So, instead of argue that for both new and old UI,
we just change that in the new UI.

+    // another page.
+    if (!base::FeatureList::IsEnabled(translate::kTranslateUI2016Q2) &&
+        !GetLanguageState().HasLanguageChanged()) {
       return;
+    }
 
     if (!triggered_from_menu) {
       if (web_contents()->GetBrowserContext()->IsOffTheRecord())
         return;
       if (GetTranslatePrefs()->IsTooOftenDenied(source_language))
         return;
     }
   }
   ShowBubble(step, error_type);
 }
@@ skipping 42 matching lines @@
   // Android does not support reporting language detection errors.
   NOTREACHED();
 #else
   // We'll open the URL in a new tab so that the user can tell us more.
   Browser* browser = chrome::FindBrowserWithWebContents(web_contents());
   if (!browser) {
     NOTREACHED();
     return;
   }
 
-  chrome::AddSelectedTabWithURL(
-      browser, report_url, ui::PAGE_TRANSITION_AUTO_BOOKMARK);
+  chrome::AddSelectedTabWithURL(browser, report_url,
+                                ui::PAGE_TRANSITION_AUTO_BOOKMARK);
 #endif  // defined(OS_ANDROID)
 }
 
 void ChromeTranslateClient::WebContentsDestroyed() {
   // Translation process can be interrupted.
   // Destroying the TranslateManager now guarantees that it never has to deal
   // with NULL WebContents.
   translate_manager_.reset();
 }
 
@@ skipping 51 matching lines @@
   if (browser != chrome::FindLastActive())
     return;
 
   // During auto-translating, the bubble should not be shown.
   if (step == translate::TRANSLATE_STEP_TRANSLATING ||
       step == translate::TRANSLATE_STEP_AFTER_TRANSLATE) {
     if (GetLanguageState().InTranslateNavigation())
       return;
   }
 
-  TranslateBubbleFactory::Show(
-      browser->window(), web_contents(), step, error_type);
+  TranslateBubbleFactory::Show(browser->window(), web_contents(), step,
+                               error_type);
 #else
   NOTREACHED();
 #endif
 }