Free GlyphData buffer

We make GlyphData's buffer as malloc.
So, we need to change the release method as free.

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

Merge "Fix bug when we use bitmap font." into devel/master

Fix bug when we use bitmap font.

We can use custom bitmap font without using glyph.
In this case, glyphBuffer can be RGBA.

Current logic assume that glyphBuffer only L8 if isColor is false.
This patch it that case works well

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

Make some visual use DecoratedVisualRenderer

Make some visual use DecoratedVisualRenderer s.t. use some properties as default :
 CORNER_RADIUS
 CORNER_RADIUS_POLICY
 BORDERLINE_WIDTH
 BORDERLINE_COLOR
 BORDERLINE_OFFSET
 BLUR_RADIUS

Now BORDERLINE_WIDTH and BLUR_RADIUS informations can be accessed in
dali-core level. So we can calculate update size

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

Merge "Minor reduce textlabel creation time." into devel/master

Delete useless define of default shaders

We can use #if defined(~~) instead of #ifndef ~~ #define ~~ 0
Now we can check the shader more clean environment.

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

Minor reduce textlabel creation time.

1. Reduce the multiply operations during TypeSetter
combine each glyph on bitmap.

2. Every textlabel hold it's own TextController
and each TextController try to get StyleManager's configure.

In this case, there was some operation
from const Property::Map to Property::Map. It required vector copy.
So I make that we don't copy the vector anymore.

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

Merge "Fix visual defect during text scroll" into devel/master

Sync UTC with text shape method.

Harfbuzz shape algorithm have some bug if we use Emoji.
So, make UTC don't check advance value if we test Emoji.

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

Fix visual defect during text scroll

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

Merge "DALi Version 2.1.25" into devel/master

DALi Version 2.1.25

Change-Id: I87da0bfd9384da3a3d9c3a1192d7e6234925cd11

Merge "Fix svace issue for dli-extra" into devel/master

Merge "Refactoring related-runs for the mutable-markup (Spannable)" into devel/master

Refactoring related-runs for the mutable-markup (Spannable)

We created common abstract class to inherit all run class that are needed for mutable-markup (Spannable)
  - ColorRun
  - CharacterSpacingCharacterRun
  - FontDescriptionRun
  - UnderlinedCharacterRun
  - StrikethroughCharacterRun
  - BoundedParagraphRun

Change-Id: I47cea29e0c9d833068b731402de68ae20cabb9d9

Ignore vertical line alignment when the line spacing is negative

This is to resolved cut-off in first-line when line-spacing is negative
 and vertical line alignment is bottom or middle.

Testing code:
https://github.com/wonrst/NUI-Test/blob/main/text-vertical-line-align/verticalline.cs

This patch should be preceded by the patch below:
https://review.tizen.org/gerrit/c/platform/core/uifw/dali-toolkit/+/274097

Change-Id: I80e6d838ad08f7f34d79f6018bb44ae467c7a89a

fix issue with VerticalLineAlignment with multiple font sizes

when having multi-line, every line contains text that have different font size.
VerticalLineAlignment does not work properly.

Change-Id: I89b2a1b0c3a01c6cd79dc8de3c0e42876e2b7251

Fix svace issue for dli-extra

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

Enable to use GPU masking in image visual

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

Merge "DALi Version 2.1.24" into devel/master

Merge "GPU Alpha Masking for Animated Image Visual" into devel/master

DALi Version 2.1.24

Change-Id: I84e156bccaccb2b977b1ddca2c59d6f58a75bf86

GPU Alpha Masking for Animated Image Visual

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

Fix automated test failures about the SvgVisual

Change-Id: I4aef4278e54337e484ef66a4dfa5229c407fbd5e

Merge "Apply premultiply in animated image visual" into devel/master

Merge "Fix key event propagation in text controller" into devel/master

Merge "Trigger svg loading early" into devel/master

Trigger svg loading early

Change-Id: Ie715e242d32ca68060c8503809222960f545ae60

Apply premultiply in animated image visual

animated-image-visual not properly apply premultiply.
so i make this patch to apply it.

Change-Id: I8b0a2b123ba6789f91163169ded93e07c6170e49

The FocusChanged event callback is called before the FocusLost and FocusGained event callbacks.

Change-Id: Idb145bdd0f9f74e84287ab783a34a72485667ae8

