Revert "[Tizen] Update RenderState in PreRender"

This reverts commit 3631dd7b2f385093bb9be9af30e9b9fb4e0966ae.

Revert "[Tizen] Prevent accessing invalid index"

This reverts commit fc942a3058a5059c0c37659efeaa9912b2700b66.

[Tizen] Prevent accessing invalid index

This reverts commit d3bc1595bb51ee5cee25299519a93b4e45b8e21d.

[Tizen] Update RenderState in PreRender

This reverts commit 25c956766ad187b49a8539bee3bab95627f3593f.

[Tizen] Not execute the remove callback

This reverts commit 12074b646605796042811041485adaa811f37845.

Merge branch 'devel/master' into tizen

Revert "[Tizen] Not execute the remove callback"

This reverts commit a17082700ec95801ed62b170d2ae86458b7e76cf.

Revert "[Tizen] Update RenderState in PreRender"

This reverts commit 62cfeeb09f1e213ede6115bb36e3c7157dade816.

Revert "[Tizen] Prevent accessing invalid index"

This reverts commit 0fde84c381dc73505ed8b229201ab9b278530eb8.

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>

Sort transform components in breadth first order per scene

Change-Id: Iad835c30b6d83554bc2a150516c23dbd19183937

[Tizen] Prevent accessing invalid index

This reverts commit 2a8bf0bda3509659512d86dc68870b8306b98260.

Change-Id: I9589aee858df56950a2182584171ec5ed96af82a

[Tizen] Update RenderState in PreRender

This reverts commit 752296e1312f7041d8004207191d33bacd8a63e7.

Change-Id: I6dab0d30568980adf5b20b356d1a961115a15ab9

[Tizen] Not execute the remove callback

This reverts commit 1d967f6b87ee09da810286fd4e63083cd4d7227d.

Change-Id: I9e3df08af140f1acf7ea7eb219f3575329c73fc2

Merge branch 'devel/master' into tizen

Change-Id: I115e366d41f91543676549da2abce9525913762e

Revert "[Tizen] Not execute the remove callback"

This reverts commit 05838be5234e9f0e1152a0d8792a1b477b7e02b4.

Revert "[Tizen] Update RenderState in PreRender"

This reverts commit 4e6b015f967d860214ab1ef1edff5fe9c1ba8c45.

Revert "[Tizen] Prevent accessing invalid index"

This reverts commit 2379dac2638ab20c3cbbbf643faa0af33c79d466.

Revert "[Tizen] ActorSizer::RelayoutDependentOnXXX return default value"

This reverts commit 835ef5103e6a44b106a9b04b9326984a3d8c6361.

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>

[Tizen] 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: I8e5c9ba7a456baa2a8d8c0473c06feb6da469a87
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

[Tizen] Prevent accessing invalid index

This reverts commit 4365bb3d85446b7b08f1a0516a137f55c03f9d63.

Change-Id: Id9eb1a0559ca8a4c69442bbb8c25b1c2fb5b6598

[Tizen] Update RenderState in PreRender

This reverts commit 38ba8471ea1beb0794b96056ab61c92db465766b.

Change-Id: Ie4999db1b136db2faad674b33c711d4b7ce2eef8

[Tizen] Not execute the remove callback

This reverts commit 22917c015433ed2cbbf7e6cfe9a6910e8955a65e.

Change-Id: I65501315546eade16f4e72ffd88db420fcce0c20

Merge branch 'devel/master' into tizen

Change-Id: I2c395ca4ed69d2c5bc8512ee99fb4c68ff41937c

Revert "[Tizen] Not execute the remove callback"

This reverts commit 566ab46fd835e3fb700a2ddffcd66e79983ca646.

Revert "[Tizen] Let range for loop don't touch container at RenderManager"

This reverts commit 4c4c6bf206ad121acc41ce145c88fbb54e86f5f2.

Revert "[Tizen] Don't do AABB culling if the clipping mode of the node is enabled"

This reverts commit 907d83a0396c252d5864a27e6ad0d0691a584257.

Revert "[Tizen] Update RenderState in PreRender"

This reverts commit 672d60b90f8a554134a4e327a5e0caaa0894cc7a.

Revert "[Tizen] Prevent accessing invalid index"

This reverts commit 648e934287d6bf7863c0c330d5052f0a34019771.

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

[Tizen] Prevent accessing invalid index

Change-Id: I31ce3b658bff9dadcf235d87257c0ca6bf82705b

[Tizen] Update RenderState in PreRender

This reverts commit 53962366b6f3138455ddb929b1c25a25d2e15252.

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

This reverts commit 168b7bff5966cc0a758ab92442831e8f51726c2b.

[Tizen] Let range for loop don't touch container at RenderManager

This reverts commit f5fcca88a4a97ad90572aa145a0dae3466337e4a.

[Tizen] Not execute the remove callback

This reverts commit 6f7b0d8829c7dd3a36d508c7d8fcc1e5256cd599.

Merge branch 'devel/master' into tizen

