SkRemote: add xfermodes

SkRemote: add xfermodes

Note this changes the default ID for each type from a valid non-null value to a nullptr.
All the ()++ are now ++(), so we always work with non-null IDs when we define things.

Some of this is prematurely generalized with an eye for supporting other effects and ref-counted things.

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

BUG=skia:

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

[mtklein_C, 2015-10-19 19:05:21]

Description was changed from

==========
hack in xfermodes to SkRemote

COMMIT=false

BUG=skia:
==========

to

==========
SkRemote: add xfermodes

BUG=skia:
==========

[mtklein_C, 2015-10-19 19:06:15]

Description was changed from

==========
SkRemote: add xfermodes

BUG=skia:
==========

to

==========
SkRemote: add xfermodes

Note this changes the default ID for each type from a valid non-null value to a
nullptr.

BUG=skia:
==========

[mtklein_C, 2015-10-19 19:07:28]

Description was changed from

==========
SkRemote: add xfermodes

Note this changes the default ID for each type from a valid non-null value to a
nullptr.

BUG=skia:
==========

to

==========
SkRemote: add xfermodes

Note this changes the default ID for each type from a valid non-null value to a
nullptr.   All the ()++ are now ++(), so we always work with non-null IDs when
we define things.

BUG=skia:
==========

[mtklein_C, 2015-10-19 19:20:31]

The CQ bit was checked by mtklein@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-10-19 19:20:40]

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

[mtklein, 2015-10-19 19:25:02]

Description was changed from

==========
SkRemote: add xfermodes

Note this changes the default ID for each type from a valid non-null value to a
nullptr.   All the ()++ are now ++(), so we always work with non-null IDs when
we define things.

BUG=skia:
==========

to

==========
SkRemote: add xfermodes

Note this changes the default ID for each type from a valid non-null value to a
nullptr.   All the ()++ are now ++(), so we always work with non-null IDs when
we define things.

https://gold.skia.org/search2?issue=1412223002&unt=true&query=source_type%3Dg...

BUG=skia:
==========

[mtklein, 2015-10-19 19:25:18]

Description was changed from

==========
SkRemote: add xfermodes

Note this changes the default ID for each type from a valid non-null value to a
nullptr.   All the ()++ are now ++(), so we always work with non-null IDs when
we define things.

https://gold.skia.org/search2?issue=1412223002&unt=true&query=source_type%3Dg...

BUG=skia:
==========

to

==========
SkRemote: add xfermodes

Note this changes the default ID for each type from a valid non-null value to a
nullptr.
All the ()++ are now ++(), so we always work with non-null IDs when we define
things.

https://gold.skia.org/search2?issue=1412223002&unt=true&query=source_type%3Dg...

BUG=skia:
==========

[commit-bot: I haz the power, 2015-10-19 19:34:09]

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

[commit-bot: I haz the power, 2015-10-19 19:34:10]

Dry run: This issue passed the CQ dry run.

[mtklein_C, 2015-10-19 19:57:45]

Description was changed from

==========
SkRemote: add xfermodes

Note this changes the default ID for each type from a valid non-null value to a
nullptr.
All the ()++ are now ++(), so we always work with non-null IDs when we define
things.

https://gold.skia.org/search2?issue=1412223002&unt=true&query=source_type%3Dg...

BUG=skia:
==========

to

==========
SkRemote: add xfermodes

Note this changes the default ID for each type from a valid non-null value to a
nullptr.
All the ()++ are now ++(), so we always work with non-null IDs when we define
things.

Some of this is prematurely generalized with an eye for supporting other effects
and ref-counted things.

https://gold.skia.org/search2?issue=1412223002&unt=true&query=source_type%3Dg...

BUG=skia:
==========

[mtklein_C, 2015-10-19 19:57:57]

mtklein@chromium.org changed reviewers:
+ halcanary@google.com

[hal.canary, 2015-10-20 14:47:36]

lgtm

[hal.canary, 2015-10-20 14:47:41]

The CQ bit was checked by halcanary@google.com

[commit-bot: I haz the power, 2015-10-20 14:47:58]

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

[commit-bot: I haz the power, 2015-10-20 14:49:15]

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

[commit-bot: I haz the power, 2015-10-20 14:49:16]

Try jobs failed on following builders:
  skia_presubmit-Trybot on client.skia.fyi (JOB_FAILED,
http://build.chromium.org/p/client.skia.fyi/builders/skia_presubmit-Trybot/bu...)

[mtklein_C, 2015-10-20 15:17:39]

The CQ bit was checked by mtklein@chromium.org

[mtklein_C, 2015-10-20 15:17:39]

The patchset sent to the CQ was uploaded after l-g-t-m from halcanary@google.com
Link to the patchset: https://codereview.chromium.org/1412223002/#ps80001
(title: "rebase")

[commit-bot: I haz the power, 2015-10-20 15:17:49]

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

[commit-bot: I haz the power, 2015-10-20 15:26:56]

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