| // Copyright 2016 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/ash/chrome_new_window_client.h" |
| |
| #include "chrome/browser/chromeos/arc/arc_web_contents_data.h" |
| #include "chrome/browser/chromeos/profiles/profile_helper.h" |
| #include "chrome/browser/extensions/bookmark_app_navigation_browsertest.h" |
| #include "chrome/browser/prefs/incognito_mode_prefs.h" |
| #include "chrome/browser/profiles/profile_manager.h" |
| #include "chrome/browser/ui/browser.h" |
| #include "chrome/browser/ui/browser_finder.h" |
| #include "chrome/browser/ui/browser_window.h" |
| #include "chrome/browser/ui/settings_window_manager_chromeos.h" |
| #include "chrome/browser/ui/settings_window_manager_observer_chromeos.h" |
| #include "chrome/browser/ui/tabs/tab_strip_model.h" |
| #include "chrome/test/base/in_process_browser_test.h" |
| #include "components/account_id/account_id.h" |
| #include "components/session_manager/core/session_manager.h" |
| #include "components/user_manager/known_user.h" |
| #include "components/user_manager/user_manager.h" |
| #include "content/public/test/test_navigation_observer.h" |
| #include "testing/gtest/include/gtest/gtest.h" |
| #include "ui/aura/window.h" |
| #include "url/gurl.h" |
| |
| namespace { |
| |
| constexpr char kTestUserName1[] = "test1@test.com"; |
| constexpr char kTestUser1GaiaId[] = "1111111111"; |
| constexpr char kTestUserName2[] = "test2@test.com"; |
| constexpr char kTestUser2GaiaId[] = "2222222222"; |
| |
| void CreateAndStartUserSession(const AccountId& account_id) { |
| using chromeos::ProfileHelper; |
| using session_manager::SessionManager; |
| |
| user_manager::known_user::SetProfileRequiresPolicy( |
| account_id, |
| user_manager::known_user::ProfileRequiresPolicy::kNoPolicyRequired); |
| const std::string user_id_hash = |
| ProfileHelper::GetUserIdHashByUserIdForTesting(account_id.GetUserEmail()); |
| SessionManager::Get()->CreateSession(account_id, user_id_hash, false); |
| ProfileHelper::GetProfileByUserIdHashForTest(user_id_hash); |
| SessionManager::Get()->SessionStarted(); |
| } |
| |
| // Give the underlying function a clearer name. |
| Browser* GetLastActiveBrowser() { |
| return chrome::FindLastActive(); |
| } |
| |
| } // namespace |
| |
| using ChromeNewWindowClientBrowserTest = InProcessBrowserTest; |
| |
| using ChromeNewWindowClientWebAppBrowserTest = |
| extensions::test::BookmarkAppNavigationBrowserTest; |
| |
| // Tests that when we open a new window by pressing 'Ctrl-N', we should use the |
| // current active window's profile to determine on which profile's desktop we |
| // should open a new window. |
| // |
| // Test is flaky. See https://crbug.com/884118 |
| IN_PROC_BROWSER_TEST_F(ChromeNewWindowClientBrowserTest, |
| DISABLED_NewWindowForActiveWindowProfileTest) { |
| CreateAndStartUserSession( |
| AccountId::FromUserEmailGaiaId(kTestUserName1, kTestUser1GaiaId)); |
| Profile* profile1 = ProfileManager::GetActiveUserProfile(); |
| Browser* browser1 = CreateBrowser(profile1); |
| // The newly created window should be created for the current active profile. |
| ChromeNewWindowClient::Get()->NewWindow(false /* incognito */); |
| EXPECT_EQ(GetLastActiveBrowser()->profile(), profile1); |
| |
| // Login another user and make sure the current active user changes. |
| CreateAndStartUserSession( |
| AccountId::FromUserEmailGaiaId(kTestUserName2, kTestUser2GaiaId)); |
| Profile* profile2 = ProfileManager::GetActiveUserProfile(); |
| EXPECT_NE(profile1, profile2); |
| |
| Browser* browser2 = CreateBrowser(profile2); |
| // The newly created window should be created for the current active window's |
| // profile, which is |profile2|. |
| ChromeNewWindowClient::Get()->NewWindow(false /* incognito */); |
| EXPECT_EQ(GetLastActiveBrowser()->profile(), profile2); |
| |
| // After activating |browser1|, the newly created window should be created |
| // against |browser1|'s profile. |
| browser1->window()->Show(); |
| ChromeNewWindowClient::Get()->NewWindow(false /* incognito */); |
| EXPECT_EQ(GetLastActiveBrowser()->profile(), profile1); |
| |
| // Test for incognito windows. |
| // The newly created incoginito window should be created against the current |
| // active |browser1|'s profile. |
| browser1->window()->Show(); |
| ChromeNewWindowClient::Get()->NewWindow(true /* incognito */); |
| EXPECT_EQ(GetLastActiveBrowser()->profile()->GetOriginalProfile(), profile1); |
| |
| // The newly created incoginito window should be created against the current |
| // active |browser2|'s profile. |
| browser2->window()->Show(); |
| ChromeNewWindowClient::Get()->NewWindow(true /* incognito */); |
| EXPECT_EQ(GetLastActiveBrowser()->profile()->GetOriginalProfile(), profile2); |
| } |
| |
| IN_PROC_BROWSER_TEST_F(ChromeNewWindowClientBrowserTest, IncognitoDisabled) { |
| CreateAndStartUserSession( |
| AccountId::FromUserEmailGaiaId(kTestUserName1, kTestUser2GaiaId)); |
| Profile* profile = ProfileManager::GetActiveUserProfile(); |
| EXPECT_EQ(1u, chrome::GetTotalBrowserCount()); |
| |
| // Disabling incognito mode disables creation of new incognito windows. |
| IncognitoModePrefs::SetAvailability(profile->GetPrefs(), |
| IncognitoModePrefs::DISABLED); |
| ChromeNewWindowClient::Get()->NewWindow(true /* incognito */); |
| EXPECT_EQ(1u, chrome::GetTotalBrowserCount()); |
| |
| // Enabling incognito mode enables creation of new incognito windows. |
| IncognitoModePrefs::SetAvailability(profile->GetPrefs(), |
| IncognitoModePrefs::ENABLED); |
| ChromeNewWindowClient::Get()->NewWindow(true /* incognito */); |
| EXPECT_EQ(2u, chrome::GetTotalBrowserCount()); |
| EXPECT_TRUE(GetLastActiveBrowser()->profile()->IsIncognito()); |
| } |
| |
| IN_PROC_BROWSER_TEST_F(ChromeNewWindowClientWebAppBrowserTest, OpenWebApp) { |
| InstallTestBookmarkApp(GetAppUrlHost()); |
| const GURL app_url = https_server().GetURL(GetAppUrlHost(), GetAppUrlPath()); |
| const char* key = |
| arc::ArcWebContentsData::ArcWebContentsData::kArcTransitionFlag; |
| |
| { |
| // Calling OpenWebAppFromArc for a not installed HTTPS URL should open in |
| // an ordinary browser tab. |
| const GURL url("https://www.google.com"); |
| auto observer = GetTestNavigationObserver(url); |
| ChromeNewWindowClient::Get()->OpenWebAppFromArc(url); |
| observer->WaitForNavigationFinished(); |
| |
| EXPECT_EQ(1u, chrome::GetTotalBrowserCount()); |
| EXPECT_FALSE(GetLastActiveBrowser()->is_app()); |
| content::WebContents* contents = |
| GetLastActiveBrowser()->tab_strip_model()->GetActiveWebContents(); |
| EXPECT_EQ(url, contents->GetLastCommittedURL()); |
| EXPECT_NE(nullptr, contents->GetUserData(key)); |
| } |
| |
| { |
| // Calling OpenWebAppFromArc for an installed web app URL should open in an |
| // app window. |
| auto observer = GetTestNavigationObserver(app_url); |
| ChromeNewWindowClient::Get()->OpenWebAppFromArc(app_url); |
| observer->WaitForNavigationFinished(); |
| |
| EXPECT_EQ(2u, chrome::GetTotalBrowserCount()); |
| EXPECT_TRUE(GetLastActiveBrowser()->is_app()); |
| content::WebContents* contents = |
| GetLastActiveBrowser()->tab_strip_model()->GetActiveWebContents(); |
| EXPECT_EQ(app_url, contents->GetLastCommittedURL()); |
| EXPECT_NE(nullptr, contents->GetUserData(key)); |
| } |
| } |
| |
| IN_PROC_BROWSER_TEST_F(ChromeNewWindowClientBrowserTest, OpenSettingsFromArc) { |
| class Observer : public chrome::SettingsWindowManagerObserver { |
| public: |
| void OnNewSettingsWindow(Browser* settings_browser) override { |
| ++new_settings_count_; |
| } |
| int new_settings_count_ = 0; |
| } observer; |
| |
| auto* settings = chrome::SettingsWindowManager::GetInstance(); |
| settings->AddObserver(&observer); |
| |
| // Navigating to Wi-Fi settings opens a new settings window. |
| GURL wifi_url("chrome://settings/networks/?type=WiFi"); |
| ChromeNewWindowClient::Get()->OpenUrlFromArc(wifi_url); |
| EXPECT_EQ(1, observer.new_settings_count_); |
| |
| // The window loads Wi-Fi settings (not just the settings main page). |
| content::WebContents* contents = |
| GetLastActiveBrowser()->tab_strip_model()->GetActiveWebContents(); |
| EXPECT_EQ(wifi_url, contents->GetVisibleURL()); |
| |
| settings->RemoveObserver(&observer); |
| } |