Revert "[Tizen] Not execute the remove callback"

This reverts commit a8fe876af4f2f46dccd5d361580ac37bec7aa2a9.

Revert "[Tizen] Add assertion to Vector"

This reverts commit 5270be1081ec39485fecf163937b6902e9533354.

Revert "[Tizen] Let range for loop don't touch container at RenderManager"

This reverts commit 517e77443c5143d083e826b6bbb88593e6930e10.

Revert "[Tizen] Don't do AABB culling if the clipping mode of the node is enabled"

This reverts commit b5ce6ba2a3edf1cbe163a97aa2b63ee6369fa139.

Revert "[Tizen] Use std::abort instead of assert"

This reverts commit ddef37646fa22fdca23729667212527dddad05b4.

Revert "[Tizen] Fix wrong condition"

This reverts commit 5ad74f2eaab997b989c6cc675f1fdeeaa261a496.

Revert "[Tizen] Update RenderState in PreRender"

This reverts commit a5c69f680da5cb00a9d9973584057f4ef08d1f8e.

(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

[Tizen] Update RenderState in PreRender

Change-Id: I2da4ae913ecbf48ec288b59c21beb1d74a9e973d

Refactored more code into Actor::Relayouter

Change-Id: I19e711ddc963e62a882679ca0afebfc5e25bb339

[Tizen] Fix wrong condition

Change-Id: I37cd995142424c50410877fd987267b5866c0bab

[Tizen] Use std::abort instead of assert

Change-Id: Ic43a91489ca9c396da5fd2bb5acf8c049066c7b9

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

Moved more actor methods into relayouter

Change-Id: Ia7cbec6bda7741797463a2bf571073043e628c5e

[Tizen] Let range for loop don't touch container at RenderManager

This reverts commit ca9ab5c8cbe015919dec88b7445941389c08fc8d.

[Tizen] Add assertion to Vector

This reverts commit f0903fc96fd2cf9dc7cd9250f4aa76e209e95e7f.

[Tizen] Not execute the remove callback

This reverts commit 34cc63a0dbdb2e0389f0b03c9f76825e05b6efba.

Merge branch 'devel/master' into tizen

Change-Id: Ic5f6a994f8fa6cd96e2db1bcae322fc1d7aaf3f2

Revert "[Tizen] Not execute the remove callback"

This reverts commit 4bf5f61d32fdaa626fd3da61e2820421bd5de3ec.

Revert "[Tizen] Add assertion to Vector"

This reverts commit a8b6d8e18765116c9f6d36c0baaad1d38349ddd2.

Revert "[Tizen] Let range for loop don't touch container at RenderManager"

This reverts commit 8e3802536c6d31422a6ffc54b5ada8f33923c5a3.

DALi Version 2.0.51

Change-Id: I40ccf8b1220abaf6b3bbbd9a0960b941720a3d9a

[Tizen] Let range for loop don't touch container at RenderManager

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

(ActorImpl) Reduced LOC

Change-Id: Idda1efb9b95056e0cd55b37574bb0e2e87b88db4

Ensure a FrameCallback is removed properly from a FrameCallbackInterface

Change-Id: Ib24a383db0512ea757e06137433aef9419b84a5b

[Tizen] Add assertion to Vector

This reverts commit 33a8f601e33017f8cacdb66c1261f185faab5e8f.

[Tizen] Add comment

This reverts commit 442b2b225b53fb1b04ade3193a94b3f9afd49779.

[Tizen] Remove throwing exception

This reverts commit fce5b62c25ff8e205a059e24c53b8e0161ddc2f0.

[Tizen] Not execute the remove callback

This reverts commit 023f339c1404c0cb0cb9f6f8ec759690c342a396.

Merge branch 'devel/master' into tizen

Revert "[Tizen] Not execute the remove callback"

This reverts commit c3535f3b72da0cca7af1216b1b7198fa3acad369.

Revert "[Tizen] Add temporaty log to node"

This reverts commit 776ce7a1852d540b8ab9bc42689be6037a3309bd.

Revert "[Tizen] Add assertion to Vector"

This reverts commit a3f2bd06b1b30eebd62f946b5d7e5888ffec018e.

Revert "[Tizen] Add comment"

This reverts commit 1b975b4acbfe3272a8804541336cd385605cd06c.

Revert "[Tizen] Remove throwing exception"

This reverts commit cc5832843a63cfe3a6b52ee57f73a67c095a62e4.

Revert "[Tizen] Exact planeclipping for actor's hit algorithm"

This reverts commit 60ff852dde6d7bdb368e85720154addbc60f9e74.

Revert "[Tizen] Protect dereferencing null in Node"

This reverts commit fcb81cd980326b1cfc9f5672b1211b36785537f3.

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

DALi Version 2.0.50

Change-Id: Ia32bcbb9470d0656f64acfe395a62dbf7d5c46d1

[Tizen] Protect dereferencing null in Node

Change-Id: I60506f563fefc31c9c70b7067fc45f91d6929b96

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

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