| OLD | NEW |
| 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 "chrome/browser/ui/browser_command_controller.h" | 5 #include "chrome/browser/ui/browser_command_controller.h" |
| 6 | 6 |
| 7 #include "base/command_line.h" | 7 #include "base/command_line.h" |
| 8 #include "chrome/app/chrome_command_ids.h" | 8 #include "chrome/app/chrome_command_ids.h" |
| 9 #include "chrome/browser/browser_process.h" | 9 #include "chrome/browser/browser_process.h" |
| 10 #include "chrome/browser/command_updater.h" | 10 #include "chrome/browser/command_updater.h" |
| (...skipping 155 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 166 // Enable for tabbed browser. | 166 // Enable for tabbed browser. |
| 167 EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_FULLSCREEN)); | 167 EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_FULLSCREEN)); |
| 168 | 168 |
| 169 // Enabled for app windows. | 169 // Enabled for app windows. |
| 170 browser()->app_name_ = "app"; | 170 browser()->app_name_ = "app"; |
| 171 ASSERT_TRUE(browser()->is_app()); | 171 ASSERT_TRUE(browser()->is_app()); |
| 172 browser()->command_controller()->FullscreenStateChanged(); | 172 browser()->command_controller()->FullscreenStateChanged(); |
| 173 EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_FULLSCREEN)); | 173 EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_FULLSCREEN)); |
| 174 } | 174 } |
| 175 | 175 |
| 176 TEST_F(BrowserCommandControllerTest, OldAvatarMenuEnabledForOneOrMoreProfiles) { | 176 TEST_F(BrowserCommandControllerTest, AvatarAcceleratorEnabledOnDesktop) { |
| 177 if (!profiles::IsMultipleProfilesEnabled()) | 177 if (!profiles::IsMultipleProfilesEnabled()) |
| 178 return; | 178 return; |
| 179 | 179 |
| 180 // The command line is reset at the end of every test by the test suite. | |
| 181 switches::DisableNewAvatarMenuForTesting( | |
| 182 base::CommandLine::ForCurrentProcess()); | |
| 183 ASSERT_FALSE(switches::IsNewAvatarMenu()); | |
| 184 | |
| 185 TestingProfileManager testing_profile_manager( | 180 TestingProfileManager testing_profile_manager( |
| 186 TestingBrowserProcess::GetGlobal()); | 181 TestingBrowserProcess::GetGlobal()); |
| 187 ASSERT_TRUE(testing_profile_manager.SetUp()); | 182 ASSERT_TRUE(testing_profile_manager.SetUp()); |
| 188 ProfileManager* profile_manager = testing_profile_manager.profile_manager(); | 183 ProfileManager* profile_manager = testing_profile_manager.profile_manager(); |
| 189 | 184 |
| 190 chrome::BrowserCommandController command_controller(browser()); | 185 chrome::BrowserCommandController command_controller(browser()); |
| 191 const CommandUpdater* command_updater = command_controller.command_updater(); | 186 const CommandUpdater* command_updater = command_controller.command_updater(); |
| 192 | 187 |
| 193 bool enabled = true; | 188 bool enabled = true; |
| 194 #if defined(OS_CHROMEOS) | 189 #if defined(OS_CHROMEOS) |
| 195 // Chrome OS uses system tray menu to handle multi-profiles. | 190 // Chrome OS uses system tray menu to handle multi-profiles. |
| 196 enabled = false; | 191 enabled = false; |
| 197 #endif | 192 #endif |
| 198 | 193 |
| 199 testing_profile_manager.CreateTestingProfile("p1"); | 194 testing_profile_manager.CreateTestingProfile("p1"); |
| 200 ASSERT_EQ(1U, profile_manager->GetNumberOfProfiles()); | 195 ASSERT_EQ(1U, profile_manager->GetNumberOfProfiles()); |
| 201 EXPECT_EQ(enabled, command_updater->IsCommandEnabled(IDC_SHOW_AVATAR_MENU)); | 196 EXPECT_EQ(enabled, command_updater->IsCommandEnabled(IDC_SHOW_AVATAR_MENU)); |
| 202 | 197 |
| 203 testing_profile_manager.CreateTestingProfile("p2"); | 198 testing_profile_manager.CreateTestingProfile("p2"); |
| 204 ASSERT_EQ(2U, profile_manager->GetNumberOfProfiles()); | 199 ASSERT_EQ(2U, profile_manager->GetNumberOfProfiles()); |
| 205 EXPECT_EQ(enabled, command_updater->IsCommandEnabled(IDC_SHOW_AVATAR_MENU)); | 200 EXPECT_EQ(enabled, command_updater->IsCommandEnabled(IDC_SHOW_AVATAR_MENU)); |
| 206 | 201 |
| 207 testing_profile_manager.DeleteTestingProfile("p1"); | 202 testing_profile_manager.DeleteTestingProfile("p1"); |
| 208 ASSERT_EQ(1U, profile_manager->GetNumberOfProfiles()); | 203 ASSERT_EQ(1U, profile_manager->GetNumberOfProfiles()); |
| 209 EXPECT_EQ(enabled, command_updater->IsCommandEnabled(IDC_SHOW_AVATAR_MENU)); | 204 EXPECT_EQ(enabled, command_updater->IsCommandEnabled(IDC_SHOW_AVATAR_MENU)); |
| 210 | 205 |
| 211 testing_profile_manager.DeleteTestingProfile("p2"); | 206 testing_profile_manager.DeleteTestingProfile("p2"); |
| 212 } | 207 } |
| 213 | 208 |
| 214 TEST_F(BrowserCommandControllerTest, NewAvatarMenuEnabledWhenOnlyOneProfile) { | |
| 215 if (!profiles::IsMultipleProfilesEnabled()) | |
| 216 return; | |
| 217 | |
| 218 // The command line is reset at the end of every test by the test suite. | |
| 219 switches::EnableNewAvatarMenuForTesting( | |
| 220 base::CommandLine::ForCurrentProcess()); | |
| 221 | |
| 222 TestingProfileManager testing_profile_manager( | |
| 223 TestingBrowserProcess::GetGlobal()); | |
| 224 ASSERT_TRUE(testing_profile_manager.SetUp()); | |
| 225 ProfileManager* profile_manager = testing_profile_manager.profile_manager(); | |
| 226 | |
| 227 chrome::BrowserCommandController command_controller(browser()); | |
| 228 const CommandUpdater* command_updater = command_controller.command_updater(); | |
| 229 | |
| 230 testing_profile_manager.CreateTestingProfile("p1"); | |
| 231 ASSERT_EQ(1U, profile_manager->GetNumberOfProfiles()); | |
| 232 #if defined(OS_CHROMEOS) | |
| 233 // Chrome OS uses system tray menu to handle multi-profiles. | |
| 234 EXPECT_FALSE(command_updater->IsCommandEnabled(IDC_SHOW_AVATAR_MENU)); | |
| 235 #else | |
| 236 EXPECT_TRUE(command_updater->IsCommandEnabled(IDC_SHOW_AVATAR_MENU)); | |
| 237 #endif | |
| 238 testing_profile_manager.DeleteTestingProfile("p1"); | |
| 239 } | |
| 240 | |
| 241 TEST_F(BrowserCommandControllerTest, AvatarMenuAlwaysDisabledInIncognitoMode) { | 209 TEST_F(BrowserCommandControllerTest, AvatarMenuAlwaysDisabledInIncognitoMode) { |
| 242 if (!profiles::IsMultipleProfilesEnabled()) | 210 if (!profiles::IsMultipleProfilesEnabled()) |
| 243 return; | 211 return; |
| 244 | 212 |
| 245 // Set up a profile with an off the record profile. | 213 // Set up a profile with an off the record profile. |
| 246 TestingProfile::Builder normal_builder; | 214 TestingProfile::Builder normal_builder; |
| 247 scoped_ptr<TestingProfile> original_profile = normal_builder.Build(); | 215 scoped_ptr<TestingProfile> original_profile = normal_builder.Build(); |
| 248 | 216 |
| 249 // Create a new browser based on the off the record profile. | 217 // Create a new browser based on the off the record profile. |
| 250 Browser::CreateParams profile_params( | 218 Browser::CreateParams profile_params( |
| 251 original_profile->GetOffTheRecordProfile(), chrome::GetActiveDesktop()); | 219 original_profile->GetOffTheRecordProfile(), chrome::GetActiveDesktop()); |
| 252 scoped_ptr<Browser> otr_browser( | 220 scoped_ptr<Browser> otr_browser( |
| 253 chrome::CreateBrowserWithTestWindowForParams(&profile_params)); | 221 chrome::CreateBrowserWithTestWindowForParams(&profile_params)); |
| 254 | 222 |
| 255 chrome::BrowserCommandController command_controller(otr_browser.get()); | 223 chrome::BrowserCommandController command_controller(otr_browser.get()); |
| 256 const CommandUpdater* command_updater = command_controller.command_updater(); | 224 const CommandUpdater* command_updater = command_controller.command_updater(); |
| 257 | 225 |
| 258 // Both the old style and the new style avatar menu should be disabled. | 226 // The avatar menu should be disabled. |
| 259 EXPECT_FALSE(command_updater->IsCommandEnabled(IDC_SHOW_AVATAR_MENU)); | |
| 260 if (switches::IsNewAvatarMenu()) { | |
| 261 switches::DisableNewAvatarMenuForTesting( | |
| 262 base::CommandLine::ForCurrentProcess()); | |
| 263 } else { | |
| 264 switches::EnableNewAvatarMenuForTesting( | |
| 265 base::CommandLine::ForCurrentProcess()); | |
| 266 } | |
| 267 EXPECT_FALSE(command_updater->IsCommandEnabled(IDC_SHOW_AVATAR_MENU)); | 227 EXPECT_FALSE(command_updater->IsCommandEnabled(IDC_SHOW_AVATAR_MENU)); |
| 268 // The command line is reset at the end of every test by the test suite. | 228 // The command line is reset at the end of every test by the test suite. |
| 269 } | 229 } |
| 270 | 230 |
| 271 ////////////////////////////////////////////////////////////////////////////// | 231 ////////////////////////////////////////////////////////////////////////////// |
| 272 class BrowserCommandControllerFullscreenTest; | 232 class BrowserCommandControllerFullscreenTest; |
| 273 | 233 |
| 274 // A test browser window that can toggle fullscreen state. | 234 // A test browser window that can toggle fullscreen state. |
| 275 class FullscreenTestBrowserWindow : public TestBrowserWindow, | 235 class FullscreenTestBrowserWindow : public TestBrowserWindow, |
| 276 ExclusiveAccessContext { | 236 ExclusiveAccessContext { |
| (...skipping 169 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 446 | 406 |
| 447 TEST_F(BrowserCommandControllerTest, OnSigninAllowedPrefChange) { | 407 TEST_F(BrowserCommandControllerTest, OnSigninAllowedPrefChange) { |
| 448 chrome::BrowserCommandController command_controller(browser()); | 408 chrome::BrowserCommandController command_controller(browser()); |
| 449 const CommandUpdater* command_updater = command_controller.command_updater(); | 409 const CommandUpdater* command_updater = command_controller.command_updater(); |
| 450 | 410 |
| 451 // Check that the SYNC_SETUP command is updated on preference change. | 411 // Check that the SYNC_SETUP command is updated on preference change. |
| 452 EXPECT_TRUE(command_updater->IsCommandEnabled(IDC_SHOW_SYNC_SETUP)); | 412 EXPECT_TRUE(command_updater->IsCommandEnabled(IDC_SHOW_SYNC_SETUP)); |
| 453 profile()->GetPrefs()->SetBoolean(prefs::kSigninAllowed, false); | 413 profile()->GetPrefs()->SetBoolean(prefs::kSigninAllowed, false); |
| 454 EXPECT_FALSE(command_updater->IsCommandEnabled(IDC_SHOW_SYNC_SETUP)); | 414 EXPECT_FALSE(command_updater->IsCommandEnabled(IDC_SHOW_SYNC_SETUP)); |
| 455 } | 415 } |
| OLD | NEW |