X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=blobdiff_plain;f=src%2Fhs-clientmanager.cpp;h=66fd699f49e986fd0040dec6cc2a266d8df67eac;hb=refs%2Ftags%2Fkoi_11.0.1;hp=9b62808d7b7ff37b9d4344dd905650cfb2ae1ebc;hpb=89edd734c0a347dafceba1cef1ac835e0fe61e1a;p=apps%2Fagl-service-homescreen.git

diff --git a/src/hs-clientmanager.cpp b/src/hs-clientmanager.cpp
index 9b62808..66fd699 100644
--- a/src/hs-clientmanager.cpp
+++ b/src/hs-clientmanager.cpp
@@ -17,6 +17,8 @@
 
 #include <cstring>
 #include <algorithm>
+#include <cassert>
+#include "hs-proxy.h"
 #include "hs-clientmanager.h"
 
 static const char _homescreen[] = "homescreen";
@@ -94,7 +96,10 @@ HS_ClientCtxt* HS_ClientManager::createClientCtxt(afb_req_t req, std::string app
         ctxt = new HS_ClientCtxt(appid);
         afb_req_session_set_LOA(req, 1);
         afb_req_context_set(req, ctxt, cbRemoveClientCtxt);
+
+	appid2ctxt[appid] = ctxt;
     }
+
     return ctxt;
 }
 
@@ -155,6 +160,46 @@ void HS_ClientManager::removeClientCtxt(void *data)
     appid2ctxt.erase(ctxt->id);
 }
 
+static int
+is_application_running(afb_req_t request, std::string id)
+{
+    bool app_still_running = false;
+    struct json_object *jobj = nullptr;
+
+    HS_AfmMainProxy afm_proxy;
+
+    // note this is sync, so this might block if afm-system-daemon is down
+    afm_proxy.ps(request->api, &jobj);
+
+    if (jobj) {
+	    size_t len = json_object_array_length(jobj);
+	    for (size_t i = 0; i < len; i++) {
+		struct json_object *aid;
+		struct json_object *item =
+		    json_object_array_get_idx(jobj, i);
+
+		bool isFound = json_object_object_get_ex(item, "id", &aid);
+		if (isFound) {
+		    const char *str_appid = json_object_get_string(aid);
+		    if (strcmp(str_appid, id.c_str()) == 0) {
+			app_still_running = true;
+			break;
+		    }
+		}
+	    }
+    }
+
+    if (!app_still_running) {
+        // we don't remove it from the context list as we're haven't really subscribed,
+        // and we just need to remove it from client_list, which happens here. We also
+	// return AFB_REQ_NOT_STARTED_APPLICATION which will attempt to start it (again).
+        HS_ClientManager::instance()->removeClient(id);
+        return AFB_REQ_NOT_STARTED_APPLICATION;
+    }
+
+    return 0;
+}
+
 /**
  * handle homescreen request
  *
@@ -181,12 +226,26 @@ int HS_ClientManager::handleRequest(afb_req_t request, const char *verb, const c
     else {
         std::string id(appid);
         auto ip = client_list.find(id);
-        if(ip != client_list.end()) {
+	if(ip != client_list.end()) {
+	    // for showWindow verb we need to verify if the app is (still)
+	    // running, and return the appropriate value to attempt to start it
+	    // again. This 'problem' is avoided if the application itself
+	    // subscribes and with that process, to install a callback that
+	    // automatically removes the application from client_list.
+	    // That is exactly how "subscribe" verb is handled below.
+            if (strcasecmp(verb, "showWindow") == 0) {
+                ret = is_application_running(request, id);
+                if (ret == AFB_REQ_NOT_STARTED_APPLICATION) {
+                    AFB_INFO("%s is not running. Will attempt to start it", appid);
+                    return ret;
+                }
+            }
+            AFB_INFO("%s found to be running. Forwarding request to the client", appid);
             ret = ip->second->handleRequest(request, verb);
         }
         else {
             if(!strcasecmp(verb, "subscribe")) {
-                appid2ctxt[id] = createClientCtxt(request, id);
+                createClientCtxt(request, id);
                 HS_Client* client = addClient(request, id);
                 ret = client->handleRequest(request, "subscribe");
             }