From: Radoslaw Cybulski <r.cybulski@partner.samsung.com>
Date: Mon, 19 Dec 2016 11:27:41 +0000 (+0100)
Subject: removed switch cases ATSPI_ROLE_POPUP_MENU & ATSPI_ROLE_DIALOG from generate_role_trait
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=094bac4bd29dfdd167e56eb4e23cba3d6133d15c;p=platform%2Fcore%2Faccessibility%2Fscreen-reader.git

removed switch cases ATSPI_ROLE_POPUP_MENU & ATSPI_ROLE_DIALOG from generate_role_trait

Change-Id: I36020a288c51ce00ea11ab51c8ed8d788cc69afb
---

diff --git a/src/navigator.c b/src/navigator.c
index 4a821c15..0564447f 100644
--- a/src/navigator.c
+++ b/src/navigator.c
@@ -306,23 +306,6 @@ char *generate_role_trait(AtspiAccessible * obj)
 		}
 		break;
 	}
-	case ATSPI_ROLE_POPUP_MENU: {
-		AtspiAccessible *parent = atspi_accessible_get_parent(obj, NULL);
-		AtspiRole parent_role = atspi_accessible_get_role(parent, NULL);
-		if (parent_role == ATSPI_ROLE_COMBO_BOX) break;
-		int children_count;
-		children_count = atspi_accessible_get_child_count(obj, NULL);
-		snprintf(ret, sizeof(ret), "%s, %s %d %s", _("IDS_TRAIT_CTX_POPUP"),
-			_("IDS_TRAIT_SHOWING"), children_count, _("IDS_TRAIT_ITEMS"));
-		break;
-	}
-	case ATSPI_ROLE_DIALOG: {
-		int children_count = _find_popup_list_children_count(obj);
-		if (children_count > 0)
-			snprintf(ret, sizeof(ret), "%s %d %s",
-				_("IDS_TRAIT_SHOWING"), children_count, _("IDS_TRAIT_ITEMS"));
-		break;
-	}
 	case ATSPI_ROLE_COMBO_BOX: {
 		int children_count = atspi_accessible_get_child_count(obj, NULL);
 		snprintf(ret, sizeof(ret), _("IDS_TRAIT_PD_HOVERSEL"), children_count);
@@ -375,6 +358,8 @@ char *generate_role_trait(AtspiAccessible * obj)
 	case ATSPI_ROLE_MENU_ITEM:
 	case ATSPI_ROLE_RADIO_MENU_ITEM:
 	case ATSPI_ROLE_RADIO_BUTTON:
+	case ATSPI_ROLE_POPUP_MENU:
+	case ATSPI_ROLE_DIALOG:
 	case ATSPI_ROLE_GROUPING: {
 		break;
 	}