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())
+    if (!base::FeatureList::IsEnabled(translate::kTranslateUI2016Q2) &&

[msw, 2016/05/03 22:33:34]

Why does this only bail for the old UI? Maybe add a comment?

[ftang, 2016/05/03 23:55:41]

The problem is currently it stop offer translate if user click to
the next page with the same language- which itself is unreasonable
at first.

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

I don't understand why this is okay for the new UI, but not the old UI.
Please seek review from someone in chrome/browser/translate/OWNERS
Also, please add an explanatory comment here, I like your other comments.

+        !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
 }