[M120 Migration][MM] Enable video hole when in full-screen mode in the public profile.

In accordance with the policy, video hole should only be enabled
when the video is in full-screen mode in the public profile.
Therefore, video hole should be dynamically turned on/off
based on whether the video is in full-screen mode or not.

- |esplusplayer_enable_video_hole| is used for this feature.
- Create MediaPlayerESPlusPlayerCommon because |esplusplayer_enable_video_hole|
can be used only for the public profile.
- Extract |SetBufferType| function that sets the buffer type
because the buffer type should be always 'copy' to use
|esplusplayer_enable_video_hole| in the public profile.
- Extract |PrepareVideoHole| function that calls video hole-related apis
to keep the following function call order.
esplusplayer_set_video_frame_buffer_type -> esplusplayer_set_display
-> esplusplayer_enable_video_hole -> esplusplayer_prepare_async

Reference:
https://review.tizen.org/gerrit/#/c/291803/
https://review.tizen.org/gerrit/#/c/292529/

Change-Id: Ibe85b1ad0e2885eb201f309e0630d2eeb40b69a4
Signed-off-by: Sun-woo Nam <sunny.nam@samsung.com>

[WRTjs][Service][VD] Fix wrt-service crash

When finalize ApplicationData instance on wrt-service, unexpected
acrash is happened probably with wrong instance size.

Change-Id: I5e859f467e3de27def4f7f208087aac3b7fcfdbc
Signed-off-by: DongHyun Song <dh81.song@samsung.com>

[Clang] Updated to Clang 18

Open source llvm 18.0 is built and accelerated for tizen to generate clang,
lld and llvm binaries that are required for clang build for standard profile.
The commit replaces 11.0 headers and binaries with the ones built for 18.0

Basic functionalities tested on TV and rpi4. Working Fine.

Change-Id: Iacd68612b0a04554a9f96105123c0487aa849f59
Signed-off-by: sidpaswan <s.paswan@samsung.com>

Fix for compiler and linker errors with clang 18

Fixes the following linker error:
ld.lld: error: relocation R_ARM_TLS_LE32 against blink::g_thread_specific_
cannot be used with -shared

More details related to the error can be found at:
https://reviews.llvm.org/D93331?id=311994

Change-Id: Ib9471e396415baa5eb77d8471c73433dc88d30f2
Signed-off-by: sidpaswan <s.paswan@samsung.com>

fixup! [M120 Migration][MM] Camera and WebRTC patches migration.

This commit fixes desktop build errors resulting from the parent patch.

Change-Id: I8e46faae50771ac8dd1a92d7d457096c5e73204c
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>

[VD] Disable GPU Rasterization by default

M120 enables GPU Rasterization by default, so we disable it until
it is fully verified. With this patch, improvements on GPU memory
consumption are seen for Naver.

References: https://review.tizen.org/gerrit/291676/

Change-Id: I302d249908e21923f4be8e99616d9547bf956b3e
Signed-off-by: Satyaveer Singh <satyaveer.s@samsung.com>

[M120 Migration][MM] Refactor codes for upversion migration.

Move some codes related to espp to other scopes to migrate next patches easily.
These codes are included in some patches.
https://review.tizen.org/gerrit/#/c/290566/
https://review.tizen.org/gerrit/#/c/291285/
https://review.tizen.org/gerrit/#/c/293459/

Change-Id: I7d3ddd1b81b9ef5d6836c6432bf64f42e3688b1c
Signed-off-by: Sun-woo Nam <sunny.nam@samsung.com>

[WRTjs][VD] Reduce IS_TIZEN_TV compile flag

Reduce IS_TIZEN_TV compile flag of ApplicationData class
for code readability

Change-Id: I614018414f9b30c3478ab7c9a2fc829fd09e9936
Signed-off-by: DongHyun Song <dh81.song@samsung.com>

[M120 Migration][XWalkExtension][VD] Propagates --xwalk-extension-path to renderer

Copy '--xwalk-extension-path' to renderer process to handle xwalk
extensions.

Reference Patch:
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/294333/

Change-Id: I2cbda9b081a4f4a4a8688a0785911594d617da2d
Signed-off-by: zhaosy <shiyusy.zhao@samsung.com>

[M120 Migration][WRTjs][VD] Fix camera permission denied issue

In PermissionControllerDelegate::GetPermissionResultForCurrentDocument,
PermissionStatus::DENIED is return as default, so can't use camera for webapp.

Override GetPermissionResultForCurrentDocument in WRT, return PermissionStatus::GRANTED.

Reference:
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/292389/

Change-Id: Ib98e5ec9f73c66493a4df6f8bc1ffa3cfe4a7d7e
Signed-off-by: zhaosy <shiyusy.zhao@samsung.com>

[M120 Migration][MM] Add new class for HBBTV on TV

MediaPlayerBridgeCapi generated MediaPlayerTizen directly.
Delete MediaPlayerEfl MediaPlayerManager MediaPlayerBridgeCapiAdapter.

Reference:
https://review.tizen.org/gerrit/#/c/291756/

Change-Id: I681429ccbe926d8bfd3e6d1a57eb60c56a7428b2
Signed-off-by: Sun-woo Nam <sunny.nam@samsung.com>

[M120 Migration][MM] Make video hole work properly.

This patch includes the three patches below to make video hole work properly
in WRT and so on.

1. fixup! [MM] Fix the issue that video hole is not enabled.
In the previous patch, there is an issue that video hole often doesn't work.
Notify that preferences have been changed when video hole is enabled.
https://review.tizen.org/gerrit/#/c/290642/

2. Enable video hole and remove unused mirror blur codes in WRT.
The video hole feature should be enabled for the web app in the public profile.
To enable the feature, remove unused mirror blur codes in WRT.
The mirror blur is enabled for the wearable profile, but it is unclear whether
the wearable profile will continue to be supported.
https://review.tizen.org/gerrit/#/c/291868/

3. Set window handle for video hole
Video playback will fail on webapps on standard build.
Set window handle to be used for overlay.
https://review.tizen.org/gerrit/#/c/292251/

