[Super Size] Add basic CSV output support

Desktop / Chromium - Samuel Huang [chromium.org] - 14 September 2017 11:24 EDT

This CL adds basic CSV output support to Super Size, using new command Csv() that's used like Print().

Limitations: Does not support verbose=True or recursive=True.

More work is needed, and will be addressed in follow-ups:
- Refactor Describer._DescribeSymbol() to make it more flexible.- Render Csv() for Describer._DescribeSymbol(), taking account of various use cases.

Bug: Change-Id: I821c8e1a0d46d8c947901f382d688db8f3d97373 Reviewed-on: https://chromium-review.googlesource.com/641375 Commit-Queue: Samuel Huang

050184f [Super Size] Add basic CSV output support.
tools/binary_size/libsupersize/console.py | 45 ++++-
tools/binary_size/libsupersize/describe.py | 224 +++++++++++++++++----
tools/binary_size/libsupersize/integration_test.py | 13 ++
.../libsupersize/testdata/Console.golden | 5 +-
tools/binary_size/libsupersize/testdata/Csv.golden | 57 ++++++
5 files changed, 292 insertions(+), 52 deletions(-)

Upstream: git.chromium.org


  • Share