commit 14c0f1b1e2ada315b3ee63084435b899e51c5bc5
author Gilad Arnold <garnold@chromium.org> Tue May 12 18:16:23 2015 -0700
committer Gilad Arnold <garnold@chromium.org> Wed May 13 01:27:57 2015 +0000
tree 1672ab23e8d4a4d83ecd4239552f6bcd43e1cd06
parent 41e3474eebe3748a45aba4b297a7a313a1006c6c

update_engine: Disable splitting of fragmented REPLACE_BZ operations.

The said feature was introduced in CL:269130 and violated an invariant
of the delta payload format. This leaves most of the logic in place but
disables invoking it during delta generation. It will be re-enabled once
the logic is fixed.

BUG=chromium:487436
TEST=Unit tests

Change-Id: I59bf79749f0548d2935aecf489849ac2f9cb1f1e
Reviewed-on: https://chromium-review.googlesource.com/270581
Tested-by: Gilad Arnold <garnold@chromium.org>
Commit-Queue: Gilad Arnold <garnold@chromium.org>
Reviewed-by: Richard Barnette <jrbarnette@chromium.org>

payload_generator/delta_diff_generator.cc

diff --git a/payload_generator/delta_diff_generator.cc b/payload_generator/delta_diff_generator.cc
index 14586f7..c9852fd 100644
--- a/payload_generator/delta_diff_generator.cc
+++ b/payload_generator/delta_diff_generator.cc
@@ -1396,13 +1396,6 @@
     } else if (aop.op.type() ==
                DeltaArchiveManifest_InstallOperation_Type_REPLACE) {
       TEST_AND_RETURN_FALSE(SplitReplace(aop, &fragmented_aops));
-    } else if (aop.op.type() ==
-               DeltaArchiveManifest_InstallOperation_Type_REPLACE_BZ) {
-      TEST_AND_RETURN_FALSE(SplitReplaceBz(aop,
-                                           &fragmented_aops,
-                                           target_part_path,
-                                           data_fd,
-                                           data_file_size));
     } else {
       fragmented_aops.push_back(aop);
     }