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

Unified Diff: chrome/installer/util/chrome_app_host_distribution.cc

Issue 11359013: Adding App Launcher shortcuts on install. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge with refactor_chrome_shortcut_op. Created 8 years, 1 month 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/installer/util/chrome_app_host_distribution.cc
diff --git a/chrome/installer/util/chrome_app_host_distribution.cc b/chrome/installer/util/chrome_app_host_distribution.cc
index 112678e430f84ec1fb76b1c7d2747661fc97bced..628a37515e9d3119778db3d0e22c6665f345c071 100644
--- a/chrome/installer/util/chrome_app_host_distribution.cc
+++ b/chrome/installer/util/chrome_app_host_distribution.cc
@@ -21,8 +21,11 @@
#include "installer_util_strings.h" // NOLINT
namespace {
+
+const wchar_t kAppListId[] = L"ChromeAppList";
const wchar_t kChromeAppHostGuid[] = L"{FDA71E6F-AC4C-4a00-8B70-9958A68906BF}";
-}
+
+} // namespace
ChromeAppHostDistribution::ChromeAppHostDistribution()
: BrowserDistribution(CHROME_APP_HOST) {
@@ -48,6 +51,11 @@ string16 ChromeAppHostDistribution::GetAlternateApplicationName() {
return product_name;
}
+string16 ChromeAppHostDistribution::GetBaseAppId() {
+ // Should be same as AppListController::GetAppModelId().
+ return kAppListId;
gab 2012/11/07 21:44:40 Hard-coding the value here is fine, this is what's
huangs 2012/11/07 22:14:46 Done.
+}
+
string16 ChromeAppHostDistribution::GetInstallSubDir() {
return BrowserDistribution::GetSpecificDistribution(
BrowserDistribution::CHROME_BINARIES)->GetInstallSubDir();
@@ -124,7 +132,7 @@ bool ChromeAppHostDistribution::CanSetAsDefault() {
}
bool ChromeAppHostDistribution::CanCreateDesktopShortcuts() {
- return false;
+ return true;
}
bool ChromeAppHostDistribution::GetCommandExecuteImplClsid(
« chrome/installer/setup/uninstall.cc ('K') | « chrome/installer/util/chrome_app_host_distribution.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698