Valgrind detected TextureSet leak and invalid access

Memory leak:

If a public API TextureSet gets deleted, it sends a
message to update-manager (RemoveTextureSet).
The scene graph side TextureSet gets removed from
update manager, but it never gets deleted.

Invalid memory access:

Currently when a scene graph TextureSet gets deleted
the renderers using that TextureSet are not notified.

So when a Render gets deleted it would call
TextureSet->RemoveObserver( this );

on an already deleted TextureSet.

DALi however doesn't crash at this point,
because the TextureSet is stored in a memory
 pool so the object is still reachable.

Change-Id: Icf0f5a3e55d3ba7537f40db08accad04ae4440f7

Merge "Fix to avoid DEREF_AFTER_NULL Svace issue in any.h" into devel/master

Merge "packaging/dali.spec use license macro" into devel/master

(Renderer) Fix memory leak when using blend-color

Change-Id: Iaf7a0ae464fa0c354e1111f9313972349a2b9b45

DALi Version 1.2.32

Change-Id: I578f410ee51348c1650dbd4b9f92008951f5865b

packaging/dali.spec use license macro

Change-Id: I2c4666e6c0b4662190f41f4945b5496dacc54b3b

Fix to avoid DEREF_AFTER_NULL Svace issue in any.h

Change-Id: I4e87718579bc208fccf498bc3204b109303abb84

Merge "Target Values of animation only set on ::Play()" into devel/master

Ensure width & height properties can be set individually

Change-Id: Ie8d7ee230a87f2152c8ad6320839c46bc70a2a5f

Target Values of animation only set on ::Play()

Change-Id: I834e151b4f06a08def523cf9d915fcafa79fd806

DALi Version 1.2.31

Change-Id: I409fc5db91f42d484d34ceee94e314b263b10b23

Added Getters for Animator Connectors base class

Change-Id: I90c6afe299dd565261fe733c4fbe3c53df1988d2

DALi Version 1.2.30

Change-Id: If1de12dcfcde05eabb9cd9647dbdbca115c7ef7b

Added a property to allow an actor to ignore the anchor-point for its position

In that scenario, TOP_LEFT is assumed.

Change-Id: Id57829b9488b0f3d3546354d51900941ca92affa

Merge "Fix memory leaks detected by Valgrind" into devel/master

Added Get Screen Position functionality to Actor

Screen coordinates TOP_LEFT is 0,0
This property will return the position of the Actor relative to the TOP_LEFT
The returned position is the position of the actor's Anchor point.
Actor Scaling is supported.
Actor Rotation is not supported.

Change-Id: I8519f2b82ab9f9ac8eec197e783a66dcf5ba949c

Fix memory leaks detected by Valgrind

Fixes the following memory leaks reported by Valgrind
that can occur when messages with raw pointers are sent
from event -> update (and update -> render)
and dali shutdowns before update or render has had chance
to process those messages

- AddSampler ( sampler is lost )
- AddPropertyBuffer ( property buffer is lost )
- SetPropertyBufferFormat( property buffer format is lost )
- AddGeometry ( geometry data  is lost )
- AddTexture ( texture data is lost )

event -> update
- AddCameraMessage ( Camera is lost )
- AddObjectMessage ( Object is lost )
- AddPropertyNotificationMessage ( PropertyNotification is lost )
- AddGestureMessage ( PanGesture is lost )
- AddSamplerMessage( sampler is lost )
- AddPropertyBuffer( property buffer is lost )
- SetPropertyBufferFormat( property buffer format is lost )
- SetPropertyBufferData( property data is lost )
- AddGeometry( geometry is lost)

Also removes PropertyBuffer data and format from being duplicated
on the event side.

Change-Id: Iac6d446bd2b5372cea7ec101e2e289ef3f284b4c

Merge "Added checks to Sibling Order Raise/Lower API" into devel/master

Merge "Valgrind fix for uninitialized values in Render::Sampler" into devel/master

Added checks to Sibling Order Raise/Lower API

Added checks to ensure target actor and actor raising or lowering belong to the same parent

Also checks that both actors are parented.

Change-Id: I346ead160a72cad12192cf4f6b771cf82bb43930

DALi Version 1.2.29

Change-Id: I4589137dbd69c6c6cbbdece518324b146a58b45a

Valgrind fix for uninitialized values in Render::Sampler

Valgrind reported conditional jump depends on uninit values in:
  if( mSampler.mBitField != oldSampler.mBitField ) // Texture::ApplySampler

