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

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

DALi Version 2.1.8

Change-Id: If05e41c6141e9e20dd655418b04843b9e7b72225

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

DALi Version 2.1.7

Change-Id: Iccbe522c45fd61d039458b809fa4f86f2423eb78

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

Even if only InterceptTouchEvent is registered, it is hittable

Change-Id: I724c3bae0741a3ca8537e48474254c3f460deab1

(Partial Update) Mark as not rendered if the node is transparent or culled

Change-Id: I0a6aeaa90e9da1ddb6b61d8ff8397e84a43ac668

Replace obsolete safe bool idiom with explicit operator bool

C++11 introduces 'explicit operator bool' to prevent unintended implicit
conversions to 'bool', thus making the trick with converting to a
pointer-to-member (a.k.a. "safe bool idiom") obsolete.

The explicit operator is more restrictive than 'safe bool', and it
helped uncover a bug in the test suite where object handles were
implicitly converted to bool before being sent to an std::ostream.

Change-Id: I8c9d33812bc022570f5c286d925914dba508185c

DALi Version 2.1.5

Change-Id: I4174771a77f5df2693b60fc09e585ee4af03d812

DALi Version 2.1.4

Change-Id: Id10b0af4d5b296d014947b2386ece3488caf6e5d

Dali::Vector release memory later during Copy

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

Merge "Move mItemsDirtyRects from Scene to SceneGraph::Scene" into devel/master

DALi Version 2.1.3

Change-Id: I9a39fc16eef86cd02dc9f65ccc6f912efd1993a5

Move mItemsDirtyRects from Scene to SceneGraph::Scene

The list is used in the update thread. So move it to SceneGraph::Scene

Change-Id: Ic3b374d2e3840c05d72ed8b55697db4e5eef1f13

Merge "Fixed the crash appears after closing app when use ResizePolicy::USE_NATURAL_SIZE" into devel/master

Merge "Fix the local matrix dirty flag when size is changed" into devel/master

DALi Version 2.1.2

Change-Id: I3df4c85dd1dc5537ec39dd593a4d5cbff22a93ba

Use a proper projection matrix

Change-Id: I6bed10bc639a838918919c5621d6f399db5cebe3

Fixed the crash appears after closing app when use ResizePolicy::USE_NATURAL_SIZE

There is a crash when destroying text-editor and the ResizePolicy is USE_NATURAL_SIZE.

textEditor.SetProperty(Dali::Actor::Property::WIDTH_RESIZE_POLICY, ResizePolicy::USE_NATURAL_SIZE);"
then close the application. the below exception appears.
ERROR: DALI: Assert (threadLocal) failed in: static Dali::Internal::ThreadLocalStorage& Dali::Internal::ThreadLocalStorage::Get()
Exception:
threadLocal
 thrown at static Dali::Internal::ThreadLocalStorage& Dali::Internal::ThreadLocalStorage::Get()
terminate called after throwing an instance of 'Dali::DaliException'

The solution applied at RelayoutController::Get().
Checked if ThreadLocalStorage is created then proceed normally otherwise returns null
Because the ThreadLocalStorage::Get() only retrieves STL without checking if it exists.

The caller of RelayoutController::Get() should check if RelayoutController is not null.
There is only one place use this function and already checks if RelayoutController is not null.

Change-Id: I343ef47f03aa86babb4bdd19f95c3e7ccb211dff

Merge "Fix ReuseRenderer issue when renderer is nullptr" into devel/master

DALi Version 2.1.1

Change-Id: Ie412924e57d35e2605366111e636ee568b9e402e

Fix ReuseRenderer issue when renderer is nullptr

When ClippingMode is CLIP_TO_BOUNDING_BOX and it have no children,
we add nullptr SceneGraph::Renderer into renderables.

Previous codes are assume that that renderers are not nullptr
so they just use many of memberfunctions.

This patch fix the dangerous nullptr access.

TODO : need to make UTC

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

Fix the local matrix dirty flag when size is changed

Change-Id: I009c1a17b037744e6d3c8268651af8543786bb80

Sort transform components in breadth first order per scene

Change-Id: Iad835c30b6d83554bc2a150516c23dbd19183937

Fix coverity issue

Delete useless member value

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

DALi Version 2.1.0

Change-Id: I86132cc32010fd84cb60c9dbee06f8b693e19a99

