From: Eunki, Hong <eunkiki.hong@samsung.com>
Date: Mon, 19 Dec 2022 06:59:13 +0000 (+0900)
Subject: Reduce GetCurrentSize call time
X-Git-Tag: dali_2.2.7~5^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;ds=inline;h=56316d6c12f74707f5d32e14a048527551735816;p=platform%2Fcore%2Fuifw%2Fdali-core.git

Reduce GetCurrentSize call time

Call mOwner.GetCurrentSize() only if we use it

Change-Id: I1a22b4d86ad97856a14aa367f1b3d4b62b7e31bf
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>
---

diff --git a/dali/internal/event/actors/actor-sizer.cpp b/dali/internal/event/actors/actor-sizer.cpp
index 4d8e6d2..81e3f4d 100644
--- a/dali/internal/event/actors/actor-sizer.cpp
+++ b/dali/internal/event/actors/actor-sizer.cpp
@@ -117,14 +117,7 @@ void ActorSizer::SetSizeInternal(const Vector3& size)
   // dont allow recursive loop
   DALI_ASSERT_ALWAYS(!mInsideOnSizeSet && "Cannot call SetSize from OnSizeSet");
   // check that we have a node AND the new size width, height or depth is at least a little bit different from the old one
-  Vector3 currentSize = mOwner.GetCurrentSize();
-
-  if((fabsf(mTargetSize.width - size.width) > Math::MACHINE_EPSILON_1) ||
-     (fabsf(mTargetSize.height - size.height) > Math::MACHINE_EPSILON_1) ||
-     (fabsf(mTargetSize.depth - size.depth) > Math::MACHINE_EPSILON_1) ||
-     (fabsf(mTargetSize.width - currentSize.width) > Math::MACHINE_EPSILON_1) ||
-     (fabsf(mTargetSize.height - currentSize.height) > Math::MACHINE_EPSILON_1) ||
-     (fabsf(mTargetSize.depth - currentSize.depth) > Math::MACHINE_EPSILON_1))
+  if(mTargetSize != size || mTargetSize != mOwner.GetCurrentSize())
   {
     mTargetSize = size;