Revert "Reland "Reland "Android: Revamp .build_config.json files"""
This reverts commit 7c3be0e555b22cc3c4d6f054951afb7cb6cc27ba.
Reason for revert: broke generate_gradle.py
Bug: 413331591
Bug: 413331591, 413429757
Original change's description:
> Reland "Reland "Android: Revamp .build_config.json files""
>
> This reverts commit e29eae6dd420460a8e3cb58c9b58b4917f0092de.
>
> Reason for reland: Ran mega CQ
>
> Bug: 413331591, 413429757
> Bug: 413331591
> Original change's description:
> > Revert "Reland "Android: Revamp .build_config.json files""
> >
> > This reverts commit a9c9cb734574a2a517dc4fc5518b4338287cc6c8.
> >
> > Reason for revert: https://ci.chromium.org/ui/p/chrome/builders/ci/android-builder-perf/878126/blamelist
> >
> > Bug: 413331591
> > Bug: 413331591, 413429757
> > Original change's description:
> > > Reland "Android: Revamp .build_config.json files"
> > >
> > > This reverts commit 9acd9592e904395600e063d6bbbc541c053e53db.
> > >
> > > Reason for revert: Fixed trace_event_rewritten targets
> > >
> > > Bug: 413331591, 413429757
> > > Original change's description:
> > > > Revert "Android: Revamp .build_config.json files"
> > > >
> > > > This reverts commit 96f90bea226aad4c34f98e051dc414c06f6171b0.
> > > >
> > > > Reason for revert: Broke trace_event_rewritten target
> > > >
> > > > Bug: 413331591
> > > > Original change's description:
> > > > > Android: Revamp .build_config.json files
> > > > >
> > > > > Use write_file() rather than command-line flags to pass information from
> > > > > GN -> write_build_config.py
> > > > > * Creates $target_out_dir/$target_name.params.json files
> > > > > * Reduces the size of .ninja files
> > > > > * Enables auxiliary tools that read .build_config.json files to instead
> > > > > read .params.json, which do not need to be built.
> > > > > * Reduces boiler-plate in write_build_config.py
> > > > >
> > > > > Other simplifications:
> > > > > * Remove outdated documentation about build_config values
> > > > > * Renamed some build_config.json keys to improve consistency
> > > > > * Moves uncompressed_assets and final_dex_path for bundle modules
> > > > > into the bundle's build_config (so that the module's config is
> > > > > not needed)
> > > > >
> > > > > I've attempted to update all tools that read .build_config.json files.
> > > > >
> > > > > build.ninja files: 884650045->878689015 (about 6mb smaller)
> > > > > .build_config.json: 285674537->273369616(about 12mb smaller)
> > > > >
> > > > > Bug: 413331591
> > > > > Change-Id: I6d01f4fea04b95a1d3911e0538f0e1a4090a4dcf
> > > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6479572
> > > > > Reviewed-by: Peter Wen <[email protected]>
> > > > > Owners-Override: Andrew Grieve <[email protected]>
> > > > > Commit-Queue: Andrew Grieve <[email protected]>
> > > > > Cr-Commit-Position: refs/heads/main@{#1451366}
> > > >
> > > > Bug: 413331591, 413429757
> > > > No-Presubmit: true
> > > > No-Tree-Checks: true
> > > > No-Try: true
> > > > Change-Id: Ia3ef17f7d67babaaea6b14e44c17e8738a4db26a
> > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6489352
> > > > Bot-Commit: Rubber Stamper <[email protected]>
> > > > Auto-Submit: Andrew Grieve <[email protected]>
> > > > Commit-Queue: Andrew Grieve <[email protected]>
> > > > Owners-Override: Andrew Grieve <[email protected]>
> > > > Cr-Commit-Position: refs/heads/main@{#1451423}
> > >
> > > Bug: 413331591
> > > Bug: 413331591, 413429757
> > > Change-Id: I63fcea903945f2b5c0554925e019d92f5d242ba3
> > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6490257
> > > Commit-Queue: Andrew Grieve <[email protected]>
> > > Reviewed-by: Sam Maier <[email protected]>
> > > Owners-Override: Andrew Grieve <[email protected]>
> > > Cr-Commit-Position: refs/heads/main@{#1451844}
> >
> > Bug: 413331591, 413429757
> > Bug: 413331591
> > No-Presubmit: true
> > No-Tree-Checks: true
> > No-Try: true
> > Change-Id: Id390eb968d27f44b2ef1d41753c13525caf2e555
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6490036
> > Auto-Submit: Andrew Grieve <[email protected]>
> > Owners-Override: Andrew Grieve <[email protected]>
> > Bot-Commit: Rubber Stamper <[email protected]>
> > Commit-Queue: Rubber Stamper <[email protected]>
> > Cr-Commit-Position: refs/heads/main@{#1451888}
>
> Bug: 413331591
> Bug: 413331591, 413429757
> Change-Id: I7273e510cb4521c5b1c9e74965faf1ac699b579f
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6489555
> Owners-Override: Andrew Grieve <[email protected]>
> Reviewed-by: Peter Wen <[email protected]>
> Commit-Queue: Andrew Grieve <[email protected]>
> Cr-Commit-Position: refs/heads/main@{#1452633}
Bug: 413331591, 413429757
Bug: 413331591
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Change-Id: I5959c554f1f355e9737ca34d5ee327b4aef1d123
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6494174
Owners-Override: Peter Wen <[email protected]>
Reviewed-by: Peter Wen <[email protected]>
Commit-Queue: Peter Wen <[email protected]>
Bot-Commit: Rubber Stamper <[email protected]>
Auto-Submit: Andrew Grieve <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1452726}
NOKEYCHECK=True
GitOrigin-RevId: add4fa921ec0bdc6d16cc5efcefca8771ccfa460
1 file changed
