New issue
Advanced search Search tips

Issue 788221 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 787512
Owner:
Closed: Nov 2017
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug-Regression



Sign in to add a comment

84KiB regression in resource_sizes (MonochromePublic.apk) at 518781:518781

Project Member Reported by wnwen@google.com, Nov 23 2017

Issue description

Caused by “Optimize HTML parser for speed instead of size on Android.”

Commit: b9fddd03657a4b15506833bf7678a51785094dc7

Link to size graph: https://chromeperf.appspot.com/report?sid=a097e74b1aa288511afb4cb616efe0f95ba4d347ad61d5e835072f23450938ba&num_points=10&rev=518781

Debugging size regressions is documented at: https://chromium.googlesource.com/chromium/src/+/master/docs/speed/apk_size_regressions.md#Debugging-Apk-Size-Increase

Section Sizes (Total=82.2kb (84159 bytes)):     
    .bss: 0 bytes (0 bytes) (not included in totals)                                                                     
    .data: 0 bytes (0 bytes) (0.0%)                   
    .data.rel.ro: 0 bytes (0 bytes) (0.0%)                                                                                                                                       
    .rel.dyn: -8 bytes (-8 bytes) (-0.0%)                                                                                                                                        
    .rodata: 0 bytes (0 bytes) (0.0%)                                                                                                                                            
    .text: 82.2kb (84171 bytes) (100.0%)

It looks to me that the size increase is expected, just these files weren't in android before.
Feel free to close as “Won't Fix” unless you can see some way to reduce size.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Nov 23 2017

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

(For debugging:) Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?sid=acd1c5ef448e00cbb3227e4250271267b78aee3674b3ccf68b1128130cd3d11f


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

Android Builder

Comment 2 by wnwen@chromium.org, Nov 23 2017

Cmd to see size analysis:

tools/binary_size/diagnose_bloat.py b9fddd03657a4b15506833bf7678a51785094dc7 --cloud

Comment 3 by wnwen@chromium.org, Nov 23 2017

Labels: OS-Android
Mergedinto: 787512
Status: Duplicate (was: Assigned)
Let's use the Bug on the commit to track this.

Sign in to add a comment