Commit a0d24437 authored by Samuel Huang's avatar Samuel Huang Committed by Commit Bot

[SuperSize] Remove unused param from diagnose_bloat.ResourceSizesDiff.

In daignose_bloat.py, class ResourceSizesDiff takes an |apk_name| param
at construction, but the value sits around and never gets used. This CL
removes the param, and simplifies callers.

Bug: 1040645
Change-Id: I8843018e400c3a12ce15178490c03a0a4b415963
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2510532Reviewed-by: default avatarAndrew Grieve <agrieve@chromium.org>
Commit-Queue: Samuel Huang <huangs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#822739}
parent 7fef7272
...@@ -127,8 +127,7 @@ class ResourceSizesDiff(BaseDiff): ...@@ -127,8 +127,7 @@ class ResourceSizesDiff(BaseDiff):
_AGGREGATE_SECTIONS = ( _AGGREGATE_SECTIONS = (
'InstallBreakdown', 'Breakdown', 'MainLibInfo', 'Uncompressed') 'InstallBreakdown', 'Breakdown', 'MainLibInfo', 'Uncompressed')
def __init__(self, apk_name, filename='results-chart.json'): def __init__(self, filename='results-chart.json'):
self._apk_name = apk_name
self._diff = None # Set by |ProduceDiff()| self._diff = None # Set by |ProduceDiff()|
self._filename = filename self._filename = filename
super(ResourceSizesDiff, self).__init__('Resource Sizes Diff') super(ResourceSizesDiff, self).__init__('Resource Sizes Diff')
...@@ -898,9 +897,7 @@ def main(): ...@@ -898,9 +897,7 @@ def main():
supersize_path, tool_prefix = paths supersize_path, tool_prefix = paths
diffs = [NativeDiff(build.size_name, supersize_path)] diffs = [NativeDiff(build.size_name, supersize_path)]
if build.IsAndroid(): if build.IsAndroid():
diffs += [ diffs += [ResourceSizesDiff()]
ResourceSizesDiff(build.apk_name)
]
diff_mngr = _DiffArchiveManager(revs, args.archive_directory, diffs, build, diff_mngr = _DiffArchiveManager(revs, args.archive_directory, diffs, build,
subrepo, args.unstripped) subrepo, args.unstripped)
consecutive_failures = 0 consecutive_failures = 0
......
...@@ -114,8 +114,8 @@ def _CreateMethodCountDelta(symbols): ...@@ -114,8 +114,8 @@ def _CreateMethodCountDelta(symbols):
_MAX_DEX_METHOD_COUNT_INCREASE, net_method_added) _MAX_DEX_METHOD_COUNT_INCREASE, net_method_added)
def _CreateResourceSizesDelta(apk_name, before_dir, after_dir): def _CreateResourceSizesDelta(before_dir, after_dir):
sizes_diff = diagnose_bloat.ResourceSizesDiff(apk_name) sizes_diff = diagnose_bloat.ResourceSizesDiff()
sizes_diff.ProduceDiff(before_dir, after_dir) sizes_diff.ProduceDiff(before_dir, after_dir)
return sizes_diff.Summary(), _SizeDelta( return sizes_diff.Summary(), _SizeDelta(
...@@ -315,8 +315,8 @@ def main(): ...@@ -315,8 +315,8 @@ def main():
# Normalized APK Size is the main metric we use to monitor binary size. # Normalized APK Size is the main metric we use to monitor binary size.
logging.info('Creating sizes diff') logging.info('Creating sizes diff')
resource_sizes_lines, resource_sizes_delta = ( resource_sizes_lines, resource_sizes_delta = (_CreateResourceSizesDelta(
_CreateResourceSizesDelta(args.apk_name, args.before_dir, args.after_dir)) args.before_dir, args.after_dir))
size_deltas.add(resource_sizes_delta) size_deltas.add(resource_sizes_delta)
metrics.add((resource_sizes_delta, _RESOURCE_SIZES_LOG)) metrics.add((resource_sizes_delta, _RESOURCE_SIZES_LOG))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment