From: Youngsun Suh <youngsun.suh@samsung.com>
Date: Thu, 26 Sep 2024 01:11:37 +0000 (+0900)
Subject: Prevent embedding a11y tree if a widget is paused
X-Git-Tag: accepted/tizen/unified/20241015.153147~2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=63482ba7f095e9966e0e40b244a62530d1021403;p=platform%2Fcore%2Fuifw%2Fwidget-viewer-dali.git

Prevent embedding a11y tree if a widget is paused

Change-Id: I71f8b015f6e381da4555d30c56a9712afbd2a9a3
---

diff --git a/widget_viewer_dali/internal/widget_view/widget_view_impl.cpp b/widget_viewer_dali/internal/widget_view/widget_view_impl.cpp
index c1258cb..5799d01 100644
--- a/widget_viewer_dali/internal/widget_view/widget_view_impl.cpp
+++ b/widget_viewer_dali/internal/widget_view/widget_view_impl.cpp
@@ -441,6 +441,11 @@ bool WidgetView::PauseWidgetInternally()
      return false;
   }
 
+  if(Dali::Accessibility::IsUp())
+  {
+    GetAccessibleObject()->OnChildrenChanged();
+  }
+
   DALI_LOG_RELEASE_INFO("WidgetView::PauseWidget: Widget is paused (%s, %s) [%p]\n", mWidgetId.c_str(), mInstanceId.c_str(), this );
 
   return true;
@@ -465,6 +470,11 @@ bool WidgetView::ResumeWidgetInternally()
      return false;
   }
 
+  if(Dali::Accessibility::IsUp())
+  {
+    GetAccessibleObject()->OnChildrenChanged();
+  }
+
   DALI_LOG_RELEASE_INFO("WidgetView::ResumeWidget: Widget is resumed (%s, %s) [%p]\n", mWidgetId.c_str(), mInstanceId.c_str(), this );
 
   return true;
@@ -1218,7 +1228,7 @@ void WidgetView::OnUpdateArea( Dali::PropertyNotification& source )
     mPaused = needPaused;
   }
 
-  if(!mPaused)
+  if(!mPaused && Dali::Accessibility::IsUp())
   {
     auto accessible = std::dynamic_pointer_cast<WidgetView::WidgetViewAccessible>(GetAccessibleObject());
     DALI_ASSERT_DEBUG(accessible);
@@ -1807,6 +1817,11 @@ void WidgetView::UpdateBuffer( struct tizen_remote_surface *surface, struct wl_b
   mRemoteSurface = surface;
 }
 
+bool WidgetView::IsPaused() const
+{
+  return mPaused;
+}
+
 bool WidgetView::LaunchWidget()
 {
   screen_connector_toolkit_ops ops;
@@ -2023,6 +2038,12 @@ void WidgetView::WidgetViewAccessible::DoGetChildren(std::vector<Dali::Accessibi
 
   auto        self       = Dali::WidgetView::WidgetView::DownCast(Self());
   auto&       selfImpl   = Dali::WidgetView::GetImplementation(self);
+  if(selfImpl.IsPaused())
+  {
+    DALI_LOG_RELEASE_INFO("Returning no children as the widget is paused.");
+    return;
+  }
+
   const auto& instanceId = selfImpl.GetInstanceId();
   int         pid        = selfImpl.GetPid();
   std::string busName    = Bridge::MakeBusNameForWidget(instanceId, pid);
diff --git a/widget_viewer_dali/internal/widget_view/widget_view_impl.h b/widget_viewer_dali/internal/widget_view/widget_view_impl.h
index 393dad1..f2663bc 100644
--- a/widget_viewer_dali/internal/widget_view/widget_view_impl.h
+++ b/widget_viewer_dali/internal/widget_view/widget_view_impl.h
@@ -331,6 +331,8 @@ public: // Internal API
    */
   void UpdateBuffer( struct tizen_remote_surface *surface, struct wl_buffer *buffer, int32_t fd);
 
+  bool IsPaused() const;
+
 public: // Internal API for WidgetViewManager
 
   /**