Merge "Ignore AnimationFinishedCallback when we call Animation::Clear() hard + Resolve bug when we call Play multiple times" into devel/master

DALi Version 2.3.26

Change-Id: I8aa7325ed7b1a0ea7980c47e6c9f062db0c3060b

Ignore AnimationFinishedCallback when we call Animation::Clear() hard + Resolve bug when we call Play multiple times

There was several bugs at AnimationPlayList.

1. Core::ProcessCoreEvents sequence is like this :
     SceneEvent -> NotificationManager -> Relayout -> Flush.
   If we call Animation::Clear() at SceneEvent timing, Notfication will send animation finished signal
   what we must not send.

   To avoid this system, let we store the cleared animations at AnimationPlayList.

   Since AnimationFinished callback required at least 1 frame times after flush,
   we can ignore whole notify events before Flush done.

2. Scene mPlayList keep the reference of Animation whenever we call Play, there
   were several problems. For example, If we call Play() 2 times and Clear(),
   that Dali::Animation never be removed.

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

Remove render once trace log

Since it print too much logs, let we remove it

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

Merge "Add FittingMode for visual fittingMode" into devel/master

Merge "Apply RenderUntil() on sorted RenderList" into devel/master

Apply RenderUntil() on sorted RenderList

Change-Id: I64a3b567984494d1f284095d68e44b7e318a04d5

Implement ProcessorOnce

Since unregister processor during Process will give overhead,
we'd better make another container to process only once time
during ProcessCoreEvent.

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

Optimize Add/Remove SG::RenderTask from SG::RenderTaskList

Let we make render task list use OrderedSet instead of vector container.

Vector container is slow if we call RemoveRenderTask.

TODO : Optimize event side render task list too.
But since it is quite complex, we'd better optimize that points
in another patch

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

DALi Version 2.3.25

Change-Id: Ib32a2c7cb1d61bff3955047cb03be7142d14693f

Control core policy as one flag

Let we collect core creation status, instead of parameter increasement.

It will be useful when we add more features of Core.

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

Performance trace for some bottleneck of update/render manager

Let we check trace the top 5 area of update : update itself / animation /
frame update callback / update renderer / transform
and top 2 area of render : render instruction process / flush

TODO : Since the name of environment need to be sync with peformance
trace tool, we'd better change it after the system prepared.

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

Merge changes Ib3f51bd8,Ibfbeca64 into devel/master

* changes:
  Pint log if animation play / stop / pause / clear / destroy
  Print top5 longest time spend AnimationFinished & FrameUpdate callbacks

Pint log if animation play / stop / pause / clear / destroy

Let we know the animation's state by id, if user want to know.

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

Print top5 longest time spend AnimationFinished & FrameUpdate callbacks

Very rarely, we got some report from performance team that
"DALI_NOTIFICATION_NOTIFY_COMPLETED" and "DALI_FRAME_CALLBACK_UPDATE" is bottleneck.

To escape the false accusation, let we collect the App side callback
execute time and print it.

This time checker already merged at tizen_8.0 branch.

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

Even when multi-touching, the captured actor is used without a hittest every time.

Change-Id: Ib934b4b5626747a9ed7036d7489f82c94e7b4b16

Merge "Remove an unneccessary argument at Process()" into devel/master

Remove an unneccessary argument at Process()

Change-Id: I62c8ab7fe11b28c67fa3fe5e277142d969af8698

Add FittingMode for visual fittingMode

Add new type for visual fittingMode

Change-Id: I17cb51564117a1c0426052dac547d0beb944e1c2

DALi Version 2.3.24

Change-Id: Ifbf8d83b68753dea68479978185e75da8fb81b49

Merge "(Partial Update) Refactorize PartialRenderingData check logic" into devel/master

Merge "Add compile time option to memory management for low spec devices (Dali::Vector, OrderedSet and memory pool)" into devel/master

Merge "Use some observer list use OrderedSet instead of Dali::Vector" into devel/master

Merge "Assert to add / remove observer during notifying" into devel/master

(Partial Update) Refactorize PartialRenderingData check logic

Since we check node's partial update data only 1 times and then update directly
when we add RenderItem.

Unfortunatly, VisualRenderer has there own update area calculation logic
(like offset and size) and Node can add multiple renderer,
The partial update data said that it is not updated for the 'second' Renderer.

So it was possible that second renderer's update area could not be applied
when we check dirty rects.

===

To fix this issue, we reset the partial update data flags for all nodes every begin of
update. And keep + calculate the partial data only 1 times.

If we check the updateness at this frame, keep this infomation and reuse it.
It will avoid useless Hash calculation.

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

DALi Version 2.3.23

Change-Id: Ie50bf0ceb4c85aafd71a46b2497e30e419e99d33

Add compile time option to memory management for low spec devices (Dali::Vector, OrderedSet and memory pool)

Let we give some option to reduce memory capacity for low spec devices.

If app create a lots of Actors and release all of them,
the memory might not be reduced, since we don't release capacity of continous array.

Most of memory bottleneck occured at Transform::Data and Node memory pool, and
Graphics::CommandBuffer.

CommandBuffer pool control at dali-adaptor. So let we control what dali-core
can reduce capacities first.

Note : 64bit OS will always use memory pool feature even if we give compile option
, due to KeyType is uint32_t.

USAGE :

 * gbs build -A armv7l --include-all --define "%enable_low_spec_memory_management 1"

 * cmake -DCMAKE_INSTALL_PREFIX=$DESKTOP_PREFIX -DENABLE_LOW_SPEC_MEMORY_MANAGEMENT=ON

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

Use some observer list use OrderedSet instead of Dali::Vector

Let we use Integration::OrderedSet instead of Dali::Vector.

Integration::OrderedSet has good point if we try to remove the items.

For BaseObject::Observer case, it will be called only 1 times per
each object's lifetime. So we have low priority of iteration overhead.

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

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