Fix coverity error in shader-generator

Change-Id: Iebff30df062f13b2ef263a10c53c0f393d9d3545

Fixed signed/unsigned comparison

Change-Id: Icc5e4c3547458d230bf90ca814c204d99c8d51bc
Signed-off-by: Adam Bialogonski <adam.b@samsung.com>

Merge "Adding a Property for ImageView's Transition Effect Setting" into devel/master

DALi Version 2.3.24

Change-Id: I6b0ebf6165d96c36acfa39bc332d8fee76326738

Adding a Property for ImageView's Transition Effect Setting

Adding a new property specifically designed for ImageView's transition effect settings.
With this feature, users will have greater control over how images are displayed and animated within our app.

Change-Id: Id53f5b0c9887c8a652cc0d64e715e9c0fd8df887

(shader-generator) Only generate shaders if input shader has changed/added

Change-Id: Ic165328a6e32876436220a2ebe3fe2356267e3cc

Merge "Fix svace issue (std::string::size_type might not 32bit)" into devel/master

Merge "Fix svace issue (Setup initialize values for GlyphRun and LineRun struct)" into devel/master

Merge "Change the precision of position in text atlas shader" into devel/master

Fix svace issue (Setup initialize values for GlyphRun and LineRun struct)

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

Fix svace issue (std::string::size_type might not 32bit)

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

Change the precision of position in text atlas shader

There are some visual artifact when too many glyhps in editable text.
Change mediump to highp to fix issue.

Change-Id: I0f89447160f6bce0107c6ae5de5efeb9b4436afc
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

Fix vertical position error when cutout enabled.

Change-Id: I611529b2540a8aaabe008c188cc18fcc6b8a613e

Merge "Introduce CUTOUT Property" into devel/master

Merge "DALi Version 2.3.23" into devel/master

DALi Version 2.3.23

Change-Id: I7618b7ecdcf0b905a1b7068e3bc9a1d517d88acd

Introduce CUTOUT Property

Change-Id: I195f3cb85415ec957b600c52080c02dcaca8020a

Returns -1 for frameCount before until animated image loading finished.

Change-Id: I368d076116b8e9716d2b3ce0f64a42c559585823
Signed-off-by: Seungho Baek <sbsh.baek@samsung.com>

Merge "Make depth-index as 'AUTO' instead of CONTENT change as default, and clamp depth-index" into devel/master

Fix ShadowMap doesn't apply blendshape

There was some issue that blendshape with position doesn't works well
when we use shadow.

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

Merge "DALi Version 2.3.22" into devel/master

DALi Version 2.3.22

Change-Id: I6bd823f92057b56174cae0a6425b7a14a6424602

Make depth-index as 'AUTO' instead of CONTENT change as default, and clamp depth-index

Let we set depth index as auto if user doesn't set visual's depth index.

Previously we use same behavior if user doesn't set, or user set depth index as CONTENT.
So if user set as 0, it might not works well.

To avoid this confuse logic, let we seperate whether user didn't set the property or not.

Furthermore, let we hard-clamp the range of DepthIndex for VisualBase.

The case of AUTO_INDEX is only for special cases, and otherwise, let we use normal values
what user set.

---

We allow to use RegisterVisual use depth-index as AUTO_INDEX. If this case, it will works
like what we don't set the depth index.

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

Merge "Do not remove-and-add default renderer if text-visual relayout" into devel/master

Merge "Let we use OrderedSet for VectorAnimationManager / SceneView" into devel/master

Do not remove-and-add default renderer if text-visual relayout

We don't need to remove and re-add the renderer.

AddRenderer / RemoveRenderer send some messages to UpdateRender thread.
After this patch, we can reduce those useless messages.

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

Add blur radius to text outline

it uses the same Gaussian blur as the text shadow.

Change-Id: I3b7d7534c68cfae5de55747158b4f74ffcaa2e88
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

Fix text autoscroll multi line issue

This patch fixes issues that arise when relayout is executed during auto-scrolling.

- relayout not executed after the scroll finish callback
- stop scrolling fails in certain situations in finish loop mode
- scroll finish being called twice during immediate mode stop

Change-Id: Ib44883380a79ad13c576d18c11171ae9f1f670c3
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

Merge "Change to use InheritedVisibilityChangedSignal" into devel/master

Let we use OrderedSet for VectorAnimationManager / SceneView

Some containers that triggered only one times could be optimized by OrderedSet feature.

Let we make Add/Remove time faster.

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

Merge "Add offset to text outline" into devel/master

Add offset to text outline

Change-Id: I280ed148384325cfa83e2d48fae15a387bc81fcd
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

DALi Version 2.3.21

Change-Id: Iee3f73027b8d239c187ed2aef18c7fa1501188aa

Change to use InheritedVisibilityChangedSignal

 - instead of VisibilityChangedSignal to control Play and Pause.

