Add a focused public API for src/record.

Add a focused public API for src/record.

BUG=skia:2378
Committed: http://code.google.com/p/skia/source/detail?r=14146

[mtklein, 2014-04-10 20:12:10]

On 2014/04/10 19:56:43, mtklein wrote:

FYI, just make the SkPlayback constructor private instead of taunting you to
find a way to break the public constructor.

[reed1, 2014-04-10 20:12:27]

Lets add some form of /** EXPERIMENTAL */ to the header and/or class names for
now. Its a good reminder to us and them.

e.g. we have that already in SkPicture.h ...     void
internalOnly_EnableOpts(bool enableOpts);

otherwise, lgtm

[mtklein, 2014-04-10 20:19:20]

On 2014/04/10 20:12:27, reed1 wrote:
> Lets add some form of /** EXPERIMENTAL */ to the header and/or class names for
> now. Its a good reminder to us and them.
> 
> e.g. we have that already in SkPicture.h ...     void
> internalOnly_EnableOpts(bool enableOpts);
> 
> otherwise, lgtm

SGTM.  Done.

[f(malita), 2014-04-10 20:30:38]

lgtm

[mtklein, 2014-04-10 22:06:56]

The CQ bit was checked by mtklein@google.com

[commit-bot: I haz the power, 2014-04-10 22:06:59]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/mtklein@chromium.org/233053005/40001

[commit-bot: I haz the power, 2014-04-10 23:31:09]

Change committed as 14146

[robertphillips, 2014-04-11 13:28:07]

Just a nit about missing headers.

https://codereview.chromium.org/233053005/diff/40001/include/record/SkRecordi...
File include/record/SkRecording.h (right):

https://codereview.chromium.org/233053005/diff/40001/include/record/SkRecordi...
include/record/SkRecording.h:1: #ifndef SkRecording_DEFINED
Header?

https://codereview.chromium.org/233053005/diff/40001/src/record/SkRecording.cpp
File src/record/SkRecording.cpp (right):

https://codereview.chromium.org/233053005/diff/40001/src/record/SkRecording.c...
src/record/SkRecording.cpp:1: #include "SkRecording.h"
Header?

https://codereview.chromium.org/233053005/diff/40001/tests/RecordingTest.cpp
File tests/RecordingTest.cpp (right):

https://codereview.chromium.org/233053005/diff/40001/tests/RecordingTest.cpp#...
tests/RecordingTest.cpp:1: #include "Test.h"
Header?