Handle getMemoryBase returning NULL in WriteTask.

Handle getMemoryBase returning NULL in WriteTask.

DM::WriteTask::fData was changed from a data to a stream in
"SkData to SkStreamAsset to avoid unneeded copying"
https://skia.googlesource.com/skia/+/a4c6094177ebde18c706cbcfbd2013d0a088e0ee
However, DM::WriteTask::draw was not updated to handle this,
resulting in segfaults when trying to write out PDF files.

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

[bungeman-skia, 2014-10-02 19:48:58]

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

[mtklein, 2014-10-02 19:54:59]

https://codereview.chromium.org/627473002/diff/1/dm/DMWriteTask.cpp
File dm/DMWriteTask.cpp (right):

https://codereview.chromium.org/627473002/diff/1/dm/DMWriteTask.cpp#newcode85
dm/DMWriteTask.cpp:85: if (stream->getMemoryBase()) {
Let's skip the if and just always call writeStream().

[bungeman-skia, 2014-10-02 20:00:11]

https://codereview.chromium.org/627473002/diff/1/dm/DMWriteTask.cpp
File dm/DMWriteTask.cpp (right):

https://codereview.chromium.org/627473002/diff/1/dm/DMWriteTask.cpp#newcode85
dm/DMWriteTask.cpp:85: if (stream->getMemoryBase()) {
On 2014/10/02 19:54:59, mtklein wrote:
> Let's skip the if and just always call writeStream().

Done.

[mtklein, 2014-10-02 20:03:29]

The CQ bit was checked by mtklein@google.com

[mtklein, 2014-10-02 20:03:29]

lgtm

[commit-bot: I haz the power, 2014-10-02 20:04:48]

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

[commit-bot: I haz the power, 2014-10-02 20:39:03]

Committed patchset #2 (id:20001) as d51ce44d18cb28df568f817b05478d24ef21c1a7