Factor out VisualBench timing code into a helper class

Factor out VisualBench timing code into a helper class

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/98d2e2f095d9a5b784a32cfc2c34b30735dedcc3

[joshualitt, 2015-10-02 16:30:30]

joshualitt@google.com changed reviewers:
+ joshualitt@google.com, jvanverth@google.com, robertphillips@google.com

[joshualitt, 2015-10-02 16:30:31]

This code moves a bunch of the timing code out of VisualLightweightBenchModule. 
In a follow on I will cleanup The interactive module.

[robertphillips, 2015-10-05 13:05:01]

lgtm + mainly dox suggestions

https://codereview.chromium.org/1375363003/diff/20001/tools/VisualBench/Timin...
File tools/VisualBench/TimingStateMachine.cpp (right):

https://codereview.chromium.org/1375363003/diff/20001/tools/VisualBench/Timin...
tools/VisualBench/TimingStateMachine.cpp:5: * found in the LICENSE file.
extra line here ?

https://codereview.chromium.org/1375363003/diff/20001/tools/VisualBench/Timin...
tools/VisualBench/TimingStateMachine.cpp:13: 
allows -> allowed ?

https://codereview.chromium.org/1375363003/diff/20001/tools/VisualBench/Timin...
tools/VisualBench/TimingStateMachine.cpp:15: DEFINE_int32(frames, 5, "Number of
frames of each skp to render per sample.");
// Each test will be tuned so that it consumes at least this much time ?
milliseconds ?

https://codereview.chromium.org/1375363003/diff/20001/tools/VisualBench/Timin...
File tools/VisualBench/TimingStateMachine.h (right):

https://codereview.chromium.org/1375363003/diff/20001/tools/VisualBench/Timin...
tools/VisualBench/TimingStateMachine.h:19: * Manages a timer via a state
machine.  Can be used by modules to time benchmarks
// The normal calling pattern for a client is:
//   ...
//   ...

?

https://codereview.chromium.org/1375363003/diff/20001/tools/VisualBench/Timin...
tools/VisualBench/TimingStateMachine.h:34: /*
They -> The ?

https://codereview.chromium.org/1375363003/diff/20001/tools/VisualBench/Timin...
tools/VisualBench/TimingStateMachine.h:41: /*
kTimingFinished_ParentEvents not kNext... ?

https://codereview.chromium.org/1375363003/diff/20001/tools/VisualBench/Timin...
tools/VisualBench/TimingStateMachine.h:42: * When TimingStateMachine returns
kNextBenchmark_ParentEvents, then the owner can call
lastMeasur_e_ment ?

https://codereview.chromium.org/1375363003/diff/20001/tools/VisualBench/Visua...
File tools/VisualBench/VisualBenchmarkStream.cpp (right):

https://codereview.chromium.org/1375363003/diff/20001/tools/VisualBench/Visua...
tools/VisualBench/VisualBenchmarkStream.cpp:36: private:
MakePath ?

https://codereview.chromium.org/1375363003/diff/20001/tools/VisualBench/Visua...
tools/VisualBench/VisualBenchmarkStream.cpp:37: static void make_path(SkPath&
path) {
Maybe have a MakeBigPath helper in sk_tools_utils ?

[joshualitt, 2015-10-05 14:16:03]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2015-10-05 14:16:04]

The patchset sent to the CQ was uploaded after l-g-t-m from
robertphillips@google.com
Link to the patchset: https://codereview.chromium.org/1375363003/#ps40001
(title: "feedback inc")

[commit-bot: I haz the power, 2015-10-05 14:16:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1375363003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1375363003/40001

[commit-bot: I haz the power, 2015-10-05 14:23:35]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/98d2e2f095d9a5b784a32cfc2c34b30735dedcc3