commit 0a52f1b9a9d99424a63105aa7ae09114f24e56ac
author Kelvin Zhang <zhangkelvin@google.com> Fri Jan 29 10:51:01 2021 -0500
committer Kelvin Zhang <zhangkelvin@google.com> Tue Feb 02 21:27:06 2021 +0000
tree f8cbe815d313c0e147f2ec0c864484096a631f6b
parent 6b45ea5c3d5e68b30d155a0eecbbaad53076b81f

Skip merge sequence and COW computation if VABC disabled

Test: treehugger
Bug: 177936022

Change-Id: I2d9e75f9c1818aa82c993b248c94b23696ea8426

payload_generator/delta_diff_generator.cc

diff --git a/payload_generator/delta_diff_generator.cc b/payload_generator/delta_diff_generator.cc
index f008d40..74014d9 100644
--- a/payload_generator/delta_diff_generator.cc
+++ b/payload_generator/delta_diff_generator.cc
@@ -106,6 +106,10 @@
     if (!snapshot_enabled || !IsDynamicPartition(new_part_.name)) {
       return;
     }
+    // Skip cow size estimation if VABC isn't enabled
+    if (!config_.target.dynamic_partition_metadata->vabc_enabled()) {
+      return;
+    }
     if (!old_part_.path.empty()) {
       auto generator = MergeSequenceGenerator::Create(*aops_);
       if (!generator || !generator->Generate(cow_merge_sequence_)) {
@@ -127,7 +131,6 @@
     for (const AnnotatedOperation& aop : *aops_) {
       *operations.Add() = aop.op;
     }
-    // TODO(177936022) Skip cow size estimation if VABC isn't enabled
     *cow_size_ = EstimateCowSize(
         source_fd,
         std::move(target_fd),