Revert "removed the unneeded comments"

This reverts commit 0cd18874fdb3c3a1d6241dc9d1506a8e3215dde5.

removed the unneeded comments

Revert "Used View instead of  Text Editor and Text Field to make one common API."

This reverts commit eafddc9a6aeeb16189d046c4b247f2c6c203f0ee.

Used View instead of  Text Editor and Text Field to make one common API.

[NUI] Make Copy, Cut and Paste APIs public

[NUI] Change the parameter of SelectText from uint to int

SelectedTextStart and SelectedTextEnd properties are int.
Change the parameter of SelectText() to int for cosistency.

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

[NUI] Remove Obsolete methods

[NUI] Removed failed testcases

[NUI] Add xaml and xamlbing testcases

[NUI] Change SelectedTextStart, SelectedTextEnd to read-only

The setter of this properties can be replaced with the
SelectText(int start, int end) method.

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

[NUI] Fix typo from synchronos to synchronous

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

[NUI] Update TCs of NUI.Devel.

Update src/Tizen.NUI/src/public/BaseComponents/VectorGraphics/CanvasView.cs

Co-authored-by: pkosko <p.kosko@samsung.com>

[NUI] VectorGraphics:Open added APIs(DrawableGroup, Picture, Gradient)

[Add ACR]
src/public/BaseComponents/VectorGraphics/CanvasView.cs
  [Add] public void RemoveDrawable(Drawable drawable)
  [Add] public void RemoveAllDrawables()

src/public/BaseComponents/VectorGraphics/Drawable.cs
  [Add] public Vector4 BoundingBox { get; }
  [Add] public void ClipPath(Drawable clip)
  [Add] public void Mask(Drawable mask, MaskType type)

src/public/BaseComponents/VectorGraphics/Shape.cs
  [Add] public Gradient FillGradient { get; set; }
  [Add] public Gradient StrokeGradient { get; set; }
  [Add] public void AddPath(PathCommands pathCommands)

src/public/BaseComponents/VectorGraphics/DrawableGroup.cs
  [Add] public DrawableGroup()
  [Add] public void AddDrawable(Drawable drawable)
  [Add] public void RemoveDrawable(Drawable drawable)
  [Add] public bool RemoveAllDrawables()

src/public/BaseComponents/VectorGraphics/Picture.cs
  [Add] public Picture()
  [Add] public void Load(string url)
  [Add] public void SetSize(Size2D size)
  [Add] public Size2D GetSize()

src/public/BaseComponents/VectorGraphics/Gradient.cs
  [Add] public SpreadType Spread { get; set; }
  [Add] public ReadOnlyCollection ColorStops { get; set; }

src/public/BaseComponents/VectorGraphics/LinearGradient.cs
  [Add] public LinearGradient()
  [Add] public void SetBounds(Position2D firstPoint, Position2D secondPoint)
  [Add] public void GetBounds(ref Position2D firstPoint, ref Position2D secondPoint)

src/public/BaseComponents/VectorGraphics/RadialGradient.cs
  [Add] public RadialGradient()
  [Add] public void SetBounds(Position2D centerPoint, float radius)
  [Add] public void GetBounds(ref Position2D centerPoint, ref float radius)

src/public/BaseComponents/VectorGraphics/ColorStop.cs
  [Add] public ColorStop(float offset, Color color)
  [Add] public float Offset { get; set; }
  [Add] public Color Color { get; set; }

src/public/BaseComponents/VectorGraphics/PathCommands.cs
  [Add] PathCommands(IEnumerable commands, IEnumerable points)
  [Add] public IEnumerable Commands { get; set; }
  [Add] public IEnumerable Points { get; set; }

src/public/BaseComponents/VectorGraphics/VectorGraphicsConstants.cs
  [Add] public enum MaskType { Alpha, AlphaInverse }
  [Add] public enum SpreadType { Pad, Reflect, Repeat }
  [Add] public enum PathCommandType { Close, MoveTo, LineTo, CubicTo }

[Changed ACR]
[Change]
src/public/BaseComponents/VectorGraphics/Shape.cs
public enum VectorGraphics.Shape.FillRuleType { Winding, EvenOdd }
->
src/public/BaseComponents/VectorGraphics/VectorGraphicsConstants.cs
public enum VectorGraphics.FillRuleType { Winding, EvenOdd }

