[ATSPI] do not make window hidden when bridge removes top level window 96/271296/1
authorShinwoo Kim <cinoo.kim@samsung.com>
Thu, 6 Jan 2022 01:02:18 +0000 (10:02 +0900)
committerShinwoo Kim <cinoo.kim@samsung.com>
Fri, 18 Feb 2022 02:31:20 +0000 (11:31 +0900)
OnAccessibilityDisabled > RemoveTopLevelWindow > WindowHidden
makes mIsShown become FALSE

After this point, if the Accessibility is Enabled then the Bridge
does not call EmitActivate in ForceUp because mIsShown is FALSE by
RemoveTopLevelWindow > WindowHidden (mentioned above)

I cannot find any reason calling WindowHidden in RemoveTopLevelWindow
Please refer to following patch

ttps://review.tizen.org/gerrit/#/c/platform/core/uifw/dali-adaptor/+/264762/8/dali/internal/accessibility/bridge/bridge-base.cpp@277

Change-Id: I6ebf2e4b94ff00867950047591539e8c8bc96c3a

dali/internal/accessibility/bridge/bridge-base.cpp

index 23c6c60..92c2000 100644 (file)
@@ -279,7 +279,6 @@ void BridgeBase::RemoveTopLevelWindow(Accessible* windowAccessible)
   {
     if(windowAccessible->GetInternalActor() == mApplication.mWindows[i].GetRootLayer())
     {
-      Dali::Accessibility::Bridge::GetCurrentBridge()->WindowHidden(mApplication.mWindows[i]);
       Dali::DevelWindow::VisibilityChangedSignal(mApplication.mWindows[i]).Disconnect(this, &BridgeBase::OnWindowVisibilityChanged);
       mApplication.mWindows[i].FocusChangeSignal().Disconnect(this, &BridgeBase::OnWindowFocusChanged);
       mApplication.mWindows.erase(mApplication.mWindows.begin() + i);