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

Unified Diff: apps/app_shim/extension_app_shim_handler_mac.cc

Issue 585123004: Mac: Give app_shim code a nicer home (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: various cleanups Created 6 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 side-by-side diff with in-line comments
Download patch
Index: apps/app_shim/extension_app_shim_handler_mac.cc
diff --git a/apps/app_shim/extension_app_shim_handler_mac.cc b/apps/app_shim/extension_app_shim_handler_mac.cc
deleted file mode 100644
index 8e903242f1762685b53ec7f745c34059e95c9d09..0000000000000000000000000000000000000000
--- a/apps/app_shim/extension_app_shim_handler_mac.cc
+++ /dev/null
@@ -1,557 +0,0 @@
-// Copyright 2013 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 "apps/app_shim/extension_app_shim_handler_mac.h"
-
-#include "apps/app_lifetime_monitor_factory.h"
-#include "apps/app_shim/app_shim_host_manager_mac.h"
-#include "apps/app_shim/app_shim_messages.h"
-#include "apps/launcher.h"
-#include "base/files/file_path.h"
-#include "base/logging.h"
-#include "chrome/browser/browser_process.h"
-#include "chrome/browser/chrome_notification_types.h"
-#include "chrome/browser/profiles/profile.h"
-#include "chrome/browser/profiles/profile_manager.h"
-#include "chrome/browser/ui/extensions/extension_enable_flow.h"
-#include "chrome/browser/ui/extensions/extension_enable_flow_delegate.h"
-#include "chrome/browser/ui/webui/ntp/core_app_launcher_handler.h"
-#include "chrome/browser/web_applications/web_app_mac.h"
-#include "chrome/common/extensions/extension_constants.h"
-#include "components/crx_file/id_util.h"
-#include "content/public/browser/notification_details.h"
-#include "content/public/browser/notification_service.h"
-#include "content/public/browser/notification_source.h"
-#include "extensions/browser/app_window/app_window.h"
-#include "extensions/browser/app_window/app_window_registry.h"
-#include "extensions/browser/app_window/native_app_window.h"
-#include "extensions/browser/extension_host.h"
-#include "extensions/browser/extension_registry.h"
-#include "ui/base/cocoa/focus_window_set.h"
-
-using extensions::AppWindow;
-using extensions::AppWindowRegistry;
-using extensions::ExtensionRegistry;
-
-namespace {
-
-typedef AppWindowRegistry::AppWindowList AppWindowList;
-
-void ProfileLoadedCallback(base::Callback<void(Profile*)> callback,
- Profile* profile,
- Profile::CreateStatus status) {
- if (status == Profile::CREATE_STATUS_INITIALIZED) {
- callback.Run(profile);
- return;
- }
-
- // This should never get an error since it only loads existing profiles.
- DCHECK_EQ(Profile::CREATE_STATUS_CREATED, status);
-}
-
-void SetAppHidden(Profile* profile, const std::string& app_id, bool hidden) {
- AppWindowList windows =
- AppWindowRegistry::Get(profile)->GetAppWindowsForApp(app_id);
- for (AppWindowList::const_reverse_iterator it = windows.rbegin();
- it != windows.rend();
- ++it) {
- if (hidden)
- (*it)->GetBaseWindow()->HideWithApp();
- else
- (*it)->GetBaseWindow()->ShowWithApp();
- }
-}
-
-bool FocusWindows(const AppWindowList& windows) {
- if (windows.empty())
- return false;
-
- std::set<gfx::NativeWindow> native_windows;
- for (AppWindowList::const_iterator it = windows.begin(); it != windows.end();
- ++it) {
- native_windows.insert((*it)->GetNativeWindow());
- }
- // Allow workspace switching. For the browser process, we can reasonably rely
- // on OS X to switch spaces for us and honor relevant user settings. But shims
- // don't have windows, so we have to do it ourselves.
- ui::FocusWindowSet(native_windows);
- return true;
-}
-
-// Attempts to launch a packaged app, prompting the user to enable it if
-// necessary. The prompt is shown in its own window.
-// This class manages its own lifetime.
-class EnableViaPrompt : public ExtensionEnableFlowDelegate {
- public:
- EnableViaPrompt(Profile* profile,
- const std::string& extension_id,
- const base::Callback<void()>& callback)
- : profile_(profile),
- extension_id_(extension_id),
- callback_(callback) {
- }
-
- virtual ~EnableViaPrompt() {
- }
-
- void Run() {
- flow_.reset(new ExtensionEnableFlow(profile_, extension_id_, this));
- flow_->StartForCurrentlyNonexistentWindow(
- base::Callback<gfx::NativeWindow(void)>());
- }
-
- private:
- // ExtensionEnableFlowDelegate overrides.
- virtual void ExtensionEnableFlowFinished() OVERRIDE {
- callback_.Run();
- delete this;
- }
-
- virtual void ExtensionEnableFlowAborted(bool user_initiated) OVERRIDE {
- callback_.Run();
- delete this;
- }
-
- Profile* profile_;
- std::string extension_id_;
- base::Callback<void()> callback_;
- scoped_ptr<ExtensionEnableFlow> flow_;
-
- DISALLOW_COPY_AND_ASSIGN(EnableViaPrompt);
-};
-
-} // namespace
-
-namespace apps {
-
-bool ExtensionAppShimHandler::Delegate::ProfileExistsForPath(
- const base::FilePath& path) {
- ProfileManager* profile_manager = g_browser_process->profile_manager();
- // Check for the profile name in the profile info cache to ensure that we
- // never access any directory that isn't a known profile.
- base::FilePath full_path = profile_manager->user_data_dir().Append(path);
- ProfileInfoCache& cache = profile_manager->GetProfileInfoCache();
- return cache.GetIndexOfProfileWithPath(full_path) != std::string::npos;
-}
-
-Profile* ExtensionAppShimHandler::Delegate::ProfileForPath(
- const base::FilePath& path) {
- ProfileManager* profile_manager = g_browser_process->profile_manager();
- base::FilePath full_path = profile_manager->user_data_dir().Append(path);
- Profile* profile = profile_manager->GetProfileByPath(full_path);
-
- // Use IsValidProfile to check if the profile has been created.
- return profile && profile_manager->IsValidProfile(profile) ? profile : NULL;
-}
-
-void ExtensionAppShimHandler::Delegate::LoadProfileAsync(
- const base::FilePath& path,
- base::Callback<void(Profile*)> callback) {
- ProfileManager* profile_manager = g_browser_process->profile_manager();
- base::FilePath full_path = profile_manager->user_data_dir().Append(path);
- profile_manager->CreateProfileAsync(
- full_path,
- base::Bind(&ProfileLoadedCallback, callback),
- base::string16(), base::string16(), std::string());
-}
-
-AppWindowList ExtensionAppShimHandler::Delegate::GetWindows(
- Profile* profile,
- const std::string& extension_id) {
- return AppWindowRegistry::Get(profile)->GetAppWindowsForApp(extension_id);
-}
-
-const extensions::Extension*
-ExtensionAppShimHandler::Delegate::GetAppExtension(
- Profile* profile,
- const std::string& extension_id) {
- ExtensionRegistry* registry = ExtensionRegistry::Get(profile);
- const extensions::Extension* extension =
- registry->GetExtensionById(extension_id, ExtensionRegistry::ENABLED);
- return extension && extension->is_platform_app() ? extension : NULL;
-}
-
-void ExtensionAppShimHandler::Delegate::EnableExtension(
- Profile* profile,
- const std::string& extension_id,
- const base::Callback<void()>& callback) {
- (new EnableViaPrompt(profile, extension_id, callback))->Run();
-}
-
-void ExtensionAppShimHandler::Delegate::LaunchApp(
- Profile* profile,
- const extensions::Extension* extension,
- const std::vector<base::FilePath>& files) {
- CoreAppLauncherHandler::RecordAppLaunchType(
- extension_misc::APP_LAUNCH_CMD_LINE_APP, extension->GetType());
- if (files.empty()) {
- apps::LaunchPlatformApp(profile, extension);
- } else {
- for (std::vector<base::FilePath>::const_iterator it = files.begin();
- it != files.end(); ++it) {
- apps::LaunchPlatformAppWithPath(profile, extension, *it);
- }
- }
-}
-
-void ExtensionAppShimHandler::Delegate::LaunchShim(
- Profile* profile,
- const extensions::Extension* extension) {
- web_app::MaybeLaunchShortcut(
- web_app::ShortcutInfoForExtensionAndProfile(extension, profile));
-}
-
-void ExtensionAppShimHandler::Delegate::MaybeTerminate() {
- AppShimHandler::MaybeTerminate();
-}
-
-ExtensionAppShimHandler::ExtensionAppShimHandler()
- : delegate_(new Delegate),
- weak_factory_(this) {
- // This is instantiated in BrowserProcessImpl::PreMainMessageLoopRun with
- // AppShimHostManager. Since PROFILE_CREATED is not fired until
- // ProfileManager::GetLastUsedProfile/GetLastOpenedProfiles, this should catch
- // notifications for all profiles.
- registrar_.Add(this, chrome::NOTIFICATION_PROFILE_CREATED,
- content::NotificationService::AllBrowserContextsAndSources());
- registrar_.Add(this, chrome::NOTIFICATION_PROFILE_DESTROYED,
- content::NotificationService::AllBrowserContextsAndSources());
-}
-
-ExtensionAppShimHandler::~ExtensionAppShimHandler() {}
-
-AppShimHandler::Host* ExtensionAppShimHandler::FindHost(
- Profile* profile,
- const std::string& app_id) {
- HostMap::iterator it = hosts_.find(make_pair(profile, app_id));
- return it == hosts_.end() ? NULL : it->second;
-}
-
-// static
-void ExtensionAppShimHandler::QuitAppForWindow(AppWindow* app_window) {
- ExtensionAppShimHandler* handler = GetInstance();
- Host* host = handler->FindHost(
- Profile::FromBrowserContext(app_window->browser_context()),
- app_window->extension_id());
- if (host) {
- handler->OnShimQuit(host);
- } else {
- // App shims might be disabled or the shim is still starting up.
- AppWindowRegistry::Get(
- Profile::FromBrowserContext(app_window->browser_context()))
- ->CloseAllAppWindowsForApp(app_window->extension_id());
- }
-}
-
-void ExtensionAppShimHandler::HideAppForWindow(AppWindow* app_window) {
- ExtensionAppShimHandler* handler = GetInstance();
- Profile* profile = Profile::FromBrowserContext(app_window->browser_context());
- Host* host = handler->FindHost(profile, app_window->extension_id());
- if (host)
- host->OnAppHide();
- else
- SetAppHidden(profile, app_window->extension_id(), true);
-}
-
-void ExtensionAppShimHandler::FocusAppForWindow(AppWindow* app_window) {
- ExtensionAppShimHandler* handler = GetInstance();
- Profile* profile = Profile::FromBrowserContext(app_window->browser_context());
- const std::string& app_id = app_window->extension_id();
- Host* host = handler->FindHost(profile, app_id);
- if (host) {
- handler->OnShimFocus(host,
- APP_SHIM_FOCUS_NORMAL,
- std::vector<base::FilePath>());
- } else {
- FocusWindows(AppWindowRegistry::Get(profile)->GetAppWindowsForApp(app_id));
- }
-}
-
-// static
-bool ExtensionAppShimHandler::ActivateAndRequestUserAttentionForWindow(
- AppWindow* app_window) {
- ExtensionAppShimHandler* handler = GetInstance();
- Profile* profile = Profile::FromBrowserContext(app_window->browser_context());
- Host* host = handler->FindHost(profile, app_window->extension_id());
- if (host) {
- // Bring the window to the front without showing it.
- AppWindowRegistry::Get(profile)->AppWindowActivated(app_window);
- host->OnAppRequestUserAttention(APP_SHIM_ATTENTION_INFORMATIONAL);
- return true;
- } else {
- // Just show the app.
- SetAppHidden(profile, app_window->extension_id(), false);
- return false;
- }
-}
-
-// static
-void ExtensionAppShimHandler::RequestUserAttentionForWindow(
- AppWindow* app_window,
- AppShimAttentionType attention_type) {
- ExtensionAppShimHandler* handler = GetInstance();
- Profile* profile = Profile::FromBrowserContext(app_window->browser_context());
- Host* host = handler->FindHost(profile, app_window->extension_id());
- if (host)
- host->OnAppRequestUserAttention(attention_type);
-}
-
-// static
-void ExtensionAppShimHandler::OnChromeWillHide() {
- // Send OnAppHide to all the shims so that they go into the hidden state.
- // This is necessary so that when the shim is next focused, it will know to
- // unhide.
- ExtensionAppShimHandler* handler = GetInstance();
- for (HostMap::iterator it = handler->hosts_.begin();
- it != handler->hosts_.end();
- ++it) {
- it->second->OnAppHide();
- }
-}
-
-void ExtensionAppShimHandler::OnShimLaunch(
- Host* host,
- AppShimLaunchType launch_type,
- const std::vector<base::FilePath>& files) {
- const std::string& app_id = host->GetAppId();
- DCHECK(crx_file::id_util::IdIsValid(app_id));
-
- const base::FilePath& profile_path = host->GetProfilePath();
- DCHECK(!profile_path.empty());
-
- if (!delegate_->ProfileExistsForPath(profile_path)) {
- // User may have deleted the profile this shim was originally created for.
- // TODO(jackhou): Add some UI for this case and remove the LOG.
- LOG(ERROR) << "Requested directory is not a known profile '"
- << profile_path.value() << "'.";
- host->OnAppLaunchComplete(APP_SHIM_LAUNCH_PROFILE_NOT_FOUND);
- return;
- }
-
- Profile* profile = delegate_->ProfileForPath(profile_path);
-
- if (profile) {
- OnProfileLoaded(host, launch_type, files, profile);
- return;
- }
-
- // If the profile is not loaded, this must have been a launch by the shim.
- // Load the profile asynchronously, the host will be registered in
- // OnProfileLoaded.
- DCHECK_EQ(APP_SHIM_LAUNCH_NORMAL, launch_type);
- delegate_->LoadProfileAsync(
- profile_path,
- base::Bind(&ExtensionAppShimHandler::OnProfileLoaded,
- weak_factory_.GetWeakPtr(),
- host, launch_type, files));
-
- // Return now. OnAppLaunchComplete will be called when the app is activated.
-}
-
-// static
-ExtensionAppShimHandler* ExtensionAppShimHandler::GetInstance() {
- return g_browser_process->platform_part()
- ->app_shim_host_manager()
- ->extension_app_shim_handler();
-}
-
-void ExtensionAppShimHandler::OnProfileLoaded(
- Host* host,
- AppShimLaunchType launch_type,
- const std::vector<base::FilePath>& files,
- Profile* profile) {
- const std::string& app_id = host->GetAppId();
-
- // The first host to claim this (profile, app_id) becomes the main host.
- // For any others, focus or relaunch the app.
- if (!hosts_.insert(make_pair(make_pair(profile, app_id), host)).second) {
- OnShimFocus(host,
- launch_type == APP_SHIM_LAUNCH_NORMAL ?
- APP_SHIM_FOCUS_REOPEN : APP_SHIM_FOCUS_NORMAL,
- files);
- host->OnAppLaunchComplete(APP_SHIM_LAUNCH_DUPLICATE_HOST);
- return;
- }
-
- if (launch_type != APP_SHIM_LAUNCH_NORMAL) {
- host->OnAppLaunchComplete(APP_SHIM_LAUNCH_SUCCESS);
- return;
- }
-
- // TODO(jeremya): Handle the case that launching the app fails. Probably we
- // need to watch for 'app successfully launched' or at least 'background page
- // exists/was created' and time out with failure if we don't see that sign of
- // life within a certain window.
- const extensions::Extension* extension =
- delegate_->GetAppExtension(profile, app_id);
- if (extension) {
- delegate_->LaunchApp(profile, extension, files);
- return;
- }
-
- delegate_->EnableExtension(
- profile, app_id,
- base::Bind(&ExtensionAppShimHandler::OnExtensionEnabled,
- weak_factory_.GetWeakPtr(),
- host->GetProfilePath(), app_id, files));
-}
-
-void ExtensionAppShimHandler::OnExtensionEnabled(
- const base::FilePath& profile_path,
- const std::string& app_id,
- const std::vector<base::FilePath>& files) {
- Profile* profile = delegate_->ProfileForPath(profile_path);
- if (!profile)
- return;
-
- const extensions::Extension* extension =
- delegate_->GetAppExtension(profile, app_id);
- if (!extension || !delegate_->ProfileExistsForPath(profile_path)) {
- // If !extension, the extension doesn't exist, or was not re-enabled.
- // If the profile doesn't exist, it may have been deleted during the enable
- // prompt. In this case, NOTIFICATION_PROFILE_DESTROYED may not be fired
- // until later, so respond to the host now.
- Host* host = FindHost(profile, app_id);
- if (host)
- host->OnAppLaunchComplete(APP_SHIM_LAUNCH_APP_NOT_FOUND);
- return;
- }
-
- delegate_->LaunchApp(profile, extension, files);
-}
-
-
-void ExtensionAppShimHandler::OnShimClose(Host* host) {
- // This might be called when shutting down. Don't try to look up the profile
- // since profile_manager might not be around.
- for (HostMap::iterator it = hosts_.begin(); it != hosts_.end(); ) {
- HostMap::iterator current = it++;
- if (current->second == host)
- hosts_.erase(current);
- }
-}
-
-void ExtensionAppShimHandler::OnShimFocus(
- Host* host,
- AppShimFocusType focus_type,
- const std::vector<base::FilePath>& files) {
- DCHECK(delegate_->ProfileExistsForPath(host->GetProfilePath()));
- Profile* profile = delegate_->ProfileForPath(host->GetProfilePath());
-
- const AppWindowList windows =
- delegate_->GetWindows(profile, host->GetAppId());
- bool windows_focused = FocusWindows(windows);
-
- if (focus_type == APP_SHIM_FOCUS_NORMAL ||
- (focus_type == APP_SHIM_FOCUS_REOPEN && windows_focused)) {
- return;
- }
-
- const extensions::Extension* extension =
- delegate_->GetAppExtension(profile, host->GetAppId());
- if (extension) {
- delegate_->LaunchApp(profile, extension, files);
- } else {
- // Extensions may have been uninstalled or disabled since the shim
- // started.
- host->OnAppClosed();
- }
-}
-
-void ExtensionAppShimHandler::OnShimSetHidden(Host* host, bool hidden) {
- DCHECK(delegate_->ProfileExistsForPath(host->GetProfilePath()));
- Profile* profile = delegate_->ProfileForPath(host->GetProfilePath());
-
- SetAppHidden(profile, host->GetAppId(), hidden);
-}
-
-void ExtensionAppShimHandler::OnShimQuit(Host* host) {
- DCHECK(delegate_->ProfileExistsForPath(host->GetProfilePath()));
- Profile* profile = delegate_->ProfileForPath(host->GetProfilePath());
-
- const std::string& app_id = host->GetAppId();
- const AppWindowList windows = delegate_->GetWindows(profile, app_id);
- for (AppWindowRegistry::const_iterator it = windows.begin();
- it != windows.end();
- ++it) {
- (*it)->GetBaseWindow()->Close();
- }
- // Once the last window closes, flow will end up in OnAppDeactivated via
- // AppLifetimeMonitor.
-}
-
-void ExtensionAppShimHandler::set_delegate(Delegate* delegate) {
- delegate_.reset(delegate);
-}
-
-void ExtensionAppShimHandler::Observe(
- int type,
- const content::NotificationSource& source,
- const content::NotificationDetails& details) {
- Profile* profile = content::Source<Profile>(source).ptr();
- if (profile->IsOffTheRecord())
- return;
-
- switch (type) {
- case chrome::NOTIFICATION_PROFILE_CREATED: {
- AppLifetimeMonitorFactory::GetForProfile(profile)->AddObserver(this);
- break;
- }
- case chrome::NOTIFICATION_PROFILE_DESTROYED: {
- AppLifetimeMonitorFactory::GetForProfile(profile)->RemoveObserver(this);
- // Shut down every shim associated with this profile.
- for (HostMap::iterator it = hosts_.begin(); it != hosts_.end(); ) {
- // Increment the iterator first as OnAppClosed may call back to
- // OnShimClose and invalidate the iterator.
- HostMap::iterator current = it++;
- if (profile->IsSameProfile(current->first.first)) {
- Host* host = current->second;
- host->OnAppClosed();
- }
- }
- break;
- }
- default: {
- NOTREACHED(); // Unexpected notification.
- break;
- }
- }
-}
-
-void ExtensionAppShimHandler::OnAppStart(Profile* profile,
- const std::string& app_id) {}
-
-void ExtensionAppShimHandler::OnAppActivated(Profile* profile,
- const std::string& app_id) {
- const extensions::Extension* extension =
- delegate_->GetAppExtension(profile, app_id);
- if (!extension)
- return;
-
- Host* host = FindHost(profile, app_id);
- if (host) {
- host->OnAppLaunchComplete(APP_SHIM_LAUNCH_SUCCESS);
- OnShimFocus(host, APP_SHIM_FOCUS_NORMAL, std::vector<base::FilePath>());
- return;
- }
-
- delegate_->LaunchShim(profile, extension);
-}
-
-void ExtensionAppShimHandler::OnAppDeactivated(Profile* profile,
- const std::string& app_id) {
- Host* host = FindHost(profile, app_id);
- if (host)
- host->OnAppClosed();
-
- if (hosts_.empty())
- delegate_->MaybeTerminate();
-}
-
-void ExtensionAppShimHandler::OnAppStop(Profile* profile,
- const std::string& app_id) {}
-
-void ExtensionAppShimHandler::OnChromeTerminating() {}
-
-} // namespace apps
« no previous file with comments | « apps/app_shim/extension_app_shim_handler_mac.h ('k') | apps/app_shim/extension_app_shim_handler_mac_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698