Add extra padding for decorated visual renderer

Since we did some anti-alias in shader area,
we need to extent update area for that anti-alias rendering

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

diff --git a/automated-tests/src/dali/utc-Dali-DecoratedVisualRenderer.cpp b/automated-tests/src/dali/utc-Dali-DecoratedVisualRenderer.cpp
index e0f8c28..4cad47c 100644 (file)
--- a/automated-tests/src/dali/utc-Dali-DecoratedVisualRenderer.cpp
+++ b/automated-tests/src/dali/utc-Dali-DecoratedVisualRenderer.cpp
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2022 Samsung Electronics Co., Ltd.
+ * Copyright (c) 2023 Samsung Electronics Co., Ltd.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -821,8 +821,8 @@ int UtcDaliDecoratedVisualRendererPartialUpdate(void)
   Actor actor = Actor::New();
   actor.AddRenderer(renderer);
   actor[Actor::Property::ANCHOR_POINT] = AnchorPoint::TOP_LEFT;
-  actor[Actor::Property::POSITION]     = Vector3(64.0f, 64.0f, 0.0f);
-  actor[Actor::Property::SIZE]         = Vector3(64.0f, 64.0f, 0.0f);
+  actor[Actor::Property::POSITION]     = Vector3(66.0f, 66.0f, 0.0f);
+  actor[Actor::Property::SIZE]         = Vector3(60.0f, 60.0f, 0.0f);
   actor.SetResizePolicy(ResizePolicy::FIXED, Dimension::ALL_DIMENSIONS);
   application.GetScene().Add(actor);
 
@@ -835,7 +835,7 @@ int UtcDaliDecoratedVisualRendererPartialUpdate(void)
   DALI_TEST_EQUALS(damagedRects.size(), 1, TEST_LOCATION);
 
   // Aligned by 16
-  Rect<int> clippingRect = Rect<int>(64, 672, 80, 80); // in screen coordinates, includes 3 last frames updates
+  Rect<int> clippingRect = Rect<int>(64, 672, 64, 64); // in screen coordinates, includes 3 last frames updates
   DALI_TEST_EQUALS<Rect<int>>(clippingRect, damagedRects[0], TEST_LOCATION);
 
   application.RenderWithPartialUpdate(damagedRects, clippingRect);
@@ -861,15 +861,15 @@ int UtcDaliDecoratedVisualRendererPartialUpdate(void)
   Property::Index index = DecoratedVisualRenderer::Property::BORDERLINE_OFFSET;
   renderer.SetProperty(index, 1.0f);
 
-  // Now current actor show as 128x128 rectangle, with center position (96, 96).
-  // So, rectangle's top left position is (32, 32), and bottom right position is (160, 160).
+  // Now current actor show as 124x124 rectangle, with center position (96, 96).
+  // So, rectangle's top left position is (34, 34), and bottom right position is (158, 158).
 
   application.SendNotification();
   damagedRects.clear();
   application.PreRenderWithPartialUpdate(TestApplication::RENDER_FRAME_INTERVAL, nullptr, damagedRects);
   DALI_TEST_EQUALS(damagedRects.size(), 1, TEST_LOCATION);
   // Aligned by 16
-  DALI_TEST_EQUALS<Rect<int>>(Rect<int>(32, 640, 144, 144), damagedRects[0], TEST_LOCATION);
+  DALI_TEST_EQUALS<Rect<int>>(Rect<int>(32, 640, 128, 128), damagedRects[0], TEST_LOCATION);
 
   application.RenderWithPartialUpdate(damagedRects, clippingRect);
 
@@ -896,7 +896,7 @@ int UtcDaliDecoratedVisualRendererPartialUpdate(void)
   application.RenderWithPartialUpdate(damagedRects, clippingRect);
   DALI_TEST_EQUALS(damagedRects.size(), 1, TEST_LOCATION);
   // Aligned by 16
-  DALI_TEST_EQUALS<Rect<int>>(Rect<int>(32, 640, 144, 144), damagedRects[0], TEST_LOCATION);
+  DALI_TEST_EQUALS<Rect<int>>(Rect<int>(32, 640, 128, 128), damagedRects[0], TEST_LOCATION);
 
   application.SendNotification();
   damagedRects.clear();
@@ -973,8 +973,8 @@ int UtcDaliDecoratedVisualRendererPartialUpdate(void)
 
   DALI_TEST_EQUALS(damagedRects.size(), 1, TEST_LOCATION);
   // Aligned by 16
-  // Now current actor show as 96x96 rectangle, with center position (96, 96).
-  DALI_TEST_EQUALS<Rect<int>>(Rect<int>(48, 656, 112, 112), damagedRects[0], TEST_LOCATION);
+  // Now current actor show as 92x92 rectangle, with center position (96, 96).
+  DALI_TEST_EQUALS<Rect<int>>(Rect<int>(48, 656, 96, 96), damagedRects[0], TEST_LOCATION);
 
   application.SendNotification();
   damagedRects.clear();
@@ -1014,8 +1014,8 @@ int UtcDaliDecoratedVisualRendererPartialUpdate(void)
 
   DALI_TEST_EQUALS(damagedRects.size(), 1, TEST_LOCATION);
   // Aligned by 16
-  // Now current actor show as 64x64 rectangle, with center position (96, 96).
-  DALI_TEST_EQUALS<Rect<int>>(Rect<int>(64, 672, 80, 80), damagedRects[0], TEST_LOCATION);
+  // Now current actor show as 60x60 rectangle, with center position (96, 96).
+  DALI_TEST_EQUALS<Rect<int>>(Rect<int>(64, 672, 64, 64), damagedRects[0], TEST_LOCATION);
 
   application.SendNotification();
   damagedRects.clear();

diff --git a/dali/internal/update/rendering/scene-graph-visual-renderer.cpp b/dali/internal/update/rendering/scene-graph-visual-renderer.cpp
index fe290e6..560fd25 100644 (file)
--- a/dali/internal/update/rendering/scene-graph-visual-renderer.cpp
+++ b/dali/internal/update/rendering/scene-graph-visual-renderer.cpp
@@ -137,13 +137,16 @@ Vector4 AnimatableVisualProperties::GetVisualTransformedUpdateArea(BufferIndex u
       const float borderlineOffset = decoratedVisualProperties->mBorderlineOffset.Get(updateBufferIndex);
       const float blurRadius       = decoratedVisualProperties->mBlurRadius.Get(updateBufferIndex);
 
+      // Extra padding information for anti-alias
+      const float extraPadding = 2.0f;
+
       DALI_LOG_INFO(gSceneGraphRendererLogFilter, Debug::Verbose, "borderline width  %5.3f\n", borderlineWidth);
       DALI_LOG_INFO(gSceneGraphRendererLogFilter, Debug::Verbose, "borderline offset %5.3f\n", borderlineOffset);
       DALI_LOG_INFO(gSceneGraphRendererLogFilter, Debug::Verbose, "blur radius       %5.3f\n", blurRadius);
 
       // D coefficients be used only decoratedVisual.
       // It can be calculated parallely with visual transform.
-      decoratedCoefficient.coefD = std::max((1.0f + Dali::Clamp(borderlineOffset, -1.0f, 1.0f)) * borderlineWidth, 2.0f * blurRadius);
+      decoratedCoefficient.coefD = std::max((1.0f + Dali::Clamp(borderlineOffset, -1.0f, 1.0f)) * borderlineWidth, 2.0f * blurRadius) + extraPadding;
     }
 
     // Update coefD so we can use this value out of this scope.