tree 87adf8d232fd7f111935dd41d386082fde16f444
parent c4ad1ebc33abc088aca2909ba5cbaf7ae5e5659f
author Sen Jiang <senj@google.com> 1498092401 -0700
committer Sen Jiang <senj@google.com> 1498513834 -0700

Only split operation if it has multiple dst extents.

FragmentOperations() will split operations so that every operation will
have only one dst extent, but if the operation already meet this
requirement, it should be a no-op and we shouldn't waste time recompressing
it. This can save a lot of time if there's huge replace operation that
only has one dst extent.

As a side effect of not splitting those operations, if they are SOURCE_COPY
operations, they will not have src_length and dst_length set anymore,
because those were set in SplitSourceCopy() before this change, but these
lengths are never actually used in delta performer since the very first
implementation of PerformSourceCopyOperation():
https://chromium-review.googlesource.com/c/263747/
So it's safe to remove it from the manifest. We might as well just remove
src_length and dst_length in all SOURCE_COPY operations (even for those
are splitted) to reduce payload size a little bit.

Bug: 62470452
Test: generated a payload, all operations are still the same except
SOURCE_COPY don't have src_length and dst_length any more

Change-Id: I019ca9056479165e76cc25d7a4ba453088398928