Assert to add / remove observer during notifying

Let we assert the cases when observer list try to be changed during notifying.

But, allow to remove observer during notifying, only for Object cases.

Until now, we just change the observer list directly, so if notifying, then
memory corruption occured.

To avoid this case, follow as ProcessorInterface did, copy the observer lists
and check whether it is valid or not, only if somebody touch original list.

Note : For BaseObject::Impl case, it is not opened
and it didn't send notify when scene on/off.
The only observer for BaseObject::Impl is WeakHandle. and OnDestroy() didnt'
send callback that timing.

Note2 : PropertyOwner case, it is also only for update-render thread for DALi.
We can control that non of callback change the observers for this thing.

So let we assume BaseObject::Impl::RemoveObserver and
PropertyOwner::RemoveObserver will not be called
during desturct object.

TODO : Should we also control Object::RemoveObserver case?
TODO : Should we remove observer copy?

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

Merge "Fix several hit issue for offscreen rendering" into devel/master

Support for recognizing legacy shaders.

Legacy shaders should be bypassed through the shader processing
pipeline as they are.

Change-Id: I04b2d15c5e95edaaf78ddff3e5c3ca99096fcc2e

Merge "DALi Version 2.3.22" into devel/master

DALi Version 2.3.22

Change-Id: I2cf0620aa085c2c9de12bc47519570164f6d2e19

Fix several hit issue for offscreen rendering

 - Previous implementation
   - MappingActor is not hittable by default, but it was used to check
     whether it is hitted from OnScreen RenderTask or not in HitTestRenderTaskList.
   - There is nothing to check the layer including MappingActor is consuming hit.

 - Current implementation
   - For the OnScreen hit result, check whether there is a mappingActor of OffScreen
     hit results that can be hit in front of the OnScreen hit result.
     If it is, returns the OffScreen hit results.
   - If the OnScreen hit result is layer and the layer consumes hit(the layer must
     not be hittable), returns the OffScreen hit results.
   - If there is no hit in OnScreen but there are hit results from OffScreen
     RenderTask, returns the top OffScreen hit results.

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

Print logs if dali skip rendering

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

Do not allow to add duplicated renderer into actor

Internal::Actor allow to add same renderer, but SceneGraph::Node doesn't allow.

If we do something like this, the logic might be breakdown.

actor.AddRenderer(renderer);
actor.AddRenderer(renderer);
actor.RemoveRenderer(renderer);

///< actor will have renderer, but actor's node doesn't have renderer.

To match with node's logic, let we don't allow to push-back the duplicated renderer.
Instead, just check the duplication and return the index.

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

Add RenderTask::RenderUntil()

Change-Id: I264e29303deedaf0f8539137b26e7d0c3e8454ce

Add Inherited Visibility Changed signal

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

Merge "DALi Version 2.3.21" into devel/master

DALi Version 2.3.21

Change-Id: I9ebe010c86446d8cae0e243144673306e0235ac9

Merge "Add log in EmitSingleTap" into devel/master

Merge "Allows updating MinimumDistance and MinimumPanEvents of pan gestures." into devel/master

Add VIRTUAL_REMOCON and VIRTUAL_MOUSE device types

Change-Id: I67f3ae35d364253b22a1d218df6c8c1bda848e97

Add log in EmitSingleTap

Change-Id: Ic5c4831daa355195c272c298d478b7d1519a20b4

Allows updating MinimumDistance and MinimumPanEvents of pan gestures.

Change-Id: Ibe985928ae9e68f70fc4faf573284ce1ee956fb5

Merge "Minor reduce IntrusivePtr<Actor>'s meanless Reference and Unreference" into devel/master

Merge "Move OrderedSet as integration-api, so dali-toolkit can use it." into devel/master

DALi Version 2.3.20

Change-Id: Ib80a171ab80256a0a5cdb5e18484c79fdbbe138e

Minor reduce IntrusivePtr<Actor>'s meanless Reference and Unreference

During Actor.Remove() API call, we try to increase whole child actor's
reference and unreference again while iteration.

It might give "minor" overhead. So let we don't do it.

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

