supersize: Add "diff" command

supersize: Add "diff" command

And marginally improved LazyPath output_directory logic to look for
build.ninja rather than "Release" in the name.

BUG=681694
Review-Url: https://codereview.chromium.org/2817813003
Cr-Commit-Position: refs/heads/master@{#464462}
Committed: https://chromium.googlesource.com/chromium/src/+/98d419b9b3e36419388ff6ce042da6e123fc841f

[agrieve, 2017-04-13 14:03:51]

Description was changed from

==========
supersize: Add "diff" command

And marginally improved LazyPath output_directory logic to look for
build.ninja rather than "Release" in the name.

BUG=681694
==========

to

==========
supersize: Add "diff" command

And marginally improved LazyPath output_directory logic to look for
build.ninja rather than "Release" in the name.

BUG=681694
==========

[agrieve, 2017-04-13 14:03:52]

agrieve@chromium.org changed reviewers:
+ estevenson@chromium.org

[agrieve, 2017-04-13 14:04:04]

On 2017/04/13 14:03:52, agrieve wrote:
> mailto:agrieve@chromium.org changed reviewers:
> + mailto:estevenson@chromium.org

	🚢

[estevenson, 2017-04-13 16:54:25]

lgtm

https://codereview.chromium.org/2817813003/diff/40001/tools/binary_size/READM...
File tools/binary_size/README.md (right):

https://codereview.chromium.org/2817813003/diff/40001/tools/binary_size/READM...
tools/binary_size/README.md:48: tools/binary_size/supersize console chrome.size
--query='Print(size_info)'
nit: exclude "diff" from comment.

[agrieve, 2017-04-13 17:40:50]

https://codereview.chromium.org/2817813003/diff/40001/tools/binary_size/READM...
File tools/binary_size/README.md (right):

https://codereview.chromium.org/2817813003/diff/40001/tools/binary_size/READM...
tools/binary_size/README.md:48: tools/binary_size/supersize console chrome.size
--query='Print(size_info)'
On 2017/04/13 16:54:25, estevenson wrote:
> nit: exclude "diff" from comment.

Done.

[agrieve, 2017-04-13 17:41:43]

The CQ bit was checked by agrieve@chromium.org

[agrieve, 2017-04-13 17:41:43]

The patchset sent to the CQ was uploaded after l-g-t-m from
estevenson@chromium.org
Link to the patchset: https://codereview.chromium.org/2817813003/#ps60001
(title: "review commetns")

[commit-bot: I haz the power, 2017-04-13 17:42:29]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-13 17:54:33]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1492105303364700,
"parent_rev": "b797bf2e48843177bc5657622bb7fd1f09bb9e24", "commit_rev":
"98d419b9b3e36419388ff6ce042da6e123fc841f"}

[commit-bot: I haz the power, 2017-04-13 17:55:28]

Description was changed from

==========
supersize: Add "diff" command

And marginally improved LazyPath output_directory logic to look for
build.ninja rather than "Release" in the name.

BUG=681694
==========

to

==========
supersize: Add "diff" command

And marginally improved LazyPath output_directory logic to look for
build.ninja rather than "Release" in the name.

BUG=681694

Review-Url: https://codereview.chromium.org/2817813003
Cr-Commit-Position: refs/heads/master@{#464462}
Committed:
https://chromium.googlesource.com/chromium/src/+/98d419b9b3e36419388ff6ce042d...
==========

[commit-bot: I haz the power, 2017-04-13 17:55:29]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/98d419b9b3e36419388ff6ce042d...