[Tizen] 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 branch 'devel/master' into tizen

Change-Id: I494c84dbdfbcb8790776d33e3d1a7c6246b6dc83

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

This reverts commit 6b643f87928bd5afececac7901b39ca05d62ed4b.

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

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

This reverts commit 3b0fba86dfcad3ae5841679e1c1790db563d7301.

Change-Id: I91350d5f72b826bc7b6fad4920b89079e4fc7d53

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

This reverts commit 3b0fba86dfcad3ae5841679e1c1790db563d7301.

Change-Id: I91350d5f72b826bc7b6fad4920b89079e4fc7d53

Merge branch 'devel/master' into tizen

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

Merge branch 'devel/master' into tizen

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

Merge branch 'devel/master' into tizen

Change-Id: I99f7799ba6b938a1ffb294f8793e9048b2157a56

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

Merge branch 'devel/master' into tizen

Change-Id: I8acdcd16aaf3caa04bd9aff85e883bd694620ccd

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>

Merge branch 'devel/master' into tizen

Change-Id: Ib9190ec8d1aba1afd3db69201bff7bb0efedbac6

Revert "[Tizen] Make Dali::InstrusivePtr able to compare with nullptr"

This reverts commit 7f65f68f7fc73f55c6e88809a344a930e8574bcf.

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

[Tizen] 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>

[Tizen] Not execute the remove callback

This reverts commit 3c690e990412acf2174e85467bf8f60b89ed649a.

Merge branch 'devel/master' into tizen

Revert "[Tizen] Not execute the remove callback"

This reverts commit a2e9ce7ec4971860c6e41853873eae7455bd999a.

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>

[Tizen] Not execute the remove callback

This reverts commit 8e457cd26e7fdd41c8a6a9483226755c3e49b503.

Change-Id: Ib0c764153076ea3918db6ff60f2c41dcc60b376e

Merge branch 'devel/master' into tizen

Change-Id: Icbe53d117677a4c2d4e47939bb3fb9362b2e589e

Revert "[Tizen] Not execute the remove callback"

This reverts commit 8a70172996a59e3d50af5bcc62d818b98598e300.

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

[Tizen] Not execute the remove callback

This reverts commit 63a02032b00033cacc6834507c594f43322e8c7b.

Merge branch 'devel/master' into tizen

Revert "[Tizen] Not execute the remove callback"

This reverts commit 187dc40447c6095e257f76730c78c7f62fb7d1ca.

Merge "Add stride to PixelData" into devel/master

Added prefix matching to test case execution

Change-Id: I5a836aca4c788a347b383a0b3d23771e22ff8f2b

Merge "If MaximumTapsRequired is 1, it is not waiting for a multi-tap, so a Tap gesture send a single tap." into devel/master

DALi Version 2.1.9

Change-Id: I7e8ba8a3683ccd489e8c6afcdcb0072fe8db2b29

Add stride to PixelData

Change-Id: Ida564386962e1afed91d3f6805b5a14f446a6937

If MaximumTapsRequired is 1, it is not waiting for a multi-tap, so a Tap gesture send a single tap.

Even if set to tapGestureDetector.SetMaximumTapsRequired(1);
When user tap multiple times quickly, the tap gesture event now fires only once every maximum Allowed Time (500ms).

If MaximumTapsRequired is 1, user want to receive tap gesture events immediately because we are not waiting for a multi-tap.

Change-Id: I9651facb9650982e2e0dc414974677c531481bf8

[Tizen] Not execute the remove callback

This reverts commit e56c5650f0f350a721c60bed4701e6b20b1e1236.

Merge branch 'devel/master' into tizen

Revert "[Tizen] Not execute the remove callback"

This reverts commit 433b4a4e79ff9171e54b92c3cd8036bed8adde34.

DALi Version 2.1.8

Change-Id: If05e41c6141e9e20dd655418b04843b9e7b72225

Merge "Adding Handle::ReserveCustomProperties method" into devel/master

DALi Version 2.1.7

Change-Id: Iccbe522c45fd61d039458b809fa4f86f2423eb78

[Tizen] Not execute the remove callback

This reverts commit 158d381240f8ed85ae9eb30feda650e883a3409b.

Merge branch 'devel/master' into tizen

Change-Id: I31e4cd9f6dc8ea2e3f40345573c0793ff23322f0

Revert "[Tizen] Not execute the remove callback"

This reverts commit e836be5c957a0a7285bfe26b466e69aa840ac9b8.

Adding Handle::ReserveCustomProperties method

Change-Id: I2baa3a77e2db557a6a755e66105530cbe72ac442

(Partial Update) Use world color to check if the item is updated

Change-Id: I2d1ca5df07cd1c0456f77c178d60e2bf463448ed

Merge "(Partial update) Change resetting the updated flag" into devel/master

(Partial update) Change resetting the updated flag

Reset the updated flag after calculating all damaged area of the scene

Change-Id: I6abf26012f3053689abb461d42c2960b09886895

Initialized SG::Scene::mGraphicsController in constructor

Change-Id: I813f9643c1e4e92a6e2f01a342a2c6981b479f7a

Merge "Changed SceneHolder's RenderTarget initialization" into devel/master

Changed SceneHolder's RenderTarget initialization

SceneHolder currently generates Graphics::RenderTarget in the wrong
thread (Event thread) which can cause crashes in EglController if
there is a heavy load, as Surface/Context pair vector can be modified
in one thread whilst being read in another.

Instead, set up the RenderTarget create info struct, and pass that
through the existing messages instead of RenderTarget. When the
SceneGraph::Scene gets it's second stage Initialize() method
called (by RenderManager), then use that CreateInfo struct to create
the RenderTarget. All modifications to the queues/vectors are now done
in the same thread, and should prevent crashes.

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

DALi Version 2.1.6

Change-Id: I666c7a31240d367c229ad8516407091e1131c802

Merge "Decrease property registration time" into devel/master

Decrease property registration time

For animated custom properties attached to objects, such as uniforms
attached to visual renderers, the registration time is too long.

One of the culprits is the Object::RegisterProperty() method iterating
over all the preceding property names to ensure the uniqueness of the
property name.

In the visual case, we can remove this check, as we can ensure that
each call to RegisterProperty is only done once per uniform.

Added a method RegisterUniqueProperty (which should really be called
RegisterPropertyWithoutUniquenessCheck, but hey!) to perform the same
registration without the name lookup checks.

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

Merge "(Partial Update) Mark as not rendered if the node is transparent or culled" into devel/master

Merge "Even if only InterceptTouchEvent is registered, it is hittable" into devel/master

[Tizen] Not execute the remove callback

This reverts commit e8ea79c5eaa2fa50dd661c0cea119d14e42e1bee.

Merge branch 'devel/master' into tizen

Revert "[Tizen] Not execute the remove callback"

This reverts commit 5247a924873aca7fc58798e409edd5614d5af775.

Even if only InterceptTouchEvent is registered, it is hittable

Change-Id: I724c3bae0741a3ca8537e48474254c3f460deab1