Fix key event propagation in text controller

The KeyEvent::UP of DALI_KEY_BACK is not propagated from text controller,
this interferes with the behavior of the parent view in some cases.

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

Merge "Add text selection popup style" into devel/master

Merge "DALi Version 2.1.23" into devel/master

DALi Version 2.1.23

Change-Id: I1699d66d1291c2cc98b22a5bd4ada63b25aa10be

[AT-SPI] Make DevelControl::GetAccessibilityRelations() equivalent to Accessible::GetRelationSet()

The previous std::vector<std::vector<Address>> return type, in which the
RelationType enumeration value served as an index to the outer vector,
can be changed to std::vector<Relation>, which is more compact (there
are very few relations per control in practice). Also, the relation
target type is now Accessible*, which is more useful than Address.

Change-Id: I9f7b86d7415e1711b8a81cd66e271ff9cdffd5c2

Add text selection popup style

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

Fix text render issue when use tiling

Due to the "uTextColorAnimatable" uniform values are not registered
into tiling renderer, It show some strange color when we required
some tiling text feature.

This patch make that all other renderers in mRenderList also have
there own constraint relative with TextLabel's TEXT_COLOR index
so tilied renderes also rendering well

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

Store VisualUrl's hash result, and reuse it

When we request load of animated image visual,
It will generate the image url's hash for every frame.

This patch cache the VisualUrl's hash result so
We don't need to re-calculate hash for the same image url.

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

Merge "Change conflict of some visual's Impl namespace" into devel/master

Merge "Optimize some text-typesetter rendering operations" into devel/master

Merge "Fix Textlabel bug if size height is bigger than 16384" into devel/master

Optimize some text-typesetter rendering operations

1. Make pixelwise iterations range more tightly.
ex) for(i=0;~~) if(i<X) continue; --> for(i=X;~~)

2. Make some pixelwise operations use precalculated color
ex) in DrawBackgroundColor, we don't change the target color.
So, we can specificate that color will be writed for each pixel.

It can reduce some float point data operations.

3. Make CombineImageBuffer reuse the memory.
CombineImageBuffer only used in Typesetter class. (It was private API)
and this API never use it's member value.
And also, All that API caller replace one of pixel buffer
either top or bottom.

So now, I move this API in unnamed namespace function +
store the result into already exist pixel buffer
so we can reduce some memory allocation time.

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

Merge "Make highp for some variable for 3d model shader" into devel/master

Fix Textlabel bug if size height is bigger than 16384

Instance fix bug.
text-style shader requied both style texture and overlay style texture.
But for some special case, overlay style texture didnt applied.

TODO : Actually, we need to shrink text relative shaders.
It should be done in future.

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

Merge "Support asynchronous svg loading" into devel/master

Make highp for some variable for 3d model shader

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

Make shader version be top of the code

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

Change conflict of some visual's Impl namespace

ConnectionTracker now use struct Impl.
So, some namespaces conflict with that names usage.

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

Modify default shader according to the gltf pbr spec

 - Add uColorFactor.
   In glTF, color factor can be used as a color when there isn't color texture,
   but it is also used to be multiplied with base color that is sampled from color texture.
   In previous implementation, when there isn't color texture, color factor is converted
   as single pixel texture.
   If the texture is from color factor, we shouldn't multiply color factor again.
   But, we couldn't notify whether the texture is from image file or color factor.

 - Add vertex color

 - Add uMetallicFactor, uRoughnessFactor

 - Use #ifdef instead of single value texture.

 - Use pre-computed brdf texture.

 - Modify alpha mode

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

Merge "Apply EncodedImageBuffer with Atlas" into devel/master

Merge "DALi Version 2.1.22" into devel/master

DALi Version 2.1.22

Change-Id: I89d8996908f0852cd10939a7223c6a02036cedc5

Support asynchronous svg loading

Change-Id: I46e977fa00fd9a173663b253867e198ac9c40cc8

1. Rather than DoMoveFocusWithinLayoutControl() being done from the parent If the currently focused actor is also a layout Control,
DoMoveFocusWithinLayoutControl() should be performed starting from the currently focused actor.

2. If the actor returned through GetNextKeyboardFocusableActor() is itself, it becomes an infinite loop.
So It should recurse DoMoveFocusWithinLayoutControl() only when not yourself.

