Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(835)

Unified Diff: chrome/browser/ui/cocoa/translate/translate_bubble_controller_unittest.mm

Issue 151283006: Mac OS X: Show the Translate icon on Omnibox (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@issue-307352-translate-bubble-2
Patch Set: Nico's review Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/translate/translate_bubble_controller_unittest.mm
diff --git a/chrome/browser/ui/cocoa/translate/translate_bubble_controller_unittest.mm b/chrome/browser/ui/cocoa/translate/translate_bubble_controller_unittest.mm
new file mode 100644
index 0000000000000000000000000000000000000000..ea214a66961edc4199ea528ac94ff65f203ff9f8
--- /dev/null
+++ b/chrome/browser/ui/cocoa/translate/translate_bubble_controller_unittest.mm
@@ -0,0 +1,33 @@
+// 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/ui/cocoa/translate/translate_bubble_controller.h"
+
+#import "chrome/browser/ui/browser.h"
+#import "chrome/browser/ui/cocoa/browser_window_controller.h"
+#import "chrome/browser/ui/cocoa/cocoa_profile_test.h"
+#include "chrome/common/url_constants.h"
+
+@interface TranslateBubbleController (ForTests)
+
+- (id)initWithBrowserWindowController:(BrowserWindowController*)controller
+ model:(scoped_ptr<TranslateBubbleModel>)model
+ webContents:(content::WebContents*)webContents;
+
+@end
+
+class TranslateBubbleControllerTest : public CocoaProfileTest {
+ public:
+ virtual void SetUp() OVERRIDE {
+ CocoaProfileTest::SetUp();
+ }
+
+ virtual void TearDown() OVERRIDE {
+ CocoaProfileTest::TearDown();
+ }
+};
+
+TEST_F(TranslateBubbleControllerTest, Show) {
groby-ooo-7-16 2014/02/11 00:43:55 Please do add this test. See e.g. new_credit_card_
hajimehoshi 2014/02/25 09:31:07 Done.
+ // fixme: Implement the test to show the bubble and check the state.
+}

Powered by Google App Engine
This is Rietveld 408576698