Change-Id: Ic98abd6f5cddd68a1cfc18b4df351b11e6e91936
Signed-off-by: Seungho Baek <sbsh.baek@samsung.com>

Fixes analysis hub issues

Change-Id: I1b55b6d635bec59ecf843fc56574044a2ccfcbab
Signed-off-by: Adam Bialogonski <adam.b@samsung.com>

Merge "[AT-SPI] Use Accessible::IsHighlighted()" into devel/master

Merge "Add INSET Property to TextField and TextEditor" into devel/master

Add INSET Property to TextField and TextEditor

Add the REMOVE_FRONT_INSET and REMOVE_BACK_INSET properties
to TextField and TextEditor as well, which were in TextLabel.

Change-Id: Ie5a417f22fb24423fde139e57c078dc45337099f

Merge "Text controller checks data type of the clipboard" into devel/master

Text controller checks data type of the clipboard

IsClipboardEmpty works meaningfully.
Temporary code has been applied to paste html type.
Once multiple types and data can be stored in the clipboard, this code should be removed.

Change-Id: I29b279acf19dd1c3397568b55e0567d314c41990
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

Do not create unneccessary sampler

We can 'skip' sampler for TextureSet if we want to use default wrap mode.

So usual cases, we don't need to create and set Sampler objects.

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

Merge "Remove TextureManager / VectorAnimationManaver lifecycle observing" into devel/master

Merge "Less call mAnimatedImageLoading.GetImageCount() who might load file synchronously" into devel/master

DALi Version 2.3.20

Change-Id: If3d8b1e01d9c5a1c9061b59be05c05f52c47064c

Remove TextureManager / VectorAnimationManaver lifecycle observing

Since we can use Dali::Adaptor::IsAvailable(), we don't need to check
whether manager itself is alive or not.

Since both of them are member value of VisualFactoryCache, and VisualFactory
is singletone class what will be destructed at Adaptor stop timing,
Dali::Adaptor::IsAvailabe() is enough.

Add/Remove observer will give some overhead for usual cases.
So just remove it, and check adaptor vailables intead.

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

Less call mAnimatedImageLoading.GetImageCount() who might load file synchronously

Since AnimatedImageLoading might not load imagefile if we use
cached texture.

In this case, If we try to call 'GetImageCount()' or similar API, it will
try to load file synchronously, what user never notify.

For example, when we call DoCreatePropertyMap, it will call that API
what we can use some other cached value.

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

[AT-SPI] Use Accessible::IsHighlighted()

This helper allows to simplify multiple comparisons of Self() to
GetCurrentlyHighlightedActor() to simply checking IsHighlighted().

Some new usages of IsHighlighted() are also introduced before event
emission in order to reduce D-Bus traffic.

Change-Id: I5d8906db22c84764de3e8df631df4d4cf8062c2f

Disconnect DevelWindow::VisibilityChangedSignal when off scene

Change-Id: I0b7faee8af24b04a496c4f5695767b454493fce6
Signed-off-by: Seungho Baek <sbsh.baek@samsung.com>

Merge "Remove BaseHandle / BaseObject TypeTraits is trivialgit" into devel/master

Merge "Fix potential of UTC fail after notify patch merged" into devel/master

Reduce the number of UpdateOnce call after rasterization

Previous code request UpdateOnce by each visuals.
But this don't need to spend the number of requests.
So let we just use only one single request API for it.

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

Fix potential of UTC fail after notify patch merged

A lots of vector animated image visual UTC failed due to 'ForceRenderOnce' required event thread trigger.

Since force render once might execute with 'load complete' or 'animation finished' signal,
Several UTC might not synchronized.

Let we change those cases and now works well anyway.

List of UTC failed:

Test case UtcDaliAnimatedVectorImageVisualGetPropertyMap01 failed: 1
Test case UtcDaliVisualFactoryGetAnimatedVectorImageVisual03 failed: 1
Test case UtcDaliVisualFactoryGetAnimatedVectorImageVisual04 failed: 1
Test case UtcDaliAnimatedVectorImageVisualPlayRange failed: 1
Test case UtcDaliAnimatedVectorImageVisualUpdateProperty failed: 1
Test case UtcDaliAnimatedVectorImageVisualNotifyAfterRasterization failed: 1

For example log:

[Pass case]
BEGIN: AsyncTask[VectorAnimationTask] Process
Add event trigger callback with id : 0
Trigger!
END: AsyncTask[VectorAnimationTask] Process
Adaptor::UnregisterProcessor : VectorAnimationManager
BEGIN: AsyncTask[VectorAnimationTask] Process
mNeedForceRenderOnceTrigger due to mCurrentFrame is not start (0 -> 1)
Add event trigger callback with id : 1110
END: AsyncTask[VectorAnimationTask] Process
INFO: scene-graph-scene.cpp: SetSurfaceRect(154) > update surfce rect in scene-graph, from width[0], height[0], to width[480], height[800]. Changed count [1]
Execute callback with id : 0
Execute callback with id : 1110
Trigger finished
Adaptor::RegisterProcessor : VectorAnimationManager
Adaptor::UnregisterProcessor : VectorAnimationManager
BEGIN: AsyncTask[VectorAnimationTask] Process
END: AsyncTask[VectorAnimationTask] Process

