Cleanup of SkPath code to remove SK_OLD_EMPTY_PATH_BEHAVIOR and other

Cleanup of SkPath code to remove SK_OLD_EMPTY_PATH_BEHAVIOR and other
dead code. The patch also renames kAfterClose_SegmentState for the
SkPath::Iter class, to kEmptyContour_SegmentState which more
accurately reflects the meaning of the state. The Iter can be in that
state initially, before anything has been seen, let alone a close.

BUG=445

TEST=Covered by existing tests.
Committed: https://code.google.com/p/skia/source/detail?r=3074

[reed1, 2012-01-18 19:20:41]

Looks like remove, plus a change for kEmptyContour .... That new code is covered
by unittests too? Can you briefly describe that change as part of the CL?

[Stephen Chenney, 2012-01-18 19:26:56]

On 2012/01/18 19:20:41, reed1 wrote:
> Looks like remove, plus a change for kEmptyContour .... That new code is
covered
> by unittests too? Can you briefly describe that change as part of the CL?

I updated the CL description. Everything is extremely well tested at this point
with the existing path drawing and iteration tests. We test all sorts of paths
that would hit the old code paths, and now hit the new ones. SampleApp Fuzzer is
doing fine too.

[reed1, 2012-01-18 21:08:44]

lgtm