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

Side by Side Diff: chrome/browser/ui/cocoa/wrench_menu/wrench_menu_controller_unittest.mm

Issue 1408643002: [Sync] Componentize synced_tab_delegate (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix GN, self review Created 5 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/command_line.h" 5 #include "base/command_line.h"
6 #include "base/mac/scoped_nsobject.h" 6 #include "base/mac/scoped_nsobject.h"
7 #include "base/run_loop.h" 7 #include "base/run_loop.h"
8 #include "base/strings/sys_string_conversions.h" 8 #include "base/strings/sys_string_conversions.h"
9 #include "base/strings/utf_string_conversions.h" 9 #include "base/strings/utf_string_conversions.h"
10 #include "chrome/app/chrome_command_ids.h" 10 #include "chrome/app/chrome_command_ids.h"
11 #include "chrome/browser/sync/profile_sync_service_factory.h" 11 #include "chrome/browser/sync/profile_sync_service_factory.h"
12 #include "chrome/browser/sync/sessions/sessions_sync_manager.h" 12 #include "chrome/browser/sync/sessions/sessions_sync_manager.h"
13 #include "chrome/browser/ui/browser_list.h" 13 #include "chrome/browser/ui/browser_list.h"
14 #include "chrome/browser/ui/browser_list_observer.h" 14 #include "chrome/browser/ui/browser_list_observer.h"
15 #include "chrome/browser/ui/cocoa/cocoa_profile_test.h" 15 #include "chrome/browser/ui/cocoa/cocoa_profile_test.h"
16 #include "chrome/browser/ui/cocoa/run_loop_testing.h" 16 #include "chrome/browser/ui/cocoa/run_loop_testing.h"
17 #import "chrome/browser/ui/cocoa/toolbar/toolbar_controller.h" 17 #import "chrome/browser/ui/cocoa/toolbar/toolbar_controller.h"
18 #import "chrome/browser/ui/cocoa/view_resizer_pong.h" 18 #import "chrome/browser/ui/cocoa/view_resizer_pong.h"
19 #import "chrome/browser/ui/cocoa/wrench_menu/wrench_menu_controller.h" 19 #import "chrome/browser/ui/cocoa/wrench_menu/wrench_menu_controller.h"
20 #include "chrome/browser/ui/sync/browser_synced_window_delegates_getter.h" 20 #include "chrome/browser/ui/sync/browser_synced_window_delegates_getter.h"
21 #include "chrome/browser/ui/toolbar/recent_tabs_builder_test_helper.h" 21 #include "chrome/browser/ui/toolbar/recent_tabs_builder_test_helper.h"
22 #include "chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h" 22 #include "chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h"
23 #include "chrome/browser/ui/toolbar/wrench_menu_model.h" 23 #include "chrome/browser/ui/toolbar/wrench_menu_model.h"
24 #include "chrome/common/chrome_switches.h" 24 #include "chrome/common/chrome_switches.h"
25 #include "chrome/grit/generated_resources.h" 25 #include "chrome/grit/generated_resources.h"
26 #include "chrome/test/base/testing_profile.h" 26 #include "chrome/test/base/testing_profile.h"
27 #include "components/sync_driver/local_device_info_provider_mock.h" 27 #include "components/sync_driver/local_device_info_provider_mock.h"
28 #include "components/sync_sessions/fake_sync_sessions_client.h"
28 #include "grit/theme_resources.h" 29 #include "grit/theme_resources.h"
29 #include "sync/api/fake_sync_change_processor.h" 30 #include "sync/api/fake_sync_change_processor.h"
30 #include "sync/api/sync_error_factory_mock.h" 31 #include "sync/api/sync_error_factory_mock.h"
31 #include "testing/gmock/include/gmock/gmock.h" 32 #include "testing/gmock/include/gmock/gmock.h"
32 #include "testing/gtest/include/gtest/gtest.h" 33 #include "testing/gtest/include/gtest/gtest.h"
33 #include "testing/gtest_mac.h" 34 #include "testing/gtest_mac.h"
34 #include "testing/platform_test.h" 35 #include "testing/platform_test.h"
35 #include "ui/base/l10n/l10n_util.h" 36 #include "ui/base/l10n/l10n_util.h"
36 #include "ui/base/resource/resource_bundle.h" 37 #include "ui/base/resource/resource_bundle.h"
37 38
(...skipping 28 matching lines...) Expand all
66 } 67 }
67 68
68 void SetUp() override { 69 void SetUp() override {
69 CocoaProfileTest::SetUp(); 70 CocoaProfileTest::SetUp();
70 ASSERT_TRUE(browser()); 71 ASSERT_TRUE(browser());
71 72
72 controller_.reset([[WrenchMenuController alloc] initWithBrowser:browser()]); 73 controller_.reset([[WrenchMenuController alloc] initWithBrowser:browser()]);
73 fake_model_.reset(new MockWrenchMenuModel); 74 fake_model_.reset(new MockWrenchMenuModel);
74 75
75 manager_.reset(new browser_sync::SessionsSyncManager( 76 manager_.reset(new browser_sync::SessionsSyncManager(
76 profile(), 77 &fake_sync_sessions_client_, profile(), local_device_.get(),
77 local_device_.get(), 78 scoped_ptr<browser_sync::LocalSessionEventRouter>(new DummyRouter()),
78 scoped_ptr<browser_sync::LocalSessionEventRouter>(
79 new DummyRouter()),
80 scoped_ptr<browser_sync::SyncedWindowDelegatesGetter>( 79 scoped_ptr<browser_sync::SyncedWindowDelegatesGetter>(
81 new browser_sync::BrowserSyncedWindowDelegatesGetter()))); 80 new browser_sync::BrowserSyncedWindowDelegatesGetter())));
82 manager_->MergeDataAndStartSyncing( 81 manager_->MergeDataAndStartSyncing(
83 syncer::SESSIONS, 82 syncer::SESSIONS,
84 syncer::SyncDataList(), 83 syncer::SyncDataList(),
85 scoped_ptr<syncer::SyncChangeProcessor>( 84 scoped_ptr<syncer::SyncChangeProcessor>(
86 new syncer::FakeSyncChangeProcessor), 85 new syncer::FakeSyncChangeProcessor),
87 scoped_ptr<syncer::SyncErrorFactory>( 86 scoped_ptr<syncer::SyncErrorFactory>(
88 new syncer::SyncErrorFactoryMock)); 87 new syncer::SyncErrorFactoryMock));
89 } 88 }
(...skipping 15 matching lines...) Expand all
105 104
106 WrenchMenuController* controller() { 105 WrenchMenuController* controller() {
107 return controller_.get(); 106 return controller_.get();
108 } 107 }
109 108
110 base::scoped_nsobject<WrenchMenuController> controller_; 109 base::scoped_nsobject<WrenchMenuController> controller_;
111 110
112 scoped_ptr<MockWrenchMenuModel> fake_model_; 111 scoped_ptr<MockWrenchMenuModel> fake_model_;
113 112
114 private: 113 private:
114 sync_sessions::FakeSyncSessionsClient fake_sync_sessions_client_;
115 scoped_ptr<browser_sync::SessionsSyncManager> manager_; 115 scoped_ptr<browser_sync::SessionsSyncManager> manager_;
116 scoped_ptr<sync_driver::LocalDeviceInfoProviderMock> local_device_; 116 scoped_ptr<sync_driver::LocalDeviceInfoProviderMock> local_device_;
117 }; 117 };
118 118
119 TEST_F(WrenchMenuControllerTest, Initialized) { 119 TEST_F(WrenchMenuControllerTest, Initialized) {
120 EXPECT_TRUE([controller() menu]); 120 EXPECT_TRUE([controller() menu]);
121 EXPECT_GE([[controller() menu] numberOfItems], 5); 121 EXPECT_GE([[controller() menu] numberOfItems], 5);
122 } 122 }
123 123
124 TEST_F(WrenchMenuControllerTest, DispatchSimple) { 124 TEST_F(WrenchMenuControllerTest, DispatchSimple) {
(...skipping 132 matching lines...) Expand 10 before | Expand all | Expand 10 after
257 BrowserRemovedObserver observer; 257 BrowserRemovedObserver observer;
258 // This is normally called by ToolbarController, but since |controller_| is 258 // This is normally called by ToolbarController, but since |controller_| is
259 // not owned by one, call it here. 259 // not owned by one, call it here.
260 [controller_ browserWillBeDestroyed]; 260 [controller_ browserWillBeDestroyed];
261 CloseBrowserWindow(); 261 CloseBrowserWindow();
262 observer.WaitUntilBrowserRemoved(); 262 observer.WaitUntilBrowserRemoved();
263 // |controller_| is released in TearDown(). 263 // |controller_| is released in TearDown().
264 } 264 }
265 265
266 } // namespace 266 } // namespace
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698