Add ship sample

Add ship sample

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/2f5bb3abfacadc8bdbc95381ab9d4fa218994a9f

[jvanverth1, 2015-10-02 19:04:49]

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

[robertphillips, 2015-10-02 21:29:18]

lgtm + suggestions

https://codereview.chromium.org/1359033005/diff/40001/samplecode/SampleShip.cpp
File samplecode/SampleShip.cpp (right):

https://codereview.chromium.org/1359033005/diff/40001/samplecode/SampleShip.c...
samplecode/SampleShip.cpp:12: #include "SkCanvas.h"
Do we need SkDrawable.h or SkPath.h ?

https://codereview.chromium.org/1359033005/diff/40001/samplecode/SampleShip.c...
samplecode/SampleShip.cpp:44: for (int y = 0; y < GRID; y++) {
Where is 960 coming from? Can it be a informatively named constant ?

https://codereview.chromium.org/1359033005/diff/40001/samplecode/SampleShip.c...
samplecode/SampleShip.cpp:49: fXform[currIndex] =
SkRSXform::MakeFromRadians(2.0f, SK_ScalarPI*0.5f,
add a space ?

https://codereview.chromium.org/1359033005/diff/40001/samplecode/SampleShip.c...
samplecode/SampleShip.cpp:94: meanTime /= 32.f;
Use SkString here ?

[jvanverth1, 2015-10-05 17:04:49]

https://codereview.chromium.org/1359033005/diff/40001/samplecode/SampleShip.cpp
File samplecode/SampleShip.cpp (right):

https://codereview.chromium.org/1359033005/diff/40001/samplecode/SampleShip.c...
samplecode/SampleShip.cpp:12: #include "SkCanvas.h"
On 2015/10/02 21:29:18, robertphillips wrote:
> Do we need SkDrawable.h or SkPath.h ?

Done.

https://codereview.chromium.org/1359033005/diff/40001/samplecode/SampleShip.c...
samplecode/SampleShip.cpp:44: for (int y = 0; y < GRID; y++) {
On 2015/10/02 21:29:18, robertphillips wrote:
> Where is 960 coming from? Can it be a informatively named constant ?

Done.

https://codereview.chromium.org/1359033005/diff/40001/samplecode/SampleShip.c...
samplecode/SampleShip.cpp:49: fXform[currIndex] =
SkRSXform::MakeFromRadians(2.0f, SK_ScalarPI*0.5f,
On 2015/10/02 21:29:18, robertphillips wrote:
> add a space ?

Done.

https://codereview.chromium.org/1359033005/diff/40001/samplecode/SampleShip.c...
samplecode/SampleShip.cpp:94: meanTime /= 32.f;
On 2015/10/02 21:29:18, robertphillips wrote:
> Use SkString here ?

Done.

[jvanverth1, 2015-10-05 17:04:52]

The CQ bit was checked by jvanverth@google.com

[jvanverth1, 2015-10-05 17:04:52]

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/1359033005/#ps60001
(title: "Update for comments")

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

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

[commit-bot: I haz the power, 2015-10-05 17:06:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-10-05 17:06:56]

Try jobs failed on following builders:
  Build-Win-MSVC-x86_64-Debug-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Win-MSVC-x86_6...)

[jvanverth1, 2015-10-05 17:58:35]

The CQ bit was checked by jvanverth@google.com

[jvanverth1, 2015-10-05 17:58:35]

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/1359033005/#ps80001
(title: "Address Win compile warnings")

[commit-bot: I haz the power, 2015-10-05 17:59:00]

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

[commit-bot: I haz the power, 2015-10-05 18:05:10]

Committed patchset #5 (id:80001) as
https://skia.googlesource.com/skia/+/2f5bb3abfacadc8bdbc95381ab9d4fa218994a9f