fuzzer fixes

fuzzer fixes

Fix path bugs exposed by the path fuzzer.

Changes to existing gm and samplecode files defer their calls to construct
SkPath objects until the first draw instead of at test initialization.

Add an experimental call to SkPath to validate the internal SkPathRef.

Fix SkPath::addPoly to set the last moveto after adding a close verb.

Fix stroke to handle failures when computing the unit normal.

Add a unit test for the unit normal failure.

R=reed@google.com

Committed: https://skia.googlesource.com/skia/+/63c684a8a609d39da11b4a656223cebf52ca85dc

[caryclark, 2015-02-25 15:46:40]

The CQ bit was checked by caryclark@google.com

[commit-bot: I haz the power, 2015-02-25 15:46:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/953383002/1

[commit-bot: I haz the power, 2015-02-25 15:46:54]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from valid reviewer(s) till 2015-02-25 21:46 UTC

[commit-bot: I haz the power, 2015-02-25 15:47:01]

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

[commit-bot: I haz the power, 2015-02-25 15:47:01]

Presubmit check for 953383002-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Since the CL is editing public API, you must have an LGTM from one of:
('reed@chromium.org', 'reed@google.com', 'bsalomon@chromium.org',
'bsalomon@google.com', 'djsollen@chromium.org', 'djsollen@google.com')

[caryclark, 2015-02-25 16:07:31]

The CQ bit was checked by caryclark@google.com

[commit-bot: I haz the power, 2015-02-25 16:08:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/953383002/20001

[reed1, 2015-02-25 16:08:16]

https://codereview.chromium.org/953383002/diff/20001/gm/bitmapshader.cpp
File gm/bitmapshader.cpp (right):

https://codereview.chromium.org/953383002/diff/20001/gm/bitmapshader.cpp#newc...
gm/bitmapshader.cpp:46: virtual void onOnceBeforeDraw() {
new style for overrides:

void onOnceBeforeDraw() SK_OVERRIDE {

[commit-bot: I haz the power, 2015-02-25 16:13:33]

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

[commit-bot: I haz the power, 2015-02-25 16:13:35]

Try jobs failed on following builders:
  Test-Ubuntu13.10-GCE-NoGPU-x86_64-Release-Shared-Trybot on client.skia
(JOB_FAILED,
http://build.chromium.org/p/client.skia/builders/Test-Ubuntu13.10-GCE-NoGPU-x...)

[caryclark, 2015-02-25 16:26:01]

The CQ bit was checked by caryclark@google.com

[commit-bot: I haz the power, 2015-02-25 16:26:33]

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

[caryclark, 2015-02-25 16:30:59]

The CQ bit was unchecked by caryclark@google.com

[caryclark, 2015-02-25 16:54:13]

The CQ bit was checked by caryclark@google.com

[commit-bot: I haz the power, 2015-02-25 16:54:44]

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

[commit-bot: I haz the power, 2015-02-25 17:04:05]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from valid reviewer(s) till 2015-02-25 22:54 UTC

[commit-bot: I haz the power, 2015-02-25 17:04:10]

Committed patchset #4 (id:60001) as
https://skia.googlesource.com/skia/+/63c684a8a609d39da11b4a656223cebf52ca85dc