[NUI.XamlBuild] Remove unused event : Element.DescendantAdded / Removed
authorEunki, Hong <eunkiki.hong@samsung.com>
Tue, 7 Nov 2023 12:58:10 +0000 (21:58 +0900)
committerJaehyun Cho <jaehyun0cho@gmail.com>
Wed, 15 Nov 2023 08:56:46 +0000 (17:56 +0900)
Those API was useless if we don't need to invoke those events.
And those try to climb-up to parent root every Add / Remove API.

It will spend heavy time if the scene tree is complex.

Those event handerl are internal, and nobody use them.
So just remove codes.

Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>
src/Tizen.NUI.XamlBuild/src/internal/XamlBinding/Element.cs
src/Tizen.NUI/src/public/XamlBinding/Element.cs

index 8eb2d08..be186b6 100755 (executable)
@@ -471,11 +471,12 @@ namespace Tizen.NUI.Binding
 
             child.ApplyBindings(skipBindingContext: false, fromBindingContextChanged:true);
 
-            ChildAdded?.Invoke(this, new ElementEventArgs(child));
+            // 2023-11-08 : Just ignore Tizen.NUI don't using logics.
+            // ChildAdded?.Invoke(this, new ElementEventArgs(child));
 
-            OnDescendantAdded(child);
-            foreach (Element element in child.Descendants())
-                OnDescendantAdded(element);
+            // OnDescendantAdded(child);
+            // foreach (Element element in child.Descendants())
+            //     OnDescendantAdded(element);
         }
 
         /// <summary>
@@ -488,11 +489,12 @@ namespace Tizen.NUI.Binding
         {
             child.Parent = null;
 
-            ChildRemoved?.Invoke(child, new ElementEventArgs(child));
+            // 2023-11-08 : Just ignore Tizen.NUI don't using logics.
+            // ChildRemoved?.Invoke(child, new ElementEventArgs(child));
 
-            OnDescendantRemoved(child);
-            foreach (Element element in child.Descendants())
-                OnDescendantRemoved(element);
+            // OnDescendantRemoved(child);
+            // foreach (Element element in child.Descendants())
+            //     OnDescendantRemoved(element);
         }
 
         /// <summary>
index b206d4f..13a3d2d 100755 (executable)
@@ -372,11 +372,12 @@ namespace Tizen.NUI.Binding
 
             child.ApplyBindings(skipBindingContext: false, fromBindingContextChanged: true);
 
-            ChildAdded?.Invoke(this, new ElementEventArgs(child));
+            // 2023-11-08 : Just ignore Tizen.NUI don't using logics.
+            // ChildAdded?.Invoke(this, new ElementEventArgs(child));
 
-            OnDescendantAdded(child);
-            foreach (Element element in child.Descendants())
-                OnDescendantAdded(element);
+            // OnDescendantAdded(child);
+            // foreach (Element element in child.Descendants())
+            //     OnDescendantAdded(element);
         }
 
         /// <summary>
@@ -395,11 +396,12 @@ namespace Tizen.NUI.Binding
 
             child.Parent = null;
 
-            ChildRemoved?.Invoke(child, new ElementEventArgs(child));
+            // 2023-11-08 : Just ignore Tizen.NUI don't using logics.
+            // ChildRemoved?.Invoke(child, new ElementEventArgs(child));
 
-            OnDescendantRemoved(child);
-            foreach (Element element in child.Descendants())
-                OnDescendantRemoved(element);
+            // OnDescendantRemoved(child);
+            // foreach (Element element in child.Descendants())
+            //     OnDescendantRemoved(element);
         }
 
         /// <summary>