Pass application ids via AcceptConnection & ConnectToApplication callback.

Pass application ids via AcceptConnection & ConnectToApplication callback.

R=sky@chromium.org
BUG=

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

[sky, 2016-01-12 20:57:44]

LGTM

https://codereview.chromium.org/1578473002/diff/140001/mojo/shell/public/inte...
File mojo/shell/public/interfaces/application.mojom (right):

https://codereview.chromium.org/1578473002/diff/140001/mojo/shell/public/inte...
mojo/shell/public/interfaces/application.mojom:26: Initialize(Shell shell,
string url, uint32 id);
nit: document what |id| is.

[Ben Goodger (Google), 2016-01-12 22:31:47]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-01-12 22:31:47]

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

[commit-bot: I haz the power, 2016-01-12 22:32:58]

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

[commit-bot: I haz the power, 2016-01-12 22:50:49]

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

[commit-bot: I haz the power, 2016-01-12 22:50:50]

Try jobs failed on following builders:
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Ben Goodger (Google), 2016-01-12 23:11:53]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-01-12 23:11:53]

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

[commit-bot: I haz the power, 2016-01-12 23:12:54]

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

[commit-bot: I haz the power, 2016-01-12 23:30:41]

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

[commit-bot: I haz the power, 2016-01-12 23:30:42]

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

[Ben Goodger (Google), 2016-01-13 00:57:50]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-01-13 00:57:51]

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

[commit-bot: I haz the power, 2016-01-13 00:58:24]

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

[commit-bot: I haz the power, 2016-01-13 01:23:46]

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

[commit-bot: I haz the power, 2016-01-13 01:23:47]

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

[Ben Goodger (Google), 2016-01-13 04:17:30]

The CQ bit was checked by ben@chromium.org

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

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

[commit-bot: I haz the power, 2016-01-13 04:35:08]

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

[commit-bot: I haz the power, 2016-01-13 04:35:08]

Try jobs failed on following builders:
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)

[Ben Goodger (Google), 2016-01-13 20:24:40]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-01-13 20:24:40]

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

[commit-bot: I haz the power, 2016-01-13 20:25:24]

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

[commit-bot: I haz the power, 2016-01-13 23:39:13]

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

[commit-bot: I haz the power, 2016-01-13 23:39:13]

Try jobs failed on following builders:
  cast_shell_linux on tryserver.chromium.linux (JOB_TIMED_OUT, no build URL)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_TIMED_OUT, no build URL)

[Ben Goodger (Google), 2016-01-26 22:01:34]

The CQ bit was checked by ben@chromium.org

[commit-bot: I haz the power, 2016-01-26 22:03:41]

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

[commit-bot: I haz the power, 2016-01-26 22:08:04]

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

[commit-bot: I haz the power, 2016-01-26 22:08:04]

Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Ben Goodger (Google), 2016-01-27 17:51:43]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-01-27 17:51:43]

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

[commit-bot: I haz the power, 2016-01-27 17:54:50]

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

[commit-bot: I haz the power, 2016-01-27 19:05:18]

Committed patchset #13 (id:240001)

[commit-bot: I haz the power, 2016-01-27 19:06:17]

Description was changed from

==========
Pass application ids via AcceptConnection & ConnectToApplication callback.

R=sky@chromium.org
BUG=
==========

to

==========
Pass application ids via AcceptConnection & ConnectToApplication callback.

R=sky@chromium.org
BUG=

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

[commit-bot: I haz the power, 2016-01-27 19:06:18]

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