Fix an issue where TapGesture Events are not being received.

If multiple Tap GestureDetectors are registered, you may not receive gesture events depending on the setting of MaximumTapsRequired.

example)
TapGestureDetector detector1 = TapGestureDetector::New();
detector1.SetMaximumTapsRequired(1u);
detector1.Attach(actor1);

TapGestureDetector detector2 = TapGestureDetector::New();
detector2.SetMaximumTapsRequired(2u);
detector2.Attach(actor2);

When actor1 is tapped multiple times, I expect to receive a gesture event on every tap because MaximumTapsRequired is 1u, but I don't get the event every tap.

It is because the MaximumTapsRequred of detector2 is set to 2u, the MaximumTaps of the tap-gesture-recognizer is set to 2u.

So, if there is a double tap, the tap event is not delivered to detector1 because it exceeds the MaximumTaps of detector1.

For this reason, the check of MaximumTapsRequired is modified to be checked by each detector.

Change-Id: I297debdf7ff680affafc763d4af3db323f80a095

Merge "DALi Version 2.1.29" into devel/master

DALi Version 2.1.29

Change-Id: Icb5bea910e30730302e73906f873469afb3146f4

Add InterceptKeyEvent

Intercepts KeyEvents in the window before dispatching KeyEvents to the control.
If a KeyEvent is consumed, no KeyEvent is delivered to the control.

Change-Id: If6ee2b2fb03d936a7840b4c984129a31bb8b948d

Add GetPixelFormat to Texture

Change-Id: Ib9cf0b03f409c909e7c7187d8ea19661296d36d5

Merge "DALi Version 2.1.28" into devel/master

DALi Version 2.1.28

Change-Id: I7cea4233901b6d16f8edba3eb3be48317ae49e23

Add default logs in key event processor

Change-Id: I3b8352cb57afe5b6de381f6285d7020786df0578

Enable ENABLE_BACKTRACE

Change-Id: Ib992346a029132000b2558b41a934c2b6115b647

Fixed SVACE error in pan-gesture-processor.cpp

Change-Id: Ia4b8d58e4dcffd1f4da85a0a5a4c8fd077693532

Fix logical bug on Pan MotionEventAge

Fix incorrect operation direction of age check-up.

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

Merge "DALi Version 2.1.27" into devel/master

DALi Version 2.1.27

Change-Id: I079653c71810ddbf0469601c0e2f594c23fb630a

Discard some motion during pan gesture

When event spend so long time, a lot of events are queued.

Ecore cannot skip these event, so we make some heuristic
threshold of discard motion events on pan gesture.

We can setup this value by PanGestuerDetector's member vaule.
The name is "MaximumMotionEventAge".
the time scale is millisecond.

Default age is UINT_MAX ms (~= 49 days)
That mean, we will not discard every motion events.
(Ecore event callback also use uint32_t type in ms.)

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

Make mScene nullptr when the Actor is disconnected from Scene recursively

Change-Id: I2ef30d3ba471d87baae32d082e91426990d46463

DALi Version 2.1.26

Change-Id: Ie428ca744a9fbb266251cb3cdc36eafa3022214b

Minor optimize during iterate of uniform map

Dali::Vector.End() operation is heavy.
We don't need to calculate this value every iteration. So just precalculate
the end-of-iterate. It will increase performance.

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

DALi Version 2.1.25

Change-Id: Ia40d6c528558b1c2b7223acf67240ef63e43a91a

Merge "DALi Version 2.1.24" into devel/master

DALi Version 2.1.24

Change-Id: I8fe305f73b9987fd87366d06cb3c9225d3375231

Revrert "Enable assert alwayd in Dali::Vector"

Due to the DALI_ASSERT_ALWAYS, Dali::Vector try to check
the range of index every times.
It might make some performance reducing.

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

Merge "DALi Version 2.1.23" into devel/master

DALi Version 2.1.23

Change-Id: Ie34538906bedaf4a85e012130fe9c11a0ca78fb0

Fix when shader changeness not applied issue

If RenderList was reused, SG::Renderer::UpdateUniformMap didn't called.