[Fail case]
BEGIN: AsyncTask[VectorAnimationTask] Process
Add event trigger callback with id : 0
Trigger!
END: AsyncTask[VectorAnimationTask] Process
Execute callback with id : 0
Trigger finished
BEGIN: AsyncTask[VectorAnimationTask] Process
mNeedForceRenderOnceTrigger due to mCurrentFrame is not start (0 -> 1)
Add event trigger callback with id : 1110
Trigger!
END: AsyncTask[VectorAnimationTask] Process
Adaptor::RegisterProcessor : VectorAnimationManager
Adaptor::UnregisterProcessor : VectorAnimationManager
Warning, TestGraphicsReflection::GetSamplers is unimplemented

Execute callback with id : 1110
Trigger finished
Test failed in utc-Dali-AnimatedVectorImageVisual.cpp:1452, checking 1 == 0

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

Merge "(Vector) Add NOTIFY_AFTER_RASTERIZATION property for low fps file + Use UpdateOnce instead of KeepRendering" into devel/master

Merge "Add imgSrc to GetAttribute return map if available" into devel/master

Add imgSrc to GetAttribute return map if available

Change-Id: I9f9df3905ab13442e5ff12a04414484a25f3d41b

(Vector) Add NOTIFY_AFTER_RASTERIZATION property for low fps file + Use UpdateOnce instead of KeepRendering

Let we change renderer rendering behavior is IF_REQUIRED.

It will be useful if app try to render lottie file which has less than 60fps.

+

Let we use UpdateOnce() API for force-rendering instead of KeepRendering().

KeepRendering will make ProcessCoreEvents forcely, which might not be need for animated vector cases.

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

Remove BaseHandle / BaseObject TypeTraits is trivialgit

Since Dali::Vector doesn't call destructor, BaseHandle / BaseObject
shoudl 'not' be used as Dali::Vector.

To avoid some memory leak mistake, let we remove some mis implementations

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

Force call KeepRendering when lottie animation stopped, or frame changed

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

Revert "Do not call malloc during terminate"

This reverts commit cc221e4f8a66b6748e5c737e5dbde82c68db8d48.

Change-Id: Ib0dd44b9ca2a82738bfb1cc1a12528803327b1fa

DALi Version 2.3.19

Change-Id: I496a510356a3c578585d9898198d4080f980f1e7

Fix Crash when the InsertText event is executed multiple times between Relayout.

When using IME, InsertText may be executed multiple times within one relayout.
Fix the problem that length of the text cannot be calculated correctly.

Change-Id: Id47f7bbacf90a098055fc7b6ae0d4931a9facde2

Merge "Fix bug in transition effect" into devel/master

(Vector) Let we send finished signal only for matched sent play/stop from visual

It is possible that AnimationFinished trigger comes "after" event thread Play action comes.

If then, user's Play action be skipped.

To avoid this case, let we give the id when play state changed to Play / Pause / Stop.
And send finished signal only if the id is matched what visual know.

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

Fix bug in transition effect

Image view provides transitoin effect when image is replaced.
but this function has some bugs, so i fixed them.

Below is the issue list
1.Sometimes, some of view play transition effect when image is not replaced.
2.Sometimes, Don't play transition effect when image is replaced.
3.Focus items show transition effect even if resource is not changed.

Change-Id: I610ef472c3046177c5e3f76dd4d854497805168c

DALi Version 2.3.18

Change-Id: I104ee21bd9fe23e4fe03f4edfde6681fc11c20ac

Fix hit-test RenderTask error

There is a problem that the hit-test does not work properly because the
RenderTask is always fetched with a fixed index from the RenderTaskList.
It is fixed by passing the corresponding SceneView's RenderTask.

Change-Id: Id114c839455613fc4ed6437e2cb8482ee6677be9
Signed-off-by: huiyu.eun <huiyu.eun@samsung.com>

Remove unused AccessibilityActivate method

Change-Id: I2ab634421bf9eb81327f7fee0c96a1f6ccac30b8

DALi Version 2.3.17

Change-Id: I9f4993940f0b0ab230d3eaf88013c72cdb739b66

Merge "Apply premultiply on initial time in animated visual" into devel/master

DALi Version 2.3.16

Change-Id: Ib3f94717e7af660bc16ab47fffb235db0457c63f

Apply premultiply on initial time in animated visual