4. Add onMove logic for window while playing video
For mini-player, window canbe moved. while we move window, we
need to set new geometry for video element.
https://review.tizen.org/gerrit/#/c/295353/

5. Get a current viewport rect to set media geometry.
When a webview was moved, a new viewport rect should be passed to player
to set media geometry for video hole.
Therefore, call |SetPlayerMediaGeometry| in |OnWebViewMoved| callback.
Additionally, remove |ResetMediaGeometry| because there is already logic
for setting deferred.
https://review.tizen.org/gerrit/#/c/295430/

6. Add willdraw logic for web app (partially)
Use willdraw to commit wl_buffer to efl while rotation is done
https://review.tizen.org/gerrit/#/c/294798/

Change-Id: I1787dadff487562c0cffaab8cfdccace83330fdb
Signed-off-by: Sun-woo Nam <sunny.nam@samsung.com>

[M120 Migration][WRTjs] Default path for 'ready-to-show'

In order to show the WebApp, a "ready-to-show" message must be generated. The
message should be sent after the first frame is drawn and the layout has been
completed with minimal content. This patch ensures that this timing is met.

Reference:
[1] Modify to ensure |DidFirstVisuallyNonEmptyPaint| callback is called.
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/293610/

[2] fixup! Modify to ensure |DidFirstVisuallyNonEmptyPaint| callback is called.
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/294060/

[3] fixup!(2) Modify to ensure |DidFirstVisuallyNonEmptyPaint| callback is called.
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/294317/

[4] Check presentatoin feedback condition for |DidFirstVisuallyNonEmptyPaint|.
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/294610/

[5] Do not ignore first frame for |DidFirstVisuallyNonEmptyPaint|.
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/295637/

Change-Id: I42979d84c6f3047824bc3e4bec034411b03b23e7
Signed-off-by: YongGeol Jung <yg48.jung@samsung.com>

[M120 Migration][MM] Camera and WebRTC patches migration.

This commit includes commits below.

- Add MJPEG support
Add support for MJPEG camera stream, jpeg frames are internally sent
with PIXEL_FORMAT_ENCODED instead of PIXEL_FORMAT_MJPEG, which avoids
the decoding in default camera pipeline and helps redirecting frames
towards the esplusplayer.
https://review.tizen.org/gerrit/#/c/290488/
https://review.tizen.org/gerrit/#/c/291867/ (partial)

- [GUM] Clean internal tizen codes for camera.
https://review.tizen.org/gerrit/#/c/290574/

- [GUM] Get lens orientation
Camera lenses may give rotated video streams. So, in application lens
orientation is obtained and is used for frame rotation.
https://review.tizen.org/gerrit/#/c/291838/

- Implementation of lazy start
Add lazy_start to capture params when request is generated by
user_media_processor to create camera in suspended state and to create
capture client in paused state.
https://review.tizen.org/gerrit/#/c/291188/

- Fix issues in video capture device tizen
We need keep MJPEG and ENCODED
Fix missing type in function GetCaptureApiTypeString()
https://review.tizen.org/gerrit/#/c/291169/

- Support H.264 codec for the public profile.
Set ffmpeg_branding to "Chrome" for enabling H.264 decoders.
https://review.tizen.org/gerrit/#/c/294042/

Change-Id: I02076dfa9c98ec344cbbdc18847d6a99a781d1a1
Signed-off-by: Sun-woo Nam <sunny.nam@samsung.com>

Add IS_TIZEN_DA to fix build break in standard build.

This patch fixes build break caused by the patch below.
https://review.tizen.org/gerrit/#/c/305076/

Change-Id: I278839c1027c91e3af31a35e6988b63716943891
Signed-off-by: Sun-woo Nam <sunny.nam@samsung.com>

[M120][XWalkExtension] Remove EWK_BRINGUP in XWalkExtension

Remove EWK_BRINGUP in XWalkExtension

Change-Id: I9e8dfc0f18e0c5ff511ca7fb7534eae59258fa28
Signed-off-by: zhaosy <shiyusy.zhao@samsung.com>

Fix HTLMInputElement.type not returing "datetime" string for <input type="datetime">

InputType::FormControlType() virtual method is removed in m120 upstream, because
if which 'datetime-local' is returned as string from base impl. Made necesasary
changes to fix the issue.

Change-Id: I439c0e257808f983cbe980c3c7188e208791b80e
Signed-off-by: Gajendra N <gajendra.n@samsung.com>

[M120 Migration][XWalkExtension] Add ewk interface to load XWalk extensions

This interface set a path located xwalk extension json profile file.

 - ewk_context_set_xwalk_extension_profile(context, path)
 - i.e.,
   ewk_context_set_xwalk_extension_profile(
      ctx, "/usr/lib/tizen-extensions-crosswalk-ewk/");

Reference Patch:
https://review.tizen.org/gerrit/292805/

Change-Id: I2ddf3e6bdb2ebaf60bcbb360aa6b2a89e0d51b16
Signed-off-by: liwei90727 <wei90727.li@samsung.com>

[M120 Migration][WRTjs][Service] Fix some coverity issues

Add initialize for fix some coverity issues.

Reference:
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/305090/

Change-Id: I48a84938fa030fdec0f34a12cd5801b2ab76c06b
Signed-off-by: zhaosy <shiyusy.zhao@samsung.com>

[M120][WRTjs][VD] Suspend media tasks without 500ms delay

For webapps, when app is hide, WRT suspend app after 500ms.
But for media, there is resource conflict in this case:
  1.Play H5 video in app A
  2.Play H5 video in app B
  3.Resume app A, media resource conflict

Reason:
  In step3, app B is not suspend, so media resource of app B is not released.
  Then when app A is fail to play video.

So when app is hide, notify media app is suspend without 500ms delay,
then media will release resource right now.

Reference:
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/304611/