So Render::Renderer will render with not-updated uniform-map
(But uniform map change counter is updated.)

This patch make sure that renderer's uniformMap changeness in Prepare time.
So it will be applied well before render call.

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

Merge "base-signal connect and disconnect in O(1)" into devel/master

Minor optimize during build uniform index maps

If node don't have any uniform properies,
we don't need to create about that node's index map.
Because that mean, this node will use only for renderItem's uniform.

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

Keyframe->GetValue works on O(log n)

Previous code iterate whole key-frame to get interpolated progress value.
This patch make this job in O(log n).

So we can run animation faster even it have near 500 key-frames.

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

base-signal connect and disconnect in O(1)

Make base-signal control the connection in O(1)
by std::unordered_map and std::list<>::iterator.
and make connection-tracker control the connection in O(1)
by std::unordered_map

std::unordered_map can found duplicated value fast
and std::list<>::iterator can access & remove value fast.
And also, std::list can keep ordered by inputed time.

TODO : Remove the dependency of std::vector header in dali-signal.h
is quite big process. Will be done in other patch

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

Merge "Apply Visual's transform + borderline properties for partial update" into devel/master

DALi Version 2.1.22

Change-Id: I4f1ac3a9a654f2b1469d6d8a1f189e646df30ebc

Apply Visual's transform + borderline properties for partial update

Previously, PartialUpdate cannot know the vertex positions
after transform the visual.
But now, we can use VisualRender's animatable properties.

This patch re-calculate the partial update size
so we can consider Visual Transform.

And more, We make DecoratedVisualRenderer so we can use
CornerRadius + etc as default property
and also can re-calculate update size consider with BorderlineWidth or BlurRadius

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

Delete Renderers before Shaders

Change-Id: Ib7744acb7f4bed8032c50f7a388f74027afb55b2

Remove duplicated scene object deletion from VisualRenderer

Change-Id: I50ef48dd8d5253360f2bc0b564ad6e84395f329a

Merge "Add API to UpdateProxy" into devel/master

DALi Version 2.1.21

Change-Id: I87bb688c0b7635d23bc6f0d85237ba14549dfb34

Add API to UpdateProxy

Change-Id: Icd9ef2011a6381ccad62a9f3edea750e392500ae

Merge "DALi Version 2.1.20" into devel/master

Merge "Simplifying UniformMap updating" into devel/master

DALi Version 2.1.20

Change-Id: I1c92a82a41be6d385c7ab307b24edc04b25b730e

Fix memory leak of visual renderer

SceneGraph::Renderer didn't remove the memory of
ANimatableVisualProperties.

This patch make AnimatableVIsualProperties as OwnerPointer
so SceneGraph::Renderer will delete it's memory well.

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

Simplifying UniformMap updating

Removed scene-graph ConnectionChangePropagator - now only need to know
if shader is attached to renderer, and that is already done through a
different mechanism.

Changed CollectedUniformMap into it's own struct. Now has a change
counter that increments any time the map is altered.

Instead of having lots of flags indicating if uniform maps have
changed, UniformMap now updates it's change counter whenever a
property is registered with the PropertyOwner (e.g. Node, Renderer or
Shader).

During Update, SceneGraph::Renderer collects the uniform maps into one
only if the renderer is added to a RenderList, i.e. if it's going to
be rendered. At this point, the change counter is updated.
Nothing is done if the renderer is not visible.

During Render, the Render::Renderer checks if the node uniform map
change counter or the renderer data provider's uniform map change
counter are different, if so, then it knows it needs to update the
uniform locations for newly registered properties.

Change-Id: I0ff861a7e97736dae059d94f54618195d0471343

Calculate hash by Dali::Vector<std::uint8_t>

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

Merge "DALi Version 2.1.19" into devel/master

DALi Version 2.1.19

Change-Id: I9be4ce8eef03b28d4e208cd648e1210446967801

Merge "Relayout only if the actor is on Scene" into devel/master

Lock uniform buffer only 1 times per each render + minor fixup of uniforms

