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

Side by Side Diff: chrome/installer/util/util_constants.h

Issue 10806086: Fix multi-install update regression. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: addressed review comments Created 8 years, 4 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/installer/util/chrome_binaries_operations.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 // 4 //
5 // Defines all install related constants that need to be used by Chrome as 5 // Defines all install related constants that need to be used by Chrome as
6 // well as Chrome Installer. 6 // well as Chrome Installer.
7 7
8 #ifndef CHROME_INSTALLER_UTIL_UTIL_CONSTANTS_H_ 8 #ifndef CHROME_INSTALLER_UTIL_UTIL_CONSTANTS_H_
9 #define CHROME_INSTALLER_UTIL_UTIL_CONSTANTS_H_ 9 #define CHROME_INSTALLER_UTIL_UTIL_CONSTANTS_H_
10 10
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
63 // multi-install installation of the same product 63 // multi-install installation of the same product
64 // on the system. 64 // on the system.
65 READY_MODE_OPT_IN_FAILED, // 36. Failed to opt-into Chrome Frame. 65 READY_MODE_OPT_IN_FAILED, // 36. Failed to opt-into Chrome Frame.
66 READY_MODE_TEMP_OPT_OUT_FAILED, // 37. Failed to temporarily opt-out of 66 READY_MODE_TEMP_OPT_OUT_FAILED, // 37. Failed to temporarily opt-out of
67 // Chrome Frame. 67 // Chrome Frame.
68 READY_MODE_END_TEMP_OPT_OUT_FAILED, // 38. Failed to end temporary opt-out 68 READY_MODE_END_TEMP_OPT_OUT_FAILED, // 38. Failed to end temporary opt-out
69 // of Chrome Frame. 69 // of Chrome Frame.
70 CONFLICTING_CHANNEL_EXISTS, // 39. A multi-install product on a different 70 CONFLICTING_CHANNEL_EXISTS, // 39. A multi-install product on a different
71 // update channel exists. 71 // update channel exists.
72 READY_MODE_REQUIRES_CHROME, // 40. Chrome Frame in ready-mode requires Chrome 72 READY_MODE_REQUIRES_CHROME, // 40. Chrome Frame in ready-mode requires Chrome
73 REQUIRES_MULTI_INSTALL, // 41. --multi-install was missing from the 73 APP_HOST_REQUIRES_MULTI_INSTALL, // 41. --multi-install was missing from the
74 // command line. 74 // command line.
75 APPLY_DIFF_PATCH_FAILED, // 42. Failed to apply a diff patch. 75 APPLY_DIFF_PATCH_FAILED, // 42. Failed to apply a diff patch.
76 INCONSISTENT_UPDATE_POLICY, // 43. Inconsistent update policy GP settings. 76 INCONSISTENT_UPDATE_POLICY, // 43. Inconsistent update policy GP settings.
77 APP_HOST_REQUIRES_USER_LEVEL, // 44. --system-level is forbidden.
78 APP_HOST_REQUIRES_BINARIES, // 45. No Chrome binaries at either level.
79 // Friendly reminder: note the COMPILE_ASSERT below.
77 }; 80 };
78 81
79 82
80 // If the following compile assert fires it means that the InstallStatus 83 // Existing InstallStatus values must not change. Always add to the end.
81 // enumeration changed which will break the contract between the old 84 COMPILE_ASSERT(installer::APP_HOST_REQUIRES_BINARIES == 45,
82 // chrome installed and the new setup.exe that is trying to upgrade.
83 COMPILE_ASSERT(installer::INCONSISTENT_UPDATE_POLICY == 43,
84 dont_change_enum); 85 dont_change_enum);
85 86
86 // The type of an update archive. 87 // The type of an update archive.
87 enum ArchiveType { 88 enum ArchiveType {
88 UNKNOWN_ARCHIVE_TYPE, // Unknown or uninitialized. 89 UNKNOWN_ARCHIVE_TYPE, // Unknown or uninitialized.
89 FULL_ARCHIVE_TYPE, // Full chrome.7z archive. 90 FULL_ARCHIVE_TYPE, // Full chrome.7z archive.
90 INCREMENTAL_ARCHIVE_TYPE // Incremental or differential archive. 91 INCREMENTAL_ARCHIVE_TYPE // Incremental or differential archive.
91 }; 92 };
92 93
93 // Stages of an installation reported through Google Update on failure. 94 // Stages of an installation reported through Google Update on failure.
(...skipping 125 matching lines...) Expand 10 before | Expand all | Expand 10 after
219 extern const wchar_t kChromeChannelCanary[]; 220 extern const wchar_t kChromeChannelCanary[];
220 extern const wchar_t kChromeChannelDev[]; 221 extern const wchar_t kChromeChannelDev[];
221 extern const wchar_t kChromeChannelBeta[]; 222 extern const wchar_t kChromeChannelBeta[];
222 extern const wchar_t kChromeChannelStable[]; 223 extern const wchar_t kChromeChannelStable[];
223 224
224 extern const size_t kMaxAppModelIdLength; 225 extern const size_t kMaxAppModelIdLength;
225 226
226 } // namespace installer 227 } // namespace installer
227 228
228 #endif // CHROME_INSTALLER_UTIL_UTIL_CONSTANTS_H_ 229 #endif // CHROME_INSTALLER_UTIL_UTIL_CONSTANTS_H_
OLDNEW
« no previous file with comments | « chrome/installer/util/chrome_binaries_operations.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698