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

Side by Side Diff: chrome/browser/extensions/app_host_installer.h

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: Changed AppHostInstaller interface; handled thread compilcation in AppHostInstaller; added error me… 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef CHROME_BROWSER_EXTENSIONS_APP_HOST_INSTALLER_H_
6 #define CHROME_BROWSER_EXTENSIONS_APP_HOST_INSTALLER_H_
7
8 #include "base/basictypes.h"
9 #include "base/callback_forward.h"
10
11 namespace extensions {
12
13 class Extension;
14 class AppHostInstallerImpl;
15
16 class AppHostInstaller {
17 public:
18 AppHostInstaller();
19 ~AppHostInstaller();
20
21 // Installs App Host, then calls (directly or via post) |next_step|
erikwright (departed) 2012/10/05 02:21:56 Installs the App Host if it is required and not pr
huangs 2012/10/05 19:46:08 Done.
22 // to proceed with installation, on the caller thread.
23 // |next_step| takes a bool parameter to be notified of whether or not
24 // App Host installation was successful.
25 void AppHostInstaller::InstallAppHostIfNecessaryThenCall(
erikwright (departed) 2012/10/05 02:21:56 Remove the "ThenCall". The presence of a callback
huangs 2012/10/05 19:46:08 Done.
26 const Extension& extension,
27 const base::Callback<void(bool)>& final_step);
28
29 private:
30 // The implementation.
31 AppHostInstallerImpl* impl_;
huangs 2012/10/04 22:56:42 Not using scoped_ptr for this, since AppHostInstal
32 DISALLOW_COPY_AND_ASSIGN(AppHostInstaller);
33 };
34
35 } // namespace extensions
36
37 #endif // CHROME_BROWSER_EXTENSIONS_APP_HOST_INSTALLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698