Revert of Third attempt to land change to remove NativeViewportService and

mojo/shell/in_process_dynamic_service_runner.cc

Index: mojo/shell/in_process_dynamic_service_runner.cc
diff --git a/mojo/shell/in_process_dynamic_service_runner.cc b/mojo/shell/in_process_dynamic_service_runner.cc
index c98ef0f8fd3d6ed6a35febfe468377651bedf595..25431a845f2de569c42d743e84cb6214e8982d1e 100644
--- a/mojo/shell/in_process_dynamic_service_runner.cc
+++ b/mojo/shell/in_process_dynamic_service_runner.cc
@@ -112,9 +112,8 @@
 #if !defined(COMPONENT_BUILD)
         // Warn on this really weird case: The library requires the GLES2
         // control functions, but doesn't require the GLES2 implementation.
-        LOG(WARNING) << app_path_.value()
-                     << " has MojoSetGLES2ControlThunks, "
-                        "but doesn't have MojoSetGLES2ImplThunks.";
+        LOG(WARNING) << "App library has MojoSetGLES2ControlThunks, but "
+                        "doesn't have MojoSetGLES2ImplThunks.";
 #endif
       }
 
@@ -145,8 +144,9 @@
       LOG(ERROR) << "MojoMain returned an error: " << result;
   } while (false);
 
-  app_completed_callback_runner_.Run();
+  bool success = app_completed_callback_runner_.Run();
   app_completed_callback_runner_.Reset();
+  LOG_IF(ERROR, !success) << "Failed post run app_completed_callback";
 }
 
 }  // namespace shell