Use unique_ptr

[apps/agl-service-windowmanager.git] / src / applist.cpp

diff --git a/src/applist.cpp b/src/applist.cpp
index 2575849..4e31c03 100644 (file)
--- a/src/applist.cpp
+++ b/src/applist.cpp
@@ -32,8 +32,6 @@ AppList::AppList()
 {
 }
 
-AppList::~AppList() {}
-
 void AppList::addClient(const string &appid, const string &role)
 {
     shared_ptr<WMClient> client = std::make_shared<WMClient>(appid, role);
@@ -93,7 +91,40 @@ unsigned AppList::currentRequestNumber() const
     return this->current_req;
 }
 
-// Is this function necessary ?
+WMError AppList::lookUpFloatingSurface(unsigned pid, unsigned *surface)
+{
+    WMError ret = WMError::NO_ENTRY;
+
+    for (auto itr = this->floating_surfaces.begin(); itr != this->floating_surfaces.end(); ++itr)
+    {
+        if(pid == itr->pid){
+            *surface = itr->surface_id;
+            itr = this->floating_surfaces.erase(itr);
+            ret = WMError::SUCCESS;
+            HMI_DEBUG("wm", "Erase surface %d", *surface);
+            break;
+        }
+    }
+    return ret;
+}
+
+WMError AppList::lookUpFloatingSurface(const std::string &appid, unsigned *surface)
+{
+    return WMError::SUCCESS;
+}
+
+WMError AppList::appendRole(const std::string &id, const std::string &role, unsigned surface)
+{
+    WMError wm_err = WMError::NO_ENTRY;
+    if (this->contains(id))
+    {
+        auto x = this->lookUpClient(id);
+        x->addSurface(role, surface);
+        wm_err = WMError::SUCCESS;
+    }
+    return wm_err;
+}
+
 unsigned AppList::getRequestNumber(const string &appid) const
 {
     for (const auto &x : this->req_list)
@@ -122,11 +153,6 @@ unsigned AppList::addAllocateRequest(WMRequest req)
     return req.req_num; // return 1; if you test time_expire
 }
 
-bool AppList::requestFinished() const
-{
-    return this->req_list.empty();
-}
-
 struct WMTrigger AppList::getRequest(unsigned req_num)
 {
     for (const auto &x : this->req_list)
@@ -149,9 +175,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 : this->req_list)
     {
         if (req_num != x.req_num)
@@ -159,7 +185,7 @@ bool AppList::setAction(unsigned req_num, const struct WMAction &action)
             continue;
         }
         x.sync_draw_req.push_back(action);
-        result = true;
+        result = WMError::SUCCESS;
         break;
     }
 
@@ -173,9 +199,9 @@ bool AppList::setAction(unsigned req_num, const struct WMAction &action)
  * otherwise (visible is false) app should be invisible. Then enddraw_finished param is set to true.
  * This function doesn't support actions for focus yet.
  */
-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.req_num)
@@ -186,7 +212,7 @@ bool AppList::setAction(unsigned req_num, const string &appid, const string &rol
         WMAction action{appid, role, area, visible, edraw_f};
 
         x.sync_draw_req.push_back(action);
-        result = true;
+        result = WMError::SUCCESS;
         break;
     }
     return result;