Eliminate ShellClientFactoryConnection & just have the ApplicationManager do it.

Eliminate ShellClientFactoryConnection & just have the ApplicationManager do it.

R=sky@chromium.org
BUG=

Committed: https://crrev.com/1a4170931b004d98c0e910d74be48bc8e78eed04
Cr-Commit-Position: refs/heads/master@{#376192}

[sky, 2016-02-18 03:30:59]

LGTM

[Ben Goodger (Google), 2016-02-18 03:49:10]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-02-18 03:49:10]

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/1706063002/#ps40001
(title: ".")

[commit-bot: I haz the power, 2016-02-18 03:50:25]

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

[commit-bot: I haz the power, 2016-02-18 03:50:27]

This CL has an open dependency (Issue 1705323003 Patch 1). Please resolve the
dependency and try again.

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

The CQ bit was checked by ben@chromium.org

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

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/1706063002/#ps80001
(title: ".")

[commit-bot: I haz the power, 2016-02-18 15:00:36]

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

[commit-bot: I haz the power, 2016-02-18 15:14:03]

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

[commit-bot: I haz the power, 2016-02-18 15:14:05]

Try jobs failed on following builders:
  cast_shell_android on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  chromeos_x86-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)

[Ben Goodger (Google), 2016-02-18 15:17:05]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-02-18 15:17:06]

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/1706063002/#ps100001
(title: ".")

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

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

[commit-bot: I haz the power, 2016-02-18 16:12:04]

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

[commit-bot: I haz the power, 2016-02-18 16:12:05]

Try jobs failed on following builders:
  android_chromium_gn_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  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-02-18 16:21:34]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-02-18 16:21:34]

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/1706063002/#ps120001
(title: ".")

[commit-bot: I haz the power, 2016-02-18 16:22:06]

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

[commit-bot: I haz the power, 2016-02-18 17:29:11]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2016-02-18 17:31:32]

Description was changed from

==========
Eliminate ShellClientFactoryConnection & just have the ApplicationManager do it.

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

to

==========
Eliminate ShellClientFactoryConnection & just have the ApplicationManager do it.

R=sky@chromium.org
BUG=

Committed: https://crrev.com/1a4170931b004d98c0e910d74be48bc8e78eed04
Cr-Commit-Position: refs/heads/master@{#376192}
==========

[commit-bot: I haz the power, 2016-02-18 17:31:34]

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