Make GrCircleBlurFragmentProcessor::onIsEqual include the position

Make GrCircleBlurFragmentProcessor::onIsEqual include the position

Previously we would fail to draw two adjacent blurred circles if they differed
only by their position.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1824693003

Committed: https://skia.googlesource.com/skia/+/ddf0713f0ba4ea75ca49a4ed6b4249eef72da1ce

[abarth-chromium, 2016-03-21 22:19:33]

Description was changed from

==========
Make GrCircleBlurFragmentProcessor::onIsEqual include the position

Previously we would fail to draw two adjacent blurred circles if they differed
only by their position.
==========

to

==========
Make GrCircleBlurFragmentProcessor::onIsEqual include the position

Previously we would fail to draw two adjacent blurred circles if they differed
only by their position.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[abarth, 2016-03-21 22:21:12]

abarth@google.com changed reviewers:
+ abarth@google.com, robertphillips@google.com

[abarth, 2016-03-21 22:21:12]

Hey, circles had the same bug as RRects.  (See
https://bugs.chromium.org/p/skia/issues/detail?id=5014 and
https://codereview.chromium.org/1743553003.)

Here's some Flutter code that reproduces the issue:

MaskFilter filter = new MaskFilter.blur(BlurStyle.normal, 1.0);
canvas.drawCircle(new Point(24.0, 50.0), 8.0, new Paint()..color = new
Color(0xFFFF0000)..maskFilter = filter);
canvas.drawCircle(new Point(87.0, 50.0), 8.0, new Paint()..color = new
Color(0xFFFF0000)..maskFilter = filter);

[robertphillips, 2016-03-22 13:05:58]

lgtm

[robertphillips, 2016-03-22 13:06:06]

The CQ bit was checked by robertphillips@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-03-22 13:06:11]

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

[commit-bot: I haz the power, 2016-03-22 14:16:06]

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

[commit-bot: I haz the power, 2016-03-22 14:16:07]

Dry run: This issue passed the CQ dry run.

[abarth, 2016-03-22 16:13:35]

The CQ bit was checked by abarth@google.com

[commit-bot: I haz the power, 2016-03-22 16:13:46]

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

[commit-bot: I haz the power, 2016-03-22 16:14:43]

Description was changed from

==========
Make GrCircleBlurFragmentProcessor::onIsEqual include the position

Previously we would fail to draw two adjacent blurred circles if they differed
only by their position.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Make GrCircleBlurFragmentProcessor::onIsEqual include the position

Previously we would fail to draw two adjacent blurred circles if they differed
only by their position.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/ddf0713f0ba4ea75ca49a4ed6b4249eef72da1ce
==========

[commit-bot: I haz the power, 2016-03-22 16:14:43]

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