Fix use-after-free in TextureLayerImplTest.ResourceNotFreedOnGpuRasterToggle.

Fix use-after-free in TextureLayerImplTest.ResourceNotFreedOnGpuRasterToggle.

The callback uses a local variable 'released', but 'released' had
a more narrow scope than the owner of the callback (and it's called
from the destructor, when 'released' is already out of scope)

BUG=649897
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Committed: https://crrev.com/360dee282db84d1e63414c9e8d471a7465a8441d
Cr-Commit-Position: refs/heads/master@{#439888}

[krasin1, 2016-12-20 21:44:23]

Description was changed from

==========
Fix use-after-free in TextureLayerImplTest.ResourceNotFreedOnGpuRasterToggle.

The callback uses a local variable 'released', but 'released' had
a more narrow scope than the owner of the callback (and it's called
from the destructor, when 'released' is already out of scope)

BUG=649897
==========

to

==========
Fix use-after-free in TextureLayerImplTest.ResourceNotFreedOnGpuRasterToggle.

The callback uses a local variable 'released', but 'released' had
a more narrow scope than the owner of the callback (and it's called
from the destructor, when 'released' is already out of scope)

BUG=649897
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[krasin1, 2016-12-20 21:45:52]

krasin@chromium.org changed reviewers:
+ ccameron@chromium.org

[krasin1, 2016-12-20 21:45:58]

The CQ bit was checked by krasin@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-20 21:46:41]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[ccameron, 2016-12-20 21:47:07]

lgtm

[krasin1, 2016-12-20 21:58:42]

The CQ bit was unchecked by krasin@chromium.org

[krasin1, 2016-12-20 21:58:46]

The CQ bit was checked by krasin@chromium.org

[krasin1, 2016-12-20 21:59:05]

On 2016/12/20 21:47:07, ccameron wrote:
> lgtm

Thank you for the instant review!

[commit-bot: I haz the power, 2016-12-20 21:59:17]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-20 22:07:07]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482271126203380, "parent_rev":
"3c2f50251637165ae33dac90295876ecf91c87a3", "commit_rev":
"e8c31cbee908a607cb52d36688900896c719546a"}

[commit-bot: I haz the power, 2016-12-20 22:07:38]

Description was changed from

==========
Fix use-after-free in TextureLayerImplTest.ResourceNotFreedOnGpuRasterToggle.

The callback uses a local variable 'released', but 'released' had
a more narrow scope than the owner of the callback (and it's called
from the destructor, when 'released' is already out of scope)

BUG=649897
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Fix use-after-free in TextureLayerImplTest.ResourceNotFreedOnGpuRasterToggle.

The callback uses a local variable 'released', but 'released' had
a more narrow scope than the owner of the callback (and it's called
from the destructor, when 'released' is already out of scope)

BUG=649897
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2596593003
==========

[commit-bot: I haz the power, 2016-12-20 22:07:39]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-20 22:11:16]

Description was changed from

==========
Fix use-after-free in TextureLayerImplTest.ResourceNotFreedOnGpuRasterToggle.

The callback uses a local variable 'released', but 'released' had
a more narrow scope than the owner of the callback (and it's called
from the destructor, when 'released' is already out of scope)

BUG=649897
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2596593003
==========

to

==========
Fix use-after-free in TextureLayerImplTest.ResourceNotFreedOnGpuRasterToggle.

The callback uses a local variable 'released', but 'released' had
a more narrow scope than the owner of the callback (and it's called
from the destructor, when 'released' is already out of scope)

BUG=649897
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Committed: https://crrev.com/360dee282db84d1e63414c9e8d471a7465a8441d
Cr-Commit-Position: refs/heads/master@{#439888}
==========

[commit-bot: I haz the power, 2016-12-20 22:11:18]

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