Merge branch 'devel/master' into tizen

Change-Id: Ibb68ab138f893b33dad8011c2d4bbd3f49d14162

Revert "[Tizen] Not execute the remove callback"

This reverts commit 300650e4973cd617dbf7e08e297e12bd7078a0e9.

Revert "[Tizen] Print backtrace when an exception occurs"

This reverts commit fc0f36cc2f9a308161d26120e8d2da38e2bb5f1f.

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>

[Tizen] Print backtrace when an exception occurs

This reverts commit 14676776439633d75fb8912f343f411d6de5a61a.

Change-Id: I2f3aac87040ae117b2c3e8a7de2c3ae2726f35a4

[Tizen] Not execute the remove callback

This reverts commit bff3fdc6aae849fb4ed5b3e2b6eb52ae70912a58.

Change-Id: Iec715ee4d05d574442d7133c5bf6513b0cd3ad8d

Merge branch 'devel/master' into tizen

Change-Id: I2c00f932b5279f2a71840c842da6d1bc73fcf5b2

Revert "[Tizen] Not execute the remove callback"

This reverts commit b3fa6fd4b2e1cd1ef58b2f0cd966d4a29fe31f01.

Revert "[Tizen] Relayout only if the actor is on Scene"

This reverts commit 5a557529859e267b142cecc05b6b02f0f8d649c2.

Revert "[Tizen] Print backtrace when an exception occurs"

This reverts commit 148c1248ededd97dc0cdf49d75d59e7ed83db267.

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>

[Tizen] Print backtrace when an exception occurs

Change-Id: I7857019045fbc7e837b3978bdf36d59c771f6f90

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

[Tizen] Relayout only if the actor is on Scene

This reverts commit 38e24a9add7c7ccc1082946534ec1f6c2ad8a31d.

Merge branch 'devel/master' into tizen

Change-Id: I370df81194fe62eb3b41d5ba4bb32c2cb98f7010

Revert "[Tizen] Relayout only if the actor is on Scene"

This reverts commit 63cb396de8cce7f12def4c9d1259f23d70d37b22.

[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

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>

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.

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

[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