Mac OS X: Show the Translate icon on Omnibox

chrome/browser/ui/cocoa/translate/translate_bubble_controller.h

+// 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.
+
+#import <Cocoa/Cocoa.h>
+
+#include "base/memory/scoped_ptr.h"
+#import "chrome/browser/ui/cocoa/base_bubble_controller.h"
+#include "chrome/browser/ui/translate/translate_bubble_model.h"
+#include "components/translate/core/common/translate_errors.h"
+
+@class BrowserWindowController;
+
+namespace content {
+class WebContents;
+}
+
+@interface TranslateBubbleController : BaseBubbleController {

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

Needs a class comment

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

Done.

+ @private
+  content::WebContents* webContents_;
+  scoped_ptr<TranslateBubbleModel> model_;
+
+  // Whether the translation is acutually executed.

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

typo "acutually"

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

Done.

+  BOOL translateExecuted_;
+}
+
+// TODO: WebContents Observer?
+
++ (void)showBubbleWithBrowserWindowController:(BrowserWindowController*)controller
+                                  webContents:(content::WebContents*)webContents
+                                    viewState:(TranslateBubbleModel::ViewState
+                                               )viewState
+                                    errorType:(TranslateErrors::Type)errorType;

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

`git cl format` produces

+ (void)
    showBubbleWithBrowserWindowController:(BrowserWindowController *)controller
                              webContents:(content::WebContents *)webContents
                                viewState:
                                    (TranslateBubbleModel::ViewState)viewState
                                errorType:(TranslateErrors::Type)errorType;


which looks ever so slightly nicer

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

Done.

+
+@end