New issue
Advanced search Search tips

Issue 744743 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

48 kb regression in resource_sizes (MonochromePublic.apk) at 486970:486970

Project Member Reported by estevenson@chromium.org, Jul 17 2017

Issue description

Caused by “Coverage counting path renderer”

Commit: ebce2b652a48bd3d3195dcf03379d723c8f47cd4 (Part of a Skia roll - actual CL was https://skia-review.googlesource.com/c/21541)

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

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

The growth came entirely from native code.

It's not clear to me whether or not this increase was expected. Please have a look and either:

Close as “Won't Fix” with a short justification, or land a revert / fix-up.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Jul 17 2017

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

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


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

Android Builder
Description: Show this description
Cc: csmartdalton@google.com
Labels: OS-Android
Owner: estevenson@chromium.org
Status: Assigned (was: Untriaged)
Assigning to csmartdalton@ (can't seem to make you the owner as you aren't a chromium project member).

Please see the attached symbol diff for more details.
diff_results.txt
37.3 KB View Download
Status: WontFix (was: Assigned)
The coverage counting path render is a sizable change. 32k of new code and 13k of new GPU shader strings both sound reasonable.

Sign in to add a comment