More installer refactoring in the interest of fixing some bugs and cleaning t...

chrome/installer/util/helper.h

Index: chrome/installer/util/helper.h
===================================================================
--- chrome/installer/util/helper.h	(revision 71802)
+++ chrome/installer/util/helper.h	(working copy)
@@ -16,17 +16,6 @@
 
 namespace installer {
 
-// Checks if a distribution is currently installed as part of a multi-install.
-bool IsInstalledAsMulti(bool system_install, BrowserDistribution* dist);
-
-// Retrieves the command line switches for uninstalling the distribution.
-// Note that the returned CommandLine object does not include a "program".
-// Only the switches should be used.
-// Returns true if the product is installed and the uninstall switches
-// were successfully retrieved, otherwise false.
-bool GetUninstallSwitches(bool system_install, BrowserDistribution* dist,
-                          CommandLine* cmd_line_switches);
-
 // This function returns the install path for Chrome depending on whether its
 // system wide install or user specific install.
 // system_install: if true, the function returns system wide location
@@ -40,23 +29,19 @@
 // that it can be overriden with a command line parameter.
 FilePath GetChromeUserDataPath(BrowserDistribution* dist);
 
-// This is a workaround while we unify Chrome and Chrome Frame installation
-// folders.  Right now, Chrome Frame can be installed into two different
-// folders: 1) A special "Chrome Frame" folder next to Chrome's folder
-// 2) The same folder as Chrome is installed into.
-// Right now this function will only return Chrome's installation folder
-// if Chrome Frame is not already installed or if Chrome Frame is installed
-// in multi_install mode.
-// If multi_install is false or if CF is installed in single mode, then the
-// returned path will be the "Chrome Frame" subfolder of either the user or
-// system default installation folders.
-FilePath GetChromeFrameInstallPath(bool multi_install, bool system_install,
-                                   BrowserDistribution* dist);
+// Returns the distribution corresponding to the current process's binaries.
+// In the case of a multi-install product, this will be the CHROME_BINARIES
+// distribution.
+BrowserDistribution* GetBinariesDistribution(bool system_install);
 
 // Returns the app guid under which the current process receives updates from
 // Google Update.
 std::wstring GetAppGuidForUpdates(bool system_install);
 
+// Returns the rename command for the current process and populates |app_guid|
+// with the location at which it was found.

[robertshield, 2011/01/20 23:36:34]

app_guid doesn't actually get a location though right, it only gets the guid?

[grt (UTC plus 2), 2011/01/21 05:27:51]

This code has been removed.

+std::wstring GetRenameCommand(std::wstring* app_guid);
+
 }  // namespace installer
 
 #endif  // CHROME_INSTALLER_UTIL_HELPER_H_