[AT-SPI] remove isRoot param from Accessible::Get 50/274750/1
authorShinwoo Kim <cinoo.kim@samsung.com>
Mon, 9 May 2022 01:54:57 +0000 (10:54 +0900)
committerShinwoo Kim <cinoo.kim@samsung.com>
Mon, 9 May 2022 01:54:57 +0000 (10:54 +0900)
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

automated-tests/src/dali-toolkit-internal/dali-toolkit-test-utils/accessibility-test-utils.cpp

index 8a7d7f1..843a26c 100644 (file)
@@ -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();