Translate: Hide the bubble if the user denies translating 2 times within 24 hours

chrome/browser/ui/views/frame/browser_view.cc

Index: chrome/browser/ui/views/frame/browser_view.cc
diff --git a/chrome/browser/ui/views/frame/browser_view.cc b/chrome/browser/ui/views/frame/browser_view.cc
index a467ff2a85e006b5a8fc1f7d577bcbc83dd47bf5..9b18de83886558feedc59629dff401aca5d2f76b 100644
--- a/chrome/browser/ui/views/frame/browser_view.cc
+++ b/chrome/browser/ui/views/frame/browser_view.cc
@@ -1166,6 +1166,15 @@ void BrowserView::ShowTranslateBubble(content::WebContents* web_contents,
     content::RenderViewHost* rvh = web_contents->GetRenderViewHost();
     if (rvh->IsFocusedElementEditable())
       return;
+
+    if (IsOffTheRecord())

[sky, 2014/06/19 15:46:46]

This code is not platform specific. Can it be moved to non-platform specific
location? Perhaps browser_commands.cc:Translate?

[hajimehoshi, 2014/06/20 11:38:45]

Done. Moved to ChromeTranslateClient.

BrowserCommands is now used only when clicking the translate icon. Clicking icon
calls ShowTranslateBubble directly for now. The other triggers (the context menu
and automatic translation) uses ChromeTranslateClient. It seems better to unify
these ways to use ChromeTranslateClient.

+      return;
+
+    PrefService* pref = browser_->profile()->GetOriginalProfile()->GetPrefs();
+    scoped_ptr<TranslatePrefs> translate_prefs =
+        ChromeTranslateClient::CreateTranslatePrefs(pref);
+    if (translate_prefs->IsBubbleHidden())
+      return;
   }
 
   ChromeTranslateClient* chrome_translate_client =