Make cc::DrawingDisplayItem hold a const SkPicture.

Make cc::DrawingDisplayItem hold a const SkPicture.

This lets us remove const_cast<SkPicture*> from callers (i.e. Blink) which
hold SkPicture through const pointers.

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/7994711ab6906f943da80049d13868530c07be1a
Cr-Commit-Position: refs/heads/master@{#368444}

[jbroman, 2016-01-08 20:25:26]

Description was changed from

==========
Make cc::DrawingDisplayItem hold a const SkPicture.

This lets us remove const_cast<SkPicture*> from callers (i.e. Blink) which
hold SkPicture through const pointers.
==========

to

==========
Make cc::DrawingDisplayItem hold a const SkPicture.

This lets us remove const_cast<SkPicture*> from callers (i.e. Blink) which
hold SkPicture through const pointers.
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[jbroman, 2016-01-08 20:27:25]

Description was changed from

==========
Make cc::DrawingDisplayItem hold a const SkPicture.

This lets us remove const_cast<SkPicture*> from callers (i.e. Blink) which
hold SkPicture through const pointers.
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Make cc::DrawingDisplayItem hold a const SkPicture.

This lets us remove const_cast<SkPicture*> from callers (i.e. Blink) which
hold SkPicture through const pointers.

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[jbroman, 2016-01-08 20:27:47]

jbroman@chromium.org changed reviewers:
+ danakj@chromium.org

[jbroman, 2016-01-08 20:27:47]

Any reason this holds onto a non-const SkPicture? I was about to write another
const_cast for SPv2, but then I decided I'd do this instead (CQ-willing).

[danakj, 2016-01-08 20:51:36]

LGTM

[jbroman, 2016-01-08 20:54:57]

The CQ bit was checked by jbroman@chromium.org

[commit-bot: I haz the power, 2016-01-08 20:55:37]

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

[commit-bot: I haz the power, 2016-01-08 22:07:29]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-08 22:08:24]

Description was changed from

==========
Make cc::DrawingDisplayItem hold a const SkPicture.

This lets us remove const_cast<SkPicture*> from callers (i.e. Blink) which
hold SkPicture through const pointers.

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Make cc::DrawingDisplayItem hold a const SkPicture.

This lets us remove const_cast<SkPicture*> from callers (i.e. Blink) which
hold SkPicture through const pointers.

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/7994711ab6906f943da80049d13868530c07be1a
Cr-Commit-Position: refs/heads/master@{#368444}
==========

[commit-bot: I haz the power, 2016-01-08 22:08:25]

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