Factor out DumpRecord method from dump_record tool for later use

Factor out DumpRecord method from dump_record tool for later use

Committed: http://code.google.com/p/skia/source/detail?r=14751

[hal.canary, 2014-05-15 15:07:15]

ptal

[mtklein, 2014-05-15 15:44:08]

https://codereview.chromium.org/282233003/diff/1/tools/DumpRecord.cpp
File tools/DumpRecord.cpp (right):

https://codereview.chromium.org/282233003/diff/1/tools/DumpRecord.cpp#newcode11
tools/DumpRecord.cpp:11: #include "LazyDecodeBitmap.h"
Should be you can prune a bunch of these includes here and in dump_record?

https://codereview.chromium.org/282233003/diff/1/tools/DumpRecord.cpp#newcode112
tools/DumpRecord.cpp:112:
////////////////////////////////////////////////////////////////////////////////
These things are just noise :(

https://codereview.chromium.org/282233003/diff/1/tools/DumpRecord.h
File tools/DumpRecord.h (right):

https://codereview.chromium.org/282233003/diff/1/tools/DumpRecord.h#newcode15
tools/DumpRecord.h:15: printing out each draw command and timing information. */
... printing each draw command and runtime in us to stdout?

https://codereview.chromium.org/282233003/diff/1/tools/DumpRecord.h#newcode18
tools/DumpRecord.h:18: bool timeWithCommand);
Explain timeWithCommand?

[hal.canary, 2014-05-15 15:59:27]

https://codereview.chromium.org/282233003/diff/1/tools/DumpRecord.cpp
File tools/DumpRecord.cpp (right):

https://codereview.chromium.org/282233003/diff/1/tools/DumpRecord.cpp#newcode11
tools/DumpRecord.cpp:11: #include "LazyDecodeBitmap.h"
On 2014/05/15 15:44:08, mtklein wrote:
> Should be you can prune a bunch of these includes here and in dump_record?

Done.

https://codereview.chromium.org/282233003/diff/1/tools/DumpRecord.cpp#newcode112
tools/DumpRecord.cpp:112:
////////////////////////////////////////////////////////////////////////////////
On 2014/05/15 15:44:08, mtklein wrote:
> These things are just noise :(

Done.

https://codereview.chromium.org/282233003/diff/1/tools/DumpRecord.h
File tools/DumpRecord.h (right):

https://codereview.chromium.org/282233003/diff/1/tools/DumpRecord.h#newcode15
tools/DumpRecord.h:15: printing out each draw command and timing information. */
On 2014/05/15 15:44:08, mtklein wrote:
> ... printing each draw command and runtime in us to stdout?

Done.

https://codereview.chromium.org/282233003/diff/1/tools/DumpRecord.h#newcode18
tools/DumpRecord.h:18: bool timeWithCommand);
On 2014/05/15 15:44:08, mtklein wrote:
> Explain timeWithCommand?

Done.

[mtklein, 2014-05-15 16:04:04]

lgtm

[commit-bot: I haz the power, 2014-05-15 16:04:29]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/halcanary@google.com/282233003/20001

[commit-bot: I haz the power, 2014-05-15 16:10:54]

Change committed as 14751