Adaptor: Fix Klocwork issues

[Problem] Klocwork issues

[Cause] 1. Data member not initialized in the constructor or non-used data member
        2. Floating point expression is test for equality
        3. Constant condition in conditional expression
        4. Memory leak
        5. syntax shortcuts: variable used twice in one expression where one usage is subject to side-effects
        6. Unreachable code

[Solution]

Change-Id: I815b8d9ae73c6bc08b807d898e197db90c6c08a4

Merge "Remove handle::operator=(NULL) as it is duplicating handle.Reset() functionality and thus redundant" into tizen

[dali_1.0.16] Merge branch 'tizen'

Change-Id: I30d51cf954a3e263620b9cdc9829d1dc7400b8e2

DALi Version 1.0.16

Change-Id: Ieb72ee27d4f07331dfbb8acfcc8998dfffa5ffd0

Remove handle::operator=(NULL) as it is duplicating handle.Reset() functionality and thus redundant

Change-Id: Iabc6220d316a3bdca55fa1edab6ff4733fb36532

Add handling for context loss and regain behaviour

[Problem] Dali doesn't handle context loss
[Cause] Never tested.
[Solution]

Fixed ReplaceSurface handling for window surface case. Fixed window destructor to ensure
indicator is correctly removed.

Added new application policy for dealing with context loss. If the application
doesn't support context loss, then Dali must instead retain resource data or have
some means to re-load it.

Added a new data retention policy to PlatformAbstraction. This allows Core to query
what to do with bitmap data and mesh data on context loss / resume.

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

Removed mutex locking in render thread.

Communication is now through the UpdateRenderSync object, which has a single mutex.

[Problem]
Need to add new interface onto render thread without using messages.
Current mechanism for ReplaceSurface API uses a mutex lock around a
double buffered data store.

Adding new interface using this method with a different mutex around a
different data-store could easily lead to deadlock.

[Solution]
Remove all mutexs from within RenderThread. There are 2 communication
points in RenderThread from UpdateRenderSync: RenderSyncWithUpdate(),
which triggers the start of the render, and RenderFinished(), which
informs that the render has finished.

Utilize RenderSyncWithUpdate to pass requests from UpdateRenderSync
into render thread. Any responses should be written back into these
requests.

Modify UpdateRenderController to talk with UpdateRenderSync rather
than directly to RenderThread.  Event requests will block until the
next render pass, it is UpdateRenderSync's responsibility to ensure
that the update/render threads are awake and available to process.

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

Merge "fix unsafe multithreaded boolean read/write" into tizen

Merge "Removed unnecessary mutex lock" into tizen

fix unsafe multithreaded boolean read/write

Change-Id: I60d6263ea9b0740fcce2a19e83a0398424cd0784

Dont keep update unnecessarily running when even thread notifications are being processed (requires Adaptor patch as well)

Change-Id: I6fb6fc3a5b1f0406ef7180193eb01f4debab1959

Removed unnecessary mutex lock

Profiler flagged this function up.
For dali-demo it gets called over 2500 times on start-up ( by reserve message
slot).
As it's only ever called from a single thread the mutex is not needed.

Change-Id: I3e6784aa62d118360244891684c4e09d0896b3ef

moving of COMPILE_TIME_ASSERT to common

Change-Id: Ibdb4d7bc6bf9f1779875f6ed8aeb9ed11b740ef3

Stop exporting internal symbols

Change-Id: I11baf0831b4faca59ecf94030593f2f84431f5d0

[dali_1.0.15] Merge branch 'tizen'

Change-Id: Ia80069a7933bdbb49c791662f539937a3049c963

Merge "DALi Version 1.0.15" into tizen

DALi Version 1.0.15

Change-Id: Ia44ae18a89802345f3f425fd12a6c7b35417307b

Removed unnecessary use of osstream and ofstream

saves 1KB from dali-adaptor library on ARM

Change-Id: I07677bbaee9c7a7075818c5297074bfdffb06770

moving the environment options to Adaptor (where they are handled)

Change-Id: I953fcbf66c9da2da18f817a24a3868506fea254c

Add required headers when these files are included individually

Change-Id: Icff2a20f0d91630bead981e3a55ca3a880928387

Cleaned up handling for optional Assimp package.

The only place that needs to change if assimp is required or not is configure.ac.

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

Add support for glProgramBinary extension in GLES 2.0

[problem]     glProgramBinary is disabled at the moment
[cause]       Missing implementation
[solution]    Add support for glProgramBinary extension in GLES 2.0

Change-Id: I1fc798009f9cbc6a933258db2d9209d70dc0fd56
Signed-off-by: Adeel Kazmi <adeel.kazmi@samsung.com>

Merge "add missing update keep updating reasons to logging" into tizen

add missing update keep updating reasons to logging

