Make GrAtlasTextBlob::run own effects it points to.

Make GrAtlasTextBlob::run own effects it points to.

BUG=chromium:608566
Currently the run has bare pointers and the effects can be destroyed while a run is pointing at them.

GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1993213003

Committed: https://skia.googlesource.com/skia/+/8b6fa5e8a9016ebbf3a03009abc3845b71452550

[bsalomon, 2016-05-19 22:28:39]

Description was changed from

==========
Make GrAtlasTextBlob own effects it points to.

Currently the run has bare pointers and the effects can be destroyed while a run
is pointing at them.
==========

to

==========
Make GrAtlasTextBlob own effects it points to.

Currently the run has bare pointers and the effects can be destroyed while a run
is pointing at them.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[bsalomon, 2016-05-19 22:31:26]

bsalomon@google.com changed reviewers:
+ bungeman@google.com, reed@google.com

[bsalomon, 2016-05-19 22:31:26]

I looked at making SkScalerContextEffects us sk_sp and move it into
SkScalerContext but it was a fairly large change. This will likely have to be
cherry picked back and I wanted to limit the scope of the change.

[bsalomon, 2016-05-19 22:31:38]

Description was changed from

==========
Make GrAtlasTextBlob own effects it points to.

Currently the run has bare pointers and the effects can be destroyed while a run
is pointing at them.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Make GrAtlasTextBlob::run own effects it points to.

Currently the run has bare pointers and the effects can be destroyed while a run
is pointing at them.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[bsalomon, 2016-05-19 22:31:43]

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

[commit-bot: I haz the power, 2016-05-19 22:31:50]

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

[bungeman-skia, 2016-05-19 22:44:27]

lgtm

[bungeman-skia, 2016-05-19 22:45:01]

The description should probably point to the BUG.

[commit-bot: I haz the power, 2016-05-19 22:51:40]

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

[commit-bot: I haz the power, 2016-05-19 22:51:41]

Dry run: This issue passed the CQ dry run.

[bsalomon, 2016-05-19 23:22:14]

Description was changed from

==========
Make GrAtlasTextBlob::run own effects it points to.

Currently the run has bare pointers and the effects can be destroyed while a run
is pointing at them.
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Make GrAtlasTextBlob::run own effects it points to.

BUG=chromium:608566

Currently the run has bare pointers and the effects can be destroyed while a run
is pointing at them.

GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[bsalomon, 2016-05-19 23:22:25]

On 2016/05/19 22:45:01, bungeman-skia wrote:
> The description should probably point to the BUG.

Yep, done.

[bsalomon, 2016-05-19 23:22:41]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2016-05-19 23:22:53]

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

[commit-bot: I haz the power, 2016-05-19 23:23:50]

Description was changed from

==========
Make GrAtlasTextBlob::run own effects it points to.

BUG=chromium:608566

Currently the run has bare pointers and the effects can be destroyed while a run
is pointing at them.

GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Make GrAtlasTextBlob::run own effects it points to.

BUG=chromium:608566

Currently the run has bare pointers and the effects can be destroyed while a run
is pointing at them.

GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/8b6fa5e8a9016ebbf3a03009abc3845b71452550
==========

[commit-bot: I haz the power, 2016-05-19 23:23:51]

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