New issue
Advanced search Search tips

Issue 735135 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Jun 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

49kb regression in resource_sizes (MonochromePublic.apk) at 480875:480875

Project Member Reported by estevenson@chromium.org, Jun 20 2017

Issue description

All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=735135

Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?keys=agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgjtmBuwoM


Bot(s) for this bug's original alert(s):

Android Builder
Owner: rouslan@chromium.org
Status: Assigned (was: Untriaged)
tools/binary_size/diagnose_bloat.py 1aea7837a31d25fb5e7422c93a2efbd1a4b8c388 --cloud -v

Summary:
Normalized APK size: +50,302 bytes

Details:
   +50,302 bytes MonochromePublic.apk_Specifics normalized apk size
   +26,060 bytes MonochromePublic.apk_InstallSize Estimated installed size
   +19,774 bytes MonochromePublic.apk_Uncompressed Native resources (l10n) size
   +16,000 bytes MonochromePublic.apk_InstallSize APK size
   +10,437 bytes MonochromePublic.apk_InstallBreakdown Java code size
    +9,071 bytes MonochromePublic.apk_TransferSize Transfer size (deflate)
    +7,635 bytes MonochromePublic.apk_InstallBreakdown Native resources (l10n) size
    +6,755 bytes MonochromePublic.apk_Breakdown Native resources (l10n) size
    +4,096 bytes MonochromePublic.apk_Uncompressed Native code size
    +4,096 bytes MonochromePublic.apk_Breakdown Native code size
    +4,096 bytes MonochromePublic.apk_InstallBreakdown Native code size
    +4,096 bytes MonochromePublic.apk_Specifics main lib size
    +3,716 bytes MonochromePublic.apk_Uncompressed Compiled Android resources size
    +3,716 bytes MonochromePublic.apk_Breakdown Compiled Android resources size
    +3,716 bytes MonochromePublic.apk_InstallBreakdown Compiled Android resources size
    +3,060 bytes MonochromePublic.apk_Uncompressed Java code size
    +3,060 bytes MonochromePublic.apk_Specifics main dex size
    +1,257 bytes MonochromePublic.apk_Breakdown Java code size
      +256 bytes MonochromePublic.apk_MainLibInfo data
      +248 bytes MonochromePublic.apk_DexCache DexCache
      -172 bytes MonochromePublic.apk_MainLibInfo text
       +28 entries MonochromePublic.apk_Dex fields
       +21 entries MonochromePublic.apk_Dex strings
       +13 entries MonochromePublic.apk_Dex methods


It looks like this is just normal growth due to a large CL.
  * Much of the growth comes from Native and Android resources (strings)
  * 4kb from native code size
  * 3kb from Java code size (and the normalized apk size accounts for extra growth due to odex)

rouslan@ - this is mostly just an FYI since it seems most of this growth is expected. Feel free to close this as WontFix unless there's something you think we can improve on.
Status: WontFix (was: Assigned)
Thank you for the heads up.
Project Member

Comment 5 by 42576172...@developer.gserviceaccount.com, Jun 20 2017


=== BISECT JOB RESULTS ===
NO Perf regression found, tests failed to produce values

Bisect Details
  Configuration: android_nexus7_perf_bisect
  Benchmark    : resource_sizes
  Metric       : MonochromePublic.apk_Specifics/normalized apk size


To Run This Test
  src/build/android/resource_sizes.py --chromium-output-directory {CHROMIUM_OUTPUT_DIR} --chartjson {CHROMIUM_OUTPUT_DIR}/apks/MonochromePublic.apk

Debug Info
  https://chromeperf.appspot.com/buildbucket_job_status/8976243143608983536

Is this bisect wrong?
  https://chromeperf.appspot.com/bad_bisect?try_job_id=6094585467502592


| O O | Visit http://www.chromium.org/developers/speed-infra/perf-bug-faq
|  X  | for more information addressing perf regression bugs. For feedback,
| / \ | file a bug with component Speed>Bisection.  Thank you!
Project Member

Comment 7 by 42576172...@developer.gserviceaccount.com, Jun 20 2017


=== BISECT JOB RESULTS ===
NO Perf regression found, tests failed to produce values

Bisect Details
  Configuration: android_nexus7_perf_bisect
  Benchmark    : resource_sizes
  Metric       : MonochromePublic.apk_Specifics/normalized apk size


To Run This Test
  src/build/android/resource_sizes.py --chromium-output-directory {CHROMIUM_OUTPUT_DIR} --chartjson {CHROMIUM_OUTPUT_DIR}/apks/MonochromePublic.apk

Debug Info
  https://chromeperf.appspot.com/buildbucket_job_status/8976238041808422976

Is this bisect wrong?
  https://chromeperf.appspot.com/bad_bisect?try_job_id=6094585467502592


| O O | Visit http://www.chromium.org/developers/speed-infra/perf-bug-faq
|  X  | for more information addressing perf regression bugs. For feedback,
| / \ | file a bug with component Speed>Bisection.  Thank you!

Sign in to add a comment