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

Adding Devel Property Registration macro

And missing test cases.

Change-Id: I47626b6dd9ef2eca4fbbfb482c3c6073612b7f5b

Merge "Remove some dead typedefs from progress-value.h" into devel/master

Merge "Changed log message to inform number of textures != than active sampler from ERROR to WARNING" into devel/master

Merge "Added sibling order property to the actor's property table" into devel/master

Stop calling destroy methods of texture and FrameBuffers on RenderManager destructor

Textures and FrameBuffers GL resources where being destroyed on RenderManager destructor
which is called from event thread causing crashes on some GL drivers.

Removed destruction of texture and framebuffer resources since they will be cleaned up
automatically when the GL context is destroyed

Change-Id: I8bd4d79399554179819e091e662ea0c9a6e0bc5f

Added sibling order property to the actor's property table

Added a check in GetDefaultProperty to return a null Value if index
is greater or equal to number of default properties

Change-Id: Ie92a96f26fba7cdbd27395ca1c0f664eaf4b3c97

Remove some dead typedefs from progress-value.h

Change-Id: Ia67e7802aa6f6940705bb30bbcd83fd190e79361

Changed log message to inform number of textures != than active sampler from ERROR to WARNING

Caused false error for SVG visual which creates a renderer but doesn't add it to an actor
until the actual texture is created

Change-Id: I7dec2873e6bb49bc36418ecad1d038d72334ce34

Merge "Prevent API generate warning log" into devel/master

Added travis.yml for auto-building patches

The Travis build runs in an Ubuntu 14.04 VM, and has the necessary dependencies
to build dali-core and to build and execute the test cases (this does not require
testkit-lite anymore, but does still use it's scripts)

After building and running, it concatenates the coverage .info files
and uploads them to coveralls.io/github/dalihub/dali-core.

Updated coverage script to stop running genhtml (to reduce
time when uploading lcov data to coveralls).

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

DALi Version 1.2.17

Change-Id: I98d020cea446af271a5c5c287e014a082759d3be

Merge "Remove dead code from tests." into devel/master

Merge "Add missing includes." into devel/master

Merge "Removed ShaderEffect" into devel/master

Removed ShaderEffect

Change-Id: I428f525da8b70c98dc55795f6a67be56bd8ddaa8

Remove dead code from tests.

Change-Id: I11c0c9f49fe9cf8235e80a84b790c04f4ee9c75d

Add missing includes.

Change-Id: I31f1ecba458022e7d584d28defcf47233b54b175

Added sibling order property to Actor

- Added a sibling order to allow the user to control drawing order in LAYER_UI

Change-Id: Ife3995b46d61ebe7abbc39b6035834b7a581c674

Remove unused custom-uniform.cpp

Change-Id: I2235f44d83aae27492a2c2d8a49c217ff9d07f15

Remove unused member variable in Render::Renderer

Change-Id: I962ee97d7fbd0ccef6de11ec45e805879fc3ba2f

Merge "Replace ASSERT_ALWAYS( false && "message ) with a more meaningful and less code producing DALI_ABORT" into devel/master