mojo: Add async waiter for python bindings

mojo: Add async waiter for python bindings

BUG=415486
R=sdefresne@chromium.org,cmasone@chromium.org

Committed: https://crrev.com/992eab604392410cf364186724380fb7cd235568
Cr-Commit-Position: refs/heads/master@{#295480}

[sdefresne, 2014-09-18 08:37:01]

https://codereview.chromium.org/573253002/diff/1/mojo/public/python/src/pytho...
File mojo/public/python/src/python_system_helper.cc (right):

https://codereview.chromium.org/573253002/diff/1/mojo/public/python/src/pytho...
mojo/public/python/src/python_system_helper.cc:98:
internal::SharedPtr<mojo::Callback<void(MojoResult)> > self =
Why don't you DCHECK that wait_id_ is in callbacks_?

https://codereview.chromium.org/573253002/diff/1/mojo/public/python/src/pytho...
mojo/public/python/src/python_system_helper.cc:103: PyObject* empty_tuple =
Py_BuildValue("(i)", mojo_result);
nit: empty_tuple is not empty, maybe rename to args_tuple?

[qsr, 2014-09-18 08:56:11]

https://codereview.chromium.org/573253002/diff/1/mojo/public/python/src/pytho...
File mojo/public/python/src/python_system_helper.cc (right):

https://codereview.chromium.org/573253002/diff/1/mojo/public/python/src/pytho...
mojo/public/python/src/python_system_helper.cc:98:
internal::SharedPtr<mojo::Callback<void(MojoResult)> > self =
On 2014/09/18 08:37:01, sdefresne wrote:
> Why don't you DCHECK that wait_id_ is in callbacks_?

Done.

https://codereview.chromium.org/573253002/diff/1/mojo/public/python/src/pytho...
mojo/public/python/src/python_system_helper.cc:103: PyObject* empty_tuple =
Py_BuildValue("(i)", mojo_result);
On 2014/09/18 08:37:00, sdefresne wrote:
> nit: empty_tuple is not empty, maybe rename to args_tuple?

Done.

[sdefresne, 2014-09-18 09:14:29]

lgtm

[qsr, 2014-09-18 12:21:14]

The CQ bit was checked by qsr@chromium.org

[commit-bot: I haz the power, 2014-09-18 12:22:14]

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

[commit-bot: I haz the power, 2014-09-18 12:32:18]

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

[commit-bot: I haz the power, 2014-09-18 12:32:19]

Try jobs failed on following builders:
  linux_chromium_gn_dbg on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[qsr, 2014-09-18 14:08:41]

The CQ bit was checked by qsr@chromium.org

[commit-bot: I haz the power, 2014-09-18 14:09:58]

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

[commit-bot: I haz the power, 2014-09-18 14:20:42]

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

[commit-bot: I haz the power, 2014-09-18 14:20:43]

Try jobs failed on following builders:
  linux_chromium_gn_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[qsr, 2014-09-18 15:22:13]

The CQ bit was checked by qsr@chromium.org

[commit-bot: I haz the power, 2014-09-18 15:23:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/573253002/40001

[commit-bot: I haz the power, 2014-09-18 16:24:01]

Committed patchset #3 (id:40001) as 490a0d19f8fa851d2828b1c0f6276f76f9b1c8e6

[commit-bot: I haz the power, 2014-09-18 16:24:42]

Patchset 3 (id:??) landed as
https://crrev.com/992eab604392410cf364186724380fb7cd235568
Cr-Commit-Position: refs/heads/master@{#295480}