// Copyright 2014 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. #import #include #include "apps/app_shim/app_shim_handler_mac.h" #include "apps/app_shim/app_shim_host_manager_mac.h" #include "apps/app_shim/extension_app_shim_handler_mac.h" #include "apps/switches.h" #include "base/auto_reset.h" #include "base/callback.h" #include "base/files/file_path_watcher.h" #include "base/mac/foundation_util.h" #include "base/mac/launch_services_util.h" #include "base/mac/mac_util.h" #include "base/mac/scoped_nsobject.h" #include "base/path_service.h" #include "base/process/launch.h" #include "base/strings/sys_string_conversions.h" #include "base/test/test_timeouts.h" #include "chrome/browser/apps/app_browsertest_util.h" #include "chrome/browser/browser_process.h" #include "chrome/browser/extensions/extension_test_message_listener.h" #include "chrome/browser/profiles/profile.h" #include "chrome/browser/web_applications/web_app_mac.h" #include "chrome/common/chrome_paths.h" #include "chrome/common/chrome_switches.h" #include "chrome/common/mac/app_mode_common.h" #include "content/public/test/test_utils.h" #include "extensions/browser/app_window/native_app_window.h" #include "extensions/browser/extension_registry.h" #import "ui/events/test/cocoa_test_event_utils.h" namespace { // General end-to-end test for app shims. class AppShimInteractiveTest : public extensions::PlatformAppBrowserTest { protected: AppShimInteractiveTest() : auto_reset_(&g_app_shims_allow_update_and_launch_in_tests, true) {} private: // Temporarily enable app shims. base::AutoReset auto_reset_; DISALLOW_COPY_AND_ASSIGN(AppShimInteractiveTest); }; // Watches for changes to a file. This is designed to be used from the the UI // thread. class WindowedFilePathWatcher : public base::RefCountedThreadSafe { public: WindowedFilePathWatcher(const base::FilePath& path) : observed_(false) { content::BrowserThread::PostTask( content::BrowserThread::FILE, FROM_HERE, base::Bind(&WindowedFilePathWatcher::Watch, this, path)); } void Wait() { if (observed_) return; run_loop_.reset(new base::RunLoop); run_loop_->Run(); } protected: friend class base::RefCountedThreadSafe; virtual ~WindowedFilePathWatcher() {} void Watch(const base::FilePath& path) { watcher_.Watch( path, false, base::Bind(&WindowedFilePathWatcher::Observe, this)); } void Observe(const base::FilePath& path, bool error) { content::BrowserThread::PostTask( content::BrowserThread::UI, FROM_HERE, base::Bind(&WindowedFilePathWatcher::StopRunLoop, this)); } void StopRunLoop() { observed_ = true; if (run_loop_.get()) run_loop_->Quit(); } private: base::FilePathWatcher watcher_; bool observed_; scoped_ptr run_loop_; DISALLOW_COPY_AND_ASSIGN(WindowedFilePathWatcher); }; // Watches for an app shim to connect. class WindowedAppShimLaunchObserver : public apps::AppShimHandler { public: WindowedAppShimLaunchObserver(const std::string& app_id) : app_mode_id_(app_id), observed_(false) { apps::AppShimHandler::RegisterHandler(app_id, this); } void Wait() { if (observed_) return; run_loop_.reset(new base::RunLoop); run_loop_->Run(); } // AppShimHandler overrides: virtual void OnShimLaunch(Host* host, apps::AppShimLaunchType launch_type, const std::vector& files) OVERRIDE { // Remove self and pass through to the default handler. apps::AppShimHandler::RemoveHandler(app_mode_id_); apps::AppShimHandler::GetForAppMode(app_mode_id_) ->OnShimLaunch(host, launch_type, files); observed_ = true; if (run_loop_.get()) run_loop_->Quit(); } virtual void OnShimClose(Host* host) OVERRIDE {} virtual void OnShimFocus(Host* host, apps::AppShimFocusType focus_type, const std::vector& files) OVERRIDE {} virtual void OnShimSetHidden(Host* host, bool hidden) OVERRIDE {} virtual void OnShimQuit(Host* host) OVERRIDE {} private: std::string app_mode_id_; bool observed_; scoped_ptr run_loop_; DISALLOW_COPY_AND_ASSIGN(WindowedAppShimLaunchObserver); }; NSString* GetBundleID(const base::FilePath& shim_path) { base::FilePath plist_path = shim_path.Append("Contents").Append("Info.plist"); NSMutableDictionary* plist = [NSMutableDictionary dictionaryWithContentsOfFile:base::mac::FilePathToNSString(plist_path)]; return [plist objectForKey:base::mac::CFToNSCast(kCFBundleIdentifierKey)]; } bool HasAppShimHost(Profile* profile, const std::string& app_id) { return g_browser_process->platform_part() ->app_shim_host_manager() ->extension_app_shim_handler() ->FindHost(profile, app_id); } } // namespace // Watches for NSNotifications from the shared workspace. @interface WindowedNSNotificationObserver : NSObject { @private base::scoped_nsobject bundleId_; BOOL notificationReceived_; scoped_ptr runLoop_; } - (id)initForNotification:(NSString*)name andBundleId:(NSString*)bundleId; - (void)observe:(NSNotification*)notification; - (void)wait; @end @implementation WindowedNSNotificationObserver - (id)initForNotification:(NSString*)name andBundleId:(NSString*)bundleId { if (self = [super init]) { bundleId_.reset([[bundleId copy] retain]); [[[NSWorkspace sharedWorkspace] notificationCenter] addObserver:self selector:@selector(observe:) name:name object:nil]; } return self; } - (void)observe:(NSNotification*)notification { DCHECK_CURRENTLY_ON(content::BrowserThread::UI); NSRunningApplication* application = [[notification userInfo] objectForKey:NSWorkspaceApplicationKey]; if (![[application bundleIdentifier] isEqualToString:bundleId_]) return; [[[NSWorkspace sharedWorkspace] notificationCenter] removeObserver:self]; notificationReceived_ = YES; if (runLoop_.get()) runLoop_->Quit(); } - (void)wait { if (notificationReceived_) return; runLoop_.reset(new base::RunLoop); runLoop_->Run(); } @end namespace apps { // Shims require static libraries http://crbug.com/386024. #if defined(COMPONENT_BUILD) #define MAYBE_Launch DISABLED_Launch #define MAYBE_RebuildShim DISABLED_RebuildShim #else #define MAYBE_Launch Launch #define MAYBE_RebuildShim RebuildShim #endif // Test that launching the shim for an app starts the app, and vice versa. // These two cases are combined because the time to run the test is dominated // by loading the extension and creating the shim. IN_PROC_BROWSER_TEST_F(AppShimInteractiveTest, MAYBE_Launch) { // Install the app. const extensions::Extension* app = InstallPlatformApp("minimal"); // Use a WebAppShortcutCreator to get the path. web_app::WebAppShortcutCreator shortcut_creator( web_app::GetWebAppDataDirectory(profile()->GetPath(), app->id(), GURL()), web_app::ShortcutInfoForExtensionAndProfile(app, profile()), extensions::FileHandlersInfo()); base::FilePath shim_path = shortcut_creator.GetInternalShortcutPath(); EXPECT_FALSE(base::PathExists(shim_path)); // Create the internal app shim by simulating an app update. FilePathWatcher // is used to wait for file operations on the shim to be finished before // attempting to launch it. Since all of the file operations are done in the // same event on the FILE thread, everything will be done by the time the // watcher's callback is executed. scoped_refptr file_watcher = new WindowedFilePathWatcher(shim_path); web_app::UpdateAllShortcuts(base::string16(), profile(), app); file_watcher->Wait(); NSString* bundle_id = GetBundleID(shim_path); // Case 1: Launch the shim, it should start the app. { ExtensionTestMessageListener launched_listener("Launched", false); CommandLine shim_cmdline(CommandLine::NO_PROGRAM); shim_cmdline.AppendSwitch(app_mode::kLaunchedForTest); ProcessSerialNumber shim_psn; ASSERT_TRUE(base::mac::OpenApplicationWithPath( shim_path, shim_cmdline, kLSLaunchDefaults, &shim_psn)); ASSERT_TRUE(launched_listener.WaitUntilSatisfied()); ASSERT_TRUE(GetFirstAppWindow()); EXPECT_TRUE(HasAppShimHost(profile(), app->id())); // If the window is closed, the shim should quit. pid_t shim_pid; EXPECT_EQ(noErr, GetProcessPID(&shim_psn, &shim_pid)); GetFirstAppWindow()->GetBaseWindow()->Close(); ASSERT_TRUE( base::WaitForSingleProcess(shim_pid, TestTimeouts::action_timeout())); EXPECT_FALSE(GetFirstAppWindow()); EXPECT_FALSE(HasAppShimHost(profile(), app->id())); } // Case 2: Launch the app, it should start the shim. { base::scoped_nsobject ns_observer; ns_observer.reset([[WindowedNSNotificationObserver alloc] initForNotification:NSWorkspaceDidLaunchApplicationNotification andBundleId:bundle_id]); WindowedAppShimLaunchObserver observer(app->id()); LaunchPlatformApp(app); [ns_observer wait]; observer.Wait(); EXPECT_TRUE(GetFirstAppWindow()); EXPECT_TRUE(HasAppShimHost(profile(), app->id())); // Quitting the shim will eventually cause it to quit. It actually // intercepts the -terminate, sends an AppShimHostMsg_QuitApp to Chrome, // and returns NSTerminateLater. Chrome responds by closing all windows of // the app. Once all windows are closed, Chrome closes the IPC channel, // which causes the shim to actually terminate. NSArray* running_shim = [NSRunningApplication runningApplicationsWithBundleIdentifier:bundle_id]; ASSERT_EQ(1u, [running_shim count]); ns_observer.reset([[WindowedNSNotificationObserver alloc] initForNotification:NSWorkspaceDidTerminateApplicationNotification andBundleId:bundle_id]); [base::mac::ObjCCastStrict( [running_shim objectAtIndex:0]) terminate]; [ns_observer wait]; EXPECT_FALSE(GetFirstAppWindow()); EXPECT_FALSE(HasAppShimHost(profile(), app->id())); } } #if defined(ARCH_CPU_64_BITS) // Tests that a 32 bit shim attempting to launch 64 bit Chrome will eventually // be rebuilt. IN_PROC_BROWSER_TEST_F(AppShimInteractiveTest, MAYBE_RebuildShim) { // Get the 32 bit shim. base::FilePath test_data_dir; PathService::Get(chrome::DIR_TEST_DATA, &test_data_dir); base::FilePath shim_path_32 = test_data_dir.Append("app_shim").Append("app_shim_32_bit.app"); EXPECT_TRUE(base::PathExists(shim_path_32)); // Install test app. const extensions::Extension* app = InstallPlatformApp("minimal"); // Use WebAppShortcutCreator to create a 64 bit shim. web_app::WebAppShortcutCreator shortcut_creator( web_app::GetWebAppDataDirectory(profile()->GetPath(), app->id(), GURL()), web_app::ShortcutInfoForExtensionAndProfile(app, profile()), extensions::FileHandlersInfo()); shortcut_creator.UpdateShortcuts(); base::FilePath shim_path = shortcut_creator.GetInternalShortcutPath(); NSMutableDictionary* plist_64 = [NSMutableDictionary dictionaryWithContentsOfFile:base::mac::FilePathToNSString( shim_path.Append("Contents").Append("Info.plist"))]; // Copy 32 bit shim to where it's expected to be. // CopyDirectory doesn't seem to work when copying and renaming in one go. ASSERT_TRUE(base::DeleteFile(shim_path, true)); ASSERT_TRUE(base::PathExists(shim_path.DirName())); ASSERT_TRUE(base::CopyDirectory(shim_path_32, shim_path.DirName(), true)); ASSERT_TRUE(base::Move(shim_path.DirName().Append(shim_path_32.BaseName()), shim_path)); ASSERT_TRUE(base::PathExists( shim_path.Append("Contents").Append("MacOS").Append("app_mode_loader"))); // Fix up the plist so that it matches the installed test app. NSString* plist_path = base::mac::FilePathToNSString( shim_path.Append("Contents").Append("Info.plist")); NSMutableDictionary* plist = [NSMutableDictionary dictionaryWithContentsOfFile:plist_path]; NSArray* keys_to_copy = @[ base::mac::CFToNSCast(kCFBundleIdentifierKey), base::mac::CFToNSCast(kCFBundleNameKey), app_mode::kCrAppModeShortcutIDKey, app_mode::kCrAppModeUserDataDirKey, app_mode::kBrowserBundleIDKey ]; for (NSString* key in keys_to_copy) { [plist setObject:[plist_64 objectForKey:key] forKey:key]; } [plist writeToFile:plist_path atomically:YES]; base::mac::RemoveQuarantineAttribute(shim_path); // Launch the shim, it should start the app and ultimately connect over IPC. // This actually happens in multiple launches of the shim: // (1) The shim will fail and instead launch Chrome with --app-id so that the // app starts. // (2) Chrome launches the shim in response to an app starting, this time the // shim launches Chrome with --app-shim-error, which causes Chrome to // rebuild the shim. // (3) After rebuilding, Chrome again launches the shim and expects it to // behave normally. ExtensionTestMessageListener launched_listener("Launched", false); CommandLine shim_cmdline(CommandLine::NO_PROGRAM); ASSERT_TRUE(base::mac::OpenApplicationWithPath( shim_path, shim_cmdline, kLSLaunchDefaults, NULL)); // Wait for the app to start (1). At this point there is no shim host. ASSERT_TRUE(launched_listener.WaitUntilSatisfied()); EXPECT_FALSE(HasAppShimHost(profile(), app->id())); // Wait for the rebuilt shim to connect (3). This does not race with the app // starting (1) because Chrome only launches the shim (2) after the app // starts. Then Chrome must handle --app-shim-error on the UI thread before // the shim is rebuilt. WindowedAppShimLaunchObserver(app->id()).Wait(); EXPECT_TRUE(GetFirstAppWindow()); EXPECT_TRUE(HasAppShimHost(profile(), app->id())); } #endif // defined(ARCH_CPU_64_BITS) } // namespace apps