systrace: line test and regex fix

[joelagnel]

I promised @derkling I would write this so here you go.

Signed-off-by: Joel Fernandes joelaf@google.com

[joelagnel]

@derkling @bjackman @sinkap looks good to you?

[bjackman]

Yep LGTM

[joelagnel]

Threw in the regex fix here too

[joelagnel]

Updated with few more fixes:

• Parsing issue with systrace for 'pid' column
• Fixed initial systrace test case

[derkling]

This PR remembered me we should add also some testcase to cover some example of possible android injected events.

Can you add a patch where you extend libs/trappy/tests/trace_systrace.html with an example of each possible event we currently generate from user-space and correspondingly a test case which check we are parsing all the pid, func and data fields as expected?

[joelagnel]

Added a few more test cases and fixed up some style comments.

[joelagnel]

@sinkap over to you. All changes requested are done

[joelagnel]

fixed last comment, good to go

[sinkap]

Merged.