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

Unified Diff: chrome/browser/extensions/app_host_installer.cc

Issue 11054006: Make application shortcuts point to app_host.exe, install App Host during app installation. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Extracting app_host_installer_impl_win.*; proposing simplified flow to switch between threads for A… Created 8 years, 2 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: chrome/browser/extensions/app_host_installer.cc
diff --git a/chrome/browser/extensions/app_host_installer.cc b/chrome/browser/extensions/app_host_installer.cc
new file mode 100644
index 0000000000000000000000000000000000000000..07ac5c92db9cb74f2dc26dffb3fd19ea80891781
--- /dev/null
+++ b/chrome/browser/extensions/app_host_installer.cc
@@ -0,0 +1,41 @@
+// Copyright (c) 2012 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/extensions/app_host_installer.h"
+
+#include "base/bind.h"
erikwright (departed) 2012/10/05 21:18:23 not required.
huangs 2012/10/07 01:42:25 Done.
+#include "base/callback.h"
+#include "base/compiler_specific.h"
erikwright (departed) 2012/10/05 21:18:23 not required
huangs 2012/10/07 01:42:25 Done.
+#include "chrome/common/extensions/extension.h"
erikwright (departed) 2012/10/05 21:18:23 not required
huangs 2012/10/07 01:42:25 Done (forward declared).
+
+#if defined(OS_WIN)
+#include "chrome/browser/extensions/app_host_installer_impl_win.h"
+#endif
+
+namespace extensions {
+
+AppHostInstaller::AppHostInstaller()
+ :
+#if defined(OS_WIN)
+ impl_(new AppHostInstallerImpl()) {}
+#else
+ impl_(0) {}
+#endif
+
+AppHostInstaller::~AppHostInstaller() {
+ if (impl_)
+ delete impl_;
+}
+
+void AppHostInstaller::InstallAppHostIfNecessary(
+ const Extension& extension,
+ const base::Callback<void(bool)>& completion_callback) {
+#if defined(OS_WIN)
+ impl_->InstallAppHostIfNecessary(extension, completion_callback);
+#else
+ completion_callback.Run(true);
+#endif
+}
+
+} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698