[NUI][ATSPI] remove duplicated calculation for states
authorShinwoo Kim <cinoo.kim@samsung.com>
Fri, 21 Jan 2022 06:33:17 +0000 (15:33 +0900)
committerdongsug-song <35130733+dongsug-song@users.noreply.github.com>
Tue, 8 Feb 2022 04:41:28 +0000 (13:41 +0900)
The HIGHLIHGTED state is calculated on parent class ControlAccessible
The get of IsHighlighted is doing exactly same calculation.

And HIGHLIGHTABLE, FOCUSABLE, and DEFUNCT as well.

Regarding VISIBLE was always TRUE. It is not correct state.
So this patch is removing the line related to the VISIBLE as well

src/Tizen.NUI/src/public/BaseComponents/ViewAccessibility.cs

index 3f0bbdd..72e513f 100755 (executable)
@@ -628,14 +628,9 @@ namespace Tizen.NUI.BaseComponents
         {
             AccessibilityStates accessibilityStates = (AccessibilityStates)states;
 
-            FlagSetter(ref accessibilityStates, AccessibilityStates.Highlightable, this.AccessibilityHighlightable);
-            FlagSetter(ref accessibilityStates, AccessibilityStates.Focusable, this.Focusable);
             FlagSetter(ref accessibilityStates, AccessibilityStates.Focused, this.State == States.Focused);
-            FlagSetter(ref accessibilityStates, AccessibilityStates.Highlighted, this.IsHighlighted);
             FlagSetter(ref accessibilityStates, AccessibilityStates.Enabled, this.State != States.Disabled);
             FlagSetter(ref accessibilityStates, AccessibilityStates.Sensitive, this.Sensitive);
-            FlagSetter(ref accessibilityStates, AccessibilityStates.Visible, true);
-            FlagSetter(ref accessibilityStates, AccessibilityStates.Defunct, !this.IsOnWindow);
 
             return accessibilityStates;
         }