When hoverevent of geometry type is used, DispatchHoverMotion is not handled properly when false. This is fixed.

Change-Id: Ibcd68776f55a6779485a36d2d17b030801225b7f

If IsDispatchHoverMotion is false, the move event was not dispatched. So consumedActor should keep the previous LastConsumedActor value.

Change-Id: Ifbadde1b21abfca728c9e91be81a8f57c4650a6e

DALi Version 2.4.9

Change-Id: I33fb7c331cf4aab53d6a787060cc409d12ef99f9

Allow to create renderer without geometry and shader

Let we make constructor renderer without geometry and shader.

Also, check some codes s.t. we can allow nullptr mGeometry and mShader.

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

Clear pipelie cache if geometry destroyed or buffer changed

We also cache pipeline cache the Render::Geometry by the raw pointer.
If someone use duplicated pointer, it might return invalid pipeline
with unmatched vertexInputState.

To avoid this issue, let we erase cache if Render::Geometry destroyed,
same as Render::Program

 - This is the commit message #2:

Reset cached pipeline if geometry buffer changed

Until now, we don't re-cache the geometry
if the vertex buffer added/removed, or data changed.

Since the vertex attribute might be changed if we try to use
same geometry, the rendering result
show some non-common results.

To fix this cache issue, let we ensure to reset the cached infomations
if the vertex buffer / indices buffer changed.

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

DALi Version 2.4.8

Change-Id: I42df6d888f5101778ba29ad50cafed19e33e9746

Merge "Make PixelData Stride return byte size, instead of pixel" into devel/master

Increase warning message threshold for animator