Change-Id: Idf7f68f6462f8c2ce75eaa15fe10dfcc34941991

[dali_1.0.14] Merge branch 'tizen'

Change-Id: I477121b578e10d2d71adb1e3fdf85168240030f9

DALi Version 1.0.14

Change-Id: I6956e055641c846769674094b46b7041cc396636

Use vector wrapper

Also Use map directly in automated tests

Change-Id: Id52c1fb676842270d53fff92fa9fb63423943cb1

make frametime state flags properly atomic read/write

Change-Id: I8b3f1680c6ef88e52cc6598904afc21e42b2b942

[dali_1.0.13] Merge branch 'tizen'

Change-Id: Id6a176017a73ec185c94ce2d67f5b67e80953463

DALi Version 1.0.13

Change-Id: Iab9b1e9317c7b6cbcdfa0f5d212c86e892005ee3

Merge "(ImfManager) Initialise ImfManager only when required rather than at startup" into tizen

(Build) Ensure efl-assist is not used in UBUNTU profile

Change-Id: I73ad523c199509ac3f26ddf70cba5a92757b61ef

[dali_1.0.12] Merge branch 'tizen'

Change-Id: I7668aa8c3d852ae7d943161359e6eeda1f4aa91e

DALi Version 1.0.12

Change-Id: If26bbf75700169acf3a3e3cd1064b4ee264ea36c

(ImfManager) Initialise ImfManager only when required rather than at startup

Also remove ImfManager implementation from Wayland as it was not being initialised propertly anyway
(we always returned an empty handle as we never registered the singleton)

Change-Id: I39e44f35ccca8fa193b7a21324ca78027bda76fa

Event handling for hover event

Change-Id: If80f5cf4facd93bedcdb4ea8f76d98ffe14bb0e1

[dali_1.0.11] Merge branch 'tizen'

Change-Id: I294122fc6c65d63dd4134205028063a0d7358e96

DALi Version 1.0.11

Change-Id: I194076af7c2f2c474330998f2b3110af47065d8e

Add spec and manifest files for TV platform

Change-Id: Ifb3d187432de7dd7d5891f915a4a8740385121c8

[dali_1.0.10] Merge branch 'tizen'

Signed-off-by: Lee Morgan <Lee.morgan@partner.samsung.com>
Change-Id: I4fe15a7e78ba074ed6c2f673616a8d1ff53c8b5b
Signed-off-by: Lee Morgan <Lee.morgan@partner.samsung.com>

DALi Version 1.0.10

Signed-off-by: Lee Morgan <Lee.morgan@partner.samsung.com>
Change-Id: Ibde990f6f09e0a1a38ac311f890aa0bd5d595ccb
Signed-off-by: Lee Morgan <Lee.morgan@partner.samsung.com>

Merge "Remove Adaptor class from Public API" into tizen

Remove unused functions

Change-Id: Ie25ac47e434da0ee838fc408bf473881c74a9fcc
Signed-off-by: Yury Usishchev <y.usishchev@samsung.com>

Remove Adaptor class from Public API

Change-Id: Ie166cc3225dec2eb68c2cc4cb43a3545f671337e

(Automated Tests) Updated test suite utils

Change-Id: I15a976bf9ef324c0cc4cb4c01d45963d2ed26370

Fix build break for Tizen TV

Change-Id: I9e6c618433784ed86e2705c93eeec3af6aedffbd

Merge "(AutomatedTests) Synchronise TestGLAbstraction" into tizen

[dali_1.0.9] Merge branch 'tizen'

Change-Id: Idbf8d1e6df075605a5f50200ca5a4adde76e77cc

DALi Version 1.0.9

Change-Id: I87dcc8297fa90918a82322212e42f7ee93300145

(AutomatedTests) Synchronise TestGLAbstraction

Change-Id: Iba80899937e9b72eb413ad5918325aa6bcb06e8f

Merge "Reduced level of unnecessary debug" into tizen

(Resource Text) Remove unused functions

Change-Id: Ice8f8fc1ddda6fa08f50b98e79a71400a047af79

Reduced level of unnecessary debug

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

(Automated Tests) Fix compiler warnings in some test cases

Change-Id: I91eb66e9a01c39951b80341f73e71cc185da86af

(Build) Fix checks for gles20, glyphcache install folder

Tizen Common does not have "gles20", but only "glesv2" and
"egl" in all repos. We have a macro to replace it in the
specfile, but not the autoconf files. Fixing.

The glyphcache install folder seems to have changed in
latest releases ("/opt/usr" to "/usr"). Fixing.

Use the %configure macro instead of "./configure". Simplify
the specfile by using a conditional variable.

Signed-off-by: Manuel Bachmann <manuel.bachmann@open.eurogiciel.org>
Change-Id: Icc88ee08b2f373be92e1a9be0f85656da032f99e

