Update MagnifierImageFilter to sk_sp

Update MagnifierImageFilter to sk_sp

TBR=reed@google.com

GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1864843002

Committed: https://skia.googlesource.com/skia/+/11171f3b66ba6aba561fcaafbc51eaa354b73353

[robertphillips, 2016-04-05 23:28:19]

Description was changed from

==========
Update MagnifierImageFilter to sk_sp
==========

to

==========
Update MagnifierImageFilter to sk_sp
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[robertphillips, 2016-04-05 23:29:48]

Description was changed from

==========
Update MagnifierImageFilter to sk_sp
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Update MagnifierImageFilter to sk_sp

TBR=reed@google.com

GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[robertphillips, 2016-04-05 23:29:48]

robertphillips@google.com changed reviewers:
+ fmalita@chromium.org, reed@google.com

[robertphillips, 2016-04-05 23:29:55]

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

[commit-bot: I haz the power, 2016-04-05 23:29:56]

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

[commit-bot: I haz the power, 2016-04-05 23:42:55]

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

[commit-bot: I haz the power, 2016-04-05 23:42:56]

Dry run: This issue passed the CQ dry run.

[robertphillips, 2016-04-06 18:22:21]

ping

[f(malita), 2016-04-07 13:33:42]

lgtm

https://codereview.chromium.org/1864843002/diff/1/fuzz/FilterFuzz.cpp
File fuzz/FilterFuzz.cpp (right):

https://codereview.chromium.org/1864843002/diff/1/fuzz/FilterFuzz.cpp#newcode584
fuzz/FilterFuzz.cpp:584: make_image_filter());
Third arg should be nullptr for equivalence with existing code, but I presume
this is intended as a fix.

[robertphillips, 2016-04-07 14:02:05]

https://codereview.chromium.org/1864843002/diff/1/fuzz/FilterFuzz.cpp
File fuzz/FilterFuzz.cpp (right):

https://codereview.chromium.org/1864843002/diff/1/fuzz/FilterFuzz.cpp#newcode584
fuzz/FilterFuzz.cpp:584: make_image_filter());
Right - I believe the prior behavior was incorrect.

[robertphillips, 2016-04-07 14:20:06]

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

[commit-bot: I haz the power, 2016-04-07 14:20:17]

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

[commit-bot: I haz the power, 2016-04-07 14:30:48]

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

[commit-bot: I haz the power, 2016-04-07 14:30:49]

Dry run: This issue passed the CQ dry run.

[robertphillips, 2016-04-07 14:33:11]

The CQ bit was checked by robertphillips@google.com

[commit-bot: I haz the power, 2016-04-07 14:33:20]

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

[commit-bot: I haz the power, 2016-04-07 14:34:19]

Description was changed from

==========
Update MagnifierImageFilter to sk_sp

TBR=reed@google.com

GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Update MagnifierImageFilter to sk_sp

TBR=reed@google.com

GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/11171f3b66ba6aba561fcaafbc51eaa354b73353
==========

[commit-bot: I haz the power, 2016-04-07 14:34:20]

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