Mac OS X: Show the Translate icon on Omnibox

chrome/browser/ui/cocoa/browser_window_cocoa.mm

 // Copyright (c) 2012 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/ui/cocoa/browser_window_cocoa.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/mac/mac_util.h"
 #import "base/mac/sdk_forward_declarations.h"
 #include "base/message_loop/message_loop.h"
 #include "base/prefs/pref_service.h"
 #include "base/strings/sys_string_conversions.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/download/download_shelf.h"
 #include "chrome/browser/extensions/tab_helper.h"
 #include "chrome/browser/fullscreen.h"
 #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"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/browser_window_state.h"
 #import "chrome/browser/ui/cocoa/browser/avatar_base_controller.h"
 #import "chrome/browser/ui/cocoa/browser/avatar_menu_bubble_controller.h"
 #import "chrome/browser/ui/cocoa/browser/edit_search_engine_cocoa_controller.h"
 #import "chrome/browser/ui/cocoa/browser/password_generation_bubble_controller.h"
 #import "chrome/browser/ui/cocoa/browser_window_controller.h"
@@ skipping 243 matching lines @@
 
 void BrowserWindowCocoa::UpdateLoadingAnimations(bool should_animate) {
   // Do nothing on Mac.
 }
 
 void BrowserWindowCocoa::SetStarredState(bool is_starred) {
   [controller_ setStarredState:is_starred ? YES : NO];
 }
 
 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,
                                             content::WebContents* new_contents,
                                             int index,
                                             int reason) {
   // TODO(pkasting): Perhaps the code in
   // TabStripController::activateTabWithContents should move here?  Or this
   // should call that (instead of TabStripModelObserverBridge doing so)?  It's
   // not obvious to me why Mac doesn't handle tab changes in BrowserWindow the
@@ skipping 178 matching lines @@
 void BrowserWindowCocoa::ShowBookmarkBubble(const GURL& url,
                                             bool already_bookmarked) {
   [controller_ showBookmarkBubbleForURL:url
                       alreadyBookmarked:(already_bookmarked ? YES : NO)];
 }
 
 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)
 void BrowserWindowCocoa::ShowOneClickSigninBubble(
     OneClickSigninBubbleType type,
     const base::string16& email,
     const base::string16& error_message,
     const StartSyncCallback& start_sync_callback) {
   WebContents* web_contents =
         browser_->tab_strip_model()->GetActiveWebContents();
@@ skipping 249 matching lines @@
   // further, both by another H, i.e. WebContentsView's height becomes
   // A.height() + H and RenderWidgetHostView's height becomes A.height() + 2H.
   // Strangely, the RenderWidgetHostView for the previous navigation entry also
   // gets enlarged by H.
   // I believe these "automatic" resizing are caused by setAutoresizingMask of
   // of the cocoa view in WebContentsViewMac, which defeats the purpose of
   // WebContentsDelegate::GetSizeForNewRenderView i.e. to prevent resizing of
   // RenderWidgetHostView in (2) and (3).
   return 0;
 }