Move OrderedSet as integration-api, so dali-toolkit can use it.

Let we move the OrderedSet class as integration-api so dali-toolkit can use it.

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

Do not call RemoveObserver when we don't add it for PropertyResetter

It is possible that PropertyResetter destructed before Initialize()
when app terminate case.

If then, it is possible to got some crash when we try to access to
'destroyed' property owner, and call RemoveObserver();

To avoid this case, let we make ensure that we call initalized so
it is possible to call RemoveObserver().

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

Merge "Seperate KeepRendering reason more detail" into devel/master

DALi Version 2.3.19

Change-Id: I569d4803c7ca83bc4f955b35545665f3ec04c4e6

Seperate KeepRendering reason more detail

Let we said more detail reason why we need to keep rendering.
KeepRendering called vs FrameCallbackInterface return true vs Renderer behaviour is Continuously.

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

DALi Version 2.3.18

Change-Id: I27f0ea2fcd707fc371d79aeeb8c61532c90ae1cf

1. Fixes a bug where events do not occur when longPressGestures overlap. This is when using gestures with FeedTouch.
2. Remove unnecessary conditional statements.

Change-Id: If8a4e876cb0b6031e744b6f0b8a3beef6d28517d

Add SetTime in TouchEvent

Change-Id: Iad92d4c0e3e07fded7f1fdbb44e59d967502f2c9

Merge "Add SetTapMaximumMotionAllowedDistance" into devel/master

DALi Version 2.3.17

Change-Id: Icdfb9fe6152f2eae50bb2ab5a57570e111c30270

Add SetTapMaximumMotionAllowedDistance

Change-Id: I5c8dd3304774f73d2bda7ca5566406b05039e233

actorCoordinates were missing from TapGesture when FeedTouch was used

Change-Id: Icfc319322f02920c76923cdaffce3b69354bcf92

Updated README.md to include recursive pre-requisite

Change-Id: Ic40db23478b503520e228bd5734bf4ca406d6cbb

(HitTest) Overlay actors should not be clipped if CLIP_TO_BOUNDING_BOX is selected

Change-Id: Id76d47a773766e19f969d73bb912e5aa6518380e

DALi Version 2.3.16

Change-Id: I964b2baeaf0a5dd83ac6f00c624c5e22e73a7684

Merge "Clean initialization and shutdown of UniformBufferManager" into devel/master

Clean initialization and shutdown of UniformBufferManager

Ensure all unique ptrs are null initialized
Clean down UBO buffers in UpdateRender thread ContextDestroyed,
rather than waiting to clean down in event thread.

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

Merge "Fix build issues for windows platform" into devel/master

Allow to set renderpass tag even RenderTask removed from RenderTaskList

There was potential of crash if we set RenderPassTag after RenderTask removed.

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

SVACE Fix: Initialise all members in SceneGraph::Animation

Change-Id: I96c3d9e6606afd0dcab84053620fcadec64c685d

Merge "Append Stopped animation notify list at Update Animation time" into devel/master

Get the id of animation and render task

Let we support to get the id of Animation and RenderTask, follow by NotifyInterface notify id.

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

Append Stopped animation notify list at Update Animation time

Let we insert animation notification list at Animate() time, instead of
immediate.

This will make insure we will not notify signal for destroyed, cleared animation.
And it will increase performance minor.

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

Merge "Unregister all processors if we need" into devel/master

DALi Version 2.3.15

Change-Id: I5d758ab48c230b0b4a4c1893991628c7ad180a6a

Unregister all processors if we need

Let we ensure that all processor are unregistered after call
Core::UnregisterProcessors() API.

It will remove both Processor, and PostProcessor.

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

Merge "Add NoIntercept KeyEvent modifier" into devel/master

Remove useless flag at SceneGraph::Renderer.mDirtyFlag

We can use PropertyOwner::Updated() instead of mDirtyFlag.

Let we make use PropertyOwner system, instead of Renderer's own member value.

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

DALi Version 2.3.14

Change-Id: If6cd46a0c4579fdacc804665c4375fc0019e5bcd

