• Samuel Huang's avatar
    Reland "[SuperSize] Add multiple Container support using .ssargs files." · d20a3819
    Samuel Huang authored
    This reverts commit fc8e613a.
    
    Reason for revert: Fixed the issue.
    
    Original change's description:
    > Revert "[SuperSize] Add multiple Container support using .ssargs files."
    > 
    > This reverts commit 19d7a780.
    > 
    > Reason for revert: Breaking CQ (see bug)
    > 
    > Original change's description:
    > > [SuperSize] Add multiple Container support using .ssargs files.
    > > 
    > > This CL adds SuperSize support for multiple containers. Details:
    > > * Introduce .ssargs file, which is a text file to specify multiple
    > >   containers for SuperSize-archive to process into a single .size file.
    > >   * Container file types are auto-detected.
    > >   * Container files are relative to .ssargs file location, unless
    > >     absolute paths are given.
    > > * Introduce Version 1.1 .size file format:
    > >   * Header fields contain Container specs, now absorbing metadata and
    > >     section_sizes.
    > >   * Per-section data become per-container-per-section.
    > > * For compatibility: Running SuperSize-archive on one container still
    > >   generates Version 1.0 .size file format.
    > > * Update SuperSize-console output (including diffs) to accommodate.
    > > 
    > > After this CL, basic Trichrome support is available. A recommended
    > > .ssargs file (3 lines) for the official build is as follows:
    > > 
    > > TrichromeLibraryGoogle.apk --name Library
    > > TrichromeChromeGoogle.minimal.apks --name Chrome --java-only
    > > TrichromeWebViewGoogle.apk --name WebView --java-only
    > > 
    > > Bug: 1040645
    > > Change-Id: I67df00731d07660163410b59c57854bf56ea651e
    > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2248860
    > > Commit-Queue: Samuel Huang <huangs@chromium.org>
    > > Reviewed-by: Andrew Grieve <agrieve@chromium.org>
    > > Cr-Commit-Position: refs/heads/master@{#780594}
    > 
    > TBR=huangs@chromium.org,agrieve@chromium.org
    > 
    > Change-Id: Ic3175ea3e8ede41e3183c12b5a00a4a09be55a75
    > No-Try: True
    > Bug: 1040645, 1097572
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2256593
    > Reviewed-by: Andrew Grieve <agrieve@chromium.org>
    > Commit-Queue: Andrew Grieve <agrieve@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#780639}
    
    TBR=huangs@chromium.org,agrieve@chromium.org
    
    # Not skipping CQ checks because this is a reland.
    
    Bug: 1040645, 1097572
    Change-Id: I95a20b271d7522a1aa70cc3247ad9a1fab911dce
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2257279Reviewed-by: default avatarAndrew Grieve <agrieve@chromium.org>
    Commit-Queue: Samuel Huang <huangs@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#780788}
    d20a3819
Console.golden 14.7 KB