Suggestions and merge in the other CL.

Suggestions and merge in the other CL.

I had some suggestions on the subset CL, and took the opportunity to rebase it
against head and merge in the other color type CL.

BUG=skia:

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

[scroggo, 2015-01-30 21:15:08]

scroggo@google.com changed reviewers:
+ scroggo@google.com

[scroggo, 2015-01-30 21:15:09]

The CQ bit was checked by scroggo@google.com

[scroggo, 2015-01-30 21:15:10]

lgtm

https://codereview.chromium.org/893703002/diff/1/dm/DMSrcSink.h
File dm/DMSrcSink.h (right):

https://codereview.chromium.org/893703002/diff/1/dm/DMSrcSink.h#newcode72
dm/DMSrcSink.h:72: Path fPath;
The other CL made this const. Is it deliberately not const?

[commit-bot: I haz the power, 2015-01-30 21:15:51]

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

[mtklein, 2015-01-30 21:16:41]

mtklein@google.com changed reviewers:
+ mtklein@google.com

[mtklein, 2015-01-30 21:16:42]

https://codereview.chromium.org/893703002/diff/1/dm/DMSrcSink.h
File dm/DMSrcSink.h (right):

https://codereview.chromium.org/893703002/diff/1/dm/DMSrcSink.h#newcode72
dm/DMSrcSink.h:72: Path fPath;
On 2015/01/30 21:15:09, scroggo wrote:
> The other CL made this const. Is it deliberately not const?

Oh no, not really.  It doesn't make much of a difference when all the methods
are const anyway.

[commit-bot: I haz the power, 2015-01-30 21:22:26]

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