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

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

Issue 1926403002: Remove BrowserDistribution::SHORTCUT_APP_LAUNCHER (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@20160406-Mac-EnableAppLauncher0
Patch Set: lint headers Created 4 years, 7 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/installer/util/google_chrome_distribution.cc
diff --git a/chrome/installer/util/google_chrome_distribution.cc b/chrome/installer/util/google_chrome_distribution.cc
index 2d7524ffea083a46fe6b67946ef42dc82687051e..6a688f6c2e55425292598b5d41c7648a4f9dfb00 100644
--- a/chrome/installer/util/google_chrome_distribution.cc
+++ b/chrome/installer/util/google_chrome_distribution.cc
@@ -158,24 +158,11 @@ base::string16 GoogleChromeDistribution::GetBaseAppName() {
return L"Google Chrome";
}
-base::string16 GoogleChromeDistribution::GetShortcutName(
- ShortcutType shortcut_type) {
- int string_id = IDS_PRODUCT_NAME_BASE;
- switch (shortcut_type) {
- case SHORTCUT_APP_LAUNCHER:
- string_id = IDS_APP_LIST_SHORTCUT_NAME_BASE;
- break;
- default:
- DCHECK_EQ(SHORTCUT_CHROME, shortcut_type);
- break;
- }
- return installer::GetLocalizedString(string_id);
+base::string16 GoogleChromeDistribution::GetShortcutName() {
+ return installer::GetLocalizedString(IDS_PRODUCT_NAME_BASE);
}
-int GoogleChromeDistribution::GetIconIndex(ShortcutType shortcut_type) {
- if (shortcut_type == SHORTCUT_APP_LAUNCHER)
- return icon_resources::kAppLauncherIndex;
- DCHECK_EQ(SHORTCUT_CHROME, shortcut_type);
+int GoogleChromeDistribution::GetIconIndex() {
return icon_resources::kApplicationIndex;
}
« no previous file with comments | « chrome/installer/util/google_chrome_distribution.h ('k') | chrome/installer/util/google_chrome_distribution_dummy.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698