Reland of Messaging: Avoid calling non-thread-safe functions in MessagePort::messageAvailable()

Reland of Messaging: Avoid calling non-thread-safe functions in MessagePort::messageAvailable() (patchset #1 id:1 of https://codereview.chromium.org/2833673002/ )

Reason for revert:
The CL that this CL depends on was relanded:
https://codereview.chromium.org/2832763002/

Original issue's description:
> Revert of Messaging: Avoid calling non-thread-safe functions in MessagePort::messageAvailable() (patchset #2 id:20001 of https://codereview.chromium.org/2818073002/ )
> 
> Reason for revert:
> [1] that this CL depends on needs to be reverted:
> https://codereview.chromium.org/2831843002/
> 
> Original issue's description:
> > Messaging: Avoid calling non-thread-safe functions in MessagePort::messageAvailable()
> > 
> > As implementation comments in MessagePort::messageAvailable(), it can be called
> > from another thread and should not call non-thread-functions. However, the
> > current implementation wrongly calls ContextLifecycleObserver's
> > GetExecutionContext() that is not a thread-safe function to post a task to the
> > context thread.
> > 
> > To avoid that, this CL replaces ExecutionContext::PostTask() with
> > WebTaskRunner::PostTask(). The task runner is captured in the ctor of
> > MessagePort called on the context thread.
> > 
> > BUG=694925
> > 
> > Review-Url: https://codereview.chromium.org/2818073002
> > Cr-Commit-Position: refs/heads/master@{#465886}
> > Committed: https://chromium.googlesource.com/chromium/src/+/829907ac0d3d2ed39d8752d22a6d0e45df1e8325
> 
> TBR=tzik@chromium.org,yuryu@chromium.org,kinuko@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=694925
> 
> Review-Url: https://codereview.chromium.org/2833673002
> Cr-Commit-Position: refs/heads/master@{#465890}
> Committed: https://chromium.googlesource.com/chromium/src/+/7cbe41ace5629ba6b9ec5502803f02fb2f09b0fd

TBR=tzik@chromium.org,yuryu@chromium.org,kinuko@chromium.org
BUG=694925
Review-Url: https://codereview.chromium.org/2836473002
Cr-Commit-Position: refs/heads/master@{#466223}
Committed: https://chromium.googlesource.com/chromium/src/+/90cbb1bd3931fb8ee58c729bbbcdb32f4915dd1f

[nhiroki, 2017-04-20 23:01:46]

Created Reland of Messaging: Avoid calling non-thread-safe functions in
MessagePort::messageAvailable()

[nhiroki, 2017-04-20 23:02:31]

Description was changed from

==========
Reland of Messaging: Avoid calling non-thread-safe functions in
MessagePort::messageAvailable() (patchset #1 id:1 of
https://codereview.chromium.org/2833673002/ )

Reason for revert:
The CL that this CL depends on was relanded:
https://codereview.chromium.org/2832763002/

Original issue's description:
> Revert of Messaging: Avoid calling non-thread-safe functions in
MessagePort::messageAvailable() (patchset #2 id:20001 of
https://codereview.chromium.org/2818073002/ )
> 
> Reason for revert:
> [1] that this CL depends on needs to be reverted:
> https://codereview.chromium.org/2831843002/
> 
> Original issue's description:
> > Messaging: Avoid calling non-thread-safe functions in
MessagePort::messageAvailable()
> > 
> > As implementation comments in MessagePort::messageAvailable(), it can be
called
> > from another thread and should not call non-thread-functions. However, the
> > current implementation wrongly calls ContextLifecycleObserver's
> > GetExecutionContext() that is not a thread-safe function to post a task to
the
> > context thread.
> > 
> > To avoid that, this CL replaces ExecutionContext::PostTask() with
> > WebTaskRunner::PostTask(). The task runner is captured in the ctor of
> > MessagePort called on the context thread.
> > 
> > BUG=694925
> > 
> > Review-Url: https://codereview.chromium.org/2818073002
> > Cr-Commit-Position: refs/heads/master@{#465886}
> > Committed:
https://chromium.googlesource.com/chromium/src/+/829907ac0d3d2ed39d8752d22a6d...
> 
> TBR=tzik@chromium.org,yuryu@chromium.org,kinuko@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=694925
> 
> Review-Url: https://codereview.chromium.org/2833673002
> Cr-Commit-Position: refs/heads/master@{#465890}
> Committed:
https://chromium.googlesource.com/chromium/src/+/7cbe41ace5629ba6b9ec5502803f...

TBR=tzik@chromium.org,yuryu@chromium.org,kinuko@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=694925
==========

to

==========
Reland of Messaging: Avoid calling non-thread-safe functions in
MessagePort::messageAvailable() (patchset #1 id:1 of
https://codereview.chromium.org/2833673002/ )

Reason for revert:
The CL that this CL depends on was relanded:
https://codereview.chromium.org/2832763002/

Original issue's description:
> Revert of Messaging: Avoid calling non-thread-safe functions in
MessagePort::messageAvailable() (patchset #2 id:20001 of
https://codereview.chromium.org/2818073002/ )
> 
> Reason for revert:
> [1] that this CL depends on needs to be reverted:
> https://codereview.chromium.org/2831843002/
> 
> Original issue's description:
> > Messaging: Avoid calling non-thread-safe functions in
MessagePort::messageAvailable()
> > 
> > As implementation comments in MessagePort::messageAvailable(), it can be
called
> > from another thread and should not call non-thread-functions. However, the
> > current implementation wrongly calls ContextLifecycleObserver's
> > GetExecutionContext() that is not a thread-safe function to post a task to
the
> > context thread.
> > 
> > To avoid that, this CL replaces ExecutionContext::PostTask() with
> > WebTaskRunner::PostTask(). The task runner is captured in the ctor of
> > MessagePort called on the context thread.
> > 
> > BUG=694925
> > 
> > Review-Url: https://codereview.chromium.org/2818073002
> > Cr-Commit-Position: refs/heads/master@{#465886}
> > Committed:
https://chromium.googlesource.com/chromium/src/+/829907ac0d3d2ed39d8752d22a6d...
> 
> TBR=tzik@chromium.org,yuryu@chromium.org,kinuko@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=694925
> 
> Review-Url: https://codereview.chromium.org/2833673002
> Cr-Commit-Position: refs/heads/master@{#465890}
> Committed:
https://chromium.googlesource.com/chromium/src/+/7cbe41ace5629ba6b9ec5502803f...

TBR=tzik@chromium.org,yuryu@chromium.org,kinuko@chromium.org
BUG=694925
==========

[nhiroki, 2017-04-20 23:03:11]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2017-04-20 23:03:25]

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, 2017-04-20 23:05:57]

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

[commit-bot: I haz the power, 2017-04-20 23:05:58]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[nhiroki, 2017-04-20 23:53:04]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2017-04-20 23:53:25]

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, 2017-04-21 01:59:26]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492732384775790, "parent_rev":
"cbc7bdbd3e2ca861dc1d0bb63dfc3fcabb1f9fb9", "commit_rev":
"90cbb1bd3931fb8ee58c729bbbcdb32f4915dd1f"}

[commit-bot: I haz the power, 2017-04-21 02:00:13]

Description was changed from

==========
Reland of Messaging: Avoid calling non-thread-safe functions in
MessagePort::messageAvailable() (patchset #1 id:1 of
https://codereview.chromium.org/2833673002/ )

Reason for revert:
The CL that this CL depends on was relanded:
https://codereview.chromium.org/2832763002/

Original issue's description:
> Revert of Messaging: Avoid calling non-thread-safe functions in
MessagePort::messageAvailable() (patchset #2 id:20001 of
https://codereview.chromium.org/2818073002/ )
> 
> Reason for revert:
> [1] that this CL depends on needs to be reverted:
> https://codereview.chromium.org/2831843002/
> 
> Original issue's description:
> > Messaging: Avoid calling non-thread-safe functions in
MessagePort::messageAvailable()
> > 
> > As implementation comments in MessagePort::messageAvailable(), it can be
called
> > from another thread and should not call non-thread-functions. However, the
> > current implementation wrongly calls ContextLifecycleObserver's
> > GetExecutionContext() that is not a thread-safe function to post a task to
the
> > context thread.
> > 
> > To avoid that, this CL replaces ExecutionContext::PostTask() with
> > WebTaskRunner::PostTask(). The task runner is captured in the ctor of
> > MessagePort called on the context thread.
> > 
> > BUG=694925
> > 
> > Review-Url: https://codereview.chromium.org/2818073002
> > Cr-Commit-Position: refs/heads/master@{#465886}
> > Committed:
https://chromium.googlesource.com/chromium/src/+/829907ac0d3d2ed39d8752d22a6d...
> 
> TBR=tzik@chromium.org,yuryu@chromium.org,kinuko@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=694925
> 
> Review-Url: https://codereview.chromium.org/2833673002
> Cr-Commit-Position: refs/heads/master@{#465890}
> Committed:
https://chromium.googlesource.com/chromium/src/+/7cbe41ace5629ba6b9ec5502803f...

TBR=tzik@chromium.org,yuryu@chromium.org,kinuko@chromium.org
BUG=694925
==========

to

==========
Reland of Messaging: Avoid calling non-thread-safe functions in
MessagePort::messageAvailable() (patchset #1 id:1 of
https://codereview.chromium.org/2833673002/ )

Reason for revert:
The CL that this CL depends on was relanded:
https://codereview.chromium.org/2832763002/

Original issue's description:
> Revert of Messaging: Avoid calling non-thread-safe functions in
MessagePort::messageAvailable() (patchset #2 id:20001 of
https://codereview.chromium.org/2818073002/ )
> 
> Reason for revert:
> [1] that this CL depends on needs to be reverted:
> https://codereview.chromium.org/2831843002/
> 
> Original issue's description:
> > Messaging: Avoid calling non-thread-safe functions in
MessagePort::messageAvailable()
> > 
> > As implementation comments in MessagePort::messageAvailable(), it can be
called
> > from another thread and should not call non-thread-functions. However, the
> > current implementation wrongly calls ContextLifecycleObserver's
> > GetExecutionContext() that is not a thread-safe function to post a task to
the
> > context thread.
> > 
> > To avoid that, this CL replaces ExecutionContext::PostTask() with
> > WebTaskRunner::PostTask(). The task runner is captured in the ctor of
> > MessagePort called on the context thread.
> > 
> > BUG=694925
> > 
> > Review-Url: https://codereview.chromium.org/2818073002
> > Cr-Commit-Position: refs/heads/master@{#465886}
> > Committed:
https://chromium.googlesource.com/chromium/src/+/829907ac0d3d2ed39d8752d22a6d...
> 
> TBR=tzik@chromium.org,yuryu@chromium.org,kinuko@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=694925
> 
> Review-Url: https://codereview.chromium.org/2833673002
> Cr-Commit-Position: refs/heads/master@{#465890}
> Committed:
https://chromium.googlesource.com/chromium/src/+/7cbe41ace5629ba6b9ec5502803f...

TBR=tzik@chromium.org,yuryu@chromium.org,kinuko@chromium.org
BUG=694925

Review-Url: https://codereview.chromium.org/2836473002
Cr-Commit-Position: refs/heads/master@{#466223}
Committed:
https://chromium.googlesource.com/chromium/src/+/90cbb1bd3931fb8ee58c729bbbcd...
==========

[commit-bot: I haz the power, 2017-04-21 02:00:14]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/90cbb1bd3931fb8ee58c729bbbcd...