[ATSPI] do not make window hidden when bridge removes top level window 80/268980/4
authorShinwoo Kim <cinoo.kim@samsung.com>
Thu, 6 Jan 2022 01:02:18 +0000 (10:02 +0900)
committerShinwoo Kim <cinoo.kim@samsung.com>
Thu, 6 Jan 2022 01:21:34 +0000 (10:21 +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 9f88d3c..0f0bc43 100644 (file)
@@ -246,7 +246,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);