Problem was down to Sampler having a anonymous union:

  union
  {
    unsigned int mBitfield;

    struct
    {
      FilterMode  mMinificationFilter   : 4;    ///< The minify filter
      FilterMode  mMagnificationFilter  : 4;    ///< The magnify filter
      WrapMode    mSWrapMode            : 4;    ///< The horizontal wrap mode
      WrapMode    mTWrapMode            : 4;    ///< The vertical wrap mode
      WrapMode    mRWrapMode            : 4;    ///< The vertical wrap mode
    };
  };

Fundamentally the size of the bit field struct
can be larger than the size of the unsigned int mBitfield, so using mBitfield to compare Samplers
isn't reliable.

The issue valgrind picked up on is, only 20 bits of the
mBitfield are being set in the sampler constructor
  Sampler()
  :mMinificationFilter(FilterMode::DEFAULT),
   mMagnificationFilter(FilterMode::DEFAULT),
   mSWrapMode(WrapMode::DEFAULT),
   mTWrapMode(WrapMode::DEFAULT),
   mRWrapMode(WrapMode::DEFAULT)
  {}

Solution is to remove the union

Change-Id: Id66afc05d21377a1363448ef10654cd1ab4bb365

Merge "Fix Message Buffer leak and added DEBUG_ASSERT for leaking nodes" into devel/master

Merge "(Actor) Fix documentation in sibling order methods" into devel/master

(Actor) Fix documentation in sibling order methods

Change-Id: Ib3ffe6e8c8f5e1f249f0a6f5e5439e174f883a4f

Check memory being free'd from a memory pool, belongs to it.

Memory leak detected by Valgrind due to Scene Graph Layers
being deleted from a memory pool, even though they were heap
allocated.

Assert currently fires on shutting down dali-demo, and when
using text-editor.example

Leak fixed in
https://review.tizen.org/gerrit/#/c/115115/

Change-Id: I1254b56a3f3be17d662fb0622ebe407874df71cc

Fix Message Buffer leak and added DEBUG_ASSERT for leaking nodes

Current message queue is not cleared when update is destroyed
This is 1 cause of Node leak.
Second cause of Node leak is InstallRootMessage passes a layer
as a raw pointer. If update doesn't run ( doesn't run in the test
cases), then we leak the layer (node).

Also added DEBUG_ASSERT to check for leaking scene graph Nodes

Change-Id: Ibfbb73ee840cbeaf33104da6769945cf9b3aafbc

Fix memory leak, scene graph layers are never deleted from memory

Reported by Valgrind.
We have a memory pool for Nodes which works fine
for creating /deleting nodes.

Scene Graph Layers inherit from Nodes, but don't use
the memory pool.

However when either a Node or a Layer is deleted, the overloaded
Node operator delete( void* ptr ) is called, which tries to free
the Node from the memory pool.
Unfortunately for Layer it was never in the memory pool so no
memory is free'd.

Fix includes:
node destructor is protected
layer destructor is private
node delete operator removed

Change-Id: Icfe83f35b4f29d5b774cde392ff976ad299481c5

Adding Raise Lower API to Actor

Devel API to allow the raising and lowering of the Actor's sibbling level.

Change-Id: I86c03d43d962441fd295dbf024092d42e78d9c0e

Merge "Removed redundant resource loading & rendering code" into devel/master

DALi Version 1.2.28

Change-Id: If7991a9a2a637f145136669dffc6908f927661c1

Add Opacity property for Actor

Change-Id: I7fefcf77e512b4d0954666d4187d116dc5d518bd

Removed redundant resource loading & rendering code

Change-Id: I369e0e907f452125eb97059f853e9e16fa31f158

Test harness: Moved CompareType<> templates to own header

Change-Id: I367a21562a7c6362a15a13bfcc6fc8fffeb37c3b

Merge "DALi Version 1.2.27" into devel/master

DALi Version 1.2.27

Change-Id: If054c3c0e9547efc5e03b843cd19004fceee43e1

Merge "Update doxygen comments" into devel/master

Update doxygen comments

- For document public release, Native API reference is checked by Lionbridge.

Change-Id: I4a1a1a364975a56e3c4a04664e8170ff950d612b
Signed-off-by: Seoyeon Kim <seoyeon2.kim@samsung.com>

Actor size property should return preferred size if size is fixed as set by SetSize

Change-Id: I8662795c67ef377c6417ef4ff7abde394d28cc94

Merge "Delete ThreadLocalStorage on shutdown" into devel/master

