Remove unnecessary CanvasAsyncBlobCreator keep alive protection.

Remove unnecessary CanvasAsyncBlobCreator keep alive protection.

Closures keep their GCed arguments and |this| alive until completed,
hence manual keep-alive handling on top of that isn't required
for CanvasAsyncBlobCreator.

R=haraken
BUG=

Committed: https://crrev.com/e0a85c1a76e808e0c0d0452600550f934ff46fc1
Cr-Commit-Position: refs/heads/master@{#387260}

[sof, 2016-04-12 14:28:45]

sigbjornf@opera.com changed reviewers:
+ haraken@chromium.org

[sof, 2016-04-12 14:28:47]

please take a look.

Following up on
https://codereview.chromium.org/1878463002/diff/60001/third_party/WebKit/Sour...

[sof, 2016-04-12 14:35:52]

Description was changed from

==========
Remove unnecessary CanvasAsyncBlobCreator keep alive protections.

Closures keep their GCed arguments and |this| alive until completed,
hence manual keep-alive handling on top of that isn't required
for CanvasAsyncBlobCreator.

R=
BUG=
==========

to

==========
Remove unnecessary CanvasAsyncBlobCreator keep alive protection.

Closures keep their GCed arguments and |this| alive until completed,
hence manual keep-alive handling on top of that isn't required
for CanvasAsyncBlobCreator.

R=
BUG=
==========

[Justin Novosad, 2016-04-12 15:21:39]

junov@chromium.org changed reviewers:
+ junov@chromium.org, xlai@chromium.org

[Justin Novosad, 2016-04-12 15:21:39]

+xlai  PTAL

IIRC, the self reference was necessary in the pre-oilpan  world.

[haraken, 2016-04-12 15:54:06]

LGTM

[sof, 2016-04-14 05:21:35]

Description was changed from

==========
Remove unnecessary CanvasAsyncBlobCreator keep alive protection.

Closures keep their GCed arguments and |this| alive until completed,
hence manual keep-alive handling on top of that isn't required
for CanvasAsyncBlobCreator.

R=
BUG=
==========

to

==========
Remove unnecessary CanvasAsyncBlobCreator keep alive protection.

Closures keep their GCed arguments and |this| alive until completed,
hence manual keep-alive handling on top of that isn't required
for CanvasAsyncBlobCreator.

R=haraken
BUG=
==========

[sof, 2016-04-14 05:22:32]

Sending this one along now; will follow up on any later feedback from canvas
owners, of course.

[sof, 2016-04-14 05:22:38]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-04-14 05:22:49]

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

[commit-bot: I haz the power, 2016-04-14 08:21:26]

Description was changed from

==========
Remove unnecessary CanvasAsyncBlobCreator keep alive protection.

Closures keep their GCed arguments and |this| alive until completed,
hence manual keep-alive handling on top of that isn't required
for CanvasAsyncBlobCreator.

R=haraken
BUG=
==========

to

==========
Remove unnecessary CanvasAsyncBlobCreator keep alive protection.

Closures keep their GCed arguments and |this| alive until completed,
hence manual keep-alive handling on top of that isn't required
for CanvasAsyncBlobCreator.

R=haraken
BUG=
==========

[commit-bot: I haz the power, 2016-04-14 08:21:28]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-14 08:22:34]

Description was changed from

==========
Remove unnecessary CanvasAsyncBlobCreator keep alive protection.

Closures keep their GCed arguments and |this| alive until completed,
hence manual keep-alive handling on top of that isn't required
for CanvasAsyncBlobCreator.

R=haraken
BUG=
==========

to

==========
Remove unnecessary CanvasAsyncBlobCreator keep alive protection.

Closures keep their GCed arguments and |this| alive until completed,
hence manual keep-alive handling on top of that isn't required
for CanvasAsyncBlobCreator.

R=haraken
BUG=

Committed: https://crrev.com/e0a85c1a76e808e0c0d0452600550f934ff46fc1
Cr-Commit-Position: refs/heads/master@{#387260}
==========

[commit-bot: I haz the power, 2016-04-14 08:22:35]

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

[Justin Novosad, 2016-04-18 19:36:00]

On 2016/04/14 05:22:32, sof wrote:
> Sending this one along now; will follow up on any later feedback from canvas
> owners, of course.

I was OOO part of last week. Catching-up now. post-commit lgtm