Change-Id: I2aa28df76568d9aee86f5f9cbf68c95e840bb869
Signed-off-by: zhaosy <shiyusy.zhao@samsung.com>
Signed-off-by: DongHyun Song <dh81.song@samsung.com>
(cherry picked from commit ee9a656b5306a78cfc166e5065b82ddd0b1eeb45)

[M120][WRTjs][VD] Fix app is paused issue when VSS app is launched

1.InitializeVSS early for hosted app to make sure aul app com connection is created.
2.In custom case, hide VSS is too later, it caused app window is showing before VSS
  video is stopped, then VSS recieve visibility change event and send INTERRUPT event
  to app, so app is paused.
  So hide VSS in HideSplashScreen(Followed M94 logic).

Reference:
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/303450/

Change-Id: I096dff53987b2b5cdda73d6dfe2c29266f6ac9f7
Signed-off-by: zhaosy <shiyusy.zhao@samsung.com>
(cherry picked from commit 3156f9570c64ed582530c98569c6948d7084d83d)

[M120][WRTjs] Fix ApplicationData instance thread safety issue

When app is launched, ApplicationData instance maybe is created
by multi-threads(main thread, prelaunch worker, reallaunch worker)
at the same time.
So add lock in ApplicationData::GetInstance.

Reference:
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/304102/

Change-Id: I883711947700e3cc973f2ad2ce4deee989708517
Signed-off-by: DongHyun Song <dh81.song@samsung.com>

[M120][WRTjs] Fix double dbus close issue

Change the way to end dbus listening to fix double calls of
dbus_connection_close()

++
wrt-service application handling is not sensitive for
launch performance, so 500ms interval polling looks proper.
In fact, uv-task runner is polling as 500ms either.

Reference:
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/303807/

Change-Id: Icfcfc70a84333d3a8b8f1e0805bafee61b265bc8
Signed-off-by: DongHyun Song <dh81.song@samsung.com>
(cherry picked from commit d5b98c76536c01ba00086727620dbf71fcd331b0)
(cherry picked from commit 329f0ee54feb80e6c7f2af3efefb23f5a1f9f54b)

[M120][WRTjs] Add flag to check context_data initialized or not

To avoid visit the invalid uninitialized memory, need add
flag to check context_data initialized or not.

Reference:
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/304681/

Change-Id: I91c9d4c45259591896dbc0f15fa7689cd51d0d0b
Signed-off-by: Chunling Ye <chunling.ye@samsung.com>
(cherry picked from commit ce59489a22b4f7b3a1e7184520a49b697c6251e5)

[M120][WRTjs][Service] Refactor the dbus logic for wrt-service

Move the dbus related logic to WrtServiceManager, and then
waitting all init work have done, start RegisterDbusHandler.

Reference:
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/304105/

Change-Id: I277242e5e3a421b46383cff3740e33cc35b5a2b5
Signed-off-by: Chunling Ye <chunling.ye@samsung.com>
(cherry picked from commit cf81c129f223890fa12bcb1a6ce8d6a5b3fe23dc)

fixup! [WRTjs] Enable WRTjs

This patch adds empty config.h file to resolve build break.

Reference:
https://github.com/nodejs/node/blob/main/deps/base64/base64/lib/config.h

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

[WRTjs] Enable WRTjs

. electron v28.1.0
  + node modules : @types/node, @types/webpack-env, buffer, husky, process,
                   timers-browserify, ts-loader, typescript, webpack,
                   webpack-cli, wrapper-webpack-plugin
  + electron.d.ts : type definitions for electron
. node.js v18.18.2
  + electron's patches
  - benchmark, doc, test, tools/node_modules
  - some deps (brotli, icu-small, icu-tmp, ngtcp2, npm, openssl, v8, zlib)
. wrt
  : from tizen.beta branch (778559908f17bfec2f28c202c44df1f6bef28a5c)

Change-Id: I7bbaeb2a3d20cc4ee23ffdb9b0220ffbff399bb9
Signed-off-by: SangYong Park <sy302.park@samsung.com>

[M120][Tizen][Onscreen] Fix build errors for TV profile

This patch fixes TV Onscreen build issues for M120.

Change-Id: I779c25f22364f863fe08b352b628e1cb24d68d9c
Signed-off-by: venu.musham <venu.musham@samsung.com>
Signed-off-by: v-saha <v.saha@samsung.com>

Fix build error for Tizen multimedia migration.

The codes that were changed during the update to m120 are applied.

The features below are enabled.
tizen_multimedia
tizen_audio_io
tizen_web_speech_recognition

tizen_tbm_support is still disabled.

Change-Id: Idfe6f0b877107d00298f4ab772108abfb65b90c2
Signed-off-by: Sun-woo Nam <sunny.nam@samsung.com>

Add separate out directory for onscreen chrome build

This commit adds changes to create separate out directory
for onscreen chrome build(desktop and tizen).

Desktop: out.chrome.x64
Tizen: out.chrome.tz_*

Change-Id: I208719e025e2b84b8bf7b6d1cf2d1308207dbc23
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>

Accelerate node binary for x86_64 tizen platform

M120 tizen build was randomly hanging as node binary was not
accelerated using qemu-accel. Below change adds node binary
built with qemu-accel which helps fixing random hangs and speeds
up the compilation speed of the same.

Change-Id: I53107cfee1712c43c51e7a9543284aed3ad024b9
Signed-off-by: uzair <uzair.jaleel@samsung.com>

Use proper naming for handles that represent |ewk_view| and its child |main_layout|

In ewk layer, at many places, names like "evas_object()", "evas_object_"
etc., are used to represent |ewk_view| which causes confusion during
debug since there are many evas objects created in efl port.
|ewk_view| is a global object created through ewk_view_add* APIs and is
passed as first argument in most of the ewk APIs, hence it has its own
significance. This patch names it properly across the efl port codebase
for understandibility.

Also, same case with its child object |main_layout|, which is used with
names like "native_view", "parent_view" etc. The child object is created
with api elm_layout_add(), so better to name it like "efl_main_layout_",
since aura::Window is also named with NativeView.

