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

chrome/installer/setup/install.h

Index: chrome/installer/setup/install.h
===================================================================
--- chrome/installer/setup/install.h	(revision 71802)
+++ chrome/installer/setup/install.h	(working copy)
@@ -13,7 +13,6 @@
 
 #include "base/version.h"
 #include "chrome/installer/util/product.h"
-#include "chrome/installer/util/master_preferences.h"
 #include "chrome/installer/util/util_constants.h"
 
 class DictionaryValue;
@@ -24,7 +23,7 @@
 
 class InstallationState;
 class InstallerState;
-class Package;
+class MasterPreferences;
 
 // This function installs or updates a new version of Chrome. It returns
 // install status (failed, new_install, updated etc).
@@ -48,11 +47,8 @@
     const InstallerState& installer_state,
     const FilePath& setup_path, const FilePath& archive_path,
     const FilePath& install_temp_path, const FilePath& prefs_path,
-    const installer::MasterPreferences& prefs, const Version& new_version,
-    const Package& package);
+    const installer::MasterPreferences& prefs, const Version& new_version);

[robertshield, 2011/01/20 22:06:06]

For readability, can you move each of these parameters to its own line?

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

Done.

 
-
-
 }  // namespace installer
 
 #endif  // CHROME_INSTALLER_SETUP_INSTALL_H_