[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.

(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

(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

[Tizen] Not execute the remove callback

Merge branch 'devel/master' into tizen

Revert "[Tizen] Not execute the remove callback"

This reverts commit b2824eb4b14d16c540502b2b976d327a83ba2911.

Revert "[Tizen] Update RenderState in PreRender"

This reverts commit c4471f68c5a18cab9dc3ca09a0cdd5f177d010d6.

Revert "[Tizen] Prevent accessing invalid index"

This reverts commit d805ad413cd1a3245287d82d961455999b09374d.

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>

[Tizen] Prevent accessing invalid index

This reverts commit 9cfaf269db610b0e0a73d75c41281f83747e04fe.

[Tizen] Update RenderState in PreRender

This reverts commit 2bc15ea4f730faa946501a98cb6572b1f38fc7e7.

[Tizen] Not execute the remove callback

This reverts commit 0db9bb9e1353273f98444ccf30e2f8549256422b.

Merge branch 'devel/master' into tizen

Revert "[Tizen] Not execute the remove callback"

This reverts commit 39a2cc902ee9ba212d87c20a8ab3f0c18bea6ab8.

Revert "[Tizen] Update RenderState in PreRender"

This reverts commit ac0a697e61c9298b7141fc9a5ad78ea7ac48beee.

Revert "[Tizen] Prevent accessing invalid index"

This reverts commit be7619f2f1269ce30259f237014a1b0273711de5.

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

[Tizen] Prevent accessing invalid index

This reverts commit 61c65ff82e80b793effc25abf729a2d3e4480356.

[Tizen] Update RenderState in PreRender

This reverts commit ea5d55a02720dc0225d4aae6492267f60834fe81.

[Tizen] Not execute the remove callback

This reverts commit ece8a3036f71f6aceee2293641973564f122e0aa.

Merge branch 'devel/master' into tizen

Revert "[Tizen] Not execute the remove callback"

This reverts commit 2056f31206c52657967b9b20e157cf6491e9af6b.

Revert "[Tizen] Update RenderState in PreRender"

This reverts commit 3631dd7b2f385093bb9be9af30e9b9fb4e0966ae.

Revert "[Tizen] Prevent accessing invalid index"

This reverts commit fc942a3058a5059c0c37659efeaa9912b2700b66.

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

DALi Version 2.1.2

Change-Id: I3df4c85dd1dc5537ec39dd593a4d5cbff22a93ba

[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.

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

[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.