commit | 59d07c9b15987d630eda963a767ad1a3ecbc0067 | [log] [tgz] |
---|---|---|
author | Zijie He <zijiehe@google.com> | Mon Feb 24 23:41:38 2025 |
committer | LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> | Mon Feb 24 23:41:38 2025 |
tree | 8ebb35b6ddb7b4154cd6a8871630984fe16e551b | |
parent | a3e93fe37487342b8fd27fcea9cf9c2f138ad5c9 [diff] |
Revert "Reland "android-binary-size: Create arm64 supersize report for large changes"" This reverts commit 7f0c209cc4e1064bfc1e4473a1cc01b5aa6746ae. Reason for revert: Break fuchsia-binary-size try, https://ci.chromium.org/ui/p/chromium/builders/try/fuchsia-binary-size?limit=200&createdBefore=1740124800&cursor=id%3E8722398656659645041. Original change's description: > Reland "android-binary-size: Create arm64 supersize report for large changes" > > This reverts commit 9cbd00dce9b8598edf572c096c9ff241d2b17f0b. > > Reason for reland: Used led to ensure it works. > > Original change's description: > > Revert "android-binary-size: Create arm64 supersize report for large changes" > > > > This reverts commit d4a06904863377c9df8a05d671c80b26cb3a8865. > > > > Reason for revert: Broke fuschia size bot: > > > > https://ci.chromium.org/ui/p/chromium/builders/try/fuchsia-binary-size/1005130/overview > > > > Original change's description: > > > android-binary-size: Create arm64 supersize report for large changes > > > > > > It's been happening on occasion where arm32 size is fine, but arm64 size > > > is not. In these cases, it's frustrating to not have the SuperSize > > > report readily available. This changes fixes that. > > > > > > There should be no immediate change, as it's guarded by a recipe > > > property that will need to be set (and will also allow testing via > > > tryjobs). > > > > > > Bug: 383540547 > > > Change-Id: I1214911b5ce60d0d69966ca14deefd9cdd4c3d89 > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/build/+/6197099 > > > Commit-Queue: Haiyang Pan <hypan@google.com> > > > Auto-Submit: Andrew Grieve <agrieve@chromium.org> > > > Reviewed-by: Haiyang Pan <hypan@google.com> > > > > Bug: 383540547 > > Change-Id: I60dce750d255f50eb4994ffe80236854013f75bd > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/build/+/6208328 > > Commit-Queue: Haiyang Pan <hypan@google.com> > > Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> > > Reviewed-by: Haiyang Pan <hypan@google.com> > > Commit-Queue: Andrew Grieve <agrieve@chromium.org> > > Commit-Queue: Zijie He <zijiehe@google.com> > > Auto-Submit: Andrew Grieve <agrieve@chromium.org> > > Reviewed-by: Zijie He <zijiehe@google.com> > > Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> > > Bug: 383540547 > Change-Id: I7c5b5cb4828252c3f950b5e6dea86c37f20d0a1d > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/build/+/6208330 > Reviewed-by: Haiyang Pan <hypan@google.com> > Commit-Queue: Andrew Grieve <agrieve@chromium.org> Bug: 383540547 Change-Id: I1661bdd35a91c15f06dfbc7c63880299aeee30c4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/build/+/6298454 Reviewed-by: Erik Staab <estaab@chromium.org> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Auto-Submit: Zijie He <zijiehe@google.com> Commit-Queue: Erik Staab <estaab@chromium.org>
If you're here to make a change to ‘recipes’ (the code located in the recipes
directory), please take a look at the README for more information pertaining to recipes.
The preferred style is PEP8 with two-space indent. Functions use lowercase_with_underscores
, with the exception of the special functions RunSteps
and RunTests
in recipes. Use yapf (git cl format --no-clang-format
) to autoformat new code.