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

Side by Side Diff: tools/binary_size/README.md

Issue 2834103002: diagnose_apk_bloat.py: handle more rev options. (Closed)
Patch Set: add missing --cloud to readme Created 3 years, 8 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 unified diff | Download patch
« no previous file with comments | « no previous file | tools/binary_size/diagnose_apk_bloat.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Tools for analyzing Chrome's binary size 1 # Tools for analyzing Chrome's binary size
2 2
3 # Super Size 3 # Super Size
4 4
5 Collect, archive, and analyze Chrome's binary size. 5 Collect, archive, and analyze Chrome's binary size.
6 6
7 ## "archive" 7 ## "archive"
8 8
9 Collect size information and dump it into a `.size` file. Mainly consists of 9 Collect size information and dump it into a `.size` file. Mainly consists of
10 symbol information parsed from a linker .map file. 10 symbol information parsed from a linker .map file.
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
57 ### Example Usage: 57 ### Example Usage:
58 58
59 tools/binary_size/supersize diff before.size after.size --all 59 tools/binary_size/supersize diff before.size after.size --all
60 60
61 # diagnose_apk_bloat.py 61 # diagnose_apk_bloat.py
62 62
63 Determine the cause of binary size bloat for a patch. 63 Determine the cause of binary size bloat for a patch.
64 64
65 ## Example Usage: 65 ## Example Usage:
66 66
67 # Sync, build, and diff for HEAD and HEAD^. 67 # Build and diff HEAD^ and HEAD.
68 tools/binary_size/diagnose_apk_bloat.py 68 tools/binary_size/diagnose_apk_bloat.py HEAD
69 69
70 # Diff using downloaded build artifacts. 70 # Diff OTHERREV and REV using downloaded build artifacts.
71 tools/binary_size/diagnose_apk_bloat.py --cloud --rev-with-patch REV 71 tools/binary_size/diagnose_apk_bloat.py REV --reference-rev OTHERREV --cloud
72
73 # Build and diff contiguous revs in range OTHERREV..REV for src/v8.
74 tools/binary_size/diagnose_apk_bloat.py REV --reference-rev OTHERREV --subre po v8 --all
72 75
73 # Display detailed usage info (there are many options). 76 # Display detailed usage info (there are many options).
74 tools/binary_size/diagnose_apk_bloat.py -h 77 tools/binary_size/diagnose_apk_bloat.py -h
75 78
76 # Roadmap for Super Size: 79 # Roadmap for Super Size:
77 80
78 Tracked in https://crbug.com/681694 81 Tracked in https://crbug.com/681694
79 82
80 1. More `archive` features: 83 1. More `archive` features:
81 84
(...skipping 19 matching lines...) Expand all
101 104
102 1. Integrate with `resource_sizes.py` so that it tracks size of major 105 1. Integrate with `resource_sizes.py` so that it tracks size of major
103 components separately: chrome vs blink vs skia vs v8. 106 components separately: chrome vs blink vs skia vs v8.
104 1. Speed up some steps (like normalizing names) via multiprocessing. 107 1. Speed up some steps (like normalizing names) via multiprocessing.
105 1. Add dependency graph info, perhaps just on a per-file basis. 108 1. Add dependency graph info, perhaps just on a per-file basis.
106 109
107 # Roadmap for diagnose_apk_bloat.py: 110 # Roadmap for diagnose_apk_bloat.py:
108 1. More `diagnose_apk_bloat.py` features: 111 1. More `diagnose_apk_bloat.py` features:
109 112
110 * Add more diff types (pak files, Java symbols, native symbols). 113 * Add more diff types (pak files, Java symbols, native symbols).
111 * Support performing diffs on all revs in a given range.
112 * Support local builds for revs before supersize existed. 114 * Support local builds for revs before supersize existed.
OLDNEW
« no previous file with comments | « no previous file | tools/binary_size/diagnose_apk_bloat.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698