Commit d4982e87 authored by grt@chromium.org's avatar grt@chromium.org

Split UNPACKING_FAILED exit code out from UNCOMPRESSION_FAILED.

To ease failure diagnosis.

BUG=385421
R=gab@chromium.org

Review URL: https://codereview.chromium.org/338273002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277780 0039d316-1c4b-4281-b951-d872f2087c98
parent 96c2de57
......@@ -1468,10 +1468,10 @@ InstallStatus InstallProductsHelper(
unpack_path.value(),
NULL)) {
installer_state.WriteInstallerResult(
UNCOMPRESSION_FAILED,
UNPACKING_FAILED,
IDS_INSTALL_UNCOMPRESSION_FAILED_BASE,
NULL);
return UNCOMPRESSION_FAILED;
return UNPACKING_FAILED;
}
VLOG(1) << "unpacked to " << unpack_path.value();
......
......@@ -92,13 +92,15 @@ enum InstallStatus {
CPU_NOT_SUPPORTED, // 54. Current OS not supported
REENABLE_UPDATES_SUCCEEDED, // 55. Autoupdates are now enabled.
REENABLE_UPDATES_FAILED, // 56. Autoupdates could not be enabled.
UNPACKING_FAILED, // 57. Unpacking the (possibly patched)
// uncompressed archive failed.
// Friendly reminder: note the COMPILE_ASSERT below.
};
// Existing InstallStatus values must not change. Always add to the end.
COMPILE_ASSERT(installer::REENABLE_UPDATES_FAILED == 56,
COMPILE_ASSERT(installer::UNPACKING_FAILED == 57,
dont_change_enum);
// The type of an update archive.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment