New issue
Advanced search Search tips

Issue 763057 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

53kb regression in resource_sizes (MonochromePublic.apk) at 500312:500312

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

Issue description

Caused by "Media Controls: Replace painter with CSS."
Commit: 8a3b36ef9e6c6747876175f48f8cb24e6573dc30

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

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

Based on the graph: 8kb of native code, 45kb of .pak

It looks like this increase was probably unexpected or might be avoidable.

Please have a look and either:
* Close as “Won't Fix” with a short justification, or
* Land a revert / fix-up.

One idea here would be to add compress="gzip" to IDR_UASTYLE_MEDIA_CONTROLS_CSS
 
All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=763057

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


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

Android Builder
Status: Fixed (was: Assigned)
Commit has been reverted.

Sign in to add a comment