Mac OS X: Show the Translate icon on Omnibox

chrome/browser/ui/cocoa/browser_window_cocoa.mm

Index: chrome/browser/ui/cocoa/browser_window_cocoa.mm
diff --git a/chrome/browser/ui/cocoa/browser_window_cocoa.mm b/chrome/browser/ui/cocoa/browser_window_cocoa.mm
index 4287c001157e61e8229bba4a18a6b49dcd4a8b16..261432e6d0fd164f29dc48233d90e0d93b4e83e6 100644
--- a/chrome/browser/ui/cocoa/browser_window_cocoa.mm
+++ b/chrome/browser/ui/cocoa/browser_window_cocoa.mm
@@ -20,6 +20,7 @@
 #include "chrome/browser/password_manager/password_manager.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/shell_integration.h"
+#include "chrome/browser/translate/translate_tab_helper.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_command_controller.h"
 #include "chrome/browser/ui/browser_commands_mac.h"
@@ -283,7 +284,7 @@ void BrowserWindowCocoa::SetStarredState(bool is_starred) {
 }
 
 void BrowserWindowCocoa::SetTranslateIconToggled(bool is_lit) {
-  NOTIMPLEMENTED();
+  [controller_ setTranslateIconToggled:is_lit ? YES : NO];

[Nico, 2014/02/05 06:07:18]

I think you can omit ` ? YES : NO`

[hajimehoshi, 2014/02/05 11:08:03]

Done.

 }
 
 void BrowserWindowCocoa::OnActiveTabChanged(content::WebContents* old_contents,
@@ -482,7 +483,14 @@ void BrowserWindowCocoa::ShowTranslateBubble(
       content::WebContents* contents,
       TranslateBubbleModel::ViewState view_state,
       TranslateErrors::Type error_type) {
-  NOTIMPLEMENTED();
+  TranslateTabHelper* translate_tab_helper =
+      TranslateTabHelper::FromWebContents(contents);
+  LanguageState& language_state = translate_tab_helper->GetLanguageState();
+  language_state.SetTranslateEnabled(true);
+
+  [controller_ showTranslateBubbleForWebContents:contents
+                                       viewState:view_state
+                                       errorType:error_type];
 }
 
 #if defined(ENABLE_ONE_CLICK_SIGNIN)