Update SkComposeImageFilter to sk_sp

Update SkComposeImageFilter to sk_sp

TBR=reed@google.com

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

Committed: https://skia.googlesource.com/skia/+/491fb17cab07574a9d5656c1c97ad0acc3af5c70

[robertphillips, 2016-03-29 21:11:41]

Description was changed from

==========
Update SkComposeImageFilter to sk_sp
==========

to

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

[robertphillips, 2016-03-29 21:20:51]

Description was changed from

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

to

==========
Update SkComposeImageFilter 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-03-29 21:20:51]

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

[robertphillips, 2016-03-29 21:54:22]

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

[commit-bot: I haz the power, 2016-03-29 21:54:29]

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

[commit-bot: I haz the power, 2016-03-29 21:56:59]

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

[commit-bot: I haz the power, 2016-03-29 21:57:00]

Dry run: Try jobs failed on following builders:
  Build-Mac-Clang-x86_64-Release-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Mac-Clang-x86_...)

[robertphillips, 2016-03-30 14:44:32]

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

[commit-bot: I haz the power, 2016-03-30 14:44:43]

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

[commit-bot: I haz the power, 2016-03-30 15:02:19]

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

[commit-bot: I haz the power, 2016-03-30 15:02:20]

Dry run: This issue passed the CQ dry run.

[f(malita), 2016-03-30 18:04:38]

lgtm

https://codereview.chromium.org/1841233002/diff/60001/include/effects/SkCompo...
File include/effects/SkComposeImageFilter.h (right):

https://codereview.chromium.org/1841233002/diff/60001/include/effects/SkCompo...
include/effects/SkComposeImageFilter.h:22: sk_sp<SkImageFilter> inputs[2] = {
outer, inner };
nit: I think we can std::move outer/inner here.

[robertphillips, 2016-03-30 18:23:27]

https://codereview.chromium.org/1841233002/diff/60001/include/effects/SkCompo...
File include/effects/SkComposeImageFilter.h (right):

https://codereview.chromium.org/1841233002/diff/60001/include/effects/SkCompo...
include/effects/SkComposeImageFilter.h:22: sk_sp<SkImageFilter> inputs[2] = {
outer, inner };
On 2016/03/30 18:04:38, f(malita) wrote:
> nit: I think we can std::move outer/inner here.

Done.

[robertphillips, 2016-03-30 18:23:38]

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

[commit-bot: I haz the power, 2016-03-30 18:23:49]

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

[commit-bot: I haz the power, 2016-03-30 18:39:44]

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

[commit-bot: I haz the power, 2016-03-30 18:39:45]

Dry run: This issue passed the CQ dry run.

[robertphillips, 2016-03-30 19:31:54]

The CQ bit was checked by robertphillips@google.com

[robertphillips, 2016-03-30 19:31:55]

The patchset sent to the CQ was uploaded after l-g-t-m from fmalita@chromium.org
Link to the patchset: https://codereview.chromium.org/1841233002/#ps80001
(title: "Address code review comments")

[commit-bot: I haz the power, 2016-03-30 19:32:01]

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

[commit-bot: I haz the power, 2016-03-30 19:33:00]

Description was changed from

==========
Update SkComposeImageFilter 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 SkComposeImageFilter 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/+/491fb17cab07574a9d5656c1c97ad0acc3af5c70
==========

[commit-bot: I haz the power, 2016-03-30 19:33:01]

Committed patchset #5 (id:80001) as
https://skia.googlesource.com/skia/+/491fb17cab07574a9d5656c1c97ad0acc3af5c70