Fix typo error

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

Do not recreate SG::Animation when we call Clear + Redefine Clear behavior

Let we make Clear don't destroy & re-create the SG::Animation.

And also, we don't need to cache current loop count at event side.
Let we ask it to scene graph side, Same as other values are doing now.

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

Fix svace : Assert when malloc matrix failed

It is possible to fail malloc memory. Let we notify this case.

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

Fix svace : assert when we fail to register animatable properties

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

DALi Version 2.3.13

Change-Id: Ic5d63355961a026b33214e70ab0f229f7916fadd

Add NoIntercept KeyEvent modifier

This modifier prevents the event from being intercepted. This, in turn, allows
the Accessibility module to intercept all events and then feed them back if not
consumed by the Screen Reader, without entering an infinite interception loop.

Change-Id: I667d5be40b1180c992852e2af025eb3b916742e8

Add updated area function to update-proxy

Add updated area function to update-proxy for partial rendering.
It is to improve the performance of update area hint.

Change-Id: I29960bd17e488147a5f382fe591346768ea686b2

DALi Version 2.3.12

Change-Id: Id01839aa0cf90116286e2d23dbe091794fa6f1f9

Merge "Do not re-create SG::Animation when we call Clear to empty animation + Clear reset some values" into devel/master

Merge "Fix coverity issue : auto cause copy" into devel/master

Merge "Change the DIRECTION_HORIZONTAL value." into devel/master

Safety guard for BaseObject during SetProperty

It is possible to delete self during SetProperty.
To avoid this case, let we keep the handle during
SetProperty / OnSetProperty / PropertySetSignal emit.

TODO : Is SetProperty is the only case to be break down?

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

Change the DIRECTION_HORIZONTAL value.

There is no difference in actual operation,
but the values ​​of DIRECTION_LEFT and DIRECTION_HORIZONTAL are matched just as DIRECTION_UP and DIRECTION_VERTICAL are the same.

Change-Id: I2c39ab7c48cb57e66ac5992a1e4134be7bed100b

Fix coverity issue : auto cause copy

Make some auto as const auto& if we can

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

Added ExecutionMode to RenderCallback.

ExecutionMode is a hint that tells graphics backend whether it should isolate native graphics API calls from main DALi rendering pipeline (ie. by creating individual GL context for the RenderCallback) or should inject calls directly into the DALi main rendering pipeline which will alter API state.

ExecutionMode::UNSAFE - injects calls directly, unsafe
ExecutionMode::ISOLATED - creates isolated context

Change-Id: Iac679a87ee47b95c623182d669d26664fdda7cc9

Merge "When a hover event enters a new actor, events are delivered in the order of start -> leave, but are modified to be delivered in the order of leave -> started." into devel/master

Do not re-create SG::Animation when we call Clear to empty animation + Clear reset some values

Since we always destroy & create scene obeject whenever we call Clear(),
It will be make some useless memory increasement when user call
Clear() infinitely.

To avoid this cases, let we re-create that object only if we call Play() before
or at least 1 animator exist.

And also, make Clear() API reset some sensitive values, like CurrentLoop and State.

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

Merge "DALi Version 2.3.11" into devel/master

DALi Version 2.3.11

Change-Id: I6cf2537b57fa1e81ba245372f9222615caff1fce

Merge "Support to get shader language version for shader static API" into devel/master

Merge "Cache traveler only if the node was child of root node" into devel/master

Merge "Added wait/client wait methods to sync abstraction" into devel/master

Merge "Change Layer's ReuseRenderers check only if we need" into devel/master

When a hover event enters a new actor, events are delivered in the order of start -> leave, but are modified to be delivered in the order of leave -> started.

Change-Id: Ib917930b4e48b3ded41f10cebce62af41046ddce

Cache traveler only if the node was child of root node

To avoid cache node infinitly, let we cache only if the node was child of root.
If then, we don't worry about memory increasement.

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

Added wait/client wait methods to sync abstraction

Note, sync abstraction isn't used by dali-core, but still
has a test implementation, so not going to move it to
dali-adaptor in this patchset.

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