1. Don't lock-unlock Graphics::Memory for each properties.
This patch lock stand-alone uniform buffer map and unlock only few times during rendering.
(We called this API as ReadyToLockUniformBuffer and UnlockUniformBuffer)

It will reduce Renderer::Render time near 6%

2. Don't convert from std::string_vew to std::string
when we get uniform map info from shader

3. Make BufferPropertiesFlags as input of UniformBuffer Constructor

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

Merge "Fix to do not update state of render task in case of uploadOnly" into devel/master

Fix to do not update state of render task in case of uploadOnly

 - And removed unused uploadOnly parameter in the render-manager

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

Relayout only if the actor is on Scene

We're already checking whether the actor is on scene in RelayoutController::Relayout() before it is added to the mRelayoutStack.
But there is a case that the actor is removed from the scene in the previous OnRelayout.
So check whether the actor is on scene again.

Change-Id: Ia2b64e7f39f24d67b65b9689c720a7ef359d54fd

Merge "Add CHROMINANCE pixel format" into devel/master

DALi Version 2.1.18

Change-Id: I044099ba626788b31036834921d153a2fa0fa6eb

Add CHROMINANCE pixel format

Change-Id: I6c66da9db49874c7273e2580964423dc23c36e90

Merge "Changed handling of uniforms of arrays of structs" into devel/master

Changed handling of uniforms of arrays of structs

A) For uniforms of the form "basename[index].element", the array index handling
didn't work.

In GL (on Ubuntu) the shader reflection for such uniforms provides
individual locations for each element of the array/struct;

B) For uniforms of the form "basename[index]", where basename is a basic
type (float/vecN, etc), the GL shader reflection produces only 1 location
for the basename, and provides the element count.

Mapping properties to such uniforms is done by having a Property per
element. For the first case, no index/array handling is needed.
For the second case, each property needs to match to the basename without
the array subscript, and also store it's array index.

Modified property setup to remove array index for properties of the first type.

Modified uniform lookup to handle the different hashes appropriately.

Modified test graphics to ensure the uniform reflection can be set up in the
same way as GL provides.

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

Merge "Added New() function creating Renderer with RenderCallback" into devel/master

DALi Version 2.1.17

Change-Id: I30ecdbddcd58603a6b1a4c2adf13b7311cbd3b0f

Added New() function creating Renderer with RenderCallback

Change-Id: I6f82838ed654342723cf043e43e8d7caba4125f7

Merge "Level up Dali::Internal::FreeList as Dali::FreeList" into devel/master

Merge "Use update object size in CalculateActorScreenPosition" into devel/master

Merge "Hit-test can hit more than 32 depth clipping actor." into devel/master

Level up Dali::Internal::FreeList as Dali::FreeList

Open devel Dali::Internal::FreeList as Dali::FreeList
so we can use this container in another packages i.e. dali-toolkit's texture-manager

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

Merge "Fixing culling for magnifier use case" into devel/master

Fixing culling for magnifier use case

Should use the camera's canvas size, not the viewport.

Added canvas size to left/right/top/bottom clipping distance
in camera for perspective projection

Fixed bug that would always force using perspective projection
Fixed bug with modifying scene object directly in event thread.

Top and Bottom clipping plane values for perspective have been
made consistent with values for Ortho projection; however, this
currently forces top to be +ve and bottom to be -ve when set
using just size. Test cases are inconsistent in this regard; when
set via l/r/t/b/n/f API, top and bottom are -ve and +ve respectively.
  Have used fabsf when re-calculating bounding box dimensions to
allow for this inconsistency.

Change-Id: I95e80e4c09f17a3240cccf16df23f09e8d9ad9cf

Use update object size in CalculateActorScreenPosition

Change-Id: I475166706a0eb4defd1761ce56d185da7fe753a1

Hit-test can hit more than 32 depth clipping actor.

When clipping mode is CLIP_TO_BOUNDING_BOX, the limitation of
clipping depth doesn't exist. We can clipping near 2^15-depth.
But current clipping algorithm only allow 31-depth of clipping.

