commit f526156e5fbc46203d097cdaa11f51df2eafae54
author Yifan Hong <elsk@google.com> Tue Mar 10 10:28:10 2020 -0700
committer Yifan Hong <elsk@google.com> Tue Mar 10 18:22:24 2020 -0700
tree f2da6ca0b753248a7acfabe709abe9270734714b
parent 0f821c70b88e088fe9794cda9251b3a92d27444e

ShouldSkipOperation -> OptimizeOperation

For SOURCE_COPY operations like
  563412 -> 123456
OptimizeOperation optimizes it to
  5612 -> 1256
and skip writing blocks that does not need to be written for snapshot
partitions.

Bug: 148623880
Test: update_engine_unittests
Test: apply incremental OTA

Change-Id: Ifd2c3851f703f272a74c8f0e9a1c9a82dbcce3e3

dynamic_partition_control_android.h

diff --git a/dynamic_partition_control_android.h b/dynamic_partition_control_android.h
index c842efc..1ff59a9 100644
--- a/dynamic_partition_control_android.h
+++ b/dynamic_partition_control_android.h
@@ -35,8 +35,9 @@
   ~DynamicPartitionControlAndroid();
   FeatureFlag GetDynamicPartitionsFeatureFlag() override;
   FeatureFlag GetVirtualAbFeatureFlag() override;
-  bool ShouldSkipOperation(const std::string& partition_name,
-                           const InstallOperation& operation) override;
+  bool OptimizeOperation(const std::string& partition_name,
+                         const InstallOperation& operation,
+                         InstallOperation* optimized) override;
   void Cleanup() override;
 
   bool PreparePartitionsForUpdate(uint32_t source_slot,