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

Unified Diff: apps/app_launch_for_metro_restart_win.cc

Issue 12450014: Show an InfoBar when trying to start Packaged Apps from Metro mode. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: feedback Created 7 years, 9 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
« no previous file with comments | « apps/app_launch_for_metro_restart_win.h ('k') | apps/apps.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: apps/app_launch_for_metro_restart_win.cc
diff --git a/apps/app_launch_for_metro_restart_win.cc b/apps/app_launch_for_metro_restart_win.cc
new file mode 100644
index 0000000000000000000000000000000000000000..1fa0cc9e36ce044a8fe7fa3c38ceb6122953d36d
--- /dev/null
+++ b/apps/app_launch_for_metro_restart_win.cc
@@ -0,0 +1,95 @@
+// 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_launch_for_metro_restart_win.h"
+
+#include "apps/pref_names.h"
+#include "base/bind.h"
+#include "base/files/file_path.h"
+#include "base/message_loop.h"
+#include "base/prefs/pref_service.h"
+#include "base/time.h"
+#include "chrome/browser/browser_process.h"
+#include "chrome/browser/extensions/api/app_runtime/app_runtime_api.h"
+#include "chrome/browser/extensions/extension_service.h"
+#include "chrome/browser/extensions/extension_system.h"
+#include "chrome/browser/extensions/platform_app_launcher.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/profiles/profile_manager.h"
+#include "win8/util/win8_util.h"
+
+using extensions::Extension;
+using extensions::ExtensionSystem;
+
+namespace apps {
+
+namespace {
+
+void LaunchAppWithId(Profile* profile,
+ const std::string& extension_id) {
+ ExtensionService* extension_service =
+ ExtensionSystem::Get(profile)->extension_service();
+ if (!extension_service)
+ return;
+
+ const Extension* extension =
+ extension_service->GetExtensionById(extension_id, false);
+ if (!extension)
+ return;
+
+ extensions::AppEventRouter::DispatchOnLaunchedEvent(profile, extension);
+}
+
+} // namespace
+
+void HandleAppLaunchForMetroRestart(Profile* profile) {
+ PrefService* prefs = g_browser_process->local_state();
+ if (!prefs->HasPrefPath(prefs::kAppLaunchForMetroRestartProfile))
+ return;
+
+ // This will be called for each profile that had a browser window open before
+ // relaunch. After checking that the preference is set, check that the
+ // profile that is starting up matches the profile that initially wanted to
+ // launch the app.
+ base::FilePath profile_dir = base::FilePath::FromUTF8Unsafe(
+ prefs->GetString(prefs::kAppLaunchForMetroRestartProfile));
+ if (profile_dir.empty() || profile->GetPath().BaseName() != profile_dir)
+ return;
+
+ prefs->ClearPref(prefs::kAppLaunchForMetroRestartProfile);
+
+ if (!prefs->HasPrefPath(prefs::kAppLaunchForMetroRestart))
+ return;
+
+ std::string extension_id = prefs->GetString(prefs::kAppLaunchForMetroRestart);
+ if (extension_id.empty())
+ return;
+
+ prefs->ClearPref(prefs::kAppLaunchForMetroRestart);
+
+ if (win8::IsSingleWindowMetroMode()) {
+ // In this case we have relaunched with the correct profile, but we are not
+ // in Desktop mode, so can not launch apps. Leave the preferences cleared so
+ // there are no surprises later.
+ return;
+ }
+
+ const int kRestartAppLaunchDelayMs = 1000;
+ MessageLoop::current()->PostDelayedTask(
+ FROM_HERE,
+ base::Bind(&LaunchAppWithId,
+ profile,
+ extension_id),
+ base::TimeDelta::FromMilliseconds(kRestartAppLaunchDelayMs));
+}
+
+void SetAppLaunchForMetroRestart(Profile* profile,
+ const std::string& extension_id) {
+ PrefService* prefs = g_browser_process->local_state();
+ prefs->SetString(prefs::kAppLaunchForMetroRestartProfile,
+ profile->GetPath().BaseName().MaybeAsASCII());
+ prefs->SetString(prefs::kAppLaunchForMetroRestart, extension_id);
+}
+
+} // namespace apps
« no previous file with comments | « apps/app_launch_for_metro_restart_win.h ('k') | apps/apps.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698