Similarly few functions are renamed as per its usage.

Below is the summary of renaming applied through this patch.

------------------------------------------------------------------------
             Before                    |            After
------------------------------------------------------------------------
EWebView::evas_object_                 | ewk_view_
::evas_object() / ::getEvasObject()    | ewk_view()

WebContentsImpl::efl_native_view_      | efl_main_layout_
WebContentsImpl::CreateEflNativeView() | CreateEflMainLayout()
               ::GetEflNativeView()    | GetEflMainLayout()

EflWindow::native_view_                | events_overlay_
EflEventHandler::native_view_          | events_overlay_
EflWindow::SetNativeViewOffscreen      | SetEventsOverlayForOffscreen()
------------------------------------------------------------------------

Change-Id: If8f719ebd0c16ff36c4d521af86e2fa951b70dab
Signed-off-by: Gajendra N <gajendra.n@samsung.com>

[M120] Fix websites loading issue for standard build

This patch fixes websites loading issue for standard build in M120.

Change-Id: I7124cd52234b2cf70592be45bda75b46c0dab9f3
Signed-off-by: Nikhil Shingne <n.shingne@samsung.com>

[M120][Tizen][Offscreen] Fix build errors for TV profile

This patch fixes TV Offscreen build issues for M120

Change-Id: Ida9265cacd16a9fecc9fac4de014ee238d4ff905
Signed-off-by: v-saha <v.saha@samsung.com>
Signed-off-by: Suhaspoornachandra <s.poornachan@samsung.com>

Reland "Revert Upgrade NodeJS binary to v16.13.0"

Engine had upgraded nodejs to v16.13.0 for other builds.
This patch upgrades for tizen profile with appropriate
node binary

Change-Id: Ia0fe2a204fad15329fcde0e82ce213b017ff171f
Signed-off-by: Suhaspoornachandra <s.poornachan@samsung.com>

EWK_BRINGUPS fixup for M120

This commit fixes some of the EWK_BRINGUPs added during M114 upversion.

Change-Id: I35402c34d802890e5fe67eab2c31ac0a5d9c765c
Signed-off-by: Nikhil Shingne <n.shingne@samsung.com>

Fix for Onscreen desktop build crash

This patch activates the enable_supervised_users switch.

Change-Id: Ica8733f9c7b38830f9ae6cffbf925b216e943a9c
Signed-off-by: sidpaswan <s.paswan@samsung.com>

EWK_BRINGUPS fixup for M120

This commit fixes some of the EWK_BRINGUPs added during M120 upversion.

Change-Id: I3e496b6fc9f11efb6332a1de20ebcf550a2440e9
Signed-off-by: Mayank Kumar <mayank.k@samsung.com>

Fix for zygote crash during browser launch

Zygote process was crashing  as its launch path overiding api
from efl port failed. Hence below change adds needed CHECK
macro and partially reverts [1] as CreateDirectory api call
failed in earlier version of tizen as well due to platform specific
"ISDIR" call failure.

[1]https://chromium-review.googlesource.com/c/chromium/src/+/4685555

Change-Id: I0a69511728fdf2d75dd0b44188e5fa8817e1d0b6
Signed-off-by: uzair <uzair.jaleel@samsung.com>

[M120][EFL][Offscreen] Fix build errors for desktop profile

This commit fixes all gn related, compilation and linker errors
for desktop build.

Change-Id: I590d3c8c33b4dee27e623e7dce9d8214d5c29397
Signed-off-by: uzair <uzair.jaleel@samsung.com>
Signed-off-by: v-saha <v.saha@samsung.com>

Upload upstream chromium 120.0.6099.5

This is a patch for upstream m120 - m114.

* chromium
dc7563588833ccad6482a3aa5848108c4387e7e5
[SHv2] Hide Safety Hub entry points in guest mode

* v8
81c506e6adcce17d4d88434b8f4a42899684ff22
Version 12.0.267.1

* skia
746dbd8c61ac4e61aab8d74e3197649921e366a6
Merge 11 release notes into RELEASE_NOTES.md

Change-Id: I29f4a234e39422997118427b479849ab50683c2f
Signed-off-by: v-saha <v.saha@samsung.com>

[M114 Migration] Refactor JavaScriptModalDialog for EWK and WRT

Original patch set wrong parent widget to JavaScriptModalDialog class.

Reference: https://review.tizen.org/gerrit/290634

Change-Id: Id1ee9acc57ea8be98064b40f99b012c5e20fb439
Signed-off-by: Pranay Chauhan <pranay.c1@samsung.com>

[M114 Migration] Apply important patches that fixed stability crashes reported on KONA

References: https://review.tizen.org/gerrit/292705

Change-Id: I0f9f18efa2729497cef84235297513184de9fb94
Signed-off-by: Nikhil Shingne <n.shingne@samsung.com>

[M114 Migration] Always set LC_NUMERIC category to 'C' locale

WebVTT's float type position was converted to incorrect css property string in
which ',' is used as decimal point when the locale of LC_NUMERIC category is DE.

blink::String::Format() should always use '.' as decimal point when converting
floats to strings regardless of current locale.
This patch always set LC_NUMERIC category to 'C' locale.

Reference: https://review.tizen.org/gerrit/291129

Change-Id: I601e1ba7e56e5649e078acbc968b5f962f449624
Signed-off-by: Pranay Chauhan <pranay.c1@samsung.com>

[M114 Migration] Remove unused js_dialogs_efl gn target

js_dialogs_efl was originally implemented for content_shell_efl.
As content_shell_efl itself is removed, js_dialogs_efl is
no longer needed.

Reference: https://review.tizen.org/gerrit/#/c/290505/

Change-Id: If93ab010cf0d2f6bbe74b2864a2fdec4df110a29
Signed-off-by: Nikhil Shingne <n.shingne@samsung.com>

Squash 3 patches and apply to beta branch

This commit combines and applies the following patches to beta branch:

https://review.tizen.org/gerrit/291694
https://review.tizen.org/gerrit/293029
https://review.tizen.org/gerrit/293259

