remove duplicate linked list adjustment

remove duplicate linked list adjustment

The list of intersection points on a curve segment may have
entries that can be safely removed when nearby points have
nearly the same t value and point value. When a path includes
very large curves as well as small ones, as is the case with
this fuzzer, additional points may lie between the similar
points that do not meet the nearby criteria.

After merging the nearby point with its doppelganger,
SkOpSegment::moveNearby() unnecessarily set the doppelganger's
next pointer to the one following the nearby point. While
this usually has no effect, since the merge already updated
the linked list, the explicit call removes the additional
outlier points from the segment.

TBR=reed@google.com
BUG=526025
Committed: https://skia.googlesource.com/skia/+/ae576b733dc427761bd254dcce5ae5a1c5d552df

[caryclark, 2015-08-31 16:40:59]

The CQ bit was checked by caryclark@google.com

[commit-bot: I haz the power, 2015-08-31 16:41:12]

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

[commit-bot: I haz the power, 2015-08-31 16:46:23]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/ae576b733dc427761bd254dcce5ae5a1c5d552df

[reed1, 2015-08-31 21:11:01]

https://codereview.chromium.org/1323813003/diff/1/tests/PathOpsBuilderTest.cpp
File tests/PathOpsBuilderTest.cpp (right):

https://codereview.chromium.org/1323813003/diff/1/tests/PathOpsBuilderTest.cp...
tests/PathOpsBuilderTest.cpp:276: DEF_TEST(fuzzTNG, reporter) {
can we tie this test to a bug or even CL? Your description in the CL was very
detalied. Perhaps some of that should be referenced from here, to justify/motive
this test.

[caryclark, 2015-09-01 11:33:01]

On 2015/08/31 21:11:01, reed1 wrote:
> https://codereview.chromium.org/1323813003/diff/1/tests/PathOpsBuilderTest.cpp
> File tests/PathOpsBuilderTest.cpp (right):
> 
>
https://codereview.chromium.org/1323813003/diff/1/tests/PathOpsBuilderTest.cp...
> tests/PathOpsBuilderTest.cpp:276: DEF_TEST(fuzzTNG, reporter) {
> can we tie this test to a bug or even CL? Your description in the CL was very
> detalied. Perhaps some of that should be referenced from here, to
justify/motive
> this test.

I'll create a separate CL to address this.