Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(13)

Unified Diff: tools/binary_size/explain_binary_size_delta_unittest.py

Issue 375063002: Polishing binary_size tool output. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@inline_measure_point_20140708
Patch Set: Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tools/binary_size/explain_binary_size_delta.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/binary_size/explain_binary_size_delta_unittest.py
diff --git a/tools/binary_size/explain_binary_size_delta_unittest.py b/tools/binary_size/explain_binary_size_delta_unittest.py
index 6a004e25a6cd3d1a66968c8d217055c0a57801c6..a5f696f1401f1de03ca199abe83f4ed6f1cc4654 100755
--- a/tools/binary_size/explain_binary_size_delta_unittest.py
+++ b/tools/binary_size/explain_binary_size_delta_unittest.py
@@ -136,10 +136,11 @@ class ExplainBinarySizeDeltaTest(unittest.TestCase):
result = output_collector.getvalue()
expected_output = """\
-Symbol statistics:
- 4 added, totalling 4000 bytes across 4 sources
- 4 removed, totalling 4000 bytes removed across 4 sources
- 4 changed, resulting in a net change of 4000 bytes \
+Total change: +4000 bytes
+=========================
+ 4 added, totalling +4000 bytes across 4 sources
+ 4 removed, totalling -4000 bytes across 4 sources
+ 4 grown, for a net change of +4000 bytes \
(4000 bytes before, 8000 bytes after) across 4 sources
8 unchanged, totalling 8000 bytes
Source stats:
@@ -149,50 +150,70 @@ Source stats:
8 partially changed.
1 completely unchanged.
Per-source Analysis:
- Source: /file_all_changed
- Change: 1000 bytes (gained 1000, lost 0)
- Changed symbols:
- changed type=t, delta=1000 bytes (was 1000 bytes, now 2000 bytes)
- Source: /file_all_deleted
- Change: -1000 bytes (gained 0, lost 1000)
- Removed symbols:
- removed type=t, size=1000 bytes
- Source: /file_new
- Change: 1000 bytes (gained 1000, lost 0)
- New symbols:
- added type=t, size=1000 bytes
- Source: /file_pair_unchanged_added
- Change: 1000 bytes (gained 1000, lost 0)
- New symbols:
- added type=t, size=1000 bytes
- Source: /file_pair_unchanged_changed
- Change: 1000 bytes (gained 1000, lost 0)
- Changed symbols:
- changed type=t, delta=1000 bytes (was 1000 bytes, now 2000 bytes)
- Source: /file_pair_unchanged_diffbuck_added
- Change: 1000 bytes (gained 1000, lost 0)
- New symbols:
- added type=@, size=1000 bytes
- Source: /file_pair_unchanged_diffbuck_changed
- Change: 1000 bytes (gained 1000, lost 0)
- Changed symbols:
- changed type=@, delta=1000 bytes (was 1000 bytes, now 2000 bytes)
- Source: /file_pair_unchanged_diffbuck_removed
- Change: -1000 bytes (gained 0, lost 1000)
- Removed symbols:
- removed type=@, size=1000 bytes
- Source: /file_pair_unchanged_removed
- Change: -1000 bytes (gained 0, lost 1000)
- Removed symbols:
- removed type=t, size=1000 bytes
- Source: /file_tetra
- Change: 1000 bytes (gained 2000, lost 1000)
- New symbols:
- added type=t, size=1000 bytes
- Removed symbols:
- removed type=t, size=1000 bytes
- Changed symbols:
- size_changed type=t, delta=1000 bytes (was 1000 bytes, now 2000 bytes)
+
+--------------------------------------------------
+ +1000 - Source: /file_new - (gained 1000, lost 0)
+--------------------------------------------------
+ New symbols:
+ +1000: added type=t, size=1000 bytes
+
+---------------------------------------------------------------------
+ +1000 - Source: /file_pair_unchanged_changed - (gained 1000, lost 0)
+---------------------------------------------------------------------
+ Grown symbols:
+ +1000: changed type=t, (was 1000 bytes, now 2000 bytes)
+
+----------------------------------------------------------------------------
+ +1000 - Source: /file_pair_unchanged_diffbuck_added - (gained 1000, lost 0)
+----------------------------------------------------------------------------
+ New symbols:
+ +1000: added type=@, size=1000 bytes
+
+-------------------------------------------------------------------
+ +1000 - Source: /file_pair_unchanged_added - (gained 1000, lost 0)
+-------------------------------------------------------------------
+ New symbols:
+ +1000: added type=t, size=1000 bytes
+
+------------------------------------------------------------------------------
+ +1000 - Source: /file_pair_unchanged_diffbuck_changed - (gained 1000, lost 0)
+------------------------------------------------------------------------------
+ Grown symbols:
+ +1000: changed type=@, (was 1000 bytes, now 2000 bytes)
+
+----------------------------------------------------------
+ +1000 - Source: /file_all_changed - (gained 1000, lost 0)
+----------------------------------------------------------
+ Grown symbols:
+ +1000: changed type=t, (was 1000 bytes, now 2000 bytes)
+
+-------------------------------------------------------
+ +1000 - Source: /file_tetra - (gained 2000, lost 1000)
+-------------------------------------------------------
+ New symbols:
+ +1000: added type=t, size=1000 bytes
+ Removed symbols:
+ -1000: removed type=t, size=1000 bytes
+ Grown symbols:
+ +1000: size_changed type=t, (was 1000 bytes, now 2000 bytes)
+
+------------------------------------------------------------------------------
+ -1000 - Source: /file_pair_unchanged_diffbuck_removed - (gained 0, lost 1000)
+------------------------------------------------------------------------------
+ Removed symbols:
+ -1000: removed type=@, size=1000 bytes
+
+----------------------------------------------------------
+ -1000 - Source: /file_all_deleted - (gained 0, lost 1000)
+----------------------------------------------------------
+ Removed symbols:
+ -1000: removed type=t, size=1000 bytes
+
+---------------------------------------------------------------------
+ -1000 - Source: /file_pair_unchanged_removed - (gained 0, lost 1000)
+---------------------------------------------------------------------
+ Removed symbols:
+ -1000: removed type=t, size=1000 bytes
"""
self.maxDiff = None
« no previous file with comments | « tools/binary_size/explain_binary_size_delta.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698