Finish migration from sameThreadBindForMojo to createBaseCallback.

Finish migration from sameThreadBindForMojo to createBaseCallback.

The sameThreadBindForMojo function is dangerous because it binds a raw
pointer into a base::Callback. If this pointer is to a heap object then
it may result in a use-after-free due to lazy sweeping. The
createBaseCallback function instead relies on WTF::bind to correctly
handle object lifetimes before converting between a WTF::Function and
base::Callback.

This patch removes sameThreadBindForMojo and converts the last few
remaining uses.

BUG=None

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

[Reilly Grant (use Gerrit), 2016-05-17 21:00:08]

reillyg@chromium.org changed reviewers:
+ pfeldman@chromium.org

[Reilly Grant (use Gerrit), 2016-05-17 21:00:09]

Please take a look.

[haraken, 2016-05-17 23:49:51]

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

[haraken, 2016-05-17 23:49:52]

LGTM

[Reilly Grant (use Gerrit), 2016-05-18 00:29:09]

The CQ bit was checked by reillyg@chromium.org

[commit-bot: I haz the power, 2016-05-18 00:29:42]

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

[kinuko, 2016-05-18 00:32:42]

kinuko@chromium.org changed reviewers:
+ kinuko@chromium.org

[kinuko, 2016-05-18 00:32:43]

Aha nice.

[commit-bot: I haz the power, 2016-05-18 00:35:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-18 00:36:36]

Description was changed from

==========
Finish migration from sameThreadBindForMojo to createBaseCallback.

The sameThreadBindForMojo function is dangerous because it binds a raw
pointer into a base::Callback. If this pointer is to a heap object then
it may result in a use-after-free due to lazy sweeping. The
createBaseCallback function instead relies on WTF::bind to correctly
handle object lifetimes before converting between a WTF::Function and
base::Callback.

This patch removes sameThreadBindForMojo and converts the last few
remaining uses.

BUG=None
==========

to

==========
Finish migration from sameThreadBindForMojo to createBaseCallback.

The sameThreadBindForMojo function is dangerous because it binds a raw
pointer into a base::Callback. If this pointer is to a heap object then
it may result in a use-after-free due to lazy sweeping. The
createBaseCallback function instead relies on WTF::bind to correctly
handle object lifetimes before converting between a WTF::Function and
base::Callback.

This patch removes sameThreadBindForMojo and converts the last few
remaining uses.

BUG=None

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

[commit-bot: I haz the power, 2016-05-18 00:36:37]

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