Eliminate mojo::Shell client lib class

Replace with mojo::Connector (already exists) now that Shell is gone.

Requires:
- Move AppRefCount to a different class, MessageLoopRef, which is optionally used by applications that wish to quit the current message loop started by application runner when the ref drops to zero.
- Changing the signature of Initialize() to take a Connector. This is what most of the change in this CL is.

BUG=

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

[Ben Goodger (Google), 2016-02-25 19:19:56]

Description was changed from

==========
Move AppRefCount to a different class

BUG=
==========

to

==========
Replace with mojo::Connector (already exists) now that Shell is gone.

Requires:
- Move AppRefCount to a different class, MessageLoopRef, which is optionally
used by applications that wish to quit the current message loop started by
application runner when the ref drops to zero.
- Changing the signature of Initialize() to take a Connector. This is what most
of the change in this CL is.


BUG=
==========

[Ben Goodger (Google), 2016-02-25 19:19:56]

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

[Ben Goodger (Google), 2016-02-25 19:20:16]

The interesting parts to look at are in mojo/shell, the rest you can just scan
some representative files.

[sky, 2016-02-25 20:30:24]

LGTM

https://codereview.chromium.org/1725353003/diff/100001/mojo/shell/public/cpp/...
File mojo/shell/public/cpp/message_loop_ref.h (right):

https://codereview.chromium.org/1725353003/diff/100001/mojo/shell/public/cpp/...
mojo/shell/public/cpp/message_loop_ref.h:5: #ifndef
MOJO_SHELL_PUBLIC_CPP_APPLICATION_RUNNER_REFERENCE_H_
nit: guards are wrong.

[Ben Goodger (Google), 2016-02-26 00:26:55]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-02-26 00:26:56]

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

[commit-bot: I haz the power, 2016-02-26 00:33:41]

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

[Ben Goodger (Google), 2016-02-26 00:49:35]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-02-26 00:49:36]

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

[commit-bot: I haz the power, 2016-02-26 00:53:17]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  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...)
  android_clang_dbg_recipe on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  android_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  cast_shell_android on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)
  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...)
  win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)

[commit-bot: I haz the power, 2016-02-26 00:54:25]

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

[commit-bot: I haz the power, 2016-02-26 01:24:19]

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

[commit-bot: I haz the power, 2016-02-26 01:24:20]

Try jobs failed on following builders:
  linux_chromium_clobber_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  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-02-26 02:31:36]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-02-26 02:31:37]

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

[commit-bot: I haz the power, 2016-02-26 02:33:20]

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

[commit-bot: I haz the power, 2016-02-26 03:01:54]

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

[commit-bot: I haz the power, 2016-02-26 03:01:56]

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

[Ben Goodger (Google), 2016-02-26 03:05:04]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-02-26 03:05:05]

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

[commit-bot: I haz the power, 2016-02-26 03:09:13]

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

[commit-bot: I haz the power, 2016-02-26 03:46:58]

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

[commit-bot: I haz the power, 2016-02-26 03:47:00]

Try jobs failed on following builders:
  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-02-26 05:59:54]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-02-26 05:59:55]

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

[commit-bot: I haz the power, 2016-02-26 06:00:47]

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

[commit-bot: I haz the power, 2016-02-26 06:05:43]

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

[commit-bot: I haz the power, 2016-02-26 06:05:44]

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...)
  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-26 06:09:57]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-02-26 06:09:58]

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

[commit-bot: I haz the power, 2016-02-26 06:10:30]

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

[commit-bot: I haz the power, 2016-02-26 07:30:33]

Description was changed from

==========
Replace with mojo::Connector (already exists) now that Shell is gone.

Requires:
- Move AppRefCount to a different class, MessageLoopRef, which is optionally
used by applications that wish to quit the current message loop started by
application runner when the ref drops to zero.
- Changing the signature of Initialize() to take a Connector. This is what most
of the change in this CL is.


BUG=
==========

to

==========
Replace with mojo::Connector (already exists) now that Shell is gone.

Requires:
- Move AppRefCount to a different class, MessageLoopRef, which is optionally
used by applications that wish to quit the current message loop started by
application runner when the ref drops to zero.
- Changing the signature of Initialize() to take a Connector. This is what most
of the change in this CL is.


BUG=
==========

[commit-bot: I haz the power, 2016-02-26 07:30:35]

Committed patchset #12 (id:220001)

[commit-bot: I haz the power, 2016-02-26 07:31:47]

Description was changed from

==========
Replace with mojo::Connector (already exists) now that Shell is gone.

Requires:
- Move AppRefCount to a different class, MessageLoopRef, which is optionally
used by applications that wish to quit the current message loop started by
application runner when the ref drops to zero.
- Changing the signature of Initialize() to take a Connector. This is what most
of the change in this CL is.


BUG=
==========

to

==========
Replace with mojo::Connector (already exists) now that Shell is gone.

Requires:
- Move AppRefCount to a different class, MessageLoopRef, which is optionally
used by applications that wish to quit the current message loop started by
application runner when the ref drops to zero.
- Changing the signature of Initialize() to take a Connector. This is what most
of the change in this CL is.

BUG=

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

[commit-bot: I haz the power, 2016-02-26 07:31:48]

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