ApplicationImpl->ShellConnection, mojom::Application->mojom::ShellClient

ApplicationImpl->ShellConnection
mojom::Application->mojom::ShellClient

R=rockot@chromium.org
BUG=

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

[Ben Goodger (Google), 2016-02-08 06:04:45]

Description was changed from

==========
ApplicationImpl->ShellClient
mojom::Application->mojom::ShellClient

R=rockot@chromium.org
BUG=
==========

to

==========
ApplicationImpl->ShellClient
mojom::Application->mojom::ShellClient

R=rockot@chromium.org
BUG=
==========

[Ben Goodger (Google), 2016-02-08 06:16:41]

The CQ bit was checked by ben@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-08 06:16:53]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1675153002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1675153002/40001

[commit-bot: I haz the power, 2016-02-08 06:20:52]

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

[commit-bot: I haz the power, 2016-02-08 06:20:54]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_compile_dbg_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  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-02-08 06:30:05]

The CQ bit was checked by ben@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-08 06:30:47]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1675153002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1675153002/60001

[Ken Rockot(use gerrit already), 2016-02-08 07:22:29]

*confused stare*

oh you mean ApplicationImpl->ShellConnection

lgtm :)

[commit-bot: I haz the power, 2016-02-08 07:33:23]

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

[commit-bot: I haz the power, 2016-02-08 07:33:24]

Dry run: Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[Ben Goodger (Google), 2016-02-08 15:31:36]

Description was changed from

==========
ApplicationImpl->ShellClient
mojom::Application->mojom::ShellClient

R=rockot@chromium.org
BUG=
==========

to

==========
ApplicationImpl->ShellConnection
mojom::Application->mojom::ShellClient

R=rockot@chromium.org
BUG=
==========

[Ben Goodger (Google), 2016-02-08 15:32:05]

On 2016/02/08 07:22:29, Ken Rockot wrote:
> *confused stare*
> 
> oh you mean ApplicationImpl->ShellConnection
> 
> lgtm :)

Er... yes.. update description.

[Ben Goodger (Google), 2016-02-08 15:34:21]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-02-08 15:34:22]

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

[commit-bot: I haz the power, 2016-02-08 15:35:36]

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

[commit-bot: I haz the power, 2016-02-08 15:47:14]

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

[commit-bot: I haz the power, 2016-02-08 15:47:14]

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

[Ben Goodger (Google), 2016-02-08 18:59:02]

ben@chromium.org changed reviewers:
+ jschuh@chromium.org

[Ben Goodger (Google), 2016-02-08 18:59:26]

jschuh: process_control.mojom (renamed a parameter)

[jschuh, 2016-02-08 19:03:30]

Rubberstamp lgtm for ipc: simple rename

[Ben Goodger (Google), 2016-02-08 19:12:52]

The CQ bit was checked by ben@chromium.org

[commit-bot: I haz the power, 2016-02-08 19:15:18]

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

[commit-bot: I haz the power, 2016-02-08 19:42:07]

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

[commit-bot: I haz the power, 2016-02-08 19:42:09]

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

[Ben Goodger (Google), 2016-02-08 22:55:14]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-02-08 22:55:15]

The patchset sent to the CQ was uploaded after l-g-t-m from rockot@chromium.org,
jschuh@chromium.org
Link to the patchset: https://codereview.chromium.org/1675153002/#ps100001
(title: ".")

[commit-bot: I haz the power, 2016-02-08 23:04:33]

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

[commit-bot: I haz the power, 2016-02-09 01:43:20]

Description was changed from

==========
ApplicationImpl->ShellConnection
mojom::Application->mojom::ShellClient

R=rockot@chromium.org
BUG=
==========

to

==========
ApplicationImpl->ShellConnection
mojom::Application->mojom::ShellClient

R=rockot@chromium.org
BUG=
==========

[commit-bot: I haz the power, 2016-02-09 01:43:21]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-02-09 01:44:51]

Description was changed from

==========
ApplicationImpl->ShellConnection
mojom::Application->mojom::ShellClient

R=rockot@chromium.org
BUG=
==========

to

==========
ApplicationImpl->ShellConnection
mojom::Application->mojom::ShellClient

R=rockot@chromium.org
BUG=

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

[commit-bot: I haz the power, 2016-02-09 01:44:53]

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