[Change]
src/public/BaseComponents/VectorGraphics/Shape.cs
public enum VectorGraphics.Shape.StrokeCapType { Square, Round, Butt }
->
src/public/BaseComponents/VectorGraphics/VectorGraphicsConstants.cs
public enum VectorGraphics.StrokeCapType { Square, Round, Butt }

[Change]
src/public/BaseComponents/VectorGraphics/Shape.cs
public enum VectorGraphics.Shape.StrokeJoinType { Bevel, Round, Miter }
->
src/public/BaseComponents/VectorGraphics/VectorGraphicsConstants.cs
public enum VectorGraphics.StrokeJoinType { Bevel, Round, Miter }

[NUI] Add NUIApplication construct.

Add NUIApplication construct with related windowType.
It is for IME Window type.

Add implemantation.

Add public APIs and the relasted test case.

[NUI] Open newly added APIs in CameraView

[NUI] Fix comments of BorderlineColor (#3531)

Currently the borderline color affected by View Opacity
But by check internally, we check it is just kind of bugs.
So during ACR, we don't tell it this is one of feature.

Bugfix will done in DALi side internally.

Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>
Co-authored-by: Eunki, Hong <eunkiki.hong@samsung.com>

[NUI] Fix svace issue of dereferencing possible null handle in TextUtils

Since PropertyMap.Find can return null, Find().Get() should be replaced with
Find()?.Get().

[NUI] Fix svace issue of dereferencing null handle in SetProperty

[NUI] Add Scale transition

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

[NUI] Fix svace issue
-Checker: API.CSHARP.CULTURE

WID:1125388 Method float.Parse(string) is culture specific.
Its usage can cause unexpected behaviour.
It is recommended to add IFormatProvider argument into invocation.

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

[WebRTC] Fix bugs (#3526)

* [WebRTC] Fix bugs ans Change MediaFileSource to internal API

[Camera] Change Default CameraDevice value (#3550)

[Camera] Add APIs for extra preview bitrate (#3527)

[Cion] Add new APIs for Advanced Companion Framework (#3472)

* [Cion] Add new APIs for Advanced Companion Framework

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
* [Cion] Remove PayloadAsyncResult.Id

This is removed from the design.

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
* Fix Cion

- Fix namespace of SecurityInfo, SecuritySafeHandle
- Fix some callback lifecycle (unmanged)
- Fix some simple getters to property
- Fix IPayload to Payload
- Fix some bad code

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
* Fix Cion

- Fix ClientBase.SendPayloadAsync to return Task
- Fix doxygen comments
- Fix some bad code

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
* Fix Cion

- Add missing doxygen comments
- Fix some bad code

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
* Fix ServerBase.SendPayloadAsync

Fix same as ClientBase.

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
* Fix Cion csproj file

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
* Fix Cion

- Fix IDisposable code
- Fix bad exception throwing
- Fix SendPayloadAsync
- Fix setting connectied peer of ClientBase

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
* Add missing public keyword

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
* Add missing API and fix doxygen comments

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
* Fix Cion

- Fix some doxygen comments.
- Remove PayloadAsyncResultSafeHandle class.
- Fix Ctor of PayloadAsyncResult.
- Fix to remove unmanaged callback after setting result.

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
* Fix PayloadAsyncResultCode.Dispose()

The property PeerInfo is a managed object.

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
* Fix Cion

- Fix ClientBase.SendData to use Marshal.Copy
- Fix SendPayloadAsync to handle error case
- Fix some bad code

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Co-authored-by: pjh9216 <jh9216.park@samsung.com>

[PackageManager][TCSACR-442] Add new APIs for resource control (#3415)

* [PackageManager] Add new APIs for resource control

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
* [PackageManager] Fix to use IEnumerable at public API

Use IEnumerable<string> instead of List<string>.

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
* Fix some doxygen comments of Package, PackageFilter class

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Co-authored-by: pjh9216 <jh9216.park@samsung.com>

[Applications][TCSACR-441] Add new APIs for resource control (#3414)

* [Applications] Add new APIs for resource control

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
* [Applications] Fix namings

Use full name instead of abbreviations.

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
* Fix ResourceControl

- Fix class summary of doxygen comment.
- Set default value of min/maxResourceVersion as null.

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Co-authored-by: pjh9216 <jh9216.park@samsung.com>

[Build] Fix common.props to drop ref assemblies (#3530)

[InputMethod] Add UnauthorizedAccessException

Signed-off-by: Jihoon Kim <jihoon48.kim@samsung.com>

[Tizen.WebView] Fix SVACE issue (#3529)

This patch fixes SVACE issue(DEREF_OF_NULL.RET.USER.PROC).

Signed-off-by: yh106.jung <yh106.jung@samsung.com>

Change internal TimeStamp's type to IntPtr (#3528)

nint is not supported by current version.
This patch Changes TimeStamp's member attribute type
to IntPtr.

Signed-off-by: ChangGyu Choi <uppletaste@gmail.com>

[Tts][TCSACR-449] Add new method for checking screen reader status (#3475)

This commit includes these changes.
* Add new method for screen reader status checking
* Remove white space to match the indentation
* Remove invalid symbol from interop
* Add feature tag on doxygen for 'IsScreenReaderOn' property
* Fix log to make easy for specifying error

Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

[Tizen.Applications.RPCPort] Add RPCPort ParcelHeader APIs (#3512)

* [rpc-port] Add new APIs

Adds:
 class:
 - Tizen.Applications.RPCPort.Parcel.Header

 Operation:
 - Tizen.Applications.RPCPort.Disconnect()
 - Tizen.Applications.RPCPort.Parcel.GetHeader()
 - Tizen.Applications.RPCPort.Parcel.Header.SetTag()
 - Tizen.Applications.RPCPort.Parcel.Header.GetTag()
 - Tizen.Applications.RPCPort.Parcel.Header.SetSeqNum()
 - Tizen.Applications.RPCPort.Parcel.Header.GetSeqNum()
 - Tizen.Applications.RPCPort.Parcel.Header.GetTimeStamp()

Signed-off-by: ChangGyu Choi <uppletaste@gmail.com>
* Fix some design

Changes:
 - Replace TimeStamp and Header scope to Parcel's outside
 - Change parameter name of SetSeqNum()
 - Change naming for visibility

Signed-off-by: ChangGyu Choi <uppletaste@gmail.com>
* Fix GetHeader()

Fix implementation that unnecesaary creating Header.

Signed-off-by: ChangGyu Choi <uppletaste@gmail.com>
* Fix to not matched description of parameter

Signed-off-by: ChangGyu Choi <uppletaste@gmail.com>
* Remove unnecessary description

Signed-off-by: ChangGyu Choi <uppletaste@gmail.com>
* Change types of Timestamp to nint

Signed-off-by: ChangGyu Choi <uppletaste@gmail.com>
* Change Disconnect()'s exception code

Signed-off-by: ChangGyu Choi <uppletaste@gmail.com>

[MediaPlayer] Fix svace issue (#3523)

[NUI] Version update (nui22042) (#3516)

Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>
Co-authored-by: Eunki, Hong <eunkiki.hong@samsung.com>

[NUI] Fix issue that objects are disposed outside callback. (#3514)

Implement more request interceptor APIs. (#3466)

[NUI] Fix to measure GridLayout's children sizes correctly

Previously, if child's measured size is 0, then GridLayout does not
update the child's measured size when the child's size is set by using
GridLayout APIs.
e.g.
SetHorizontalStretch(child, GridLayout.StretchFlags.ExpandAndFill);

Moreover, if child has its own size and ExpandAndFill is set, then
child's size is set to be its own size (its measured size) plus its
expanded size.
e.g.
parent size is 100.
child1's measured size is 50 and ExpandAndFill.
child2's measured size is 0 and ExpandAndFill.
Then, child1's size is 50 + 25 = 75 and child2's size is 25.

To resolve the above, GridLayout updates child's measured size in
OnLayout() and child's size is set to be the bigger one between its
measured size and its expanded size.

[NUI] update event descriptionfor textfield

[NUI] update event description

[NUI] remove unneeded comment

[NUI] Make SelectionChanged event for TextField & TextEditor public

Add SetDoubleTapTimeout binding

[NUI] Update TCs of NUI.Devel.

Fix setvalue and xamlstyle issues

[NUI] Fix Svace issues

[EXaml] Remove unused code

[EXaml] Refactor EXaml

[NUI] Add Slide transition

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

[NUI] Fix testhub NUI.Components coredump issue

- there was no fail but coredumps are remained after finishing NUI.Components test.
- native crash occured when empty native object handle is accessed and the TouchEvent signal is tried to be gotten.
- https://code.sec.samsung.net/jira/browse/TSDF-2266

[NUI] CanvasView: Change parameter and return type of binder function

int -> uint(unsigned int)
dali-csharp-binder patch : https://review.tizen.org/gerrit/#/c/platform/core/uifw/dali-csharp-binder/+/262968/

[NUI] Add SelectionHandleImage struct to Text Components

Add a SelectionHandleImage struct to pass data
of DALi SelectionHandleImageLeft, SelectionHandleImageRight PropertyMap.
The SelectionHandleImage struct can be used as an argument to
SetSelectionHandleImage, GetSelectionHandleImage, SetSelectionHandlePressedImage, GetSelectionHandlePressedImage,
SetSelectionHandleMarkerImage and GetSelectionHandleMarkerImage methods.

// example
var selectionHandleImage = new Tizen.NUI.Text.SelectionHandleImage();
selectionHandleImage.LeftImageUrl = "handle_downleft.png";
selectionHandleImage.RightImageUrl = "handle_downright.png";

field.SetSelectionHandleImage(selectionHandleImage);

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

[NUI] Use BindableProperty in struct get/set methods

In struct get/set methods, use BindableProperty instead of directly accessing internal properties.

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

[NUI] Add SetHiddenInput, GetHiddenInput to TextField

Add a HiddenInput struct to pass data of DALi HiddenInputSettings PropertyMap.
The HiddenInput struct can be used as an argument to
SetHiddenInput and GetHiddenInput methods.

// example
var hiddenInput = new Tizen.NUI.Text.HiddenInput();
hiddenInput.Mode = HiddenInputModeType.ShowLastCharacter;
hiddenInput.SubstituteCharacter = '★';
hiddenInput.SubstituteCount = 0;
hiddenInput.ShowLastCharacterDuration = 1000;

field.SetHiddenInput(hiddenInput);

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

[NUI] Add SetPlaceholder, GetPlaceholder to TextField, TextEditor

Add a Placeholder struct to pass data of DALi Placeholder PropertyMap.
The Placeholder struct can be used as an argument to
SetPlaceholder and GetPlaceholder methods.

// example
var placeholder = new Tizen.NUI.Text.Placeholder();
placeholder.Text = "placeholder text";
placeholder.TextFocused = "placeholder textFocused";
placeholder.Color = new Color("#45B39D");
placeholder.FontFamily = "BreezeSans";
placeholder.FontStyle = new Tizen.NUI.Text.FontStyle()
{
    Width = FontWidthType.Expanded,
    Weight = FontWeightType.ExtraLight,
    Slant = FontSlantType.Italic,
};
placeholder.PointSize = 25.0f;
//placeholder.PixelSize = 50.0f;
placeholder.Ellipsis = true;

field.SetPlaceholder(placeholder);

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

[EXaml] Remove unused comment

[EXaml] Remove unused comment

[EXaml] 1. Add copyright; 2. Add new line at the end of file

[EXaml] Support Trigger in EXaml

[NUI] Update TCs of NUI.Devel.

[WebRTC] Add CreateOffer/AnswerAsync method (#3506)

[ScreenMirroring] Fix exception bug (#3515)

[MediaPlayer] Support Video and UI sync mode (#3510)

[MediaContent] Use MediaContent internal free method (#3508)

Fix unhandled exception while calling CoRegister

[MediaContent] Support ebook format (#3374)

* [MediaContent] Support ebook format

[ScreenMirroring] Add ConnectAsync method to support port selection (#3412)

* [ScreenMirroring] Add ConnectAsync method to support port selection

[ImageUtil] Support HEIF decoder (#3447)

* [ImageUtil] Support HEIF decoder

[WebRTC] Remove wrong state check code (#3500)

[Camera] Add internal constructor (#3489)

* [Camera] Add internal constructor and code clean

Add SetSize API for NUI IME

Signed-off-by: Jihoon Kim <jihoon48.kim@samsung.com>

[NUI] Fix Layout Performance Issue (#3484)

Layout Process should be registered only once.

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

[NUI] Fix to follow c# coding rule
Follow C# Coding rule
 - reposition the type of class
 - delete unusued using state
 - follow event naming rule

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

[NUI][ATSPI] Add a function to check whether accessibility is enabled (#3463)

* [NUI][ATSPI] Add a function to check whether accessibility is enabled

- Added `IsEnabled` to check accessibility state

Signed-off-by: Seoyeon Kim <seoyeon2.kim@samsung.com>
* Change the type of Enabled to property

- Make `Enabled` both static and property.

Signed-off-by: Seoyeon Kim <seoyeon2.kim@samsung.com>

[NUI] Replace XXXList with IList<XXX>. (#3468)

[Xaml] Remove underline in member's name

[Xaml] Remove underline in member's name

[Xaml] Add MergeStyle to View

[Xaml] Remove code of VisualState

[Xaml] Modify by HQ's feedback

[Xaml] Support Trigger in Xaml

[NUI] Remove duplicate class WebContextMenuItemList (#3464)

The class WebContextMenuItemList is cenceptually same as WebContextMenu.

Signed-off-by: Jiyun Yang <ji.yang@samsung.com>

[NUI] Add FocusableChildren

[NUI] Refactoring Transition and FadeTransition

 - Change Fade to FadeTransition.
 - Remove Default AppearingTransition/DisappearingTransition of Page

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

[NUI] Allow to set Size.Depth (#3442)

Previous version set dali-side size first, and set Width/Height Specification.
But int Width/Height Specification setter, they also call Size2D property set.

Unfortunatly, Size2DProperty doesn't care about the depths of view.
So the Depth value is gone.

This patch re-ordering the function call to set dali-side size later
so Size.Depth will works well.

Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>
Co-authored-by: Eunki, Hong <eunkiki.hong@samsung.com>

[NUI] Add description for ImageView

This PR added descritoon for ImageView Property

Rebase master (#3453)

* [Tizen.System.SystemSettings] Remove lock about invoking event (#3433)

Signed-off-by: JinWang An <jinwang.an@samsung.com>
* [ScreenMirroring] Support NUI window (#3413)

Co-authored-by: JinWang An <35290168+jinwangan@users.noreply.github.com>
Co-authored-by: Haesu Gwon <haesu.gwon@samsung.com>

[WebRTC] Fix destroy bug and code clean (#3476)

* [WebRTC] Fix destroy bug and code clean

[NSD] Add new GetTxtRecords api (#3353)

* [NSD] Add new GetTxtRecords api

Related PR: https://github.com/Samsung/TizenFX/pull/2418

* Update src/Tizen.Network.Nsd/Tizen.Network.Nsd/DnssdService.cs

Co-authored-by: WonYoung Choi <wy80.choi@samsung.com>
* Update src/Tizen.Network.Nsd/Tizen.Network.Nsd/DnssdService.cs

Co-authored-by: WonYoung Choi <wy80.choi@samsung.com>
* Fix build error

Error: Tizen.Network.Nsd/DnssdService.cs(273,20): error CS0128: A local variable or function named 'txtValue' is already defined in this scope

Co-authored-by: WonYoung Choi <wy80.choi@samsung.com>

[Bluetooth][TCSACR-437] Add GATT related APIs (#3377)

BluetoothGattServer.GetAttMtu(string clientAddress)
BluetoothGattClient.ServiceChanged
ServiceChangedEventArgs
BluetoothGattServiceChangeType

Signed-off-by: Wootak Jung <wootak.jung@samsung.com>

Change API names

Add Window Test file

add new window test file and fix build error for UTC.

Change the name with meaningful.

Add input regions APIs

Add input regions APIs about adding input area and subtracting.

[NUI] Add SelectionCleared Event (#3446)

* Add SelectionCleared Event

* [NUI] rename parameters + rebase

* [NUI] remove unnecessary if statement

* [NUI] remove unnecessary use of EventArgs

Co-authored-by: abdullah <abdullahhasan10@gmail.com>

[NUI] Remove old event args from SelectionChanged and CursorPositionC… (#3467)

* [NUI] Remove old event args from SelectionChanged and CursorPositionChanged

* [NUI] remove unnecessary use of EventArgs

Co-authored-by: abdullah <abdullahhasan10@gmail.com>

[NUI] Remove ReleaseFunction enum at PixelData with legacy

PixelData buffer's allocate / release function is only need for C++ side memory.
And C# cannot control that allocation method.
So ReleaseFunction enum is unneccesary arguments at NUI.

But now, Tize.TV.FLUX use this ReleaseFunction.
So we leave it and will be deprecated later.

Signed-off-by: Eunki Hong <h.pichulia@gmail.com>

[NUI] Copy, Paste & Cut Public APIs (#3450)

[NUI] Adding the Copy, Cut and paste public APIs to NUI.

Added:
string CopyText() //Copy and return the selected text.
- This function will copy the previously selected string into the clipboard and will return it.
string CutText() //Cut and return the selected text.
- This function will cut the previously selected string into the clipboard and will return it.
void PasteText() //Paste the most recently copied/cut text.
- This function will paste the most recent string in the clipboard stack into the text control.