Replace AllowCrossThreadAccess() + ThreadSafeRefCounted pointers with PassRefPtr

Replace AllowCrossThreadAccess() + ThreadSafeRefCounted with PassRefPtr

This CL disallows AllowCrossThreadAccess() with pointers to
ThreadSafeRefCounted in platform/CrossThreadCopier.h and replaces
AllowCrossThreadAccess() with PassRefPtr in other files
where compile failed.

BUG=597856
Committed: https://crrev.com/856cfbac0b7c1f14262e00c747c0724e0595a2e8
Cr-Commit-Position: refs/heads/master@{#394079}

[hiroshige, 2016-04-22 13:20:28]

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

[commit-bot: I haz the power, 2016-04-22 13:20:39]

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

[hiroshige, 2016-04-22 14:09:56]

Description was changed from

==========
Replace AllowCrossThreadAccess() + ThreadSafeRefCounted with (Pass)RefPtr

BUG=
==========

to

==========
Replace AllowCrossThreadAccess() + ThreadSafeRefCounted with (Pass)RefPtr

BUG=
==========

[commit-bot: I haz the power, 2016-04-22 15:50:58]

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

[commit-bot: I haz the power, 2016-04-22 15:50:59]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2016-04-27 06:45:36]

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

[commit-bot: I haz the power, 2016-04-27 06:45:43]

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

[commit-bot: I haz the power, 2016-04-27 08:27:32]

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

[commit-bot: I haz the power, 2016-04-27 08:27:33]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2016-04-27 09:27:23]

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

[commit-bot: I haz the power, 2016-04-27 09:27:37]

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

[commit-bot: I haz the power, 2016-04-27 11:21:55]

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

[commit-bot: I haz the power, 2016-04-27 11:21:55]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2016-05-17 03:43:55]

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

[commit-bot: I haz the power, 2016-05-17 03:44:08]

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

[hiroshige, 2016-05-17 03:45:57]

Description was changed from

==========
Replace AllowCrossThreadAccess() + ThreadSafeRefCounted with (Pass)RefPtr

BUG=
==========

to

==========
Replace AllowCrossThreadAccess() + ThreadSafeRefCounted with PassRefPtr

This CL disallows AllowCrossThreadAccess() with pointers to
ThreadSafeRefCounted in platform/CrossThreadCopier.h and replaces
AllowCrossThreadAccess() with PassRefPtr in other files
where compile failed.

BUG=597856
==========

[hiroshige, 2016-05-17 03:45:57]

hiroshige@chromium.org changed reviewers:
+ haraken@chromium.org, kinuko@chromium.org, tzik@chromium.org

[hiroshige, 2016-05-17 03:46:13]

PTAL.

https://codereview.chromium.org/1905343002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webaudio/MediaElementAudioSourceNode.cpp
(right):

https://codereview.chromium.org/1905343002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webaudio/MediaElementAudioSourceNode.cpp:180:
createCrossThreadTask(&MediaElementAudioSourceHandler::printCORSMessage,
PassRefPtr<MediaElementAudioSourceHandler>(this), m_currentSrcString));
I merged mulitiple lines into one here, just to avoid a presubmit error:
when I replaced a line
> AllowCrossThreadAccess(this),
with a line
> PassRefPtr<MediaElementAudioSourceHandler>(this),
the presubmit script thinks it is a declaration of PassRefPtr as a local
variable (because it is PassRefPtr at the line head?) and reports an error.

Other workarounds might be:
- Introduce wrapPassRefPtr(), or
- Fix the presubmit script.

https://codereview.chromium.org/1905343002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/webaudio/OfflineAudioDestinationNode.cpp
(right):

https://codereview.chromium.org/1905343002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/webaudio/OfflineAudioDestinationNode.cpp:206:
createCrossThreadTask(&OfflineAudioDestinationHandler::notifySuspend,
PassRefPtr<OfflineAudioDestinationHandler>(this),
context()->currentSampleFrame()));
ditto.

[haraken, 2016-05-17 04:18:10]

LGTM

[kinuko, 2016-05-17 05:26:20]

lgtm

[tzik, 2016-05-17 05:28:23]

lgtm

[commit-bot: I haz the power, 2016-05-17 05:35:04]

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

[commit-bot: I haz the power, 2016-05-17 05:35:05]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2016-05-17 07:44:41]

The CQ bit was checked by hiroshige@chromium.org

[commit-bot: I haz the power, 2016-05-17 07:44:59]

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

[commit-bot: I haz the power, 2016-05-17 07:49:11]

Description was changed from

==========
Replace AllowCrossThreadAccess() + ThreadSafeRefCounted with PassRefPtr

This CL disallows AllowCrossThreadAccess() with pointers to
ThreadSafeRefCounted in platform/CrossThreadCopier.h and replaces
AllowCrossThreadAccess() with PassRefPtr in other files
where compile failed.

BUG=597856
==========

to

==========
Replace AllowCrossThreadAccess() + ThreadSafeRefCounted with PassRefPtr

This CL disallows AllowCrossThreadAccess() with pointers to
ThreadSafeRefCounted in platform/CrossThreadCopier.h and replaces
AllowCrossThreadAccess() with PassRefPtr in other files
where compile failed.

BUG=597856
==========

[commit-bot: I haz the power, 2016-05-17 07:49:12]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-05-17 07:50:37]

Description was changed from

==========
Replace AllowCrossThreadAccess() + ThreadSafeRefCounted with PassRefPtr

This CL disallows AllowCrossThreadAccess() with pointers to
ThreadSafeRefCounted in platform/CrossThreadCopier.h and replaces
AllowCrossThreadAccess() with PassRefPtr in other files
where compile failed.

BUG=597856
==========

to

==========
Replace AllowCrossThreadAccess() + ThreadSafeRefCounted with PassRefPtr

This CL disallows AllowCrossThreadAccess() with pointers to
ThreadSafeRefCounted in platform/CrossThreadCopier.h and replaces
AllowCrossThreadAccess() with PassRefPtr in other files
where compile failed.

BUG=597856

Committed: https://crrev.com/856cfbac0b7c1f14262e00c747c0724e0595a2e8
Cr-Commit-Position: refs/heads/master@{#394079}
==========

[commit-bot: I haz the power, 2016-05-17 07:50:39]

Patchset 4 (id:??) landed as
https://crrev.com/856cfbac0b7c1f14262e00c747c0724e0595a2e8
Cr-Commit-Position: refs/heads/master@{#394079}