Change-Id: I7ce10be72455c659f5b5d09c3ce303f39dd13102

Merge "Make ModelNode / Light don't work A11y + Clean code lines" into devel/master

Merge "[AT-SPI] Lazily calculate the "class" attribute" into devel/master

Make ModelNode / Light don't work A11y + Clean code lines

Let we make ModelNode and Light disable to create ControlAccessible as default.

And also, let we seperate the header file API override comes from.
It will make easy to find where it is comes from.

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

Fix svace issue phase2 : Need to check Property::Value.Get() return

It is possible that user set invalid value type.

If then, we need to skip given value setter.

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

Merge "(Vector) Change shader at visual side" into devel/master

Merge "Ensure image visual shaderType is in range" into devel/master

Merge "(AutomatedTests) Let we make adaptor invalidate if adaptor stop" into devel/master

DALi Version 2.3.15

Change-Id: Ie546eda86d7b47c8473d6aca29c86d4f38a99549

Ensure image visual shaderType is in range

Let we make assert if we try to use invalid native shader feature

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

[AT-SPI] Lazily calculate the "class" attribute

This helps streamline Control::Initialize() by moving the attribute calculation
to ControlAccessible::GetAttributes() which is where the value is really
needed. Additionally, GetAttributes() is cleaned up a little for readability.

Change-Id: Iecc2a63a200d23e4bfb30681fa00b63ca4d72c5a

(Vector) Change shader at visual side

Let we change animated vector visual shader at visual side, instead of dali-extension.
Now we can use cached shader.

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

(AutomatedTests) Let we make adaptor invalidate if adaptor stop

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

Merge "Change TransitionEffect" into devel/master

Merge "Ignore overwritable image changeness during utc test" into devel/master

Merge "The canvas view defaults to SYNCHRONOUS_LOADING." into devel/master

Change TransitionEffect

The existing effect was rather slow and unnatural.
therefore, some of the settings have been modified to make the effect more natural.

Change-Id: I42ef237dd831dd210766bf670b2d3061def69c44

Merge "Lock mutex when we queue offscreen texture at GlView" into devel/master

Ignore overwritable image changeness during utc test

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

The canvas view defaults to SYNCHRONOUS_LOADING.

A crash occurs when multiple CanvasViews are operated async.
So, we change it to operate synchronously by default.

The problem of async operation when using a multi-canvas view should be resolved later.

Change-Id: I9bb4c8165b69ebdd93753211214fcadf06a3f456

Introduce REMOVE_FRONT_INSET, REMOVE_BACK_INSET Property

Added new property to TextLabel

Example:
textLabel.SetProperty(DevelTextLabel::Property::REMOVE_FRONT_INSET, false);
textLabel.SetProperty(DevelTextLabel::Property::REMOVE_BACK_INSET, false);

When REMOVE_FRONT_INSET is set to false, the xBearing of first glyph won't be trimmed.
When REMOVE_BACK_INSET is set to false, the advance of last glyph won't be trimmed.

Change-Id: I169d87b9f83d3b204ea51601188982cda54cb9d0

Merge "Fix svace issue : Need to check Property::Value.Get() return" into devel/master

Merge "DALi Version 2.3.14" into devel/master

DALi Version 2.3.14

Change-Id: I523b759829956740ddc95706005ce408a0741041

Add CastShadow and ReceiveShadow for Model and ModelNode

 - CastShadow can be used to make an object cast shadow or not.
 - ReceiveShadow can be used to make an object be not drawn any shadow on its surface.
 - Model and ModelNode have the methods.
 - If Model's method is called, it is inherited to its current child ModelNode.
 - If ModelNode's method is called, it is only affects the ModelNode itself.
 - The ModelNode's property changes do not affect its parent Model's property.

Change-Id: I8ebeeed9ed814718a38ad7f69c43a4814a5a70ae
Signed-off-by: Seungho Baek <sbsh.baek@samsung.com>

Fix svace issue : Need to check Property::Value.Get() return

It is possible that user set invalid value type. If then, we need to skip
given value setter.

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

Let we use legacy rounded blur algorithm for gles2.0 + remove GL keyword

Since low spec device also want to use rounded blur color visual,
let we make them use some old legacy code, which required low calculation.

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

Lock mutex when we queue offscreen texture at GlView

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

Fix code comments

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

Fix many svace issues at dali-toolkit (integer overflow + etc)

This is a combination of 3 commits.

Let we remove many cases of dali-toolkit when we can be call 0u - 1u, which is undefined behavior.

Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>
(Scene3D) Fix svace issue : convert uint32_t to std::streamoff + minor

Fix svace issue when we try to convert from 32bit unsigned integer
to (might) 64bit signed integer.

+

Fix the name of function collision issue

Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>
Fix svace issue : Avoid to divide by zero cases

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