Fixed actor relayout dimension dependencies

Height dimension dependency on child should now work properly.

Retained old behaviour from before refactor - when no size
negotiation, RelayoutDependentOnChildrenBase returned true.

Change-Id: I3a375f64b9bfe6bca8f1909e53aa1e6e133afcd3

ActorSizer::RelayoutDependentOnXXX return default value

Before refactoring, when mRelayoutData doesn't exist,
RelayoutDependentOnChildrenBase return true.

Because DEFAULT state is "ResizePolicy::USE_NATURAL_SIZE" and
It's case s.t. RelayoutDependentOnChildrenBase is true.

After refactoring, if mRelayoutData is null, it always return false.

This patch just sync with previous behavior.

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

Dali::Vector release data more safe (Dali::Vector::Replace)

We got some issue when someone use mData
during Dali::Vector is on reserving.

In this case, mData = nullptr (or mData memory is deleted.) very short time.

Now, we add VectorBase::Replace(void* newData) function.
It mean, VectorBase::mData will be replaced as newData.

It can block thread integrity issue.

Note : VectorBase::Copy can have nullptr between Release() <-> Reserve()
There is no way to fix it without any additional memory.
So just keep current implement logic.

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

Let range for loop don't touch container at RenderManager

Reallocate or Resize the container during range-based-loop is UB.
So many case, we try to not add/erase some items during range-based-loop is working.

There was some codes that erase item during for loop.
So I make it out, for safety.

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

Recalculated layer's reusability flags after transform update

Layer's reusability flags were being left as true causing
some examples to fail to render properly, as render items
were being reused without being updated after a transform.

Reworked UpdateNodes to move layer reusability calculation
until after TransformManager::Update.

Change-Id: Id0e30489074ae3690984e4f95f0b3cd730d80440

Reset node animatable properties for two frames after removing the frame callback

Change-Id: I67e0ee42b2da8076fa47cadf2af1435c89a4a431

Fix the local matrix dirty flags for transform components

Change-Id: I0e54461468d2e0c923757ab9dc2a7d79f93ea2fe

Fixed node dirty flags when inheriting the flags from the parent node

Change-Id: I112641e530c0c56f8c793ddf8e263366a16cfe08

Fix the camera's projection matrix when copying it from the previous frame

Change-Id: Ic189b49d2bd824e5ad8802396bd26b5224701dc2

Add a node resetter that resets the animatable properties for two frames after node connection

Change-Id: I3a64c3719107c231085a524b05c1302977ccde10

Merge "DALi Version 2.0.53" into devel/master

DALi Version 2.0.53

Change-Id: Ib413dcfba88735cfa6a1bd33d8569ff03cf75beb

Fix SVACE error in render-manager.cpp

Change-Id: I4da240f713fef7777d5c653de52397e3f8cb9afc

Refactored actor size negotiation

Moved all size negotiation implementation to a new
ActorSizer class. This handles size negotiation and
target / animated size.

Moved static methods from ActorRelayouter into ActorSizer.

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

Merge "(Partial update) Reset the updated flag after calculating the update area" into devel/master

Merge "Don't do AABB culling if the clipping mode of the node is enabled" into devel/master

(Partial update) Reset the updated flag after calculating the update area

We had an issue that the updated flag of non-renderable actor was not reset.
It should also be reset after calculating the update area.
So reset the updated flag of all nodes.

Change-Id: I8b4d540996565ba2d0ef9e528114dad5e3de2475

Don't do AABB culling if the clipping mode of the node is enabled

It should not be culled because the clipping rect should be used.

Change-Id: I7d5895c8471720ba808d0c5919adb1b4288c5f18

Set TransformId after Initialize finished

Setup mTransformManagerData.mId after transform data initialize done.
After this patch, we can assume that
if mTransformManagerData.Id() is INVALID_TRANSFORM_ID, then
mTransformManagerData is not initialized

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

DALi Version 2.0.52

Change-Id: I2fb8a867bf1c1f2c9a52416f4fd076d583a8eb21

Refactored more code into Actor::Relayouter

Change-Id: I19e711ddc963e62a882679ca0afebfc5e25bb339

Moved more actor methods into relayouter

Change-Id: Ia7cbec6bda7741797463a2bf571073043e628c5e

DALi Version 2.0.51

