Second 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 25431a845f2de569c42d743e84cb6214e8982d1e..c98ef0f8fd3d6ed6a35febfe468377651bedf595 100644
--- a/mojo/shell/in_process_dynamic_service_runner.cc
+++ b/mojo/shell/in_process_dynamic_service_runner.cc
@@ -112,8 +112,9 @@ void InProcessDynamicServiceRunner::Run() {
 #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 library has MojoSetGLES2ControlThunks, but "
-                        "doesn't have MojoSetGLES2ImplThunks.";
+        LOG(WARNING) << app_path_.value()
+                     << " has MojoSetGLES2ControlThunks, "
+                        "but doesn't have MojoSetGLES2ImplThunks.";
 #endif
       }
 
@@ -144,9 +145,8 @@ void InProcessDynamicServiceRunner::Run() {
       LOG(ERROR) << "MojoMain returned an error: " << result;
   } while (false);
 
-  bool success = app_completed_callback_runner_.Run();
+  app_completed_callback_runner_.Run();
   app_completed_callback_runner_.Reset();
-  LOG_IF(ERROR, !success) << "Failed post run app_completed_callback";
 }
 
 }  // namespace shell