Refactor build/android/symbolize.py to function as an output filter.

Refactor build/android/symbolize.py to function as an output filter.

Doing so will make it easy to drop in as a test logfile text filter, similar to android_commands.NewLineNormalizer.

BUG=261311
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=212438

[scherkus (not reviewing), 2013-07-17 20:39:17]

care to review, bulach?

https://codereview.chromium.org/19675003/ is the follow up CL, but I want to
land this first as I'm not sure if where I'm adding my Symbolizer is the best
location

[bulach, 2013-07-18 09:07:02]

lgtm, small suggestion below:

https://codereview.chromium.org/19670004/diff/3001/build/android/symbolize.py
File build/android/symbolize.py (right):

https://codereview.chromium.org/19670004/diff/3001/build/android/symbolize.py...
build/android/symbolize.py:34: def write(self, data):
nit: it looks a bit weird to have "write(input_data)" :)
how about naming this method "symbolize"?

[scherkus (not reviewing), 2013-07-18 18:08:34]

thanks for the review!

https://codereview.chromium.org/19670004/diff/3001/build/android/symbolize.py
File build/android/symbolize.py (right):

https://codereview.chromium.org/19670004/diff/3001/build/android/symbolize.py...
build/android/symbolize.py:34: def write(self, data):
On 2013/07/18 09:07:03, bulach wrote:
> nit: it looks a bit weird to have "write(input_data)" :)
> how about naming this method "symbolize"?

The idea is that an instance of this class can act as an output text filter for
other stdout-ish / file-ish classes, similar to how StringIO emulates stdout-ish
/ file-ish API but saves the contents to a string.

[commit-bot: I haz the power, 2013-07-18 18:09:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/scherkus@chromium.org/19670004/3001

[commit-bot: I haz the power, 2013-07-18 21:37:46]

Change committed as 212438