SkPDF/SkColorSpace: use writeToICC

SkPDF/SkColorSpace: use writeToICC

.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2012133003

Committed: https://skia.googlesource.com/skia/+/99e22fbe569ef3525d4de07eceaca2200a3d6e50

[hal.canary, 2016-05-26 15:02:24]

Description was changed from

==========
SkPDF/SkColorSpace: use writeToICC

.
==========

to

==========
SkPDF/SkColorSpace: use writeToICC

.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2012133003
==========

[hal.canary, 2016-05-26 15:03:53]

halcanary@google.com changed reviewers:
+ msarett@google.com

[hal.canary, 2016-05-26 15:03:54]

PTAL.  Is there a way to avoid the cast?  Is the data valid?

[hal.canary, 2016-05-26 15:06:56]

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

[commit-bot: I haz the power, 2016-05-26 15:07:08]

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

[msarett, 2016-05-26 15:11:53]

You can use as_CSB() instead of the cast.

lgtm

https://codereview.chromium.org/2012133003/diff/1/src/pdf/SkPDFDocument.cpp
File src/pdf/SkPDFDocument.cpp (right):

https://codereview.chromium.org/2012133003/diff/1/src/pdf/SkPDFDocument.cpp#n...
src/pdf/SkPDFDocument.cpp:322: return
((SkColorSpace_Base*)srgb.get())->writeToICC();
You can use as_CSB() instead of the case.

[hal.canary, 2016-05-26 15:17:25]

The CQ bit was checked by halcanary@google.com

[hal.canary, 2016-05-26 15:17:25]

The patchset sent to the CQ was uploaded after l-g-t-m from msarett@google.com
Link to the patchset: https://codereview.chromium.org/2012133003/#ps20001
(title: "2016-05-26 (Thursday) 11:17:15 EDT")

[commit-bot: I haz the power, 2016-05-26 15:17:27]

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

[commit-bot: I haz the power, 2016-05-26 15:31:10]

Description was changed from

==========
SkPDF/SkColorSpace: use writeToICC

.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2012133003
==========

to

==========
SkPDF/SkColorSpace: use writeToICC

.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2012133003

Committed:
https://skia.googlesource.com/skia/+/99e22fbe569ef3525d4de07eceaca2200a3d6e50
==========

[commit-bot: I haz the power, 2016-05-26 15:31:11]

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