Change-Id: I0d22e91953121d1c152c02d7ef434655c5aea3ec
Signed-off-by: Mayank Kumar <mayank.k@samsung.com>

Rename WebContents related delegate classes inside tizen_src

There are around three to four |WebContents**Delegate| classes inside
tizen_src; this causes confusion during debugging the codeflow.

This commit improves readability/understandability by making below changes:
------------------------------------------------------------------------

[1] Rename WebContentsViewDelegateEwk to WebContentsViewDelegateEfl

A class inside tizen_src implementing a opensource public class should have
classname with 'Efl' suffix (instead of 'Ewk').

WebContentsImpl : WebContentsDelegate => WebContentsDelegateEfl
WebContentsViewAura : WebContentsViewDelegate => WebContentsViewDelegateEfl
------------------------------------------------------------------------

[2] Rename WebContentsEflDelegate to WebViewDelegateEfl

Currently WebContentsEflDelegate is created in EWebView and been accessed by
two other classes, hence rename it to WebViewDelegateEfl and make EWebView
own it. Other classes will just use its pointer to route delegate calls.

EWebView                       \
WebContentsImplEfl           ---  WebViewDelegateEfl (owned by EWebView)
WebContentsViewAuraHelperEfl   /
------------------------------------------------------------------------

After this patch, there exists only 2 |WebContents*DelegateEfl| classes, i.e
WebContentsDelegateEfl and WebContentsViewDelegateEfl, both are inherited
from opensource public classes respectively. Other delegate classnames will
not have prefix 'WebContents'.

Change-Id: I5dafc3f858aed6c9affc81d89a3b8512bfcfc6a9
Signed-off-by: Gajendra N <gajendra.n@samsung.com>

fixup! [M108 Migration][Accessibility] Bringup Accessibility

This commit combines the following changes:

1) Migrates below patches from M108
   https://review.tizen.org/gerrit/300648
   https://review.tizen.org/gerrit/299881

2) Removes EWK_BRINGUPs to enable the A11y flow working.

Change-Id: Ia72564c581f34ca749682fe5c0ddfdfa07079bc4
Signed-off-by: Mayank Kumar <mayank.k@samsung.com>

[M114 Migration][VD] Set the Tizen version of UA

1. Modify Default User Agent

2. [VD][UWE] Set the Tizen version of UA

3. Add UWE App Info in UserAgent only for UWE

References: https://review.tizen.org/gerrit/291510/

Change-Id: I67359cc438731050f62c3bd7b1fd736d649797bc
Signed-off-by: Satyaveer Singh <satyaveer.s@samsung.com>

[M114 Migration][Fixup] Popup window transparent issue

1.Default style is now being provided to applications which
  use their own popup edc file so that they can customise their popup
2.C_PopupBasic_WhiteBottom is the UX recommended default style
3.use C_PopupBasic_WhiteBottom instead of default to fix
  DF221005-01028, DF221005-01016

Reference: https://review.tizen.org/gerrit/289305

Change-Id: I80c48b52f04e550185f33495a4dd6f78abbd8d27
Signed-off-by: Nikhil Shingne <n.shingne@samsung.com>

[M114 Migration] Implement an overridden method in PermissionControllerDelegateEfl

Webpages fetching Geolocation coordinates for first time are supposed to request
for permission from user and show a prompt popup to allow or deny access.
But currently the popup does not appear due to unimplemented method in
PermissionControllerDelegateEfl. This patch adds the same.

Reference: https://review.tizen.org/gerrit/299972

Change-Id: I6131ba88e03d5b881eb83207987b8636a9452506
Signed-off-by: Pranay Pankajbhai Chauhan <pranay.c1@samsung.com>

[M114 Migration] Fix for missing mojom binding error during page load

This commit fixes below errors that resulted in renderer process crash
during loading of www.tumblr.com.

[1027/113315.059524:ERROR:render_process_host_impl.cc(5384)] Terminating
render process for bad Mojo message: Received bad user message: No binder
found for interface blink.mojom.BadgeService for the frame/document scope
[1027/113315.059608:ERROR:bad_message.cc(29)] Terminating renderer for bad
IPC message, reason 123

Reference: https://review.tizen.org/gerrit/291273/

Change-Id: Ibd48409d77fa69ebe16a45b33f1965383b801661
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>

Revert "Upgrade NodeJS binary to v16.13.0"

This reverts commit 41902d27495512fe250e450298a2e6adaea5f6be
due to build errors for TV profile.

Change-Id: I71230118cf2b166dad3b578d2635f69430ec78e6

fixup! [Tizen] Fix for Compiler and linker errors

This commit ensures that a valid TtsPlatform is returned to fix
segmentation fault crash related to it on following websites.
www.baidu.com
www.hao123.com
www.flipkart.com

Change-Id: I86e9f50fb7f54be8980dabc114ac45af7b7d4a8c
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>

[Autofill] Make AutofillClientEfl implement ContentAutofillClient

Made changes as per upstream patch [1]. Set ContentAutofillClient as base
class for AutofillClientEfl similar to Android and Chrome implementations.

[1] https://chromium-review.googlesource.com/c/chromium/src/+/4272332

Change-Id: I70d8604cacb61c51e60d26fb70ff6bfc197356a3
Signed-off-by: Gajendra N <gajendra.n@samsung.com>

fixup! [M108 Migration][API] Bring up autofill

This commit enables the autofill feature which was disabled during
m114 bringup. Fixes all the build errors due to changes on upstream.

With this commit, autofill related TCs PASS on NativeTCT.

Change-Id: I26024557dc91654f8bd04fa8f8cdac04565cc95d
Signed-off-by: Nikhil Shingne <n.shingne@samsung.com>

Fix native tct failures related to cookie manager

Due to changes on upstream code, now AllowFileSchemeCookies
should not be called on IO thread. This patch makes necessary changes.
With this, ewk cookie manager TCs are passing on NativeTCT.

