Upstream version 11.39.266.0

[platform/framework/web/crosswalk.git] / src / mojo / shell / in_process_dynamic_service_runner.cc

diff --git a/src/mojo/shell/in_process_dynamic_service_runner.cc b/src/mojo/shell/in_process_dynamic_service_runner.cc
index 8b38239..c98ef0f 100644 (file)
--- a/src/mojo/shell/in_process_dynamic_service_runner.cc
+++ b/src/mojo/shell/in_process_dynamic_service_runner.cc
@@ -41,8 +41,7 @@ bool SetThunks(Thunks (*make_thunks)(),
 }
 
 InProcessDynamicServiceRunner::InProcessDynamicServiceRunner(
-    Context* context)
-    : keep_alive_(context) {
+    Context* context) {
 }
 
 InProcessDynamicServiceRunner::~InProcessDynamicServiceRunner() {
@@ -113,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
       }
 
@@ -135,7 +135,7 @@ void InProcessDynamicServiceRunner::Run() {
     MojoMainFunction main_function = reinterpret_cast<MojoMainFunction>(
         app_library_.GetFunctionPointer("MojoMain"));
     if (!main_function) {
-      LOG(ERROR) << "Entrypoint MojoMain not found";
+      LOG(ERROR) << "Entrypoint MojoMain not found: " << app_path_.value();
       break;
     }
 
@@ -145,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