(Build) Ensure branch coverage is collected for later versions of lcov

Change-Id: I52200982606e14c2191470fd8bc3b60a776e9271

(Build) Combined common & application libraries

Change-Id: I974fe5a668001364489c4891ec01774f72e04bf9

Fix X11 build for Tizen Common (MSG_DOMAIN_CONTROL_ACCESS)

The X11 adaptor code uses the "MSG_DOMAIN_CONTROL_ACCESS"
macro, which was at once an alias to "ECORE_X_ATOM_E_ILLUME
_ACCESS_CONTROL" in a Tizen Mobile-specific version of
Elementary.

As Tizen Common now has a more recent version of Elementary
which contains this macro, we can directly use it ; this
shouldn't break other profiles either.

Change-Id: I2687b7303a5f7bcb2360c0c682e78a71211e6315
Signed-off-by: Manuel Bachmann <manuel.bachmann@open.eurogiciel.org>

[dali_1.0.8] Merge branch 'tizen'

Change-Id: I08db1c75ea79c859ae0127702590ad6f33ec20b3

DALi Version 1.0.8

Change-Id: Iaa436985cf759803f2e013accde96959b14fc4f8

(VSyncMonitor) Fixed build break for UBUNTU profile

Change-Id: Ie4fe3d883bf2402a722d854eed3213bed2e4f688

Merge "Redesigning render sync to handle reduced frequency" into tizen

(AutomatedTests) Added automated tests for SingletonService

Change-Id: I59e9aa7267ee699709071a3999274d3d25b98dfe

(Application) Added a New method which does not require an arglist

Change-Id: I49c7bf988e6ccf3153a00f6793da4c9822412287

Added SingletonService, removed Singleton APIs from Adaptor & removed Application::Get

Change-Id: I09191b12dfb40c54e68f3bfa811bd61b70248780

Redesigning render sync to handle reduced frequency

[Problem] Update and render timers are out of step with expected results.
[Solution] Manage frame skipping and synchronisation with VSync in a single
place.

Moved all compositor syncing to reside in PixmapRenderSurface only.

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

[dali_1.0.7] Merge branch 'tizen'

Change-Id: Ie6d2c1a3c7b88d1f08baf2b90489c30f5625ddf5

DALi Version 1.0.7

Change-Id: Iba95c3e85bdcd530973df214f9346c97cf43c67b
Signed-off-by: Agnelo Vaz <agnelo.vaz@samsung.com>

Merge "Moved FrameTime from Core to Adaptor" into tizen

Uses TextArray new type definition.

TextArray is now a Dali::Vector instead a std::vector.

Change-Id: I49930f74969b8b7b7562b57f104baf1ebcf3585a
Signed-off-by: Victor Cebollada <v.cebollada@samsung.com>

Moved FrameTime from Core to Adaptor

[Problem]  Core::Update was second-guessing VSyncMonitor.
[Solution] Moved FrameTime from Core to Adaptor; now passes in elapsed time
and last/next vsync times to Core::Update().
Updated test harness

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

[dali_1.0.6] Merge branch 'tizen'

Change-Id: Ic69941959efe20002d53e30d5967b76f1e13b164

DALi Version 1.0.6

Change-Id: I0e6d397f9b4b576b747ed409f8620d17ddcbb933

[dali_1.0.5] Merge branch 'tizen'

Change-Id: If9ba312ec08080ba8c105d0b3c28678e898f518b
Signed-off-by: Lee Morgan <Lee.morgan@partner.samsung.com>

DALi Version 1.0.5

Change-Id: Ib78687ee422a30e690daed9c3f91f63f95b2e3cf
Signed-off-by: Lee Morgan <Lee.morgan@partner.samsung.com>

Revert "Revert pc file & spec changes"

This reverts commit a87c8432f6cdf5eec93117eaf9fe1c980046b76e.

Change-Id: I0a432477303b5518d807591af78aca055d1a48c2

Revert pc file & spec changes

Reverting due to ensure matching weekly release across all repos.  Will be pushed later.

This reverts commit f85a9b4bc73ea297646152c134a6ae340634fcbb.

Change-Id: Ie6795a9e2a44a5dadf7ac616fe582053a0b1103e

Fix the plugins install path, and search path in "*.pc" files

Plugins were not always installed in a directory matching
the architecture ("/usr/lib" instead of "/usr/lib64" e.g.),
which caused build problems.

Depending on the architecture, libraries may be installed
in another directory than "/usr/lib" ("/usr/lib64" e.g.)
Reflect that in the .pc files.

Change-Id: I286907a4b3d59e3416ca96e8903665f155c07415
Signed-off-by: Manuel Bachmann <manuel.bachmann@open.eurogiciel.org>

Public API headers are installed in the correct folder

Change-Id: I8cf1b8865be9fe35a4c95dfe43de7ae10acfb911

