Makes bindings work with nested message loops

Makes bindings work with nested message loops

Dispatching a message may result in a nested message loop. When
dispatching a message we need to start a wait before dispatch, so that
if a nested message loop does result we continue dispatching messages.

BUG=none
TEST=covered by tests.
R=yzshen@chromium.org

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

[yzshen1, 2016-01-09 00:26:30]

LGTM with one comment

Thanks!

https://codereview.chromium.org/1576603002/diff/1/mojo/public/cpp/bindings/li...
File mojo/public/cpp/bindings/lib/connector.cc (right):

https://codereview.chromium.org/1576603002/diff/1/mojo/public/cpp/bindings/li...
mojo/public/cpp/bindings/lib/connector.cc:243: WaitToReadMore();
Could you please add a TODO for evaluating the perf impact of this change?

[sky, 2016-01-11 16:10:35]

https://codereview.chromium.org/1576603002/diff/1/mojo/public/cpp/bindings/li...
File mojo/public/cpp/bindings/lib/connector.cc (right):

https://codereview.chromium.org/1576603002/diff/1/mojo/public/cpp/bindings/li...
mojo/public/cpp/bindings/lib/connector.cc:243: WaitToReadMore();
On 2016/01/09 00:26:30, yzshen1 wrote:
> Could you please add a TODO for evaluating the perf impact of this change?

Done.

[sky, 2016-01-11 16:10:41]

The CQ bit was checked by sky@chromium.org

[sky, 2016-01-11 16:10:41]

The patchset sent to the CQ was uploaded after l-g-t-m from yzshen@chromium.org
Link to the patchset: https://codereview.chromium.org/1576603002/#ps20001
(title: "comment")

[commit-bot: I haz the power, 2016-01-11 16:11:04]

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

[commit-bot: I haz the power, 2016-01-11 17:17:13]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-11 17:18:32]

Description was changed from

==========
Makes bindings work with nested message loops

Dispatching a message may result in a nested message loop. When
dispatching a message we need to start a wait before dispatch, so that
if a nested message loop does result we continue dispatching messages.

BUG=none
TEST=covered by tests.
R=yzshen@chromium.org
==========

to

==========
Makes bindings work with nested message loops

Dispatching a message may result in a nested message loop. When
dispatching a message we need to start a wait before dispatch, so that
if a nested message loop does result we continue dispatching messages.

BUG=none
TEST=covered by tests.
R=yzshen@chromium.org

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

[commit-bot: I haz the power, 2016-01-11 17:18:33]

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