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

Unified 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | tools/binary_size/diagnose_apk_bloat.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/binary_size/README.md
diff --git a/tools/binary_size/README.md b/tools/binary_size/README.md
index 39252578f00aa8715c1daf0dcb907e1d55c2f927..598e53d206765c1b9dbff3ad10206676f2046edb 100644
--- a/tools/binary_size/README.md
+++ b/tools/binary_size/README.md
@@ -64,11 +64,14 @@ Determine the cause of binary size bloat for a patch.
## Example Usage:
- # Sync, build, and diff for HEAD and HEAD^.
- tools/binary_size/diagnose_apk_bloat.py
+ # Build and diff HEAD^ and HEAD.
+ tools/binary_size/diagnose_apk_bloat.py HEAD
- # Diff using downloaded build artifacts.
- tools/binary_size/diagnose_apk_bloat.py --cloud --rev-with-patch REV
+ # Diff OTHERREV and REV using downloaded build artifacts.
+ tools/binary_size/diagnose_apk_bloat.py REV --reference-rev OTHERREV --cloud
+
+ # Build and diff contiguous revs in range OTHERREV..REV for src/v8.
+ tools/binary_size/diagnose_apk_bloat.py REV --reference-rev OTHERREV --subrepo v8 --all
# Display detailed usage info (there are many options).
tools/binary_size/diagnose_apk_bloat.py -h
@@ -108,5 +111,4 @@ Tracked in https://crbug.com/681694
1. More `diagnose_apk_bloat.py` features:
* Add more diff types (pak files, Java symbols, native symbols).
- * Support performing diffs on all revs in a given range.
* Support local builds for revs before supersize existed.
« 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