Change-Id: I858b97098b829cd1a21db63cc96ca240adec3c36
Signed-off-by: Satyaveer Singh <satyaveer.s@samsung.com>

Fix for runtime error during ubrowser launch

Change-Id: I3d427726a91a0b638b23ee1032f3b749651eb400
Signed-off-by: uzair <uzair.jaleel@samsung.com>

Initialize CommandLine before Mojo core initialization

In order to initialize Mojo core, CommandLine must be initialized first.
This patch calls content::InitializeMojoCore() after
CommandLineEfl::GetDefaultPortParams() to ensure CommandLine
initialization.

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

Always initialize eldbus

eldbus has been used not only in ATX but also in clipboard,
so this patch always initailize edbus.

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

[M114] Onscreen Rendering support based on Ozone EFL

This commit fixes build errors to bringup content_shell on desktop.

Build: ./tizen_src/build/build_desktop.sh --build-chrome
Launch :
a. content shell : ./content_shell.sh
b. chrome : ./chrome.sh

Change-Id: Id77b8ea98d0ac633903c2cdee046cba1b8f135f3
Signed-off-by: Ayush Kumar <ayush.k123@samsung.com>

Revert "[M108 Migration] [SlimDownCode] Delete Redundant Modules"

This commit partially reverts 143b987df202b29f81a056cad7237efa20eea195
to add chromeos/* to resolve chrome build errors.

Change-Id: I0f16969d11e15e49721bf4676bd678e2703e020d
Signed-off-by: Ayush Kumar <ayush.k123@samsung.com>

[M114] Avoid adding same EFlWindow to the dispatcher list more than once

This commit ensures the following to fix the below CHECK failure.
1. EflWindow is added to the dispatcher list only once.
2. Previous EflEventHandler is deleted before creating a new one.

[0817/174640.034242:ERROR:observer_list.h(292)] Check failed: false.
Observers can only be added once!

Change-Id: Ibc8f06a5a76b774fd085ea7253fe44cb1dc9e1a9
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>

Upgrade NodeJS binary to v16.13.0

Engine had upgraded nodejs to v16.13.0 for other builds.
This patch upgrades for tizen profile with appropriate
node binary

Change-Id: I6a5b2bbde8586adc2d82031d5bb7121b2a8b7918
Signed-off-by: Surya Kumar <surya.kumar7@samsung.com>

fixup! Upload upstream chromium 114.0.5735.31

icudtl.dat used currently is from M108 that results in renderer crash
due to below CHECK failure.

[0802/172502.073998:FATAL:url_idna_icu.cc(57)] Check failed: false.
failed to open UTS46 data with error: U_FILE_ACCESS_ERROR. If you see
this error message in a test environment your test environment likely
lacks the required data tables for libicu. See https://crbug.com/778929.

This commit updates icudtl.dat to that from M114 to fix above failure.

Change-Id: I084f46b5a564364ef28b10fbf458bf61fc8172d9
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>

fixup! [Tizen] Fix for Compiler and linker errors

Below change add few changes which were missed out from
initial commit

Change-Id: Ic28d8a3d16e7aa159c742e7864f5a2a7f6613b38
Signed-off-by: uzair <uzair.jaleel@samsung.com>

[M114 Migration]Add repo of tizen 8.0 to fix build break on TV.

The problem is that tv build is targeted at MuseM with the previous base repo.
TV build should be targeted PontusM and tizen 8.0 by default.
So, add repos for tizen 8.0 as the default build target
and modify base repo of tizen 7.5 to be targeted at PontusM.

Reference:
https://review.tizen.org/gerrit/c/294529

Change-Id: I87cdc2d0cc88402c43265ca2a95502f5de2817e3
Signed-off-by: Sun-woo Nam <sunny.nam@samsung.com>

[Tizen] Fix for Compiler and linker errors

Below change along with [1]  fixes all the related compiler
and linker errors for tizen profile.

Build:  $ ./tizen_src/build/build_tv.sh
        $ ./tizen_src/build/build_standard_armv7l.sh

Launch: $ /usr/apps/org.tizen.chromium-efl/bin/efl_webview_app
        $ /usr/apps/org.tizen.chromium-efl/bin/ubrowser

[1] https://review.tizen.org/gerrit/c/294878

Change-Id: Id7177f267633efab02aeb3efcd548098ad037232
Signed-off-by: uzair <uzair.jaleel@samsung.com>

[PDNCF] Avoid constexpr std::string which is C++20-specific

Below change backports [1] and [2] to bringup tizen build

[1]https://chromium-review.googlesource.com/c/chromium/src/+/4521911
[2]https://chromium-review.googlesource.com/c/chromium/src/+/4523268

Change-Id: If06210edd2e73c79c745b77b6757b5e51eb3de15
Signed-off-by: uzair <uzair.jaleel@samsung.com>

[M114 Migration]Update tizen version and gbs repo to 7.5

This patch updates $LATEST_TIZEN_VERSION to 7.5 and gbs repo
urls accordingly.

Standard build repo URL has changed to:
https://download.tizen.org/snapshots/TIZEN/Tizen-{version}

Reference:
https://review.tizen.org/gerrit/c/290526
https://review.tizen.org/gerrit/c/292772

Change-Id: I6427fc989296f6d548a87cca29ddc51719a6f164
Signed-off-by: Gajendra N <gajendra.n@samsung.com>
Signed-off-by: uzair <uzair.jaleel@samsung.com>

[M114][EFL][Offscreen] Fix crashes and black screen on webview launch

This commit fixes browser/renderer crashes and black screen
on webview launch. Basic rendering works fine and google.com
loads as expected.

Build:  $./tizen_src/build/build_desktop.sh
Launch: $./out.x64/efl_webview_app.sh
        $./out.x64/ubrowser.sh -l3

Change-Id: I4283b8fa2898a01862b3bc7373b65f7693a6a157
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>

[M114][EFL][Offscreen] Fix build errors for desktop profile

This commit fixes all gn related, compilation and linker errors
for desktop build.

Change-Id: I16cbc9ae25c425cd65da09d1097ceb57c3c0282d
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>

Revert "[PDNCF] Backport ImageManager class and its dependencies"

This reverts commit d3644c0193273f708634389d6b47e18c1c8a0c24.

PDNCF patch, not required for M114.

Change-Id: I61471b0a16719483f85e8742bdd79e451ae50f85
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>

Upload upstream chromium 114.0.5735.31

This is a patch for upstream m114 - m108.

* chromium
72d3dff19e4fa332840c101a5df2f8ddd08a9427
Publish DEPS for 114.0.5735.31

* v8
ca96c940fe41670a2e1b7d56ad7ced67ce3528d1
Version 11.4.183.8

* skia
93e7d305c32c17b3fa3e531dd967bc588948a080
Merge 1 release notes into RELEASE_NOTES.md

Change-Id: Ifcb7db3364bc5a804d60fb30137d10aaf3f9972c
Signed-off-by: Ayush Kumar <ayush.k123@samsung.com>

Revert "Ensure all gl calls are submitted to GPU"

This reverts commit 530b19ef2644316dc95baaae52e3102a4d66a0af.

Reason for revert: This change is not needed anymore as https://review.tizen.org/gerrit/c/platform/framework/web/chromium-efl/+/291434 is merged

Change-Id: Ifae43e8bbc629cc197b20187d41686177065d3db

[XWalkExtension] Change a way to load XWalk extensions

This will load all *.json from given base folder, not a
specific json file.

Change-Id: I8ddc154366308f4eece7a2d8e538d321b566c928
Signed-off-by: DongHyun Song <dh81.song@samsung.com>

[M108 Aura Migration][NaCl][PPAPI] Implement backward compatibility for PPAPI UDPSocketExtensionSamsung

Ported patches from M94:
https://review.tizen.org/gerrit/#/c/280723/

Change-Id: I8d0217ee9516ecf9fceae5e5537a838dd8d73249
Signed-off-by: Xiaoshu Wei <xiaoshu.wei@samsung.com>

[M108 Migration][WRTjs][VD] Merge splash screen feature

1.Support landscape/portrait splash screen
landscape/portrait splash screen should be supported.
https://docs.tizen.org/application/tizen-studio/web-tools/config-editor/#tizen-launch-screen-2
<tizen:launch_screen ready_when="custom">
  <tizen:ls_default background_color="#00ff00" image="images/default.png" image_border="0px stretch"/>
  <tizen:ls_landscape background_color="#ff0000" image="images/landscape.png" image_border="0px stretch"/>
  <tizen:ls_portrait background_color="#0000ff" image="images/portrait.png" image_border="0px stretch"/>
</tizen:launch_screen>

2.splash screen image support SHARE path
LYNKCloud app need to download splash screen image from server.
They save image to SHARE path(/opt/usr/home/owner/share),
WRT set this image as splash screen image.

Reference:
https://review.tizen.org/gerrit/289862/
https://review.tizen.org/gerrit/289246/

Change-Id: I4cf8d770e2698af9c40a6e8242da109d048098e1
Signed-off-by: zhaosy <shiyusy.zhao@samsung.com>

[M108 Aura Migration][NaCl][W3C][Gamepad][VD] Fix: Gamepad Timestamp issue, get timestamp using accesory's timestamp

Merge from M94 patch:
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/277929/

Change-Id: Iaa50bffe1e7074971d4b3db292de62682ce4ba15
Signed-off-by: weijun <weijun5.lin@samsung.com>

[M108 Aura Migration][NaCl][PPAPI] URLResponseInfo response code 0

URLResponseInfo response code 0 if file is loaded via file:// protocol.

Ported patches from M94:
  https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/280897/

Change-Id: I6cdde453283cac23957e506fd4b5d685af71ce84
Signed-off-by: jinbei09 <jinbei09.dai@samsung.com>

[M108 Aura Migration][NaCl][PPFwk] Add error logs + SVACE/DLOG/Static analysis fix

Ported patch from tizen_7.0:
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/280926/

Change-Id: I90218f239233ab49f543279cf50cf1bf8705bb5e
Signed-off-by: jinbei09 <jinbei09.dai@samsung.com>

[M108 Aura Migration][NaCl] Recover GetPrivilegeMapping function when checking privilege

Migrated from tizen 7.0:
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/280677/

Change-Id: Ibe598e2cd12b337b204e148e296fb15bf66e8a92
Signed-off-by: jinbei09 <jinbei09.dai@samsung.com>

[M108 Aura Migration][NaCl][PPFWK] Add logs to indicate when pepper plugin process exit

Ported patch from M94:
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/279690/

Change-Id: I6384673375c7a7ad99a79d7209c7cbed63b771cc
Signed-off-by: jinbei09 <jinbei09.dai@samsung.com>

[M108 Aura Migration][NaCl][PPAPI]Fix compilation warnings in media related components.

* content/renderer/pepper/video_decoder_shim.cc
In some scenarios IsCodecSupported did not returned any value. Now it returns false by default.

Ported patch from:
 Fix compilation warnings in media related components
 https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/280521/

Change-Id: I6ccc63073536fd1ab1cbb60d49933680e59451c4
Signed-off-by: jinbei09 <jinbei09.dai@samsung.com>

[M108 Aura Migration][NaCl][PPAPI] Crash during PPB_MessageLoop destruction

When MessageLoop for PPAPI is destroyed it may happen that there are
some unrun callbacks. Dtor of such callback tries to acquire global
proxy lock.

If such destroy is called from during call to method exposed via PPAPI
to the plugin then deadlock occurs as global proxy lock is held
EnterResource object form Thunk layer.

Ported patch from Tizen 7.0:
  https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/280223/

Change-Id: If50242a099ce4c95f42148cd38c565314558359c
Signed-off-by: jinbei09 <jinbei09.dai@samsung.com>

[M108 Aura Migration][NaCl][PPAPI]Fix PPAPI unittest error

This patch fixes below PPAPI unittest error:
[1] var_value_converter_unittest.cc build error
[2] MediaStreamBufferManager unittest execute error

error[1] was already fixed in patch:
  [M108 Aura Migration][NaCl][PPFW] Enable VD Trusted Plugins functionality
  https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/289527/

Ported patch from tizen_7.0:
  https://review.tizen.org/gerrit/#/c/270688/

Change-Id: Id837625846669d5622da9ae167ac7071a97ef5e8

[M108 Migration][MPEG-H][Hbbtv][MSE] Add proper demuxing for MPEG-H codec

Migration from
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/272960/

Change-Id: Ibf667810251a6fd0425dd3f9ce197162518ef122
Signed-off-by: luyq <yanqing.lu@samsung.com>

[M108 Aura Migration] Setting demuxer memory limit

Migrated from:
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/280160/

Change-Id: Ib78ab63a36ac9744313524baf884ac5cd6d5e1e6
Signed-off-by: yanting.hong <yanting.hong@samsung.com>

[M108 Migration] Check scroll area pointer before use

-Check the pointer is not null before using,
otherwise this causes the render process crashes.

-Add some macro.

reference:
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/274002/

Change-Id: I617b8c7ee002e9b3e185aa27c17d19c9c90bb4e5
Signed-off-by: wangjing <jing124.wang@samsung.com>

[M108 Migration] Make sure device scale factor (DSF) is not less than 1.0 on any profile

On RPI4 device, due to DPI value being low (120), DSF was computed as 0.75,
this was causing some problems related to subpixel rendering on css related
TCTs. This patch makes sure the DSF value is not lesser than 1.0.

Change-Id: Ibde585f3a75793a4ac429e712a8572a0d922e488
Signed-off-by: Gajendra N <gajendra.n@samsung.com>
(cherry picked from commit db4b64a66f803008fa66f08b156f31997ba9f385)

Disable debug package for aarch64 build

Change-Id: Ib65aa864e272f4a3ce6d1f6f93ebf1b69fc0b724
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>

[M108 Aura Migration][NaCl][PPFW] Fix crash when get render process in UnregisterPepperExtensionDelegate.

EWeb View will destruct in EvasObjectDelete callback called.
If ewk_shutdown called before EvasObjectDelete, it may cause crash
during get render process in UnregisterPepperExtensionDelegate,
which is in EWeb View destructor.

This change is for DF190114-00719

from M94 Aura Patch:
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/279978/

Change-Id: I76685eb3ea0a9e2c1fcfc9aeedf476b52813ac05
Signed-off-by: pengxia <pengxia.shen@samsung.com>

[M108 Migration][VD]Implement ewk_settings_allow_file_access_from_external_url_set

Some specific case in Tizen TV, we need to enable this.
When hosted applications want to use tv device api,
load file://opt/usr/apps/pepper/webapis.js located in local.
But, they cannot access local because of security vulnerability(SOP)
Policy of webapis.js is only located in local.

Reference:
https://review.tizen.org/gerrit/282771/

Change-Id: Id7e29a72dd5571770ae196d5387d97d98eb9d326
Signed-off-by: xiafeng <feng.xia@samsung.com>

[M108 Migration][VD]Dummy Implementation for web security API

ewk_settings_web_security_enabled_set/get

Reference:
https://review.tizen.org/gerrit/#/c/256343

Change-Id: I7bcf6fa71c953bcdbe5fc5831f09114693fa6d32
Signed-off-by: jingjieli <jingjie.li@samsung.com>

Fix critical SVACE issue

This patch fixes some critical SVACE issues.

Change-Id: I48296ccf49643421f7210d8d8d0502f31f38705a
Signed-off-by: v-saha <v.saha@samsung.com>

[M108 Migration][VD] Enable Suspend & resume feature

Because RenderViewObserverEfl unavailable in M108,
so need to implement in widget.mojom

References:
  - https://review.tizen.org/gerrit/284921/

Change-Id: Ib558fcb5a21f34c0aaf7946b07c161342ee74e1a
Signed-off-by: jiangyuwei <yuwei.jiang@samsung.com>

Fix major SVACE issues

This patch fixes major SVACE issues.

Change-Id: Ie5fbe44a3aa99aa1589718a8d7d64d75453ef5d0
Signed-off-by: Ayush Kumar <ayush.k123@samsung.com>

Add IS_TIZEN guard for XWalkExtension

XWalkExtension can be enabled with IS_TIZEN and this will
fix build break for the desktop profile.

Change-Id: I0edb4a3c33a42a4637bebce0b3c74cd683f8ccf2
Signed-off-by: DongHyun Song <dh81.song@samsung.com>

[M108 Migration][VD] Support floating video window

Support the new requirement of floating video window for WebBrowser.

- Provide an api of ewk_view_floating_window_state_changed.
  Notify WebEngine the status that floating video window is on/off.
  If floating video window existed, but not current webView,
  For tabs switching case, in order to keep floating video window,
  need to prevent media resume.

- Provide "notify,video,resized" for video rect notification.
  WebEngine notify WebBrowser playing video dimension in normal webview
  through which mini tab button is displayed at correct location on video.

- Appended the document url to the paramater of "video,playing,url".
  In case video is playing inside an iframe, document URL provides
  URL loaded in iFrame to identify which document is playing video.

- Allow to run the insecure content from localhost,
  that are http://127.0.0.1 and http://localhost.

refs:
https://review.tizen.org/gerrit/283144/

Change-Id: I3c8c0f4c2279e4cadcba471250a2e6e1c67729c6
Signed-off-by: qiang.ji <qiang.ji@samsung.com>

[M108 Aura Migration][WebRTC] Camera implementation related patches

- Enable network camera feature.
- Add remote camera and camera device manager API.
- Adapt the change of camera FW.

Including following patches:
https://review.tizen.org/gerrit/#/c/280505/
https://review.tizen.org/gerrit/#/c/272718/

Change-Id: Iba32024cf344e5eb6a4285bb9020229ccdd249d1
Signed-off-by: Suhaspoornachandra <s.poornachan@samsung.com>