Change-Id: I40ccf8b1220abaf6b3bbbd9a0960b941720a3d9a

(ActorImpl) Reduced LOC

Change-Id: Idda1efb9b95056e0cd55b37574bb0e2e87b88db4

Ensure a FrameCallback is removed properly from a FrameCallbackInterface

Change-Id: Ib24a383db0512ea757e06137433aef9419b84a5b

Merge "Update the preferred size after relayoutting" into devel/master

DALi Version 2.0.50

Change-Id: Ia32bcbb9470d0656f64acfe395a62dbf7d5c46d1

Exact planeclipping for actor's hit algorithm

rayOriginLocal is difference vector betwwen the actor and the Camera(==rayOrigin).
ActorTest API's distance value is the real euclidean distance between rayOrigin and actor.
But we only need difference vector's z-axis value to clipping by View Space Plane.

So we get hitPointWorld and multiply ViewMatrix (get from renderTask's camera actor)
and get z-coordinate value at Camera Space.

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

Update the preferred size after relayoutting

It should be used in the next relayoutting

Change-Id: I3342de587942a9123ebb31cba4eac35c4e2b65ca

If the parent is intercepted, the child is informed that it is INTERRUPTED.

Change-Id: I4ef25e7bad0883a9798f86add0b911de36078dea

Merge "DALi Version 2.0.49" into devel/master

DALi Version 2.0.49

Change-Id: I99286537008bce74eb41966539bd4ec2d1915fea

Fix LogMessage format to print Filename / Functioname / Line

Add LogMessageDlogFormat macro to apply these informations :
Filename: Function(Line#) > "messages"

Previous LogMessage function always print like this
logging-tizen.cpp: LogMessage(38) > "messages"
So we fix to print actual informations

Default LogMessage(~~) will be used when someone don't need to follow up
Dlog format. (like ttrace and perfomance logger)

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

Moved actor recursion methods to actor-parent-impl

Change-Id: I01a3153dc8849425f6b8e69f60c224dc36ee7d45

Refactored out actor render container

Change-Id: Ib7f45346b92d7c9674833da5c6b54e889e8da28f

Refactored Actor screen to local conversion

Change-Id: I145644d1702e9e3f9f6571aff5651c811abbf39c

Merge "Protect dereferencing null in Node" into devel/master

Supports to acknowledge for completing  window rotation

It is waited the window rotation completing until a view finishes to draw.
The view is drawn on the other thread.
It means asychronous rendering occurs between dali render thread and the other thread when window is rotated.

Change-Id: Ia9ac33e3719a874c35c58acf91e92b96ac802c7f

Protect dereferencing null in Node

Change-Id: I60506f563fefc31c9c70b7067fc45f91d6929b96

Skip rendering if the damaged rect is empty

Change-Id: I9424cf079d68a4afd00fda11a5e01d740be8155a

DALi Version 2.0.48

Change-Id: I1ba13133446071912493467c53f22ba7573eed68

Merge "Adding photoshop blend modes to the GFX API" into devel/master

DALi Version 2.0.47

Change-Id: Ie0ca363284bc1ed55f9cc02359cd7730d20c2e9f

Adding photoshop blend modes to the GFX API

Change-Id: I9274f02b9e0098ebc06caa4c569944624bc56030

DALi Version 2.0.46

Change-Id: I9c473ddd86cd66ded7dc18d19cd143fa4d6e193c

Prevent unused attributes from binding bad location

Change-Id: I2655aa8d6e2fb42cc2aed1f06ce91cd05dc37b5d

Add Default Uniform : uActorColor

Previously, DevelRenderer always merged into uColor.a value.
When we use Borderline feature, it will occure some problems.

This patch add one more default uniform : named "uActorColor"
So now we can split MIX_COLOR.a and Actor::COLOR.a

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

DALi Version 2.0.45

Change-Id: I002cb962e0df531e8fd3f5edcdfbf14f6aae2d91

DALi Version 2.0.44

Change-Id: Ic23ea29b6caa984854d00ed525c4b3181ff66f36

Perform AABB test in the viewport space for renderers when not processing a 3D layer

Change-Id: Ib95e9fb1a4c0c4dd7b8ef8260ab9cbe2f4c3c706

(dali_env) Install the correct package depending on the Ubuntu version

Change-Id: I007d91f7e6f38abfd387c8bd4bd86a578959289f