This patch make that we don't use depthmask in hit-test algorithm
When clipping mode is enabled, and current actor hit test failed,
just skip all children's hit test.

Note : CLIP_CHILDREN is kind of rendering method, So when clipping mode is
CLIP_CHILDREN, we can't detect how it "really" clipped.
This is physically limitation, so just ignore it.

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

DALi Version 2.1.16

Change-Id: If043023658a6a6f59d220166a67cd5363d866faf

Revert "If CapturesAllTouchAfterStart() is true, it should be hit only after touchdown."

This reverts commit 3b0fba86dfcad3ae5841679e1c1790db563d7301.

Change-Id: I91350d5f72b826bc7b6fad4920b89079e4fc7d53

Fix hit-test bug when overlay2D child is not overlayed

Let's image some case like below scene tree.
root --- A (overlay2D)  --- A1 (normal)
      |- B (normal)

When A1 and B is overlaped, B will be drawn under the A1. (Because A is overlay)
But, in hit-test-algorithm, overlay2D information doesn't propagated.

This patch make hit-test can propagate overlay2D property,
so the hit test result return reasonable as what we can see.

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

Merge "TTrace enable always in tizen" into devel/master

Merge "If CapturesAllTouchAfterStart() is true, it should be hit only after touchdown. So, If the touch moves after another actor has been touched so that the current actor is hit, it should behave as if it didn't hit." into devel/master

TTrace enable always in tizen

Make ttrace enable always in tizen. Other platform keep current policy

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

DALi Version 2.1.15

Change-Id: I98023d937a38affaa836fed830c12aeeddda9b59

If CapturesAllTouchAfterStart() is true, it should be hit only after touchdown.
So, If the touch moves after another actor has been touched so that the current actor is hit, it should behave as if it didn't hit.

Change-Id: I609607ea19a5cf5f6b07b5065132f491122f7a6b

Merge "Remove element of Property::Map by the specified key." into devel/master

Remove element of Property::Map by the specified key.

Change-Id: Ife048e33a54edb6b6cebfe79b464707c0a361a45

DirectRendering:

- Added RenderCallback for native rendering
- Added new Graphics API command - DrawNative()
- Added DrawableActor

Change-Id: If286d6eddf0836c63227c870eb13e8bab252fda7

DALi Version 2.1.14

Change-Id: Ifc21ce4863a0b29e278f50410facba6158caa4a4

Merge "Add UserInteractionEnabled property on actor for controlling user interaction." into devel/master

Add a Hit-Test result events.

In the case of TouchEvent or Gesture, there is no way to propagate the event to the view below that is not related.

So, before sending an touch event, send an hitTestResult event to the view in the hit-test process to ask whether it will be hit or not.
If it returns false, it means that it will not be hit, and the hit-test continues to the next view.

Change-Id: I5b8681f569f5c3b89acc3fb23c1eada3d49b1c7b

DALi Version 2.1.13

Change-Id: I69b796cffa8be10d5e5fa21226d19e1f7535f436

Removed old context test cases

Synced with toolkit tests

Change-Id: I67f21c445cb9b17972e8994bee558d0bd0836e58

Added VisualRenderer (for visual default properties)

Rather than have registered properties for each visual,
embedded them in a specific type of VisualRenderer.

Added UniformMappings to ensure that they can be used as
uniforms. (Properties don't have to have the same name
as the uniform, but it's probably advisable. However, we
have some very specific hardcoded mapping for actor "size"
-> uniform "uSize" in render-renderer...), so it's less
confusing having property "transformSize"->uniform "size".

Changed types to ensure they match the actual uniform types
in the shaders.

Reduces creation time for each visual, as much more is done
at compilation time.

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

Add UserInteractionEnabled property on actor for controlling user interaction.

actor can be disabled touch event by setting SetSensitive(false),
and also can be disabled keyboard event by setting
SetKeyboardFocusable(false),
and touch focus on SetTouchFocusable(false).

but we do not provide universal user interaction controls to disable
actor events and behavior, so this property is neceesary.

