X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=blobdiff_plain;f=src%2Fapplist.cpp;h=6b608d0ace42313ad5e747ccec1f38ac594aa79b;hb=1e55798ce66d927f649284441202bc6ae1df328b;hp=8c13f5fee49870588fe11c6e52ef7fde3a06750c;hpb=2e78bcd71f27bc18033b24d45bc3a67b5207862b;p=apps%2Fagl-service-windowmanager.git

diff --git a/src/applist.cpp b/src/applist.cpp
index 8c13f5f..6b608d0 100644
--- a/src/applist.cpp
+++ b/src/applist.cpp
@@ -54,10 +54,10 @@ void AppList::removeClient(const string &appid)
     client_list.erase(appid);
 }
 
-bool AppList::contains(const string &appid)
+WMError AppList::contains(const string &appid)
 {
     auto result = client_list.find(appid);
-    return (client_list.end() != result) ? true : false;
+    return (client_list.end() != result) ? WMError::SUCCESS : WMError::NOT_REGISTERED;
 }
 
 void AppList::removeSurface(unsigned surface_id){
@@ -150,9 +150,9 @@ const vector<struct WMAction> &AppList::getActions(unsigned req_num)
     }
 }
 
-bool AppList::setAction(unsigned req_num, const struct WMAction &action)
+WMError AppList::setAction(unsigned req_num, const struct WMAction &action)
 {
-    bool result = false;
+    WMError result = WMError::FAIL;
     for (auto &x : req_list)
     {
         if (req_num != x.seq_num)
@@ -160,16 +160,16 @@ bool AppList::setAction(unsigned req_num, const struct WMAction &action)
             continue;
         }
         x.sync_draw_req.push_back(action);
-        result = true;
+        result = WMError::SUCCESS;
         break;
     }
 
     return result;
 }
 
-bool AppList::setAction(unsigned req_num, const string &appid, const string &role, const string &area, bool visible)
+WMError AppList::setAction(unsigned req_num, const string &appid, const string &role, const string &area, bool visible)
 {
-    bool result = false;
+    WMError result = WMError::NOT_REGISTERED;
     for (auto &x : req_list)
     {
         if (req_num != x.seq_num)
@@ -179,7 +179,7 @@ bool AppList::setAction(unsigned req_num, const string &appid, const string &rol
         WMAction action{appid, role, area, visible, false};
 
         x.sync_draw_req.push_back(action);
-        result = true;
+        result = WMError::SUCCESS;
         break;
     }
     return result;