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

Unified Diff: chrome/browser/shell_integration_win.cc

Issue 15255004: Refactor of BrowserDistribution. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rework Created 7 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/browser/shell_integration_win.cc
diff --git a/chrome/browser/shell_integration_win.cc b/chrome/browser/shell_integration_win.cc
index fb1bd643b7d061084fed2ec030ee51c53bc17a58..7cd7d73ef8ea1cc341a6b29a291ae7ed60a365e6 100644
--- a/chrome/browser/shell_integration_win.cc
+++ b/chrome/browser/shell_integration_win.cc
@@ -357,20 +357,6 @@ string16 ShellIntegration::GetAppListAppModelIdForProfile(
GetAppListAppName(), profile_path);
}
-string16 ShellIntegration::GetChromiumIconLocation() {
gab 2013/06/18 19:50:29 Also remove the method definition in shell_integra
calamity 2013/06/27 01:27:44 Done.
- // Determine the path to chrome.exe. If we can't determine what that is,
- // we have bigger fish to fry...
- base::FilePath chrome_exe;
- if (!PathService::Get(base::FILE_EXE, &chrome_exe)) {
- NOTREACHED();
- return string16();
- }
-
- return ShellUtil::FormatIconLocation(
- chrome_exe.value(),
- BrowserDistribution::GetDistribution()->GetIconIndex());
-}
-
void ShellIntegration::MigrateChromiumShortcuts() {
if (base::win::GetVersion() < base::win::VERSION_WIN7)
return;
@@ -511,7 +497,8 @@ base::FilePath ShellIntegration::GetStartMenuShortcut(
base::DIR_START_MENU,
};
BrowserDistribution* dist = BrowserDistribution::GetDistribution();
- string16 shortcut_name(dist->GetAppShortCutName());
+ string16 shortcut_name(
+ dist->GetShortcutName(BrowserDistribution::SHORTCUT_CHROME));
base::FilePath shortcut;
// Check both the common and the per-user Start Menu folders for system-level

Powered by Google App Engine
This is Rietveld 408576698