mojo: terminate apps if the shell goes away

Each application <> shell connection is represented by a ShellImpl instance
on the shell side. This CL makes the Application watch its ShellPtr for pipe
errors so it can Quit() itself if the shell goes away (shell loop destroyed).

mojo_shell_tests starts using a new method to terminate all shell connections
in this CL, and waits afterward until KeepAlive quits the loop signifying that
all apps are gone. This is done prior to destroying the shell MessageLoop so
that the test ensures apps have all died before the next test. In the future
it would be a test failure if this wasn't a no-op, but we have things in the
shell that don't quit themselves right now.

The shell itself won't wait around for apps to die (as of this CL), but
~MessageLoop will still send the message to Application sides that will quit.

** NOTE ** This does not require an app build target to explicitly add magic
*.cc files to their sources.  It requires selecting an appropriate
mojo_application_{chromium, standalone} library akin to mojo_environment_*.
We can possibly combine these two in the future.

BUG=394477

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=287680

Review URL: https://codereview.chromium.org/394903005

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@287894 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 12 insertions, 12 deletions

diff --git a/mojo/mojo_services.gypi b/mojo/mojo_services.gypi
index de07a71..3e07f8e 100644
--- a/mojo/mojo_services.gypi
+++ b/mojo/mojo_services.gypi
@@ -32,7 +32,7 @@
         'mojo_base.gyp:mojo_cpp_bindings',
         'mojo_base.gyp:mojo_environment_chromium',
         'mojo_base.gyp:mojo_utility',
-        'mojo_base.gyp:mojo_application',
+        'mojo_base.gyp:mojo_application_chromium',
         'mojo_navigation_bindings',
         'mojo_network_bindings',
         'mojo_launcher_bindings',
@@ -303,7 +303,7 @@
         '../ui/gfx/gfx.gyp:gfx_geometry',
         'mojo_base.gyp:mojo_common_lib',
         'mojo_base.gyp:mojo_environment_chromium',
-        'mojo_base.gyp:mojo_application',
+        'mojo_base.gyp:mojo_application_chromium',
         'mojo_geometry_bindings',
         'mojo_geometry_lib',
         'mojo_gles2_service',
@@ -392,7 +392,7 @@
         '../url/url.gyp:url_lib',
         'mojo_base.gyp:mojo_common_lib',
         'mojo_base.gyp:mojo_environment_chromium',
-        'mojo_base.gyp:mojo_application',
+        'mojo_base.gyp:mojo_application_chromium',
         'mojo_network_bindings',
       ],
       'export_dependent_settings': [
@@ -434,7 +434,7 @@
         '../ui/gfx/gfx.gyp:gfx_geometry',
         'mojo_base.gyp:mojo_environment_chromium',
         'mojo_base.gyp:mojo_system_impl',
-        'mojo_base.gyp:mojo_application',
+        'mojo_base.gyp:mojo_application_chromium',
         'mojo_cc_support',
         'mojo_geometry_bindings',
         'mojo_geometry_lib',
@@ -483,7 +483,7 @@
         '../url/url.gyp:url_lib',
         'mojo_base.gyp:mojo_cpp_bindings',
         'mojo_base.gyp:mojo_environment_chromium',
-        'mojo_base.gyp:mojo_application',
+        'mojo_base.gyp:mojo_application_chromium',
         'mojo_launcher_bindings',
         'mojo_network_bindings',
         '<(mojo_system_for_loadable_module)',
@@ -521,7 +521,7 @@
         '../ui/events/events.gyp:events',
         '../ui/gfx/gfx.gyp:gfx',
         '../ui/gfx/gfx.gyp:gfx_geometry',
-        'mojo_base.gyp:mojo_application',
+        'mojo_base.gyp:mojo_application_chromium',
         'mojo_base.gyp:mojo_application_bindings',
         'mojo_geometry_bindings',
         'mojo_geometry_lib',
@@ -629,7 +629,7 @@
         '../base/base.gyp:base',
         'mojo_base.gyp:mojo_environment_standalone',
         'mojo_base.gyp:mojo_utility',
-        'mojo_base.gyp:mojo_application',
+        'mojo_base.gyp:mojo_application_standalone',
         'mojo_test_service_bindings',
         '<(mojo_system_for_loadable_module)',
       ],
@@ -653,7 +653,7 @@
         '../base/base.gyp:base',
         'mojo_base.gyp:mojo_environment_standalone',
         'mojo_base.gyp:mojo_utility',
-        'mojo_base.gyp:mojo_application',
+        'mojo_base.gyp:mojo_application_standalone',
         'mojo_test_service_bindings',
         '<(mojo_system_for_loadable_module)',
       ],
@@ -707,7 +707,7 @@
             '../webkit/common/gpu/webkit_gpu.gyp:webkit_gpu',
             'mojo_base.gyp:mojo_common_lib',
             'mojo_base.gyp:mojo_environment_chromium',
-            'mojo_base.gyp:mojo_application',
+            'mojo_base.gyp:mojo_application_chromium',
             'mojo_cc_support',
             'mojo_geometry_bindings',
             'mojo_geometry_lib',
@@ -797,7 +797,7 @@
             '../ui/gl/gl.gyp:gl',
             'mojo_base.gyp:mojo_system_impl',
             'mojo_base.gyp:mojo_environment_chromium',
-            'mojo_base.gyp:mojo_application',
+            'mojo_base.gyp:mojo_application_chromium',
             'mojo_geometry_bindings',
             'mojo_geometry_lib',
             'mojo_input_events_bindings',
@@ -832,7 +832,7 @@
             '../ui/wm/wm.gyp:wm',
             'mojo_base.gyp:mojo_common_lib',
             'mojo_base.gyp:mojo_environment_chromium',
-            'mojo_base.gyp:mojo_application',
+            'mojo_base.gyp:mojo_application_chromium',
             'mojo_aura_support',
             'mojo_core_window_manager_bindings',
             'mojo_view_manager_lib',
@@ -903,7 +903,7 @@
             'mojo_base.gyp:mojo_common_lib',
             'mojo_base.gyp:mojo_environment_chromium',
             'mojo_base.gyp:mojo_system_impl',
-            'mojo_base.gyp:mojo_application',
+            'mojo_base.gyp:mojo_application_chromium',
             'mojo_dbus_service',
             'mojo_echo_bindings',
           ],