From 671b366ab609ccd58456e8debf0a20e46315b3e6 Mon Sep 17 00:00:00 2001 From: Kazumasa Mitsunari Date: Thu, 15 Nov 2018 19:32:39 +0900 Subject: Clean: Remove conversion table from old role Stop compatibility support. Remove conversion table from old role. Bug-AGL: SPEC-1947 Change-Id: Idbb0717920c2f4a0c29bfb4356e1b167ac8bce0b Signed-off-by: Kazumasa Mitsunari --- src/CMakeLists.txt | 1 - src/main.cpp | 3 +- src/window_manager.cpp | 270 +++++-------------------------------------------- src/window_manager.hpp | 4 - 4 files changed, 28 insertions(+), 250 deletions(-) (limited to 'src') diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index 21f287a..d2b16ae 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -112,7 +112,6 @@ add_custom_command(TARGET ${TARGETS_WM} POST_BUILD COMMAND cp -rf ${PROJECT_BINARY_DIR}/src/${TARGETS_WM}.so ${PROJECT_BINARY_DIR}/package/root/lib COMMAND mkdir -p ${PROJECT_BINARY_DIR}/package/root/etc COMMAND cp -f ${CMAKE_SOURCE_DIR}/conf/layers.json ${PROJECT_BINARY_DIR}/package/root/etc - COMMAND cp -f ${CMAKE_SOURCE_DIR}/conf/old_roles.json ${PROJECT_BINARY_DIR}/package/root/etc COMMAND cp -f ${CMAKE_SOURCE_DIR}/conf/areas.json ${PROJECT_BINARY_DIR}/package/root/etc ) diff --git a/src/main.cpp b/src/main.cpp index d0d7930..29d25fb 100644 --- a/src/main.cpp +++ b/src/main.cpp @@ -116,8 +116,7 @@ static void createSecurityContext(afb_req req, const char* appid, const char* ro if (!ctxt) { // Create Security Context at first time - const char *new_role = g_afb_instance->wmgr.convertRoleOldToNew(role); - WMClientCtxt *ctxt = new WMClientCtxt(appid, new_role); + WMClientCtxt *ctxt = new WMClientCtxt(appid, role); HMI_DEBUG("create session for %s", ctxt->name.c_str()); afb_req_session_set_LOA(req, 1); afb_req_context_set(req, ctxt, cbRemoveClientCtxt); diff --git a/src/window_manager.cpp b/src/window_manager.cpp index 7ab5213..cc6f410 100644 --- a/src/window_manager.cpp +++ b/src/window_manager.cpp @@ -124,11 +124,6 @@ int WindowManager::init() return -1; } - // TODO: application requests by old role, - // so create role map (old, new) - // Load old_role.db - this->loadOldRoleDb(); - // Store my context for calling callback from PolicyManager g_context = this; @@ -162,16 +157,13 @@ int WindowManager::init() result WindowManager::api_request_surface(char const *appid, char const *drawing_name) { - // TODO: application requests by old role, - // so convert role old to new - const char *role = this->convertRoleOldToNew(drawing_name); string str_id = appid; - string str_role = role; + string role = drawing_name; unsigned lid = 0; if(!g_app_list.contains(str_id)) { - lid = this->lc->getNewLayerID(str_role); + lid = this->lc->getNewLayerID(role); if (lid == 0) { // register drawing_name as fallback and make it displayed. @@ -184,20 +176,17 @@ result WindowManager::api_request_surface(char const *appid, char const *dr } this->lc->createNewLayer(lid); // add client into the db - g_app_list.addClient(str_id, lid, str_role); + g_app_list.addClient(str_id, lid, role); } // generate surface ID for ivi-shell application - auto rname = this->id_alloc.lookup(str_role); + auto rname = this->id_alloc.lookup(role); if (!rname) { // name does not exist yet, allocate surface id... - auto id = int(this->id_alloc.generate_id(str_role)); + auto id = int(this->id_alloc.generate_id(role)); this->tmp_surface2app[id] = {str_id, lid}; - // Set role map of (new, old) - this->rolenew2old[role] = string(drawing_name); - return Ok(id); } @@ -208,11 +197,8 @@ result WindowManager::api_request_surface(char const *appid, char const *dr char const *WindowManager::api_request_surface(char const *appid, char const *drawing_name, char const *ivi_id) { - // TODO: application requests by old role, - // so convert role old to new - const char *role = this->convertRoleOldToNew(drawing_name); string str_id = appid; - string str_role = role; + string role = drawing_name; unsigned sid = std::stol(ivi_id); HMI_DEBUG("This API(requestSurfaceXDG) is for XDG Application using runXDG"); @@ -231,7 +217,7 @@ char const *WindowManager::api_request_surface(char const *appid, char const *dr if(!g_app_list.contains(str_id)) { - unsigned l_id = this->lc->getNewLayerID(str_role); + unsigned l_id = this->lc->getNewLayerID(role); if (l_id == 0) { // register drawing_name as fallback and make it displayed. @@ -244,10 +230,10 @@ char const *WindowManager::api_request_surface(char const *appid, char const *dr } this->lc->createNewLayer(l_id); // add client into the db - g_app_list.addClient(str_id, l_id, str_role); + g_app_list.addClient(str_id, l_id, role); } - auto rname = this->id_alloc.lookup(str_role); + auto rname = this->id_alloc.lookup(role); if (rname) { @@ -255,26 +241,19 @@ char const *WindowManager::api_request_surface(char const *appid, char const *dr } // register pair drawing_name and ivi_id - this->id_alloc.register_name_id(str_role, sid); + this->id_alloc.register_name_id(role, sid); auto client = g_app_list.lookUpClient(str_id); client->addSurface(sid); - // Set role map of (new, old) - this->rolenew2old[role] = string(drawing_name); - return nullptr; } void WindowManager::api_activate_window(char const *appid, char const *drawing_name, char const *drawing_area, const reply_func &reply) { - // TODO: application requests by old role, - // so convert role old to new - const char *c_role = this->convertRoleOldToNew(drawing_name); - string id = appid; - string role = c_role; + string role = drawing_name; string area = drawing_area; if(!g_app_list.contains(id)) @@ -305,9 +284,8 @@ void WindowManager::api_activate_window(char const *appid, char const *drawing_n return; } - /* - * Do allocate tasks - */ + // Do allocate tasks + ret = this->checkPolicy(req_num); if (ret != WMError::SUCCESS) @@ -322,15 +300,9 @@ void WindowManager::api_activate_window(char const *appid, char const *drawing_n void WindowManager::api_deactivate_window(char const *appid, char const *drawing_name, const reply_func &reply) { - // TODO: application requests by old role, - // so convert role old to new - const char *c_role = this->convertRoleOldToNew(drawing_name); - - /* - * Check Phase - */ + // Check Phase string id = appid; - string role = c_role; + string role = drawing_name; string area = ""; //drawing_area; Task task = Task::TASK_RELEASE; unsigned req_num = 0; @@ -353,9 +325,7 @@ void WindowManager::api_deactivate_window(char const *appid, char const *drawing return; } - /* - * Do allocate tasks - */ + // Do allocate tasks ret = this->checkPolicy(req_num); if (ret != WMError::SUCCESS) @@ -369,12 +339,8 @@ void WindowManager::api_deactivate_window(char const *appid, char const *drawing void WindowManager::api_enddraw(char const *appid, char const *drawing_name) { - // TODO: application requests by old role, - // so convert role old to new - const char *c_role = this->convertRoleOldToNew(drawing_name); - string id = appid; - string role = c_role; + string role = drawing_name; unsigned current_req = g_app_list.currentRequestNumber(); bool result = g_app_list.setEndDrawFinished(current_req, id, role); @@ -437,9 +403,7 @@ result WindowManager::api_get_area_info(char const *drawing_name) { HMI_DEBUG("called"); - // TODO: application requests by old role, - // so convert role old to new - const char *role = this->convertRoleOldToNew(drawing_name); + string role = drawing_name; // Check drawing name, surface/layer id auto const &surface_id = this->id_alloc.lookup(role); @@ -572,7 +536,7 @@ void WindowManager::startTransitionWrapper(vector &actions) if ("" != act.role) { bool found; - auto const &surface_id = this->id_alloc.lookup(act.role.c_str()); + auto const &surface_id = this->id_alloc.lookup(act.role); if(surface_id == nullopt) { goto proc_remove_request; @@ -665,7 +629,7 @@ void WindowManager::emit_syncdraw(const string &role, const string &area) { struct rect rect = this->lc->getAreaSize(area); this->send_event(kListEventName[Event_SyncDraw], - role.c_str(), area.c_str(), rect.x, rect.y, rect.w, rect.h); + role, area, rect.x, rect.y, rect.w, rect.h); } void WindowManager::emit_flushdraw(const string& role) @@ -774,12 +738,7 @@ WMError WindowManager::startTransition(unsigned req_num) if (action.visible == TaskVisible::VISIBLE) { sync_draw_happen = true; - - // TODO: application requests by old role, - // so convert role new to old for emitting event - string old_role = this->rolenew2old[action.role]; - - this->emit_syncdraw(old_role, action.area); + this->emit_syncdraw(action.role, action.area); /* TODO: emit event for app not subscriber if(g_app_list.contains(y.appid)) g_app_list.lookUpClient(y.appid)->emit_syncdraw(y.role, y.area); */ @@ -797,8 +756,7 @@ WMError WindowManager::startTransition(unsigned req_num) for (const auto &x : actions) { this->lc->visibilityChange(x); - string old_role = this->rolenew2old[x.role]; - emit_deactivated(old_role.c_str()); + emit_deactivated(x.role); /* if (g_app_list.contains(x.client->appID())) { auto client = g_app_list.lookUpClient(x.client->appID()); @@ -840,17 +798,15 @@ WMError WindowManager::doEndDraw(unsigned req_num) } ret = this->lc->visibilityChange(act); - // Emit active/deactive event - string old_role = this->rolenew2old[act.role]; if(act.visible == VISIBLE) { - emit_visible(old_role.c_str()); - emit_activated(old_role.c_str()); + emit_visible(act.role); + emit_activated(act.role); } else { - emit_invisible(old_role.c_str()); - emit_deactivated(old_role.c_str()); + emit_invisible(act.role); + emit_deactivated(act.role); } if (ret != WMError::SUCCESS) @@ -870,11 +826,7 @@ WMError WindowManager::doEndDraw(unsigned req_num) { if(act_flush.visible == TaskVisible::VISIBLE) { - // TODO: application requests by old role, - // so convert role new to old for emitting event - string old_role = this->rolenew2old[act_flush.role]; - - this->emit_flushdraw(old_role.c_str()); + this->emit_flushdraw(act_flush.role); } } @@ -968,172 +920,4 @@ void WindowManager::processNextRequest() } } -const char* WindowManager::convertRoleOldToNew(char const *old_role) -{ - const char *new_role = nullptr; - - for (auto const &on : this->roleold2new) - { - std::regex regex = std::regex(on.first); - if (std::regex_match(old_role, regex)) - { - // role is old. So convert to new. - new_role = on.second.c_str(); - break; - } - } - - if (nullptr == new_role) - { - // role is new or fallback. - new_role = old_role; - } - - HMI_DEBUG("old:%s -> new:%s", old_role, new_role); - - return new_role; -} - -int WindowManager::loadOldRoleDb() -{ - // Get afm application installed dir - char const *afm_app_install_dir = getenv("AFM_APP_INSTALL_DIR"); - HMI_DEBUG("afm_app_install_dir:%s", afm_app_install_dir); - - string file_name; - if (!afm_app_install_dir) - { - HMI_ERROR("AFM_APP_INSTALL_DIR is not defined"); - } - else - { - file_name = string(afm_app_install_dir) + string("/etc/old_roles.json"); - } - - // Load old_role.db - json_object* json_obj; - int ret = jh::inputJsonFilie(file_name.c_str(), &json_obj); - if (0 > ret) - { - HMI_ERROR("Could not open old_role.db, so use default old_role information"); - json_obj = json_tokener_parse(kDefaultOldRoleDb); - } - HMI_DEBUG("json_obj dump:%s", json_object_get_string(json_obj)); - - // Perse apps - json_object* json_cfg; - if (!json_object_object_get_ex(json_obj, "old_roles", &json_cfg)) - { - HMI_ERROR("Parse Error!!"); - return -1; - } - - int len = json_object_array_length(json_cfg); - HMI_DEBUG("json_cfg len:%d", len); - HMI_DEBUG("json_cfg dump:%s", json_object_get_string(json_cfg)); - - for (int i=0; iroleold2new[old_role] = string(new_role); - } - - // Check - for(auto itr = this->roleold2new.begin(); - itr != this->roleold2new.end(); ++itr) - { - HMI_DEBUG(">>> role old:%s new:%s", - itr->first.c_str(), itr->second.c_str()); - } - - // Release json_object - json_object_put(json_obj); - - return 0; -} - -const char* WindowManager::kDefaultOldRoleDb = "{ \ - \"old_roles\": [ \ - { \ - \"name\": \"HomeScreen\", \ - \"new\": \"homescreen\" \ - }, \ - { \ - \"name\": \"Music\", \ - \"new\": \"music\" \ - }, \ - { \ - \"name\": \"MediaPlayer\", \ - \"new\": \"music\" \ - }, \ - { \ - \"name\": \"Video\", \ - \"new\": \"video\" \ - }, \ - { \ - \"name\": \"VideoPlayer\", \ - \"new\": \"video\" \ - }, \ - { \ - \"name\": \"WebBrowser\", \ - \"new\": \"browser\" \ - }, \ - { \ - \"name\": \"Radio\", \ - \"new\": \"radio\" \ - }, \ - { \ - \"name\": \"Phone\", \ - \"new\": \"phone\" \ - }, \ - { \ - \"name\": \"Navigation\", \ - \"new\": \"map\" \ - }, \ - { \ - \"name\": \"HVAC\", \ - \"new\": \"hvac\" \ - }, \ - { \ - \"name\": \"Settings\", \ - \"new\": \"settings\" \ - }, \ - { \ - \"name\": \"Dashboard\", \ - \"new\": \"dashboard\" \ - }, \ - { \ - \"name\": \"POI\", \ - \"new\": \"poi\" \ - }, \ - { \ - \"name\": \"Mixer\", \ - \"new\": \"mixer\" \ - }, \ - { \ - \"name\": \"Restriction\", \ - \"new\": \"restriction\" \ - }, \ - { \ - \"name\": \"^OnScreen.*\", \ - \"new\": \"on_screen\" \ - } \ - ] \ -}"; - } // namespace wm diff --git a/src/window_manager.hpp b/src/window_manager.hpp index 57b9fde..739abee 100644 --- a/src/window_manager.hpp +++ b/src/window_manager.hpp @@ -188,7 +188,6 @@ class WindowManager void removeClient(const std::string &appid); void exceptionProcessForTransition(); - const char* convertRoleOldToNew(char const *role); // Do not use this function void timerHandler(); @@ -223,8 +222,6 @@ class WindowManager private: std::map map_afb_event; std::unordered_map area2size; - std::unordered_map roleold2new; - std::unordered_map rolenew2old; std::shared_ptr lc; PMWrapper pmw; rect_map area_info; @@ -232,7 +229,6 @@ class WindowManager // ID allocation and proxy methods for lookup std::unordered_map tmp_surface2app; - static const char* kDefaultOldRoleDb; }; } // namespace wm -- cgit 1.2.3-korg