Android stack symbolizer: clamp padding on symbolized names at 80 cols

Change-Id: Ic77300a39d6f7cd6e58ada65393a7b2cd131cd47
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1603347
Auto-Submit: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#658392}

third_party/android_platform/README.chromium

@@ -47,4 +47,5 @@ Speedup symbolization by avoiding unnecessary APK manifest extraction loops.
 Changed the symbolizer to act in a passthrough mode when reading from stdin, so
     adb logcat can be piped through it to symbolize on the fly.
 
-Made the symbolizer agnostic to the width of pointers.
\ No newline at end of file
+Made the symbolizer agnostic to the width of pointers.
+Clamping the padding on symbolized names to 80 columns.

third_party/android_platform/development/scripts/stack_core.py

@@ -104,7 +104,7 @@ _SHARED_LIB_OFFSET_IN_APK = re.compile(' \(offset 0x(?P<offset>[0-9a-f]{0,16})\)
 
 def PrintTraceLines(trace_lines):
   """Print back trace."""
-  maxlen = max(map(lambda tl: len(tl[1]), trace_lines))
+  maxlen = min(80, max(map(lambda tl: len(tl[1]), trace_lines)))
   print
   print 'Stack Trace:'
   print '  RELADDR   ' + 'FUNCTION'.ljust(maxlen) + '  FILE:LINE'
@@ -117,7 +117,7 @@ def PrintTraceLines(trace_lines):
 
 def PrintValueLines(value_lines):
   """Print stack data values."""
-  maxlen = max(map(lambda tl: len(tl[2]), value_lines))
+  maxlen = min(80, max(map(lambda tl: len(tl[2]), value_lines)))
   print
   print 'Stack Data:'
   print '  ADDR      VALUE     ' + 'FUNCTION'.ljust(maxlen) + '  FILE:LINE'