cc: Fix nullptr dereference when capturing skps.

cc: Fix nullptr dereference when capturing skps.

We capture sk picture from PictureLayer, which happens before we had
a chance to set the image decode controller during commit. The fix here
is not to use PlaybackToCanvas, but rather use other functions to
accomplish the same goal. PlaybackToCanvas should only be called after
commit.

R=enne
BUG=581107
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/6abc838bd4fed2d0e0eef80febbda07aa0190644
Cr-Commit-Position: refs/heads/master@{#371655}

[vmpstr, 2016-01-26 21:46:58]

Description was changed from

==========
cc: Fix nullptr dereference when capturing skps.

We capture sk picture from PictureLayer, which happens before we had
a chance to set the image decode controller during commit. The fix here
is not to use PlaybackToCanvas, but rather use other functions to
accomplish the same goal. PlaybackToCanvas should only be called after
commit.

R=enne
BUG=581107
==========

to

==========
cc: Fix nullptr dereference when capturing skps.

We capture sk picture from PictureLayer, which happens before we had
a chance to set the image decode controller during commit. The fix here
is not to use PlaybackToCanvas, but rather use other functions to
accomplish the same goal. PlaybackToCanvas should only be called after
commit.

R=enne
BUG=581107
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[vmpstr, 2016-01-26 21:50:42]

Please take a look.

[enne (OOO), 2016-01-26 21:57:25]

lgtm

[vmpstr, 2016-01-26 21:57:34]

The CQ bit was checked by vmpstr@chromium.org

[commit-bot: I haz the power, 2016-01-26 21:58:31]

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

[commit-bot: I haz the power, 2016-01-26 23:27:09]

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

[commit-bot: I haz the power, 2016-01-26 23:27:09]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/8...)

[vmpstr, 2016-01-26 23:28:04]

The CQ bit was checked by vmpstr@chromium.org

[commit-bot: I haz the power, 2016-01-26 23:30:11]

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

[commit-bot: I haz the power, 2016-01-27 00:19:14]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-27 00:20:07]

Description was changed from

==========
cc: Fix nullptr dereference when capturing skps.

We capture sk picture from PictureLayer, which happens before we had
a chance to set the image decode controller during commit. The fix here
is not to use PlaybackToCanvas, but rather use other functions to
accomplish the same goal. PlaybackToCanvas should only be called after
commit.

R=enne
BUG=581107
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Fix nullptr dereference when capturing skps.

We capture sk picture from PictureLayer, which happens before we had
a chance to set the image decode controller during commit. The fix here
is not to use PlaybackToCanvas, but rather use other functions to
accomplish the same goal. PlaybackToCanvas should only be called after
commit.

R=enne
BUG=581107
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/6abc838bd4fed2d0e0eef80febbda07aa0190644
Cr-Commit-Position: refs/heads/master@{#371655}
==========

[commit-bot: I haz the power, 2016-01-27 00:20:07]

Patchset 1 (id:??) landed as
https://crrev.com/6abc838bd4fed2d0e0eef80febbda07aa0190644
Cr-Commit-Position: refs/heads/master@{#371655}