| 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 "base/macros.h" | 8 #include "base/macros.h" |
| 9 #include "build/build_config.h" | 9 #include "build/build_config.h" |
| 10 #include "chrome/app/chrome_command_ids.h" | 10 #include "chrome/app/chrome_command_ids.h" |
| (...skipping 209 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 220 | 220 |
| 221 testing_profile_manager.DeleteTestingProfile("p2"); | 221 testing_profile_manager.DeleteTestingProfile("p2"); |
| 222 } | 222 } |
| 223 | 223 |
| 224 TEST_F(BrowserCommandControllerTest, AvatarMenuAlwaysDisabledInIncognitoMode) { | 224 TEST_F(BrowserCommandControllerTest, AvatarMenuAlwaysDisabledInIncognitoMode) { |
| 225 if (!profiles::IsMultipleProfilesEnabled()) | 225 if (!profiles::IsMultipleProfilesEnabled()) |
| 226 return; | 226 return; |
| 227 | 227 |
| 228 // Set up a profile with an off the record profile. | 228 // Set up a profile with an off the record profile. |
| 229 TestingProfile::Builder normal_builder; | 229 TestingProfile::Builder normal_builder; |
| 230 scoped_ptr<TestingProfile> original_profile = normal_builder.Build(); | 230 std::unique_ptr<TestingProfile> original_profile = normal_builder.Build(); |
| 231 | 231 |
| 232 // Create a new browser based on the off the record profile. | 232 // Create a new browser based on the off the record profile. |
| 233 Browser::CreateParams profile_params( | 233 Browser::CreateParams profile_params( |
| 234 original_profile->GetOffTheRecordProfile()); | 234 original_profile->GetOffTheRecordProfile()); |
| 235 scoped_ptr<Browser> otr_browser( | 235 std::unique_ptr<Browser> otr_browser( |
| 236 chrome::CreateBrowserWithTestWindowForParams(&profile_params)); | 236 chrome::CreateBrowserWithTestWindowForParams(&profile_params)); |
| 237 | 237 |
| 238 chrome::BrowserCommandController command_controller(otr_browser.get()); | 238 chrome::BrowserCommandController command_controller(otr_browser.get()); |
| 239 const CommandUpdater* command_updater = command_controller.command_updater(); | 239 const CommandUpdater* command_updater = command_controller.command_updater(); |
| 240 | 240 |
| 241 // The avatar menu should be disabled. | 241 // The avatar menu should be disabled. |
| 242 EXPECT_FALSE(command_updater->IsCommandEnabled(IDC_SHOW_AVATAR_MENU)); | 242 EXPECT_FALSE(command_updater->IsCommandEnabled(IDC_SHOW_AVATAR_MENU)); |
| 243 // The command line is reset at the end of every test by the test suite. | 243 // The command line is reset at the end of every test by the test suite. |
| 244 } | 244 } |
| 245 | 245 |
| (...skipping 147 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 393 EXPECT_TRUE(testprofile); | 393 EXPECT_TRUE(testprofile); |
| 394 testprofile->SetGuestSession(true); | 394 testprofile->SetGuestSession(true); |
| 395 | 395 |
| 396 browser()->command_controller()->FullscreenStateChanged(); | 396 browser()->command_controller()->FullscreenStateChanged(); |
| 397 EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_OPTIONS)); | 397 EXPECT_TRUE(chrome::IsCommandEnabled(browser(), IDC_OPTIONS)); |
| 398 EXPECT_FALSE(chrome::IsCommandEnabled(browser(), IDC_IMPORT_SETTINGS)); | 398 EXPECT_FALSE(chrome::IsCommandEnabled(browser(), IDC_IMPORT_SETTINGS)); |
| 399 } | 399 } |
| 400 | 400 |
| 401 TEST_F(BrowserCommandControllerTest, IncognitoModeOnSigninAllowedPrefChange) { | 401 TEST_F(BrowserCommandControllerTest, IncognitoModeOnSigninAllowedPrefChange) { |
| 402 // Set up a profile with an off the record profile. | 402 // Set up a profile with an off the record profile. |
| 403 scoped_ptr<TestingProfile> profile1 = TestingProfile::Builder().Build(); | 403 std::unique_ptr<TestingProfile> profile1 = TestingProfile::Builder().Build(); |
| 404 Profile* profile2 = profile1->GetOffTheRecordProfile(); | 404 Profile* profile2 = profile1->GetOffTheRecordProfile(); |
| 405 | 405 |
| 406 EXPECT_EQ(profile2->GetOriginalProfile(), profile1.get()); | 406 EXPECT_EQ(profile2->GetOriginalProfile(), profile1.get()); |
| 407 | 407 |
| 408 // Create a new browser based on the off the record profile. | 408 // Create a new browser based on the off the record profile. |
| 409 Browser::CreateParams profile_params(profile1->GetOffTheRecordProfile()); | 409 Browser::CreateParams profile_params(profile1->GetOffTheRecordProfile()); |
| 410 scoped_ptr<Browser> browser2( | 410 std::unique_ptr<Browser> browser2( |
| 411 chrome::CreateBrowserWithTestWindowForParams(&profile_params)); | 411 chrome::CreateBrowserWithTestWindowForParams(&profile_params)); |
| 412 | 412 |
| 413 chrome::BrowserCommandController command_controller(browser2.get()); | 413 chrome::BrowserCommandController command_controller(browser2.get()); |
| 414 const CommandUpdater* command_updater = command_controller.command_updater(); | 414 const CommandUpdater* command_updater = command_controller.command_updater(); |
| 415 | 415 |
| 416 // Check that the SYNC_SETUP command is updated on preference change. | 416 // Check that the SYNC_SETUP command is updated on preference change. |
| 417 EXPECT_TRUE(command_updater->IsCommandEnabled(IDC_SHOW_SYNC_SETUP)); | 417 EXPECT_TRUE(command_updater->IsCommandEnabled(IDC_SHOW_SYNC_SETUP)); |
| 418 profile1->GetPrefs()->SetBoolean(prefs::kSigninAllowed, false); | 418 profile1->GetPrefs()->SetBoolean(prefs::kSigninAllowed, false); |
| 419 EXPECT_FALSE(command_updater->IsCommandEnabled(IDC_SHOW_SYNC_SETUP)); | 419 EXPECT_FALSE(command_updater->IsCommandEnabled(IDC_SHOW_SYNC_SETUP)); |
| 420 } | 420 } |
| 421 | 421 |
| 422 TEST_F(BrowserCommandControllerTest, OnSigninAllowedPrefChange) { | 422 TEST_F(BrowserCommandControllerTest, OnSigninAllowedPrefChange) { |
| 423 chrome::BrowserCommandController command_controller(browser()); | 423 chrome::BrowserCommandController command_controller(browser()); |
| 424 const CommandUpdater* command_updater = command_controller.command_updater(); | 424 const CommandUpdater* command_updater = command_controller.command_updater(); |
| 425 | 425 |
| 426 // Check that the SYNC_SETUP command is updated on preference change. | 426 // Check that the SYNC_SETUP command is updated on preference change. |
| 427 EXPECT_TRUE(command_updater->IsCommandEnabled(IDC_SHOW_SYNC_SETUP)); | 427 EXPECT_TRUE(command_updater->IsCommandEnabled(IDC_SHOW_SYNC_SETUP)); |
| 428 profile()->GetPrefs()->SetBoolean(prefs::kSigninAllowed, false); | 428 profile()->GetPrefs()->SetBoolean(prefs::kSigninAllowed, false); |
| 429 EXPECT_FALSE(command_updater->IsCommandEnabled(IDC_SHOW_SYNC_SETUP)); | 429 EXPECT_FALSE(command_updater->IsCommandEnabled(IDC_SHOW_SYNC_SETUP)); |
| 430 } | 430 } |
| OLD | NEW |