Merge "Removed troubleshooting section from README" into tizen

Removed troubleshooting section from README

- libjpegturbo.so link created by dali_env script now.

Change-Id: Ia9ad48e71620ed75c72e98a844972ed6a23da4bf

Updated documentation for stereoscopic rendering

Change-Id: Ie1ab44aaff897b02dd967282e73d04646d041941

Refactored image loaders, added public API

[Problem] No synchronous access to the image loaders from external application
[Cause] No requirement
[Solution]

Moved image loading code out of resource-thread-image.cpp into a new image-loader.cpp.

New API to allow bitmaps to be loaded synchronously.
These can then be used to instantiate a BitmapImage().

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

Merge "DALi Version 1.0.4" into tizen

[dali_1.0.4] Merge branch 'tizen'

Change-Id: Ie4edfa6323fdc01304b9150c825607dc59edbc08

DALi Version 1.0.4

Change-Id: Ifb1510ecc7a96db8ea961a10392e586a8f58fe8a

Merge "Implemented the Handle assignment operators properly" into tizen

Implemented the Handle assignment operators properly

[Problem] Some unsafe handle assignments are allowed by the compiler e.g.
Actor notAnImageActor = Actor::New();
ImageActor notSafeToUse = notAnImageActor;
[Cause] using BaseHandle::operator= is unsafe
[Solution] Implemented the Handle assignment operators properly

Change-Id: I0df98636a08f49afbf195f0ee9f7d4f7bdbfdc5a

Merge "Remove Deprecated APIs" into tizen

Remove Deprecated APIs

Change-Id: I9f70e6649fc6124d045420398fc16f9f7e0e00e6

Fix to disable all tests for in-flight cancellation off the async loading path

[problem]
When loading nine-patch images, an interruption exception could be
thrown on a thread that doesn't handle them.

[cause]
Our standard loading path involves two threads at a time coordinating on
cancellation of resource loads that have already been started.
Nine-patch images are a special case that execute image loading file IO
and decoding to bitmaps directly on the event thread, thus introducing a
third thread into this mix.
The event thread was not considered in the design or the reviews of the
late image cancelation feature.

[solution]
Make all image loading paths other than the main async one on a
background thread use a stubbed NOP implementation of the interface that
lets such loading paths interrupt themselves.
Thus the nine-patch loading path can never be interrupted.

No currently-commited demos show this issue but the following patch
fails without this one and works after it is applied:
https://review.tizen.org/gerrit/25392

Change-Id: Id5adeebb95b41eac1414e58b8ab61e560ad1a585
Signed-off-by: Andrew Cox <andrew.cox@partner.samsung.com>

Ensure a valid profile is selected

[Problem] It was possible to pass any old junk when using --enable-profile and there would be no
          error.
[Cause]   we never actually checked if it matched any of the profiles we support.
[Solutin] Ensure we check all profiles and show an error when the profile does not match any of
          the ones we support.

Change-Id: Ide85946093a06e7dd1f7ef97585a42b1bae71c81

Fix for ColorController.
Creates an internal implementation for the default color controller to avoid a crash if it's called.

[problem] Crash if the default ColorController is used.
[cause] There is no internal implementation.
[solution] Create an internal implementation.

Change-Id: I49ac335ed6db2dfc68b6dae3beea42c1b5db275e
Signed-off-by: Victor Cebollada <v.cebollada@samsung.com>

Support to build against ubuntu environment.

Change-Id: I59e1aa0adc3a3f32cde756ae816e71d955fd1a8e
Signed-off-by: Adeel Kazmi <adeel.kazmi@samsung.com>

[dali_1.0.3] Merge branch 'tizen'

Change-Id: I2fdaee520c05b07783c08691c11b8ca780b4d620
Signed-off-by: Agnelo Vaz <agnelo.vaz@samsung.com>

Remove lib-adaptor as it is not used anymore

Change-Id: I62644c25c9d1f22f2adfe43a31311692d7a96302

DALi Version 1.0.3

Change-Id: Ieeeb77b93ea1b24d3987fddc5f4d68986e2fabf0
Signed-off-by: Agnelo Vaz <agnelo.vaz@samsung.com>

Fix for crash when first resource load is synchronous

[problem]
If the first load of a resource executed by dali is synchronous,
it is treated as cancelled.

[solution]
Change initialisation of ResourceThreadBase members tracking
cancellation.

Change-Id: I61f4f86dbcb638fb385558429909d46d7bd066a3
Signed-off-by: Andrew Cox <andrew.cox@partner.samsung.com>

[dali_1.0.2] Merge branch 'tizen'

Change-Id: I9d0dc43de965f34f42dbcfcbc0e332609c07e7cd

DALi Version 1.0.2

Change-Id: If5daac36e05fef3445193b1d8cf68287b1a1a6d2