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>

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>

Do not call malloc during terminate

There was some strange crash issue during application terminate.

That crash comes at malloc.c during destruct JsonParser.

So, let we just remove some potential of mallocate memory during terminate,
what might not neccessary.

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

Assume memory allocation failed cases

Let we consider malloc return null.

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

ImageVisualShader debug script make to add custom attribute / varying / uniform

Let we allow to set some extra uniform and attribute , varying if user need to debug.

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

DALi Version 2.3.13

Change-Id: I694a7fa17d2953cb73dea911f332f3276de1352d

(Automated Tests) Use github link instead of w3 for SVG image

Change-Id: I801f86b265f0101c5c906ccfcb221c477aee9484

(Vector) Change ConditionalWait as Mutex at lottie task

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

Change  precision of corner radius relative shader codes as highp

There was some visual defect when view size is near 20,000 in target.

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

Merge "Change precision of shader variable" into devel/master

Merge "DALi Version 2.3.12" into devel/master

DALi Version 2.3.12

Change-Id: I853019d9e13691b62c471ac789e010c101fca2d1

Fix Shadow crash issue during SceneOff

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

Change precision of shader variable

Change-Id: I4138a170c0ba9bea240a0fd0fcfab36afc2d182b
Signed-off-by: seungho baek <sbsh.baek@samsung.com>

Merge "Support Scene3D on gles 2.0" into devel/master

Merge "Fix bug when we try to load two image s.t. premultyOnLoad difference" into devel/master

Merge "Emit ResourceReady if we call Reload even if it was ready visual." into devel/master

Merge changes Ic9dba03a,Id5ec87c1 into devel/master

* changes:
  Print image process duration when we trace it
  Reduce the length of log during image trace

Support Scene3D on gles 2.0

Let we support Scene3D::Model under gles 2.0 devices.

Note : Since glsl doesnt support textureLod, we cannot support roughness.

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

Added UNSAFE_RENDERING_DIRECT backend mode for GLView.

UNSAFE_RENDERING_DIRECT mode allows injecting the GL calls
directly into window context. This method is considered unsafe
and should be used cautiously as it may alter GL state of the
main DALi pipeline (ie. affect UI rendering).

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

Print image process duration when we trace it

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

Reduce the length of log during image trace

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

Fix bug when we try to load two image s.t. premultyOnLoad difference

Let we seperate cache between
"Could-be-premultiplied, but not loaded yet" VS "Do not premultiplied"

Currently, we check both of them as same item.

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

Emit ResourceReady if we call Reload even if it was ready visual.

If the visual was already ready state, and if we call Reload, the visual state
still ResourceReady. So ResourceReady signal was not be emitted.

Since someone might need to know the timing of resource loaded after Reload(),
Make we reset status as PREPARE, and wait callback again.

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

Merge "DALi Version 2.3.11" into devel/master

DALi Version 2.3.11

Change-Id: Ifb9447b21d0f483c31d2658660f19c15694704a5

Merge "Ignore corner blur radius if glsl version is low" into devel/master

Test harness sync

Change-Id: I480e0fb3b1f3a475ea02ec7be1599f9af36a8f18

Merge "Let we make WebView invisible if window is hide" into devel/master

Merge "Make uShadowBias highp" into devel/master

Merge changes Iafce95d1,I701627ff,Ic2b33c53 into devel/master

* changes:
  Apply some missing shader names
  Add precompiled shader names
  Add name of shader what we generate as default

DALi Version 2.3.10

Change-Id: I86045f8c3be35213371476eac93fcc52d3b3b615

Make uShadowBias highp

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

Apply some missing shader names

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

Add precompiled shader names

Let we allow to use precompiled shader also use shader name

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

Add name of shader what we generate as default

Let we use the feature of shader name for our dali side default shaders.

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

Let we make WebView invisible if window is hide

Since we didn't check window visibility changed during webview is on scene
+
We don't check whether webview is on scene or not,

it could be make WebEngine keep rendering infinitly.

Let we make web engine visible only if we can assume that it is visible now.

TODO : We cannot check parent visibility changed for current system correctly,
without any overhead. Todo in future

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

Merge "Make model load task priority as low" into devel/master

Merge "Create Shadow RenderTask only shadow is required." into devel/master

Ignore corner blur radius if glsl version is low

Let we make to not blur rounded corner if gles version is too low

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

Make model load task priority as low

Since gltf model load spend a lots of time, let we make it priority low

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

Create Shadow RenderTask only shadow is required.

 - By using OrderIndex, Shadow RenderTask can be created only it is required.

Change-Id: I65a14ff4db167b1aa546be81f80238008322e7b5
Signed-off-by: seungho baek <sbsh.baek@samsung.com>

Fix wrong order of linear enumeration

Since Fitting mode have FIT_HEIGHT = 4 and FIT_WIDTH = 5, we need to follow up the order of enum

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