Remove deactivate_main_surface API

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

diff --git a/src/wm_client.cpp b/src/wm_client.cpp
index 72517bd..a9ed547 100644 (file)
--- a/src/wm_client.cpp
+++ b/src/wm_client.cpp
@@ -26,10 +26,10 @@ using std::vector;
 namespace wm
 {
 
-const vector<string> kWMEvents = {
+static const vector<string> kWMEvents = {
     // Private event for applications
     "syncDraw", "flushDraw", "visible", "invisible", "active", "inactive", "error"};
-const vector<string> kErrorDescription = {
+static const vector<string> kErrorDescription = {
     "unknown-error"};
 
 static const char kKeyDrawingName[] = "drawing_name";
@@ -84,7 +84,6 @@ unsigned WMClient::surfaceID(const string &role) const
 {
     if (0 == this->role2surface.count(role))
     {
-        HMI_NOTICE("wm", "invalid role %s : appID : %s", role.c_str(), this->id.c_str());
         return INVALID_SURFACE_ID;
     }
     return this->role2surface.at(role);
@@ -107,11 +106,35 @@ unsigned WMClient::layerID() const
     return this->layer;
 }
 
+/**
+ * Set layerID the client belongs to
+ *
+ * This function set layerID the client belongs to.
+ * But this function may not used because the layer should be fixed at constructor.
+ * So this function will be used to change layer by some reasons.
+ *
+ * @param     unsigned[in] layerID
+ * @return    None
+ * @attention WMClient can't have multiple layer
+ */
 void WMClient::registerLayer(unsigned layer)
 {
     this->layer = layer;
 }
 
+/**
+ * Add the pair of role and surface to the client
+ *
+ * This function set the pair of role and surface to the client.
+ * This function is used for the client which has multi surfaces.
+ * If the model and relationship for role and surface(layer)
+ * is changed, this function will be changed
+ * Current Window Manager doesn't use this function.
+ *
+ * @param     string[in] role
+ * @param     unsigned[in] surface
+ * @return    true
+ */
 bool WMClient::addSurface(const string &role, unsigned surface)
 {
     HMI_DEBUG("wm", "Add role %s with surface %d", role.c_str(), surface);