Delete ThreadLocalStorage on shutdown

Currently DALi ThreadLocalStorage object is not deleted
on shutdown.

Reported by Valgrind / Valkyrie

Change-Id: I376f2a05c149b3ac0d81e07fa1052fd44ee6d8a7

Prepare for Tizen 4.0 Build

I've assumed that Tizen 4.0 code is going to be more similar with 3.0
than 2.0.

Please do not rely on string == compare for version check.

Change-Id: I12eeef7463c50e1be811bb28aac0cf81bb6c5698
Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

Merge "Remove batching." into devel/master

Remove batching.

Change-Id: Ia834dde43961d11dbac0b71f7770c4e00da7b4e8

Add devel API to link an object (e.g. C# custom control) with its correct type-info

Change-Id: I51407eab027b7a73d781b20d15f850ee40d05e92

Ensure TestPlatformAbstraction cannot be copied

- Fixes an SVACE issue.
- Also synchronized automated tests with other repos.

Change-Id: Ib8cc53264d74aacfcf147706d302f57bb6b822e5

DALi Version 1.2.26

Change-Id: I423b6d38c3a91115d8a96630d3edfe795d034eb9

Merge "Deprecated Layer::TREE_DEPTH_MULTIPLIER and fixed mis-usage." into devel/master

Merge "Remove dali.h from core test" into devel/master

Merge "Use TextureSet instead when sorting render items." into devel/master

Remove dali.h from core test

Change-Id: I5f89e11d27daffb275a96466b0e5c100ff8f9027

Fix SVACE issue - memory leak

- Delete the elements of mResourceRequests

Change-Id: Ib0e95cccf8ac755b2e5571a0efd2b90e15a64747

Merge "Added travis.yml for auto-building patches" into devel/master

Use TextureSet instead when sorting render items.

Change-Id: Id03460a0e99bca4c914a5f28ff397d570034dcba

Deprecated Layer::TREE_DEPTH_MULTIPLIER and fixed mis-usage.

Layer::TREE_DEPTH_MULITPLIER has the wrong value for our sorting
algorithm, and has been replaced by DevelLayer::TREE_DEPTH_MULTIPLIER,
which is now set to 1,000,000. Updated HitTestAlgorithm to also use the
correct value.

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

Merge "DALi Version 1.2.25" into devel/master

DALi Version 1.2.25

Change-Id: Ie6091edc01e7c7f93e6a2ea138e70301c5b1924e

Removed ResourceClient

Change-Id: Iecccb23503008d3c5f24203998f14d92a3a99888

Merge "Removed FrameBufferId from SceneGraph::RenderTask" into devel/master

Merge "Remove images from TextureSet" into devel/master

DALi Version 1.2.24

Change-Id: I94c051014beeae1c0283440bd23c91190d15e044

Removed FrameBufferId from SceneGraph::RenderTask

Change-Id: I5411bbffd7569a29eb35559b767e3cc5e1159eeb

Remove images from TextureSet

Change-Id: I12e3b03fc9613d0beaf3d18b0ecb62b8be83f406

Changed remaining image types to use Texture internally

- FrameBufferImage, NativeImage, EncodedBufferImage and BufferImage use Dali::Texture internally
- Removed Dali::Atlas class as it is not used

Change-Id: Ife1bf687ca314d87cf5974651857dc7242a1ce37

Delete ImageFactory and ImageFactoryCache as they are unused

Change-Id: Ia76278fc6d2c3c62e2789f46320e2fc897089603

(GCC 6.2) Remove unused functions from automated tests

Change-Id: I22092d5c6ed2c6536e26f426ae5b04ef152d86e3

DALi Version 1.2.23

Change-Id: I6454ba8a1df926dabae3b7dfed634d68af2ce792

Merge "Changed ResourceImage to use Dali::Texture internally" into devel/master

Fix undefined behaviour due to strict-aliasing.

Change-Id: I5b69e86a01d8f4dd720f685222ce2424aa664d9b

Changed ResourceImage to use Dali::Texture internally

Change-Id: Ie8e740f8549bb23e4e2caf91a837d844bc6eb89c

Merge "Remove erroneous assert, pointers to references cannot be null." into devel/master

Update doxygen tags

- @returns -> @return / @param -> @param[in] / @detail descriptions

Change-Id: I5ae9f7849030988748f0afb4ce58f7b4b06ed7ab
Signed-off-by: Seoyeon Kim <seoyeon2.kim@samsung.com>

Remove erroneous assert, pointers to references cannot be null.

Change-Id: I1556faec4cf03b5bd272230e232e1492a9616e28

DALi Version 1.2.22

Change-Id: I50eca844edc5031109806fe8a865b071de4f2369

Remove left over references to ImageActor & ShaderEffect

Change-Id: Ied2e62c240ffdf563879a54a2b53ed027038606e

Merge "DALi Version 1.2.21" into devel/master

DALi Version 1.2.21

Change-Id: Ia6031e55868f8a9c27df50fc232a5b97ee378990

Ensure clipping gets disabled properly when previously enabled

The clipping sort modifier is used to retrieve the clipping ID but is not cleared when we disable clipping.

Change-Id: Icfea7d7115fe24574edd5a058d65229621e75ae8

(Actor) Add enumeration setting support to various properties

Before, you could only set these enumeration value properties as strings which is quite annoying from code.

Change-Id: I566bd962c1bbcc43abdf15786e5c06065969ce60

DALi Version 1.2.20

Change-Id: Ia0568f0964f6e8e72d91b9a1d3016d3055159ab8

Remove deprecation warnings from TouchEvent

This pollutes the log for general touch as it's used by the TouchEventProcessor for backwards compatability
and, thus, the log is always shown even if applications are not using it.

Change-Id: Ied028740a3007914a6c68c0ffa3a66ad6ef840c9

Merge changes I822c3044,I9e0f12e7,Ibab0e11e,Icb145e7b,Ib68aba7f, ... into devel/master

* changes:
  Redundant assignments.
  Simplify logic, avoid storing raw pointer
  DaliException should be captured by reference.
  Redundant assignments and unread values.
  Members not set in assignment constructor.
  Comparison between boolean and integer.
  Redundant break, default being used in switch statement.
  Incorrect signature of assignment operator.

Move virtual destructor same position as 2.4

Signed-off-by: suhyung Eom <suhyung.eom@samsung.com>
Change-Id: Iafcc02d8c24c298a19eca37e951c6827fc6c9d65

Move virutal method's position to its original

Signed-off-by: suhyung Eom <suhyung.eom@samsung.com>
Change-Id: Iff71613b617b26d11553ccbec4ddfb3a7bb514bb

Redundant assignments.

Change-Id: I822c304423985c56150047576dfb2e442a68b10d

Simplify logic, avoid storing raw pointer

Change-Id: I9e0f12e72ec49312080a073afca7db835c5b8006

DaliException should be captured by reference.

Change-Id: Ibab0e11ed2290f979c525b0f7c6d0dc16ce4425e

Redundant assignments and unread values.

Change-Id: Icb145e7b40cde869279ce5cd6d66a22a91b33df9

Members not set in assignment constructor.

Change-Id: Ib68aba7f7c2208e496f3b64aef0caa122236f7fb

Comparison between boolean and integer.

Change-Id: I565aee867965ab67bd8d5c97e3ac620a893f4925

Redundant break, default being used in switch statement.

Change-Id: I1c9d532eec75571074b65fd110bbfd49f8ffbb30

Incorrect signature of assignment operator.

Change-Id: I5742daf33fd44d4d25a679c9d5684cdff6374ae7

DALi Version 1.2.19

Change-Id: Icb57b6cbd45320778be963cb4750eba7f79636ef

Merge "Enabling animation of Renderer properties when offstage" into devel/master

Type registry changed C# callback signatures

To save memory / simplicity we now use just 3 C# delegates to
connect between C# and C++ type registry. Previously it was
3 delegates per custom control ( Create, Property Set, Property Get).

Change-Id: I91a1da13014449f2589b6f5f5f3cea6308a4c582

Enabling animation of Renderer properties when offstage

If the renderer exists, but hasn't been staged yet, then the scene
graph object also exists, but cannot be accessed because of a spurious
check for OnStage().

Removing the OnStage check to allow animations to run on Renderers
that haven't 'quite' been staged but are very likely to in the near
future.

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

Fix bug binding native image textures

NewTexture was always binding native image textures to GL_TEXTURE_2D binding point
even if textures required a different binding point

Change-Id: If01f8d33a92ed993b19818d6ba4b804f21b67253

Apply deprecate macro and LOG

Signed-off-by: suhyung Eom <suhyung.eom@samsung.com>
Change-Id: If9e0583d52cab9a036da0a93dd924c8a02bec999

Merge "Adding Devel Property Registration macro" into devel/master

DALi Version 1.2.18

Change-Id: Idecbcc614e7857fc300971bed885c0ff74dad31e