Change-Id: I8ad1f868151337a9e13869682c99c5a19b82bb0f

Reverts the patch that prevents focus set when parent is visible when false due to backward compatibility issue.

https://review.tizen.org/gerrit/#/c/platform/core/uifw/dali-toolkit/+/274317/

Change-Id: I5a2152395238596dbccd4e6b064c990a0d802930

Merge "Cache animated image's frame index" into devel/master

Apply EncodedImageBuffer with Atlas

Allow to use atlas with EncodedImageBuffer.

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

Merge "[AT-SPI] Call EmbedAtkSocket() from WebViewAccessible" into devel/master

Cache animated image's frame index

There is no reason to multiple decoding for same-image's same-frame loading.
It just increase our memory useage and CPU usage.

This patch cache animated image's frame index so
we can re-use already decoded frame informations.

+

For more, Due to the animated image decoding can hold big amount of observers
in one frame, we need to reduce runtime of NotifyObservers API.
This patch also reduce NotifyObservers time complexity from O(N^2) to O(N).
Note : We can't add additional observer due to the mQueueLoadFlag flag is true.

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

[AT-SPI] Call EmbedAtkSocket() from WebViewAccessible

This is necessary for the ATK-based AT-SPI subtree of the web content to
know the address of its parent.

Change-Id: I98ad98bc5d016df9a827d733451a4ceda5b429c3

Delete YGNode only root of Flex::Node

YGNode remove the memory recursively internally.
If we delete childe node's YGNode, it become double free.

This patch make Flex::Node free only it's own YGNode.

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

CPU Alpha Masking for Animated Image Visual

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

Merge "[AT-SPI] remove isRoot param from Accessible::Get" into devel/master

Merge "Fix glTF loading issue." into devel/master

[AT-SPI] remove isRoot param from Accessible::Get

The Accessibility::Accessible::Get for a window could be called
before getting accessible to add window accessible.

So far, we thought that the next part creates an accessible for
window for the first time.

  accessible = Accessibility::Accessible::Get(rootLayer, true);
  bridge->AddTopLevelWindow(accessible);

However, there is a case where it is created before this part
caused by following symbol

  CSharp_Dali_Toolkit_DevelControl_GetAccessibilityStates+0x144

In this case, isRoot is set to `false`. Then window will have
incorrect accessible information.

For more information, please refer to the following.

  https://github.sec.samsung.net/tizen/atspi/issues/60

[Dependency]
https://review.tizen.org/gerrit/#/c/platform/core/uifw/dali-adaptor/+/274737/

Change-Id: Iadca9c42aa2027f372838247ac23b27d484e4c8d

Merge "Check window validity" into devel/master

DALi Version 2.1.21

Change-Id: Ifd1fa240f5e970e38c3cc3b41c263467b5e93013

Fix glTF loading issue.

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

Check window validity

Change-Id: I104882efa0cd8506388ddb92328adc4152cdc1b1

Merge "Changed default font family of 7.0 IoT UX" into devel/master

Merge "Remove ResourceReady before rasterization in SvgVisual" into devel/master

Changed default font family of 7.0 IoT UX

SamsungOneUI -> BreezeSans
SamsungOneUI font can not be used as default due to license problem

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

Merge "Allows you to specify the root when calculating the focus movement algorithm." into devel/master

Merge "[AT-SPI] ScrollView: Use child postion for ScrollToChild" into devel/master

Allows you to specify the root when calculating the focus movement algorithm.

Change-Id: Id0b1a191301008c979cc00258a75d380e25ca99e

Remove ResourceReady before rasterization in SvgVisual

It will use a default size of the content file if the size is not set.

Change-Id: I5dbfe79146a461d412c39ab5dcf37fabd94b683c

Fix svace error about initialization of member variable

 - ShaderDefinition::mUseBuiltInShader

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

Refactoring TextureManager cache as Dali::FreeList

Apply additional container as Dali::FreeList as TextureIdConverter;
From TextureId to TextureCacheIndex.
and add mTextureHashContainer, key is TextureHash and value is list of TextureId.
These things will make us access mTextureInfoContainer more faster.