When we launch benchmark.example, it print too much warning logs :(

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

Make PixelData Stride return byte size, instead of pixel

Some cases doesn't have stride byte as multiply of BytesPerPixel().
But until now, PixelBuffer and PixelData only support Stride as pixels count
(It looks since GL_UNPACK_ROW_LENGTH is pixel scale, not a bytes.)

For example, If we have 2 width image with RGB888 format, stride byte
should be 8 byte, which is not a multiply of 3 byte.

To support those cases, let we set stride value as bytes scale.

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

Merge "DALi Version 2.4.7" into devel/master

DALi Version 2.4.7

Change-Id: Id6252a283cf17a78688548f01d0263369e292bcc

Merge "(Partial Update) Ensure to renderer dirty if Geometry changed" into devel/master

Merge "Reduce IntrusivePtr<ActorPtr> reference change during PropagateAll" into devel/master

Print more debug logs for Animation

Let we print more infomations for dali animations
whenever we call API

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

(Animation) Bug fix when Cleared state animation finish normally don't emit signal

Their was some bug if

(Event Thread) Clear()
(Render Thread) Finish animation normally
(Event Thread) NotificationManager->ProcessMessages() -> HasFinished() called
...
(Event Thread) Play()
(Render Thread) Finish animation normally
(Event Thread) NotificationManager->ProcessMessages() -> HasFinished() called

progress comes.

The first HasFinished() should not emit finished signal.
But second HasFinisehd() should emit finished signal, but not.

It was bugs since HasFinished() update some values, but we don't use it.
(Cleared state should have mNotificationCount always) It make bug.

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

Reduce float hash collision by squaring valueHash

Applied valueHash^2 to reduce frequent hash collisions in similar float values.

Change-Id: I8877b24e2936e8a3cbf7da3ca52998f74d75e65c

Merge "Share partial update data info to RenderItem" into devel/master

DALi Version 2.4.6

Change-Id: I21a91c30091c3b411e9068c450921be029fd4cb1

(Partial Update) Ensure to renderer dirty if Geometry changed

Let we also make item.mRenderer is Updated() if Render::Geometry information
changed.

For example, Toolkit::TextEditor change the vertex buffer data and indicise
when some texts, or highlighted area are changed.
But if the size of each actors are not be changed, the infomation of
geometry changed not be applied to the PreRender() dirtyrect calculation.

To fix this issue, let we make ensure that geometry is updated at Upload time,
and notify it to Render::Renderer's Update() time.

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

Share partial update data info to RenderItem

Let we make RenderItem use Node's PartialUpdateData directly,
instead copy the matrix and etc.

It will reduce each item's memory, and memory copy time.

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

Reduce IntrusivePtr<ActorPtr> reference change during PropagateAll

std::atomic value changeness might gave overhead if we call it
very frequencly.

To reduce it's overhead, let we use Interal::Actor& instead Actor handle
if we can.

Note that we cannot remove Dali::Actor handle at RelayoutInfo,
since we need to guard case that Actor itself destruct during
Relayout callback at application side.

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

Reduce dlog spam per frame

Added debug filter for  animation update and transform manager.
These can be turned on by setting DALI_LOG_ANIMATION or
DALI_LOG_TRANSFORM_MANAGER environment variables to 3 before running.

(Or temporarily turned on in a build by modifying the filter
constructor arguments).

Change-Id: I1ade7dfecb5fd0b9d4a7dc8d043486af52a791bf

Check if scene has something to render

Core::PreRender(scene) now returns true if there is any render instruction
with a render list with render items for that scene.

Change-Id: Ib628f0a03dc9959fc945bdb410a7af52ec54e86b

DALi Version 2.4.5

Change-Id: If0bc883cbcb85fbcb5518e66a1bf3300310d0810

DALi Version 2.4.4

Change-Id: I02cf6f6f0652bab320bcd0ab7d7a6c5f0669c394

Merge "Changed command buffer ownership" into devel/master

DALi Version 2.4.3

Change-Id: I342647c70b396a90fabad694551d4931a684ae46

Changed command buffer ownership

Each render instruction now has a command buffer.

This enables the backend/gpu to synchronize each command buffer when
rendering to framebuffers.

Because RenderTask API is freeform, the same framebuffer can be written
to more than once in a frame using different actors, so each render pass
needs it's own command buffer.

(May still be potential issue with multiple render tasks against the
scene, e.g. magnifier isn't fixed in Vulkan by this change)

Change-Id: I367f94144ab9670c346b98a8e57ad2b54db96fd5
Signed-off-by: David Steele <david.steele@samsung.com>

Make mat2 type uniform also use matrix stride

Dali use mat2 type as Vector4 property.
But if we just use Vector4 type as mat2 uniform,
matrix stride information not be applied.

To fix this issue, let we also check matrixStride value
of uniform reflection, and use it.

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

Merge "Minor optimization (Use l-value of ShaderData + Move matrix + less hash)" into devel/master

DALi Version 2.4.2

Change-Id: I9d2c3790442d7a7825ea2b442d70278983439ba4

Minor optimization (Use l-value of ShaderData + Move matrix + less hash)

- Make SG::Shader::GetShaderData return const l-value.
  (It will not call useless refcount increase + decrease)

- Move Matrix if possible

- Less creation and hashing for PartialUpdateData
  Let we use const l-value and less creation matrix and vector.

After this patch, when render 16,000 view with color animate,
fps increased 32 --> 36

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

Support equality operator for Property::Map and Property::Array

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

Fix coverity : do not call move to const & type

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

Support DecoratedVisualRenderer borderine with blur radius

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

DALi Version 2.4.1

Change-Id: I90f0d7c3a81bd906ffca9283dc5867985057ce84

Merge "Allow to set empty data for vertex buffer" into devel/master

Allow to use moved Property::Map & Property::Array

Until now, we make those value's mImpl as nullptr.
So their was a lots of DALI_ASSERT_DEBUG() to check
whether they are moved or not.

Those codes might not be fair than other moved objects implementation.

we allow to use moved object like WeakHandle, or Dali::Vector.
Let we also allow to use them.

And let we make surely assert by DALI_ASSERT_ALWAYS if we need, instead DALI_ASSERT_DEBUG

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

Allow to set empty data for vertex buffer

Let we allow to set data with nullptr and size 0 for VertexBuffer

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

Merge "Implement CircularQueue Resize, Clear. And optimize some useless branch" into devel/master

(dali_env) Install libuv1-dev

Change-Id: I5380e39b7589fa9e5ed54df5b0f5e88dd31fa4f3

Implement CircularQueue Resize, Clear. And optimize some useless branch

There was some useless branch behaviour exist for CircularQueue implements.
Let we optimize it.

And also, make a API that we can change the max capacity of queue.

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

DALi Version 2.4.0

Change-Id: I814ba9dc36c6d35a6521f799b592f79f4850b9fc

DALi Version 2.3.54

Change-Id: Ib9a9747a02886b6b17d96da1fbfe76a593aa8119

Merge "Moved program resource hints to Controller" into devel/master

Merge "Fix typo in actor impl docs" into devel/master

Fix typo in actor impl docs

Change-Id: I251fa3aad6f582af6364a4021ffa407645697a9d

Support Property::Value::GetHash() + Implement lite equality operator for Map and Array

Let we support to get hash value of Property
+
Seperate the hash calculation function as internal,
so we can remove duplicated hash calculation logics in dali-core.
+
Support equality operasion for Property::Map and Property::Array.

TODO : For now, we don't consider Eplison when we check equality.
TODO : For now, we only compare hash value, not an exact value of array and map.
It should be supported in future.

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

Moved program resource hints to Controller

In each scene, the main command buffer also is used to inform the
backend of all used program resources; in Vk, this is used to create
descriptor pools.

Moved this resource hint from CommandBuffer::Begin() info struct, to
Controller::SetResourceBindingHint()

Change-Id: I6cd66af23ad1981f56e06f962404a2602c6e3026
Signed-off-by: David Steele <david.steele@samsung.com>

Add view offscreen rendering

Change-Id: I4e5b4378a3e953d2382ccb931a54a754c5368ab4
Signed-off-by: jmm <j0064423.lee@samsung.com>

DALi Version 2.3.53

Change-Id: I342b1233f085e8cdcf0bda6fd6371bf4838d1a66

Keep rendered buffer in framebuffer

 - Users can use RenderTask API to request and to use the buffer.

Change-Id: Iaf63a60f246df8dc48f152bc8a4b4c0b68cd1e1b
Signed-off-by: Seungho Baek <sbsh.baek@samsung.com>

(Automated Tests) Sync AddOn Manager with Toolkit

Change-Id: I72861ca0750b419ad25399b8f6bb104956e0d7dd

DALi Version 2.3.52

Change-Id: I1b1cafca5676da6955c9598ef281f4c7304420d3

Add KeyEventMonitorSignal

Change-Id: I5f598251bfa1cb1fb6661ded46030c1e0074c62f

Merge "TimeChecker filter implement for debug time duration" into devel/master

Remove outdated exclusive render task checker

Change-Id: Ia3f7b0e7fbd3d48d6b960eb80446b2eef8f50aed

TimeChecker filter implement for debug time duration

Let we define new filter to check time duration and threshold to print.
It will be useful if we want to check duration of some API

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

DALi Version 2.3.51

Change-Id: Ic5188d41ad95270fcc265b7131ce83c07fa91195

Allow to Rect<float>::Intersect Merge and Inset works well

Until now, we only use Intersect / Merge / Inset API only for Rect<int>.
Let's allow to use it for Rect<float> case.

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

Merge "Wait message queue flush 10 ms if rendering is too late" into devel/master

DALi Version 2.3.50

Change-Id: Ica3b9d8888b712d05d0b567b374ddfc7136d51e9

Wait message queue flush 10 ms if rendering is too late

Let we wait message queue be flushed if the message queue count is too big.

It will be resolved some memory limitation issues when
event thread flush queue very quickly, but render thread cannot
follow up the messages.

To awake as fast as we can, let's use promise-future system.
Since we don't need to call future.get(), let's just use future<void>

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

Do not add or get singletone container if it is removing

Let we block singletone class don't try to get singletone handle
during it is removing.

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

Merge "Support OffScreenRenderable Ordering when the sibling order is chaged." into devel/master

Support OffScreenRenderable Ordering when the sibling order is chaged.

Change-Id: I65ef042f122a114778da87ffb44d2cb2dc9c63fc
Signed-off-by: Seungho Baek <sbsh.baek@samsung.com>

DALi Version 2.3.49

Change-Id: Ic7802432284d54c5775c5a6e1486a81a60296695

Don't use mEventThreadService without checkout thread

Since member value of EventThreadService can call API at worker thread.
It might make some resource leak.

For example, we can call animation.Clear() at worker thread.
If then, only event thread side informations removed, and nothing changed
for update thread.

To avoid this kind of issue
- Let we define interface class s.t. Get EventThreadService and assert if we access here at workerthread.
- Change every local value usage to using this API.

During test, let we just assert it always.
After several weeks later, it would be change as debug.

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

Ensure updatedTextures call OnRenderFinished safety

Since we can change the updatedTextures container during OnRenderFinished() callback,
we need to move OnRenderFinished() caller into seperated container.

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

Merge "DALi Version 2.3.48" into devel/master

DALi Version 2.3.48

Change-Id: Ifbd240f24487c3c8ea78d810c952e5e0e791f933

Support Squircle factor for DecoratedVisual property : CornerSquareness

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

Merge changes If6e7b493,Iebcd2ba4 into devel/master

* changes:
  Changed Matrix3 uniform write to use stride
  Added comparison ops for Graphics::Viewport and Rect2D

Make WeakHandleBase::mImpl as unique_ptr + nullcheck for Reset and comparision

Until now, WeakHandleBase::Reset() function and == operator
don't check mImpl was nullptr.

But it could be nullptr when we use move operations.

So we should check nullptr before call mImpl->Reset() and operator==.

===

Also, to avoid this kind of leak, let we use std::unique_ptr instead of raw pointer.

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

Changed Matrix3 uniform write to use stride

If the uniform is in a block, then each row of 3 is padded
to 4 floats(16 bytes). Can get the stride from GLES/Vulkan reflection.

So, use matrixStride if not in the standalone uniform block
(which means it's gles2 and packed tightly).

Added test cases to check Mat3 uniform writing in both standalone
and uniform buffer.

Change-Id: If6e7b49318dae795972d5ae76092df48f4713cab
Signed-off-by: David Steele <david.steele@samsung.com>

DALi Version 2.3.47

Change-Id: Iead3b5d3edf64e07270662cb024d3b0cd4ea85e7

Added comparison ops for Graphics::Viewport and Rect2D

Change-Id: Iebcd2ba4792cd4e6fad6423033fb4f592d32f2fc

(AddOn Manager) New API to load addons/libraries rather than via an ENV variable

Change-Id: Ic11dc6920e390842a1bde610c1a1f732b212ff00

Merge "Fixes in handling uniform buffers." into devel/master

DALi Version 2.3.46

Change-Id: I74fe57654290c80020d46791cb51164e73bef230

Fixes in handling uniform buffers.

- Using correct binding point for uniform buffers
- Getting correct UBO alignment offset from GL/Vulkan API

Change-Id: I34c752b3bb5ee770d9992c730810c58137b6378e

Merge "Adding depth/stencil requirements" into devel/master

Merge "Fix partial update issue when clipping mode changed" into devel/master

Fix partial update issue when clipping mode changed

When some node's ancient node change cliping mode from
ClipChildren | ClipBoundingBox to Disable,

we should partial update descent nodes.

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

Do not update componentwise animatable property's Set/Bake

We don't update PropertyOwner's update state when the value was
not actuall changed.

But it doesn't reset the dirty flag of AnimatablePropertyBase.

Due to the AnimatablePropertyBase is not a clean flag,
the renderer who use constrait become always check as dirty
when we check dirty rect.

To fix this kind of issue, let we don't call OnSet and OnBake
if the componentwise value is equal.
(Since float comparision is cheap enough)

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

Adding depth/stencil requirements

During ubo/program requirements look, also check to
see if any node/render pairs require depth/stencil
buffer to be switched on for the surface. (This will
have no effect in GLES backend, or for offscreens)

In graphics API command buffer, merged stencilOp & stencilFunc into single
stencilState; as Gl and Vulkan stencil methods take different combinations
of the same params.

Had to fix some stencil clipping test cases, as order of GL method calls
had changed. (Should change to test GraphicsAPI, not GL mock)

Ensured depth clear color defaults to 1.0 (the furthest depth)

Change-Id: Ifbf4f3349b93d4c72b7d1024d40ae21680e65c4e

DALi Version 2.3.45

Change-Id: If1a29d2fd97156f9ba938668bc8a2258f729658d

(Automated Tests) Render Surface Sync

Change-Id: I84e317ba916b98f5660e49830b6cafa5389d7642

Merge "Implmement ConditionalWait::WaitUntil" into devel/master

DALi Version 2.3.44

Change-Id: I6722f644d9f0b2b9d184b9426ec4317b05a3dc55

(dali_env) Ensure correct gtest libraries are installed

Change-Id: I16bedd65bf8c42388a9d852b07190c3a0ad182d3

(dali_env) Added option to install USD & its dependencies

Change-Id: Ia4b4d4ffd599f48d32e53f81db366a5cfb27c656

Implmement ConditionalWait::WaitUntil

Let we make wait_until feature for conditional wait for dali.

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

DALi Version 2.3.43

Change-Id: Ib8814b7b42f985f81a5e97e1b07a63754887aaa1

Merge "Added clip transform matrix to the Graphics Controller" into devel/master

Merge "Fix gcc-14 build error" into devel/master

Merge "Add Integration code for Shader + Open GenerateTaggedShaderPrefix" into devel/master

Added clip transform matrix to the Graphics Controller

For GL, this matrix should not be used, so doesn't affect
render time. For Vulkan, this will change to Vulkan
clip space, so adds another matrix multiply.

(Consider moving the projection matrix setup to graphics
controller...)

Moved the viewport/scissor setting to the secondary command buffer.
(It's not an issue for GL, but is for Vulkan)

Also ensured that the viewport near/far clip values are set.

Change-Id: I4c1842fad5766d9be769fe9dce79386f84b5459a

Fix gcc-14 build error

Let we include vector wrapper for custom actor impl

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

Merge "Add CommandBufferResourceBinding add resource well" into devel/master