New issue
Advanced search Search tips

Issue 760990 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Aug 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug-Regression



Sign in to add a comment

63kb Increase in Install Size

Project Member Reported by agrieve@chromium.org, Aug 31 2017

Issue description

Caused by: "Separate all refactor code for downloads as a foreground service."
Commit: 94298d7de8ab29bc0d95970137e4c52886ca1771

Link to size graph: https://chromeperf.appspot.com/report?sid=a097e74b1aa288511afb4cb616efe0f95ba4d347ad61d5e835072f23450938ba&num_points=10&rev=498445
(Note: point has many commits, but used diagnose_bloat.py to identify this commit).

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

Looks like the growth was entirely from dex growth. Checked dex diff, and there's doesn't seem to be any unexpected classes/methods pulled in (size growth is from the large change). 

Feel free to close as “Won't Fix” unless you can see some way to reduce size.
 

Comment 1 by qin...@chromium.org, Aug 31 2017

Status: WontFix (was: Assigned)
We are in the middle of a refactoring,  all the old code will go away once the refactoring is done. Unfortuately, M62 will see the size increase.

Mark this as Won't fix, as it will be addressed in coming weeks.

Sign in to add a comment