New issue
Advanced search Search tips

Issue 880854 link

Starred by 1 user
Status: Assigned
Owner:
amalova@chromium.org
Cc:
cjgrant@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

24716 byte binary size regression in resource_sizes (MonochromePublic.apk) at 588850:588850

Project Member Reported by cjgrant@google.com, Sep 5 
Caused by "Allow update callback for existing CCT, rolls DEPS to the CL:1076648."

Commit:  b5cd7407911106ebd7dc599f75ff05099310233d

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

It's not clear whether or not this increase was expected.
Could you please  have a look and either:

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

Thanks!
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Sep 5 

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

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


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

Android Builder Perf
Project Member

Comment 2 by 42576172...@developer.gserviceaccount.com, Sep 5 

Assigning to amalova@chromium.org because this is the only CL in range:
Allow update callback for existing CCT, rolls DEPS to the CL:1076648.

Bug:  847528 
Change-Id: Ie119a50be4dc4131954cac001a70821feb94018f
Reviewed-on: https://chromium-review.googlesource.com/1077147
Commit-Queue: Anna Malova <amalova@chromium.org>
Reviewed-by: Benoit L <lizeb@chromium.org>
Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
Reviewed-by: Peter Conn <peconn@chromium.org>
Cr-Commit-Position: refs/heads/master@{#588850}
►

Sign in to add a comment