[NUI] Fix TableView TC fail issue again in proper way
authordongsug.song <dongsug.song@samsung.com>
Sun, 25 Aug 2024 07:14:22 +0000 (16:14 +0900)
committerSangHyeon Jade Lee <dltkdgus1764@gmail.com>
Wed, 4 Sep 2024 09:37:54 +0000 (18:37 +0900)
- Fix the https://github.com/Samsung/TizenFX/pull/6258 correctly
- This is for https://github.sec.samsung.net/NUI/home/issues/25
- locally applied image has been passed the TCT

src/Tizen.NUI/src/public/BaseComponents/ViewInternal.cs

index e0a94fdef3566dfd9de1b2c732895bec680250be..8e1eb8a28f463dfabae3dd04cd93f828415096da 100755 (executable)
@@ -1356,85 +1356,90 @@ namespace Tizen.NUI.BaseComponents
 
             //_mergedStyle = null;
 
-            internalMaximumSize?.Dispose();
-            internalMaximumSize = null;
-            internalMinimumSize?.Dispose();
-            internalMinimumSize = null;
-            internalMargin?.Dispose();
-            internalMargin = null;
-            internalPadding?.Dispose();
-            internalPadding = null;
-            internalSizeModeFactor?.Dispose();
-            internalSizeModeFactor = null;
-            internalCellIndex?.Dispose();
-            internalCellIndex = null;
-            internalBackgroundColor?.Dispose();
-            internalBackgroundColor = null;
-            internalColor?.Dispose();
-            internalColor = null;
-            internalPivotPoint?.Dispose();
-            internalPivotPoint = null;
-            internalPosition?.Dispose();
-            internalPosition = null;
-            internalPosition2D?.Dispose();
-            internalPosition2D = null;
-            internalScale?.Dispose();
-            internalScale = null;
-            internalSize?.Dispose();
-            internalSize = null;
-            internalSize2D?.Dispose();
-            internalSize2D = null;
-
-            panGestureDetector?.Dispose();
-            panGestureDetector = null;
-            longGestureDetector?.Dispose();
-            longGestureDetector = null;
-            pinchGestureDetector?.Dispose();
-            pinchGestureDetector = null;
-            tapGestureDetector?.Dispose();
-            tapGestureDetector = null;
-            rotationGestureDetector?.Dispose();
-            rotationGestureDetector = null;
-
-            internalCurrentParentOrigin?.Dispose();
-            internalCurrentParentOrigin = null;
-            internalCurrentAnchorPoint?.Dispose();
-            internalCurrentAnchorPoint = null;
-            internalTargetSize?.Dispose();
-            internalTargetSize = null;
-            internalCurrentSize?.Dispose();
-            internalCurrentSize = null;
-            internalCurrentPosition?.Dispose();
-            internalCurrentPosition = null;
-            internalCurrentWorldPosition?.Dispose();
-            internalCurrentWorldPosition = null;
-            internalCurrentScale?.Dispose();
-            internalCurrentScale = null;
-            internalCurrentWorldScale?.Dispose();
-            internalCurrentWorldScale = null;
-            internalCurrentColor?.Dispose();
-            internalCurrentColor = null;
-            internalCurrentWorldColor?.Dispose();
-            internalCurrentWorldColor = null;
-            internalSizeModeFactor?.Dispose();
-            internalSizeModeFactor = null;
-            internalCurrentScreenPosition?.Dispose();
-            internalCurrentScreenPosition = null;
-
-            if (visualContainers != null)
-            {
-                foreach (var visualContainer in visualContainers)
-                {
-                    visualContainer?.Dispose();
-                }
-                visualContainers = null;
-            }
-
             if (type == DisposeTypes.Explicit)
             {
                 //Called by User
                 //Release your own managed resources here.
                 //You should release all of your own disposable objects here.
+                internalMaximumSize?.Dispose();
+                internalMaximumSize = null;
+                internalMinimumSize?.Dispose();
+                internalMinimumSize = null;
+                internalMargin?.Dispose();
+                internalMargin = null;
+                internalPadding?.Dispose();
+                internalPadding = null;
+                internalSizeModeFactor?.Dispose();
+                internalSizeModeFactor = null;
+                internalCellIndex?.Dispose();
+                internalCellIndex = null;
+                internalBackgroundColor?.Dispose();
+                internalBackgroundColor = null;
+                internalColor?.Dispose();
+                internalColor = null;
+                internalPivotPoint?.Dispose();
+                internalPivotPoint = null;
+                internalPosition?.Dispose();
+                internalPosition = null;
+                internalPosition2D?.Dispose();
+                internalPosition2D = null;
+                internalScale?.Dispose();
+                internalScale = null;
+                internalSize?.Dispose();
+                internalSize = null;
+                internalSize2D?.Dispose();
+                internalSize2D = null;
+
+                panGestureDetector?.Dispose();
+                panGestureDetector = null;
+                longGestureDetector?.Dispose();
+                longGestureDetector = null;
+                pinchGestureDetector?.Dispose();
+                pinchGestureDetector = null;
+                tapGestureDetector?.Dispose();
+                tapGestureDetector = null;
+                rotationGestureDetector?.Dispose();
+                rotationGestureDetector = null;
+
+                internalCurrentParentOrigin?.Dispose();
+                internalCurrentParentOrigin = null;
+                internalCurrentAnchorPoint?.Dispose();
+                internalCurrentAnchorPoint = null;
+                internalTargetSize?.Dispose();
+                internalTargetSize = null;
+                internalCurrentSize?.Dispose();
+                internalCurrentSize = null;
+                internalCurrentPosition?.Dispose();
+                internalCurrentPosition = null;
+                internalCurrentWorldPosition?.Dispose();
+                internalCurrentWorldPosition = null;
+                internalCurrentScale?.Dispose();
+                internalCurrentScale = null;
+                internalCurrentWorldScale?.Dispose();
+                internalCurrentWorldScale = null;
+                internalCurrentColor?.Dispose();
+                internalCurrentColor = null;
+                internalCurrentWorldColor?.Dispose();
+                internalCurrentWorldColor = null;
+                internalSizeModeFactor?.Dispose();
+                internalSizeModeFactor = null;
+                internalCurrentScreenPosition?.Dispose();
+                internalCurrentScreenPosition = null;
+
+                if (visualContainers != null)
+                {
+                    foreach (var visualContainer in visualContainers)
+                    {
+                        visualContainer?.Dispose();
+                    }
+                    visualContainers = null;
+                }
+
+                foreach (View view in Children)
+                {
+                    view.InternalParent = null;
+                }
+
                 if (themeData != null)
                 {
                     themeData.selectorData?.Reset(this);
@@ -1461,11 +1466,6 @@ namespace Tizen.NUI.BaseComponents
 
             DisConnectFromSignals();
 
-            foreach (View view in Children)
-            {
-                if (view != null) { view.InternalParent = null; }
-            }
-
             backgroundExtraDataUpdatedFlag = BackgroundExtraDataUpdatedFlag.None;
 
             LayoutCount = 0;