commit 4bc21e8413d1d01f2b1bf474cbb7fa6811b40528
author Andrew Grieve <agrieve@chromium.org> Thu Jun 26 18:58:39 2025
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Jun 26 18:58:39 2025
tree 05cd6d2c189dcace47eb2d4abe460db538bac82d
parent 9f8c95031c2436e52295228c6b82191c9cc10f5a

Reland #2 "Android: Refactor write_build_config.py"

This reverts commit 95e33c7f5a86dc5a4a574ba73b5d9af9c0fca544.

Reason for reland:
 * Fixed not taking proguard configs from apk_under_test
 * Fixed cronet build (via https://chromium-review.googlesource.com/c/chromium/src/+/6680687)

Bug: 413331591
Original change's description:
> Revert "Reland "Android: Refactor write_build_config.py""
>
> This reverts commit 06d621e852be7ea84d6dfcd251b8f0c33d36a6f4.
>
> Reason for revert: Broke chrome_uiautomator_test_apk
>
> Bug: 413331591
> Original change's description:
> > Reland "Android: Refactor write_build_config.py"
> >
> > This is a reland of commit 814fb1e64612b502875b64e82a95a2ca3a4a5af9
> >
> > Original change's description:
> > > Android: Refactor write_build_config.py
> > >
> > > The main goal is to speed it up by having it read only .params.json
> > > files instead of also needing to read dependent .build_config.json
> > > files.
> > >
> > > There's also a good number of simplifications included, including
> > > fixing lists not always being topologically sorted.
> > >
> > > Running: autoninja --offline $(cat all_build_configs.txt)
> > > Before: 50 seconds
> > > After: 16 seconds
> > >
> > > This removes dependent .build_config files as inputs in most cases,
> > > so improves remote build times.
> > > Running: autoninja $(cat all_build_configs.txt)
> > > Before: 42 seconds
> > > After: 14 seconds
> > >
> > > Bug: 413331591
> > > Change-Id: I58f424ffc14943089b1f7625a417abb586e6fb92
> > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6639823
> > > Commit-Queue: Andrew Grieve <agrieve@chromium.org>
> > > Reviewed-by: Sam Maier <smaier@chromium.org>
> > > Owners-Override: Andrew Grieve <agrieve@chromium.org>
> > > Cr-Commit-Position: refs/heads/main@{#1478885}
> >
> > Bug: 413331591
> > Change-Id: Ia6ff026573c6cd31b2c3d1c00124237a1abf0230
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6679732
> > Commit-Queue: Andrew Grieve <agrieve@chromium.org>
> > Owners-Override: Andrew Grieve <agrieve@chromium.org>
> > Reviewed-by: Sam Maier <smaier@chromium.org>
> > Cr-Commit-Position: refs/heads/main@{#1479161}
>
> Bug: 413331591
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Change-Id: I0d9c1b2fb1c13a1ef6de722b4f882e7c717477b7
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6681268
> Owners-Override: Andrew Grieve <agrieve@chromium.org>
> Auto-Submit: Andrew Grieve <agrieve@chromium.org>
> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> Cr-Commit-Position: refs/heads/main@{#1479260}

Bug: 413331591
Change-Id: Ib5df512c17d40ae066f5500f620bcd74f7495000
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6681467
Auto-Submit: Andrew Grieve <agrieve@chromium.org>
Commit-Queue: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: Mohannad Farrag <aymanm@google.com>
Reviewed-by: Sam Maier <smaier@chromium.org>
Owners-Override: Andrew Grieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1479356}