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

Side by Side Diff: chrome/browser/ui/toolbar/recent_tabs_sub_menu_model_unittest.cc

Issue 2354613002: [Sync] Fix namespaces for the browser_sync component. (Closed)
Patch Set: Address comments. Created 4 years, 3 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 2012 The Chromium Authors. All rights reserved. 1 // Copyright 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 "chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h" 5 #include "chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 148 matching lines...) Expand 10 before | Expand all | Expand 10 after
159 sync_sessions::OpenTabsUIDelegate* GetOpenTabsDelegate() { 159 sync_sessions::OpenTabsUIDelegate* GetOpenTabsDelegate() {
160 return manager_.get(); 160 return manager_.get();
161 } 161 }
162 162
163 void RegisterRecentTabs(RecentTabsBuilderTestHelper* helper) { 163 void RegisterRecentTabs(RecentTabsBuilderTestHelper* helper) {
164 helper->ExportToSessionsSyncManager(manager_.get()); 164 helper->ExportToSessionsSyncManager(manager_.get());
165 } 165 }
166 166
167 private: 167 private:
168 TestingProfile testing_profile_; 168 TestingProfile testing_profile_;
169 ProfileSyncServiceMock sync_service_; 169 browser_sync::ProfileSyncServiceMock sync_service_;
170 std::unique_ptr<sync_driver::SyncPrefs> sync_prefs_; 170 std::unique_ptr<sync_driver::SyncPrefs> sync_prefs_;
171 std::unique_ptr<sync_sessions::SessionsSyncManager> manager_; 171 std::unique_ptr<sync_sessions::SessionsSyncManager> manager_;
172 std::unique_ptr<sync_driver::LocalDeviceInfoProviderMock> local_device_; 172 std::unique_ptr<sync_driver::LocalDeviceInfoProviderMock> local_device_;
173 }; 173 };
174 174
175 // Test disabled "Recently closed" header with no foreign tabs. 175 // Test disabled "Recently closed" header with no foreign tabs.
176 TEST_F(RecentTabsSubMenuModelTest, NoTabs) { 176 TEST_F(RecentTabsSubMenuModelTest, NoTabs) {
177 TestRecentTabsSubMenuModel model(NULL, browser(), NULL); 177 TestRecentTabsSubMenuModel model(NULL, browser(), NULL);
178 178
179 // Expected menu: 179 // Expected menu:
(...skipping 427 matching lines...) Expand 10 before | Expand all | Expand 10 after
607 // 2 Recently closed heaer (disabled) 607 // 2 Recently closed heaer (disabled)
608 // 3 <separator> 608 // 3 <separator>
609 // 4 No tabs from other Devices 609 // 4 No tabs from other Devices
610 610
611 TestRecentTabsSubMenuModel model(NULL, browser(), NULL); 611 TestRecentTabsSubMenuModel model(NULL, browser(), NULL);
612 EXPECT_EQ(5, model.GetItemCount()); 612 EXPECT_EQ(5, model.GetItemCount());
613 EXPECT_EQ(-1, model.GetMaxWidthForItemAtIndex(2)); 613 EXPECT_EQ(-1, model.GetMaxWidthForItemAtIndex(2));
614 EXPECT_NE(-1, model.GetMaxWidthForItemAtIndex(3)); 614 EXPECT_NE(-1, model.GetMaxWidthForItemAtIndex(3));
615 EXPECT_EQ(-1, model.GetMaxWidthForItemAtIndex(4)); 615 EXPECT_EQ(-1, model.GetMaxWidthForItemAtIndex(4));
616 } 616 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc ('k') | chrome/browser/ui/views/profiles/profile_chooser_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698