regardless of value on Sensitive or Focusable,
UserInteractionEnable has high priority over the negative action.

Change-Id: Ia339cc4c94e2fa36f80cb8672aa16d7a664c858c

Make Dali::InstrusivePtr able to compare with nullptr

After explicit operation bool() patch, Dali::InstrusivePtr == nullptr make compile error.
This patch fix that situation, so keep codes legacy

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

DALi Version 2.1.12

Change-Id: I37214142c68799c7c0f2ecea4a7ef26694c6273a

DALi Version 2.1.11

Change-Id: I949fdf526b1c5bd2c16d18751e2014389ca664a1

Merge "Implement some more speed-up : Reserve TextureBindings + don't copy geometry" into devel/master

Merge "Dali::CalculateHash support to hashing raw-buffer" into devel/master

Add Internal::IndexedMap

Make IndexedMap that key can be std::uint32_t and Internal::ConstString
IndexedMap can only do Register + Get.
We can check unique registration by this API.

IndexedMap use iterator as std::vector<std::pair<KeyType, ElementType>>::iterator.

Get API return iterator that key contained.
If found failed, return End() iterator.

Internally, we make it sorted key-index-element container.
It can search key by binary search. and less element copy action.
(ConstString can be convert const char* == size_t which can comparable)

Time complexity is O(|Element count|) for Register and
O(log|Element count|) for Get

Change some std::vector container as IndexedConstStringMap and IndexedIntegerMap
at type-info and type-register.

type-info is very important area of DALi engine.
So I append some define blocks when we meet some strange problems in future.
It will be remove when this patch is safe enough.

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

Dali::CalculateHash support to hashing raw-buffer

Current Dali::CalculateHash input std::string, but internally
convert as const char * string.
In this case, hashing calculation didn'w works well
when std::string contain \0 character middle of string.

This patch make one more API s.t. allow std::vector<std::uint8_t>
as input so we can specific the length of data what we want to hash.

It will be used in dali-toolkit texture-cache-manager in soon.

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

Implement some more speed-up : Reserve TextureBindings + don't copy geometry

Make some GraphicsCommand API's input as const vector<>&

These will not copy the internal values. and also we can assume that
inputed data didn't change in that API

+

We reserve textureBinding's capacity as the Count of textures.
Most of texture have graphics object, and it will fitin that
reserved memory area.
This patch reduce unuseful re-allocation internal std::vector.

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

Ignore insert/erase function during PreRender's itemsDirtyRects

Previous code just insert and erase when the renderer hold dirtyRect
more than 3.
Each insert & erase operation in std::vector cost O(N). and it is expensive.

This patch reduce the insert & erase operation when renderer hold dirtyRect
more than 3. We can do tge same job by just moving data linearly.

(This patch is follow up 250728)

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

Avoid temporary vector creation during Relayout

We use temp std::vector stack during NegotiateDimensions.
This patch make we use always same global static value as stack.
So we can avoid create / destroy it every frame.

(This patch is follow up 248736)

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

Merge "DALi Version 2.1.10" into devel/master

DALi Version 2.1.10

Change-Id: I255826e810ab53dc5f41a77c1e2416bfbaf4a517

Merge "(Partial Update) Set the updated flag when the Renderer is added/removed" into devel/master

Add EmitWheelEventGeneratedSignal for custom wheel event

For custom wheel events,
we first send the event to the focus manager by WheelEventGeneratedSignal.

Previously, RotaryEvents(CustomWheel type) could always be received only by window.
Now, User can receive Rotary Events in focused View as well.
It is also possible to propagate events to the parent view.

If there is no focused View, the window will receive the event.

This only applies to Rotary Event(CustomWheel type).

Change-Id: I13e2c9b725ee5a65c7408df2f4293c7b61c56035

(Partial Update) Set the updated flag when the Renderer is added/removed

Change-Id: I586777f0bda8dde74ee04d461b54e7165e74b35c

Merge "Add stride to PixelData" into devel/master

Added prefix matching to test case execution

Change-Id: I5a836aca4c788a347b383a0b3d23771e22ff8f2b