New issue
Advanced search Search tips

Issue 762359 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Jan 21
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Consolidate resource_sizes.py and sizes.py

Project Member Reported by agrieve@chromium.org, Sep 6 2017

Issue description

There are two copies of sizes.py:
https://cs.chromium.org/chromium/build/scripts/slave/chromium/sizes.py
https://cs.chromium.org/chromium/src/infra/scripts/legacy/scripts/slave/chromium/sizes.py

And resource_sizes.py is here:
https://cs.chromium.org/chromium/src/build/android/resource_sizes.py

sizes.py is used only for desktop (and cronet?), and dumps a few key ELF section sizes, as well as static initializer count.

resource_sizes.py dumps many metrics about the make-up of an APK, and also static initializer count.

size.py static initializers are checked by trybots via checked-in expectations here:
https://cs.chromium.org/chromium/src/tools/perf_expectations/perf_expectations.json

resource_sizes.py static initializers are checked via a build target:
//chrome/android:monochrome_static_initializers

It would really be great if we could consolidate as much of these scripts as possible.
 
Components: -Build Tools>BinarySize
Labels: -Performance-Size

Comment 2 by cjgrant@chromium.org, Jan 21 (2 days ago)

Status: WontFix (was: Available)
As per triage meeting discussion, closing this off.  No work is planned.

Sign in to add a comment