Support to get shader language version for shader static API

Let we support to get shader language version

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

Merge "Add shader class name so we can debug easly." into devel/master

Change Layer's ReuseRenderers check only if we need

Until now, we check ReuseRenderers value even though we make it as false.
It may required iterate O(RenderItem) times. So we can reduce useless check.

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

DALi Version 2.3.10

Change-Id: I60227547556b391ef337ea6a2688c11e2663c311

Do not travel scene tree what we don't need to

During Renderables adding cases, there was some hidden tree treversal exist
when the view visibility changed (false --> true).

This mUpdated = true; will be used when we check DamagedRects.

Since we dont need to propargate this informations to all decents,
let we propargate this visibility changed information only for this
recursive function, and forget.

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

Add shader class name so we can debug easly.

Let we allow to apply / get the name of shader.
It will be useful when we want to control default shader + debug it.

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

Merge "Optimize render task completed notify signal" into devel/master

Merge "Touch and Hover event propagrated by geometry way.(5)" into devel/master

Merge "Remove inclusion of gl-abstraction.h" into devel/master

Optimize render task completed notify signal

Let we follow up RenderTask also optimized like Animation.

The relationship between RenderTask <-> RenderTaskList is euqal with
Animation <-> AnimationPlayList. So we can use same logic.

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

Merge "Fix INTEGER_OVERFLOW coverity issues" into devel/master

Merge "Add OrderIndex to RenderTask to control rendering order between RenderTask" into devel/master

Touch and Hover event propagrated by geometry way.(5)

1. If there is a consumed actor, the intercept is sent only up to the moment before the consumed actor.
2. To propagate the gesture, check whether it has been detected and the NeedGesturePropagation flag.

Change-Id: Icfbec13c000092ae8526e0a269cd180fd7431a9e

Remove inclusion of gl-abstraction.h

Remove all #includes.

Would like to move this abstraction to dali-adaptor, but many tests use the test-gl-abstraction
layer to check that e.g. uniforms have been written. Instead, they should be modified to use the
test graphics layer, but that's a lot of work :/.

So, have just removed the usage, but left the file there.

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

Add OrderIndex to RenderTask to control rendering order between RenderTask

Change-Id: I7aa9bec7a27c04878d1842751ea64813068d582f
Signed-off-by: seungho baek <sbsh.baek@samsung.com>

Fix mis-implementation for GetLinearEnumerationName

Let we fix mis-implementation of GetLinearEnumerationName,
and add relative UTC for it.

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

Merge "Removing old temporary graphics APIs" into devel/master

DALi Version 2.3.9

Change-Id: I2a50f5f3d7de05bcad0e3c73dc8cdeb08fd16180

Fix INTEGER_OVERFLOW coverity issues

 - LinearConstraint : min value could be zero. We should branch out that case.
 - Render::Geometry : The number of indecies could be zero.
 - BaseSignal : Callback might be nullptr (even if we don't allow this cases)
 - Program : we might fail to find '[' keyword.

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

Removing old temporary graphics APIs

Change-Id: I41c5af839ab5b0807194ebe2ecedbbaa27ba033c

Removing unused threadpool

Change-Id: Ia2cccb3bee0068b15f5eff1c23d46ff62cc97b3b

Merge "Remove duplicate codes for screen position" into devel/master

Remove duplicate codes for screen position

- There are some duplicate codes when screen position and screen extent
are calculated.
- Remove these duplicate codes to make new methods for calculation

Change-Id: I5c9a50f5125933013bfd20a85c13f55a16fc4387
Signed-off-by: Seoyeon Kim <seoyeon2.kim@samsung.com>

Keep node by id + Make SceneGraphTraveler don't travel anymore

Let we keep whole nodes cache by id.
And let we make scene graph traveler use this system instead of traveling.

Previously, if we want to check whether given node is under root node,
we should search whole childrens (travel up-to-down).

Instead, let we travel down-to-up, s.t. will spend much less time than before.

And also, let we make a way to user don't use root actor for frame callback.
It is not a neccessary if we ask to node to mUpdateManger.

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