Revert of Drop unecessary use of CrossThreadPersistent by CanvasAsyncBlobCreator.

Revert of Drop unecessary use of CrossThreadPersistent by CanvasAsyncBlobCreator. (patchset #1 id:1 of https://codereview.chromium.org/2051993002/ )

Reason for revert:
Breaks WebKit Linux Leak bot: 
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Leak/builds/20274

unexpected_failures:
virtual/threaded/fast/canvas-toBlob/canvas-toBlob-file-vs-blob.html
virtual/display_list_2d_canvas/fast/canvas/canvas-toBlob-toDataURL-race-imageEncoder-jpeg.html
virtual/threaded/fast/canvas-toBlob/canvas-toBlob-invalid.html
fast/canvas/canvas-toBlob-toDataURL-race-imageEncoder-jpeg.html
virtual/gpu/fast/canvas/canvas-toBlob-toDataURL-race-imageEncoder-jpeg.html
virtual/threaded/fast/canvas-toBlob/canvas-toBlob-defaultpng.html
virtual/gpu/fast/canvas/canvas-toBlob-jpeg-medium-quality.html


Original issue's description:
> Drop unecessary use of CrossThreadPersistent by CanvasAsyncBlobCreator.
> 
> The callback object that the CanvasAsyncBlobCreator passes along to a
> background thread can be kept as a simple Member<>; no need to
> involve CrossThreadPersistent<> and risk inadvertently introducing
> leaks.
> 
> R=
> BUG=
> 
> Committed: https://crrev.com/089aa85a72d06184c60104b1c1c6f84d1e2eb334
> Cr-Commit-Position: refs/heads/master@{#399181}

TBR=oilpan-reviews@chromium.org,sigbjornf@opera.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=

[kjellander_chromium, 2016-06-14 11:50:14]

The CQ bit was checked by kjellander@chromium.org

[kjellander_chromium, 2016-06-14 11:50:14]

Created Revert of Drop unecessary use of CrossThreadPersistent by
CanvasAsyncBlobCreator.

[commit-bot: I haz the power, 2016-06-14 11:50:22]

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

[sof, 2016-06-14 11:51:35]

The CQ bit was unchecked by sigbjornf@opera.com

[sof, 2016-06-14 11:52:11]

Unchecked; leak bots will cycle green follwing https://crrev.com/399675.

[sof, 2016-06-14 11:53:33]

On 2016/06/14 11:52:11, sof wrote:
> Unchecked; leak bots will cycle green follwing https://crrev.com/399675.

cf.
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Leak/b...
- leaks there are not related to the CL in Q.

[kjellander_chromium, 2016-06-14 14:01:07]

On 2016/06/14 11:53:33, sof wrote:
> On 2016/06/14 11:52:11, sof wrote:
> > Unchecked; leak bots will cycle green follwing https://crrev.com/399675.
> 
> cf.
>
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Leak/b...
> - leaks there are not related to the CL in Q.

OK thanks. I will compare with the latest build before I revert next time.

[kjellander_chromium, 2016-06-14 14:01:14]

Description was changed from

==========
Revert of Drop unecessary use of CrossThreadPersistent by
CanvasAsyncBlobCreator. (patchset #1 id:1 of
https://codereview.chromium.org/2051993002/ )

Reason for revert:
Breaks WebKit Linux Leak bot: 
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Leak/b...

unexpected_failures:
virtual/threaded/fast/canvas-toBlob/canvas-toBlob-file-vs-blob.html
virtual/display_list_2d_canvas/fast/canvas/canvas-toBlob-toDataURL-race-imageEncoder-jpeg.html
virtual/threaded/fast/canvas-toBlob/canvas-toBlob-invalid.html
fast/canvas/canvas-toBlob-toDataURL-race-imageEncoder-jpeg.html
virtual/gpu/fast/canvas/canvas-toBlob-toDataURL-race-imageEncoder-jpeg.html
virtual/threaded/fast/canvas-toBlob/canvas-toBlob-defaultpng.html
virtual/gpu/fast/canvas/canvas-toBlob-jpeg-medium-quality.html


Original issue's description:
> Drop unecessary use of CrossThreadPersistent by CanvasAsyncBlobCreator.
> 
> The callback object that the CanvasAsyncBlobCreator passes along to a
> background thread can be kept as a simple Member<>; no need to
> involve CrossThreadPersistent<> and risk inadvertently introducing
> leaks.
> 
> R=
> BUG=
> 
> Committed: https://crrev.com/089aa85a72d06184c60104b1c1c6f84d1e2eb334
> Cr-Commit-Position: refs/heads/master@{#399181}

TBR=oilpan-reviews@chromium.org,sigbjornf@opera.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=
==========

to

==========
Revert of Drop unecessary use of CrossThreadPersistent by
CanvasAsyncBlobCreator. (patchset #1 id:1 of
https://codereview.chromium.org/2051993002/ )

Reason for revert:
Breaks WebKit Linux Leak bot: 
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Leak/b...

unexpected_failures:
virtual/threaded/fast/canvas-toBlob/canvas-toBlob-file-vs-blob.html
virtual/display_list_2d_canvas/fast/canvas/canvas-toBlob-toDataURL-race-imageEncoder-jpeg.html
virtual/threaded/fast/canvas-toBlob/canvas-toBlob-invalid.html
fast/canvas/canvas-toBlob-toDataURL-race-imageEncoder-jpeg.html
virtual/gpu/fast/canvas/canvas-toBlob-toDataURL-race-imageEncoder-jpeg.html
virtual/threaded/fast/canvas-toBlob/canvas-toBlob-defaultpng.html
virtual/gpu/fast/canvas/canvas-toBlob-jpeg-medium-quality.html


Original issue's description:
> Drop unecessary use of CrossThreadPersistent by CanvasAsyncBlobCreator.
> 
> The callback object that the CanvasAsyncBlobCreator passes along to a
> background thread can be kept as a simple Member<>; no need to
> involve CrossThreadPersistent<> and risk inadvertently introducing
> leaks.
> 
> R=
> BUG=
> 
> Committed: https://crrev.com/089aa85a72d06184c60104b1c1c6f84d1e2eb334
> Cr-Commit-Position: refs/heads/master@{#399181}

TBR=oilpan-reviews@chromium.org,sigbjornf@opera.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=
==========