SkRemote: refactoring

SkRemote: refactoring

  - Cache becomes CachingEncoder that wraps another Encoder
  - Encoders provide IDs
  - syntaxy improvements to Client
  - ID isn't really protocol sensitive.
  - I don't think we need Type::kNone.

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

BUG=skia:

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

[mtklein_C, 2015-10-21 16:41:05]

Description was changed from

==========
SkRemote: refactoring

  - Cache becomes CachingEncoder that wraps another Encoder
  - Encoders provide IDs
  - syntaxy improvements to Client

No diffs.

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

to

==========
SkRemote: refactoring

  - Cache becomes CachingEncoder that wraps another Encoder
  - Encoders provide IDs
  - syntaxy improvements to Client
  - ID isn't really protocol sensitive.

No diffs.

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

[mtklein_C, 2015-10-21 16:46:30]

Description was changed from

==========
SkRemote: refactoring

  - Cache becomes CachingEncoder that wraps another Encoder
  - Encoders provide IDs
  - syntaxy improvements to Client
  - ID isn't really protocol sensitive.

No diffs.

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

to

==========
SkRemote: refactoring

  - Cache becomes CachingEncoder that wraps another Encoder
  - Encoders provide IDs
  - syntaxy improvements to Client
  - ID isn't really protocol sensitive.
  - I don't think we need Type::kNone.

No diffs.

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

[mtklein_C, 2015-10-21 16:46:36]

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

[commit-bot: I haz the power, 2015-10-21 16:46:45]

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

[commit-bot: I haz the power, 2015-10-21 16:56:14]

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

[commit-bot: I haz the power, 2015-10-21 16:56:15]

Dry run: This issue passed the CQ dry run.

[mtklein_C, 2015-10-21 16:59:57]

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

[mtklein_C, 2015-10-21 17:00:24]

Description was changed from

==========
SkRemote: refactoring

  - Cache becomes CachingEncoder that wraps another Encoder
  - Encoders provide IDs
  - syntaxy improvements to Client
  - ID isn't really protocol sensitive.
  - I don't think we need Type::kNone.

No diffs.

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

to

==========
SkRemote: refactoring

  - Cache becomes CachingEncoder that wraps another Encoder
  - Encoders provide IDs
  - syntaxy improvements to Client
  - ID isn't really protocol sensitive.
  - I don't think we need Type::kNone.

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

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

[hal.canary, 2015-10-21 17:45:05]

The CQ bit was checked by halcanary@google.com

[hal.canary, 2015-10-21 17:45:05]

lgtm

[commit-bot: I haz the power, 2015-10-21 17:45:09]

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

[commit-bot: I haz the power, 2015-10-21 17:46:04]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/b6b8f2727f6b08bbc307aeebac23beeba1e4984f