Previous code iterate whole std::vector list to find cached texture.
This patch make we can get cacheIndex from textureId as O(1) by Dali::FreeList
and get textureId from textureHash as O(1 * collision)
by std::unordered_map<TextureHash, vector<TextureId>>.

This patch make TextureCacheIndex as a kind of structure. named TextureCacheIndexData.
This data use 4 bit as "TextueCacheIndexType" and 28 bit as "Index".
By this structure, we can use only one TextureIdConverter.
We can know what this TextureId come from ;
NormalImage or ExternalTextures or EncodedImageBuffer.

So now, we can do all API things as near O(1). Without useless iteratating.

It will reduce ImageView's SceneOn and SceneOff time.
And also, reduce AsyncLoadComplete API runtime, that animated image used
every image frame.

Note : After this patch, TextureId will not be unique anymore.
TODO : Masking feature still works on O(N). We need to make new container for it in future.

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

Merge "DALi Version 2.1.20" into devel/master

DALi Version 2.1.20

Change-Id: If7d4f644122b73eef460e41fa7ca3c261f8723ad

Merge "Fix webp&gif issue" into devel/master

Merge "Caching cropToMask in texture manager" into devel/master

Merge changes I7b63ee17,I61bd1ed2 into devel/master

* changes:
  Use default PBR shader to the scene-loader
  Fix Scene3d-View light issue

Caching cropToMask in texture manager

cropToMask need to be cahing,
so i added this patch

Change-Id: I6fecdfd8bc12a0c5d270c50da50340331ce0917f

Merge "Add ECORE_IMF_CALLBACK_SELECTION_SET to IMFContext" into devel/master

Use default PBR shader to the scene-loader

 - glTF uses built-in shader for rendering.
 - glTF features of Emission and Ambient Occlusion are added.

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

[AT-SPI] ScrollView: Use child postion for ScrollToChild

It was not able to scroll in a child of scroll view if the child is
not a child of the first depth.

(1) [scroll pane],[(class=ScrollView)],[364,199,656,36],[TextSelectionScrollView]
  (2) [table],[(class=TableView)],[364,101,901,134],[]
    (3) [push button],[(class=PushButton)],[364,101,447,134],[Select all]
      (4) [image],[(class=ImageView)(highlight=)],[364,101,447,134],[]
    (5) [unknown],[(class=Control)],[811,107,1,122],[]
    (6) [push button],[(class=PushButton)],[812,101,265,134],[Copy]
    (7) [unknown],[(class=Control)],[1077,107,1,122],[]
    (8) [push button],[(class=PushButton)],[1078,101,187,134],[Cut]

The node (8) was not able to bring in even though it has highlight.
This patch makes node (8) bring in when it grabs highlight.

Change-Id: I862a8adcc2bd6212464ffc833a2a0d0cf0a4480c

Merge "If VISIBLE is false, the focus cannot be set and code clean." into devel/master

If VISIBLE is false, the focus cannot be set and code clean.

Change-Id: I92d65ad4ab87368bce3a85d0a112955e306a104a

Fix Scene3d-View light issue

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

Merge "Resolved incorrect size when call GetTextSize(TextLabel) for one glyph" into devel/master

Merge "Enhance texteditor background drawing performance" into devel/master

Add ECORE_IMF_CALLBACK_SELECTION_SET to IMFContext

added selection callback from IMF for tizen 7.0 new feature support

adaptor: https://review.tizen.org/gerrit/#/c/platform/core/uifw/dali-adaptor/+/274302/

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

Merge "If the currently focused actor is hidden or disabled, it should lose focus." into devel/master

Merge "Block the changeness of PreMultiplied for some visuals" into devel/master

Block the changeness of PreMultiplied for some visuals

Svg, AnimatedVector, Text, and Gradient Visuals always use
PreMultiplied texture.
But PreMultiplied option is Visual::Base property.

So it was possible to change premultiplied option as false
even It's visual type is not allow it

This patch make some kind of visual that should not change the
PreMultiplied alpha flags don't change it.

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

Merge "[AT-SPI] Set child's parent in WebViewAccessible" into devel/master

If the currently focused actor is hidden or disabled, it should lose focus.

Change-Id: I9ccc144a266982085b4aae43cbd87bd1e3d2391e

Merge "fix incorrect selection highlight position with postive line spacing" into devel/master

add theme to TextSelectionPopup according to 7.0 UX

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