fix incorrect index range in emoji-helper

Length lastCharacter = startIndex + numberOfCharacters;

Previous lastCharacter's actual value is "lastCharacter + 1"
This incorrect value makes the whole code misleading.

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

Fix remote UTC SIGKILL issue.

Due to the utc can test maximum 60 seconds,
We cannot test 2 or more network sample.
(Current dali's CONNECTION_TIMEOUT_SECOND is 30 seconds)

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

Merge "Fix uint16 max issue in TextLabel" into devel/master

Fix uint16 max issue in TextLabel

If the size of TextLabel is bigger than uint16 max value,
there is a problem that assert occurs in Texture::New.

We can avoid the assert by setting the VisualTransform size to uint16 max,
but this also doesn't guaranteed rendering behavior.

To guaranteed the behavior, VisualTransform size is limitted to maxTextureSize.

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

Merge "DALi Version 2.1.27" into devel/master

DALi Version 2.1.27

Change-Id: I255331086656ae9578381d4ac1f64b269129fe00

Merge "If the default algorithm is enabled, there is no need to do "MoveFocus(Toolkit::Control::KeyboardFocus::RIGHT, deviceName);" because nextFocus is automatically found." into devel/master

If the default algorithm is enabled, there is no need to do "MoveFocus(Toolkit::Control::KeyboardFocus::RIGHT, deviceName);" because nextFocus is automatically found.

So we add conditional statements.

Change-Id: I7b4cafd7f514d38141e4918c1651f229b063d595

Merge "It may be outside the range of type int. Change to uint64_t." into devel/master

fix incorrect calculaion of natural size in text

In DoRelayout of GetNaturalSize/GetHeightForWidth,
operationsPending should inlcude LAYOUT.
Without this, incorrect size calculations will occur.

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

It may be outside the range of type int.
Change to uint64_t.

Change-Id: I41cc20a97e7213f8a27de6281bc861f6b9292867

Add observer during ProcessLoadQueue

Due to we request LoadTexture multiple time during ProcessLoadQueue,
same AnimatedImageLoading can be requested on multiple ImageLoadThread.

Decode same textureId in multiple thread is just waste the CPU resource.

This patch make we add observer in ProcessLoadQueue.

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

The FocusChanged event callback is called before the FocusLost

Change-Id: I131c7e32a4372b54b2b57c5acf53bfdc5a82c15a

Merge "[AT-SPI] Fix EditableTextControlAccessible::InsertText()" into devel/master

DALi Version 2.1.26

Change-Id: I653b9e9c59e0273ff52582a065e13d836f00cf04

[AT-SPI] Fix EditableTextControlAccessible::InsertText()

This method did not work when appending to the end of the text
(including the case where the text was initially empty).

Change-Id: I981531feb1eea3d067a06292104145d48de0a76f

Merge "Fix text AutoScroll ellipsis issue" into devel/master

Merge "Resolve incorrect position for Ellipsis when mixed LTR & RTL languages and set layout direction RTL" into devel/master

Merge "Remove useless iteration when debug mode" into devel/master

Remove useless iteration when debug mode

Some API; like fontClient.GetDescription, are very slow.
But these result values only used for debug log printing.

This patch make some useless iteration will not works for
normal debug mode + NoLogging level.

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

Fix include path value of scene-loader

Change-Id: I9753119ea72cc3db33f7251ef5d9235389f6c5a8

Merge "Free GlyphData buffer" into devel/master

Resolve incorrect position for Ellipsis when mixed LTR & RTL languages and set layout direction RTL

Here's sample code for issues:
    //Case 1:
    TextLabel textLabel = TextLabel::New("Projector استكشاف");
    textLabel.SetProperty(Dali::Toolkit::DevelTextLabel::Property::ELLIPSIS, true);
    textLabel.SetProperty(Dali::Toolkit::DevelTextLabel::Property::ELLIPSIS_POSITION, Dali::Toolkit::DevelText::EllipsisPosition::END);
    textLabel.SetProperty(Dali::Actor::Property::LAYOUT_DIRECTION, LayoutDirection::RIGHT_TO_LEFT);
    textLabel.SetProperty(Actor::Property::SIZE, Vector2(145.0f, 60.0f)); // minimize width until the ellipsis comes between two languages

    //Case 2:
    TextLabel textLabel = TextLabel::New("استكشاف Projector");
    textLabel.SetProperty(Dali::Toolkit::DevelTextLabel::Property::ELLIPSIS, true);
    textLabel.SetProperty(Dali::Toolkit::DevelTextLabel::Property::ELLIPSIS_POSITION, Dali::Toolkit::DevelText::EllipsisPosition::START);
    textLabel.SetProperty(Dali::Actor::Property::LAYOUT_DIRECTION, LayoutDirection::RIGHT_TO_LEFT);
    textLabel.SetProperty(Actor::Property::SIZE, Vector2(145.0f, 60.0f)); // minimize width until the ellipsis comes between two languages

Change-Id: Ia645d143083074383f4db3b8b2407810b69e4862

Make visualOffset as Absolute scale.

offset / uSize.xy might make some precision issue.
For example, tiling use TRANSFORM_OFFSET.
If tiling has some high value, vertex position might missmatched.

This patch will reduce precision.

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

Fix text AutoScroll ellipsis issue

Text should ellipsis while AutoScroll is running
when text length is bigger than MaxTextureSize.

this issue occurs as a side effect of another patch.
74f7af1b08ce65dde5959d22b2d533cbc64b9d2e

the height comparison condition is for another AutoScroll ellipsis issue
and this should not be removed.

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

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