Revert of [mojo-bindings] Use Watch API instead of MessagePumpMojo

Revert of [mojo-bindings] Use Watch API instead of MessagePumpMojo (patchset #12 id:210001 of https://codereview.chromium.org/1759783003/ )

Reason for revert:
one more time. mojo_app_tests flake

Original issue's description:
> [mojo-bindings] Use Watch API instead of MessagePumpMojo
> 
> This removes the C++ bindings dependency on MessagePumpMojo,
> consuming the new Watch API instead.
> 
> For convenience a new mojo::Watcher is added to the public
> Mojo C++ API library, and this is used by Connector.
> 
> BUG=590495
> R=yzshen@chromium.org
> TBR=blundell@chromium.org for rename affecting components/message_port.gypi
> TBR=jam@chromium.org - added a missing header to new url tests
> 
> Committed: https://crrev.com/d06373e7cd8b4ad725ed5c64c958f2de13585add
> Cr-Commit-Position: refs/heads/master@{#379402}
> 
> Committed: https://crrev.com/2cdb2f677c68f06b3bc54cd40f000ab961a09bbb
> Cr-Commit-Position: refs/heads/master@{#379463}

TBR=jam@chromium.org,yzshen@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=590495
Committed: https://chromium.googlesource.com/chromium/src/+/554355993ee16fa592a450f2dd00729b69988dc7

[Ken Rockot(use gerrit already), 2016-03-06 17:23:53]

Created Revert of [mojo-bindings] Use Watch API instead of MessagePumpMojo

[commit-bot: I haz the power, 2016-03-06 17:23:57]

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

[commit-bot: I haz the power, 2016-03-06 17:31:08]

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

[commit-bot: I haz the power, 2016-03-06 17:32:25]

Description was changed from

==========
Revert of [mojo-bindings] Use Watch API instead of MessagePumpMojo (patchset #12
id:210001 of https://codereview.chromium.org/1759783003/ )

Reason for revert:
one more time. mojo_app_tests flake

Original issue's description:
> [mojo-bindings] Use Watch API instead of MessagePumpMojo
> 
> This removes the C++ bindings dependency on MessagePumpMojo,
> consuming the new Watch API instead.
> 
> For convenience a new mojo::Watcher is added to the public
> Mojo C++ API library, and this is used by Connector.
> 
> BUG=590495
> R=yzshen@chromium.org
> TBR=blundell@chromium.org for rename affecting components/message_port.gypi
> TBR=jam@chromium.org - added a missing header to new url tests
> 
> Committed: https://crrev.com/d06373e7cd8b4ad725ed5c64c958f2de13585add
> Cr-Commit-Position: refs/heads/master@{#379402}
> 
> Committed: https://crrev.com/2cdb2f677c68f06b3bc54cd40f000ab961a09bbb
> Cr-Commit-Position: refs/heads/master@{#379463}

TBR=jam@chromium.org,yzshen@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=590495
==========

to

==========
Revert of [mojo-bindings] Use Watch API instead of MessagePumpMojo (patchset #12
id:210001 of https://codereview.chromium.org/1759783003/ )

Reason for revert:
one more time. mojo_app_tests flake

Original issue's description:
> [mojo-bindings] Use Watch API instead of MessagePumpMojo
>
> This removes the C++ bindings dependency on MessagePumpMojo,
> consuming the new Watch API instead.
>
> For convenience a new mojo::Watcher is added to the public
> Mojo C++ API library, and this is used by Connector.
>
> BUG=590495
> R=yzshen@chromium.org
> TBR=blundell@chromium.org for rename affecting components/message_port.gypi
> TBR=jam@chromium.org - added a missing header to new url tests
>
> Committed: https://crrev.com/d06373e7cd8b4ad725ed5c64c958f2de13585add
> Cr-Commit-Position: refs/heads/master@{#379402}
>
> Committed: https://crrev.com/2cdb2f677c68f06b3bc54cd40f000ab961a09bbb
> Cr-Commit-Position: refs/heads/master@{#379463}

TBR=jam@chromium.org,yzshen@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=590495

Committed: https://crrev.com/554355993ee16fa592a450f2dd00729b69988dc7
Cr-Commit-Position: refs/heads/master@{#379492}
==========

[commit-bot: I haz the power, 2016-03-06 17:32:26]

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

[Ken Rockot(use gerrit already), 2016-03-06 17:33:04]

Description was changed from

==========
Revert of [mojo-bindings] Use Watch API instead of MessagePumpMojo (patchset #12
id:210001 of https://codereview.chromium.org/1759783003/ )

Reason for revert:
one more time. mojo_app_tests flake

Original issue's description:
> [mojo-bindings] Use Watch API instead of MessagePumpMojo
>
> This removes the C++ bindings dependency on MessagePumpMojo,
> consuming the new Watch API instead.
>
> For convenience a new mojo::Watcher is added to the public
> Mojo C++ API library, and this is used by Connector.
>
> BUG=590495
> R=yzshen@chromium.org
> TBR=blundell@chromium.org for rename affecting components/message_port.gypi
> TBR=jam@chromium.org - added a missing header to new url tests
>
> Committed: https://crrev.com/d06373e7cd8b4ad725ed5c64c958f2de13585add
> Cr-Commit-Position: refs/heads/master@{#379402}
>
> Committed: https://crrev.com/2cdb2f677c68f06b3bc54cd40f000ab961a09bbb
> Cr-Commit-Position: refs/heads/master@{#379463}

TBR=jam@chromium.org,yzshen@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=590495

Committed: https://crrev.com/554355993ee16fa592a450f2dd00729b69988dc7
Cr-Commit-Position: refs/heads/master@{#379492}
==========

to

==========
Revert of [mojo-bindings] Use Watch API instead of MessagePumpMojo (patchset #12
id:210001 of https://codereview.chromium.org/1759783003/ )

Reason for revert:
one more time. mojo_app_tests flake

Original issue's description:
> [mojo-bindings] Use Watch API instead of MessagePumpMojo
> 
> This removes the C++ bindings dependency on MessagePumpMojo,
> consuming the new Watch API instead.
> 
> For convenience a new mojo::Watcher is added to the public
> Mojo C++ API library, and this is used by Connector.
> 
> BUG=590495
> R=yzshen@chromium.org
> TBR=blundell@chromium.org for rename affecting components/message_port.gypi
> TBR=jam@chromium.org - added a missing header to new url tests
> 
> Committed: https://crrev.com/d06373e7cd8b4ad725ed5c64c958f2de13585add
> Cr-Commit-Position: refs/heads/master@{#379402}
> 
> Committed: https://crrev.com/2cdb2f677c68f06b3bc54cd40f000ab961a09bbb
> Cr-Commit-Position: refs/heads/master@{#379463}

TBR=jam@chromium.org,yzshen@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=590495

Committed:
https://chromium.googlesource.com/chromium/src/+/554355993ee16fa592a450f2dd00...
==========

[Ken Rockot(use gerrit already), 2016-03-06 17:33:06]

Committed patchset #1 (id:1) manually as
554355993ee16fa592a450f2dd00729b69988dc7 (presubmit successful).