From 860667a3ef3ff81298754c855907bb358d5c244a Mon Sep 17 00:00:00 2001 From: Shinwoo Kim Date: Mon, 9 May 2022 10:54:57 +0900 Subject: [PATCH] [AT-SPI] remove isRoot param from Accessible::Get The Accessibility::Accessible::Get for a window could be called before getting accessible to add window accessible. So far, we thought that the next part creates an accessible for window for the first time. accessible = Accessibility::Accessible::Get(rootLayer, true); bridge->AddTopLevelWindow(accessible); However, there is a case where it is created before this part caused by following symbol CSharp_Dali_Toolkit_DevelControl_GetAccessibilityStates+0x144 In this case, isRoot is set to `false`. Then window will have incorrect accessible information. For more information, please refer to the following. https://github.sec.samsung.net/tizen/atspi/issues/60 [Dependency] https://review.tizen.org/gerrit/#/c/platform/core/uifw/dali-adaptor/+/274737/ Change-Id: Iadca9c42aa2027f372838247ac23b27d484e4c8d --- .../dali-toolkit-test-utils/accessibility-test-utils.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/automated-tests/src/dali-toolkit-internal/dali-toolkit-test-utils/accessibility-test-utils.cpp b/automated-tests/src/dali-toolkit-internal/dali-toolkit-test-utils/accessibility-test-utils.cpp index 8a7d7f1..843a26c 100644 --- a/automated-tests/src/dali-toolkit-internal/dali-toolkit-test-utils/accessibility-test-utils.cpp +++ b/automated-tests/src/dali-toolkit-internal/dali-toolkit-test-utils/accessibility-test-utils.cpp @@ -22,7 +22,7 @@ namespace Accessibility firstTime = false; auto bridge = Accessibility::Bridge::GetCurrentBridge(); Dali::Stage stage = Dali::Stage::GetCurrent(); - auto accessible = Accessibility::Accessible::Get( stage.GetRootLayer(), true ); + auto accessible = Accessibility::Accessible::Get( stage.GetRootLayer() ); bridge->AddTopLevelWindow( accessible ); bridge->SetApplicationName( "TestApp" ); bridge->Initialize(); -- 2.7.4