Fix for crash on webview resize for a new page opened in the same tab.

Currently, segmentation fault occurs on webview resize when we load a
new page on an already existing tab. In this scenario, a new RWHVEfl
is created first and then the previous RWHVEfl is destroyed. However,
the same WebContentsViewEfl(therefore the same native view) is used
for the new RWHVEfl as well.

Two resize callbacks(one for each RWHVEfl) are added to the same
native view in order of their creation.
In destructor of the previous RWHVEfl, evas_object_event_callback_del()
deletes the most recently added callback(the one for the new RWHVEfl).
When resize occurs for the new RWHVEfl, the callback for the previous
RWHVEfl(already deleted) is invoked that results in crash.

To fix this issue, we now use evas_object_event_callback_del_full()
instead of evas_object_event_callback_del().
evas_object_event_callback_del_full() deletes the callback for the
corresponding RWHVEfl only.

Bug: http://suprem.sec.samsung.net/jira/browse/CBEFL-806

Reviewed by: sm.venugopal, sns.park

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

fixup! [MM] Fix flickering when playing video.

Previous patch relied mostly on CleanUp() for releasing
used tbm_surface. But it has issues like
- It causes that many number of tbm_surfaces are pended for releasing,
because compositor sometimes does not signal to release.
This causes locking on capi-player because capi-player has constraints
on output tbm_surface counts.
- CleanUp() is cost operation.

This fixup introduces Closure to bound the lifecycle of
tbm_surface with VideoFrame by following the Chromium's VideoFrame approach.
(So no more hack is required on GLImage for releasing tbm_surface.)
When Chromium invalidates the VideoFrame, tbm_surface will also be released.

For this approach, this fixup introduces IPC message for releasing tbm_surface,
so any process/thread can request to release tbm_surface.
(On previous approach, Browser-Process can only request releasing tbm_surface).

By taking this approach, it's guaranteed that every tbm_handles are
requested to be released, so CleanUp is removed.

Bug: http://165.213.149.170/jira/browse/TSAM-676
Bug: http://165.213.149.170/jira/browse/TSAM-662
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15255
Bug: http://suprem.sec.samsung.net/jira/browse/CBEFL-835

Reviewed by: msu.koo, sm.venugopal, sns.park

Change-Id: I9c622396402ec34e0aa77e28bff7bf3e937f83eb
Signed-off-by: msu.koo <msu.koo@samsung.com>

[MM] Fix flickering when playing video.

Previously, Player deletes |tbm_surface| when next frame is delivered.
But occationally, it causes frame drops and crashs
because |tbm_surface| is released before the gpu does not finish to handle it.

This patch introduces |MediaPacketManager|
to hold media_packets which wraps tbm_surface and CleanUp regularly.

Also supplying APIs to let Renderer explicitly release the |tbm_surface|s
by indexing the |media_packet|s using |tbm_surface_h| as the key.
(Renderer only knows tbm_surface.)

CleanUp is introduced to prevent media_packet(tbm_surface) leaks
even Renderer does not request to release.
Theoritically CleanUp should do nothing because Renderer invoke
Release() for all |tbm_surface|s afer rendering so that
implementation is focused on optimizing Release() rather than CleanUp().

Bug: http://165.213.149.170/jira/browse/TSAM-653
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15255

Reviewed by: sns.park

Change-Id: I9a1e3b0710da0072b3459818690d16e5c43d6fd3
Signed-off-by: msu.koo <msu.koo@samsung.com>

[MM][MSE]CAPI code refactor

Removed unused code. Re-arranged code to improve readablity.

Bug: http://165.213.149.170/jira/browse/TSAM-663

Reviewed by: msu.koo, sm.venugopal

Change-Id: I5d481488d5f9ba1398d2d5058b2866cdd5b80f1c
Signed-off-by: sm.venugopal <sm.venugopal@samsung.com>

Refactoring code with creating OK button on popup.

The issue is that we has CreateOKButton() function in Layout class,
but in the code there is not any usage. In every function with
name started with the "show" word (e.g. showDatePopup) there is
created "OK" button the same way, without using special function
for it. It creates a redundancy of code.

Reviewed by: a1.gomes, g.czajkowski

Change-Id: Ic7a98c43de3dcdc4106187b9e2624d12d9f42cd9
Signed-off-by: Piotr Ganicz <p.ganicz@samsung.com>

Clear selection rectangle on tap gesture

On branch m42 the selection rectangle is cleared when on tap gesture is handled.
This does not happen on m47.

Because of the above WCS#41 fails when JavaScript preventDefault() Event Method
is used to cancel mouse down event. Context menu is being hidden but the
selection rectangle is still visible. This patch makes m47 to works as m42.

Original beta/m47 patch:
- http://165.213.202.130/gerrit/#/c/98569/

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15140

Reviewed by: a1.gomes

Change-Id: I71a6ffe0b0645d4e91811b2eb87ff65f9169d13b
Signed-off-by: Piotr Krysiewicz <p.krysiewicz@samsung.com>

Align the content of popups (date,month,week,time,colorpicker)

This patch is to set a proper position of swallow element on the
popup on TV platform. The solution for this issue is to send a signal
from C++ code to the edje file (control.edc) with the information
of the current platform. Depending on the platform the proper code
will be running in edc file in part "elm.swallow.datetime".
In colorpicker top_padding element was displayed as a white RECT,
due to that fact we should change the type of it  to SPACER
(it is transparent).

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14841

Reviewed by: a1.gomes, g.czajkowski

Change-Id: I1a787979581cf218fdf8d8d1d630a17a620296ab
Signed-off-by: Piotr Ganicz <p.ganicz@samsung.com>

fixup! Do not zoom-to-focused-element when long pressing.

There is another spot in the code that calls
::ScrollFocusedEditableNodeIntoRect and triggers
zoom-to-focused-element.
It is trigger on target builds, when dpad shows up.

Patch fixes it.

Original beta/m47 patch:
- http://165.213.202.130/gerrit/#/c/98597/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14911

Reviewed by: g.czajkowski

Change-Id: I7c1cfc6f8d13f8842848950a32d7a3ea78804580
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>

Set enable-prefer-compositing-to-lcd-text on ubrowser (desktop).

Our text selection implementation is based on updates that
come as part of the compositing updates (cc/ land).
One bit of this update is the ui::SelectionBound::visible property,
which holds the visibility status of a given selection bound.
For instance, this takes into account when a selection happens
on an scrollable box (e.g. <div>) or a scrollable inner frame
(e.g. <iframe>), and it gets scrolled out of the view.

The visible status is determinated by checking if the given
point is hit-test'able considering the bounds of the clipping
parent layers.
However, if a layer is not promoted as composited by a reason,
it is not taken into account on the upwards frame traversal.

Scrollable iframes get their promotion to a composited layer
controlled by a setting: preferCompositingToLCDTextEnabled()
(see CompositingReasonFinder.cpp).
This setting gets set on RenderViewImpl::Initialize and bubbled
down to blink::WebView.

Its value is controlled by:
1) command line switches kDisablePreferCompositingToLCDText and
kEnablePreferCompositingToLCDText
2) the device_scale_factor (see DeviceScaleEnsuresTextQuality).

On desktop, where the device_scale_factor is 1.0, preferCompositingToLCDTextEnabled
gets set to FALSE. Due to that, scrollable inner frames do not
get promoted to composited layers, and all the visibility set
logic of selection bounds (described above) does not work properly -
it is always TRUE.

In order to make ubrowser test what TIZEN runs, patch passes
'enable-prefer-compositing-to-lcd-text' switch at launch time.

This makes clipping work for selection bounds that go offscreen.

Original beta/m47 patch:
- http://165.213.202.130/gerrit/#/c/98862/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14720

Reviewed by: g.czajkowski

Change-Id: I3587e53c1e357107d03b9b32d9870d91de791a7e
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>

Offscreen handles are visible

Patch changes the selection controller logic a bit to
take SelectionBound::visible bit into account before
show/hide a "input" handle.

This fixes WCS TC 15, and various real world websites.

Original beta/m47 patch:
- http://165.213.202.130/gerrit/98864

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14720

Reviewed by: g.czajkowski

Change-Id: I104c0b38f583c7b0467fe224c100052e1b8fe98a
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>

Dismiss disambiguation popup when touch outside of it.

When disambiguation popup gets shown, it is only possible
to dismiss it when clicking on the zoomed area. This differs
from s-browser, where clicking outside the zoomed dismiss it.

Patch implements the same for chromium-efl.

Original beta/m47 patch:
- http://165.213.202.130/gerrit/98592

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15238

Reviewed by: g.czajkowski

Change-Id: Ieff0a1ce0d4784d30036c2ec02bdc5ca6c7ddc9b
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>

[EWK] "policy,decision,navigation" callback invoked to m47_2526

On Chromium Upstrean HandleNavigation was deprecated ([1]) and reflected
to ours ([2]). Because of this callback for "policy,navigation,decide"
was not called normally. This patch modifies this behaviour to fulfill
EWK API requirements.

[1] http://code.google.com/p/chromium/issues/detail?id=325351
[2] http://165.213.202.130/gerrit/#/c/90805/17

Together with: I922aeeb194e9c4a207716657c1b5d8b9663ed20b
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14732

Reviewed by: a.renevier, a1.gomes, djmix.kim

Change-Id: I3a01cd41861d0913b36797c5ceb21e10fcfa32a9
Signed-off-by: Piotr Pruski <p.pruski@samsung.com>

Save traces.json in $HOME for TIZEN

ubrowser app allows tracing events which are saved to
/opt/usr/media/traces.json for TIZEN. Since ubrowser can not be run on root
saving to this path fils due to lack of permission for non root user.

This patch saves traces.json in root of home directory.

Original beta/m47 patch:
- http://165.213.202.130/gerrit/#/c/98830/

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15248

Reviewed by: a1.gomes

Change-Id: Ie0cdf091518e12351d9495541e3858d4c77c9b71
Signed-off-by: Grzegorz Czajkowski <g.czajkowski@samsung.com>

Implementation of DownloadManagerDelegateEfl class

The patch implements DownloadManagerDelegateEfl::DetermineDownloadTarget,
which allows to use Chromium downloads.

We are checking whether the external download manager is available,
in order for it to be considered, before using Chromium fallback
download code.

Right now, PathService class is used to determine where downloads
should be saved on given platform based on mime type.

Original beta/m47 patch:
- http://web.sec.samsung.net/gerrit/#/c/96713/

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14817

Reviewed by: a1.gomes

Change-Id: Ia1362cad11044dab4de4e1ee7eb908ebeee3c995
Signed-off-by: Pawel Niemirski <p.niemirski@samsung.com>
(cherry picked from commit d0bde5a4fdf6f0591bf194478e1b7f3b5997bd00)

Add WebViewBrowserMessageFilter in RenderViewCreated

The renderer process is created for each domain. Communication
between the browser process and renderer processes is done
by ChannelProxy. In order to initialize communication
the RenderProcessHost inserts WebViewBrowserMessageFilter object
into the channel.

Adding WebViewBrowserMessageFilter object to ChannelProxy by
RendererProcessHost is called too early because RWHV was not created
yet (while calling RenderProcessWillLaunch). This breaks
communication.

To make sure that RenderWidgetHostView is already created,
when adding WebViewBrowserMessageFilter to the channel, this patch
moves it to WebContentsDelegateEfl::RenderViewCreated.

The issue happens ONLY after domain change, because the first instance
Instance of RWHV is created by EWebView::Init(), but every consequent
is created after EWebView::SetURL().

Original beta/m47 patch:
- http://165.213.202.130/gerrit/#/c/99073/

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15100

Reviewed by: a1.gomes

Change-Id: I058e6dc4666cb913f7256fd613528e165cbd1352
Signed-off-by: Pawel Niemirski <p.niemirski@samsung.com>

Revert "fixup! Make build_tv.sh script usable again."

This reverts commit fa1b0b3713341f8123a0e147ce48387f68d5a690.
Reverting due to problem in bot.

Change-Id: Ifd57cac9022af2167d6f892eaac7d6711cf16cd8
Signed-off-by: SeungSeop Park <sns.park@samsung.com>

[MM] Cleanup dead codes(pixmap support) on multimedia

Pixmap was introduced for Tizen 2.4 mobile,
but Pixmap support was deprecated from Tizen 3.0.
(because Tizen 3.0 moves to wayland backend from X.)

Bug: http://165.213.149.170/jira/browse/TSAM-558

Reviewed by: sm.venugopal, sns.park

Change-Id: I4a5a10d15105a764c8fcc63d340ecfcd657c3975
Signed-off-by: msu.koo <msu.koo@samsung.com>

Upgrade EFL version for desktop to 1.16.0.

EFL version is updated to 1.16+ on mobile and TV.
- With EFL 1.16, recreating surface on resize is not
needed, commit[1] revert's implementation of
recreating evasgl surface on resize.
- Desktop version is using EFL 1.13, whitescreen issue
is observed with commit[1].
commit[1]: http://165.213.202.130/gerrit/#/c/97324/

Update EFL version for desktop to 1.16.0.

Reviewed by: a1.gomes, sns.park

Change-Id: I482af687ffe06aa2805968aaaf30e25312411f1d
Signed-off-by: venu.musham <venu.musham@samsung.com>

[MM][CAPI] Copy the YUV data from TBM surface by each plane

Now copying YUV plane to shared memory uses correct start point of
each plane's pointer using libyuv function.
Before this change, there was segmentation fault because it was trying to
access complete YUV size where plane[0] was allocated of only Y size

Reference: http://165.213.202.130/gerrit/#/c/88539/

Reviewed by: msu.koo, sm.venugopal, sns.park

Change-Id: Id8b4e7ae3b267cb3b5d103a6ecb4d63ae6fc8d9a
Signed-off-by: ws29.jung <ws29.jung@samsung.com>
Signed-off-by: vm.arjun <vm.arjun@samsung.com>

[EFL] Implement orientation_lock and orientation_unlock API

This is a very basic implementation of orientation lock/unlock API:
http://www.w3.org/TR/screen-orientation/, based on current Chromium Android
implementation. Android's version has been stripped from Android specific
Start/StopAccurateListening methods, which will be provided in next patch.

This patch deprecates EWK API's ewk_view_orientation_lock_callback_set.

Original beta/m42_2311_t patch:
- http://165.213.202.130/gerrit/#/c/80821/

Original patch was partially passed to dev/master. This is supplement.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14887

Together with: Ia91c2dc181fd9c08f0027c1a50e681c349e20294

Reviewed by: a1.gomes, djmix.kim, msu.koo

Change-Id: I1617916634ee03b54c80124a724eb4e24d01e427
Signed-off-by: Andrzej Badowski <a.badowski@samsung.com>

[MM][ME][MSE] Implemented to pass over media_packet to renderer.

This patch is to enable MediaPacket on ME/MSE for Tizen 3.0.
Currently actual composition happens inside Browser Process,
(to be exact, GPU thread lives in Browser Process)
so passing the raw video frames via SharedMemory can be avoid.

This patch is to pass over the frame memory address to renderer process
to let GPU thread directly access Tizen video buffers.(aka, tbm_surface)

Bug: http://165.213.149.170/jira/browse/TSAM-560
Reference: http://165.213.202.130/gerrit/#/c/87210/

Reviewed by: msu.koo, sm.venugopal, sns.park

Change-Id: I1abf25be1c346fbf1ec046d1fd5307745756ca2f
Signed-off-by: uzair.jaleel <uzair.jaleel@samsung.com>
Signed-off-by: msu.koo <msu.koo@samsung.com>
Signed-off-by: ws29.jung <ws29.jung@samsung.com>

When selection is cleared, context menu also should be cleared.

When |ewk_view_text_selection_clear| is called, context menu is still shown
although selection was already cleared.

This patch clear context menu in |EWebView::ClearSelection|.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15223

Reviewed by: a1.gomes, sns.park

Change-Id: Ib07dcf92777f49206bd899fa14934cf27e4c1364
Signed-off-by: Sanghyup Lee <sh53.lee@samsung.com>

fixup! [MM] Handling multiple instances of media players.

This is to fix the wrong file-mode changes of previous patch.
File mode is reverted from 100755 to 100644.

Reviewed by: msu.koo, sm.venugopal, sns.park

Change-Id: I86072782c55de96d2d2d25df1a2001f5e3bfe096
Signed-off-by: msu.koo <msu.koo@samsung.com>

Fix crash when calling ewk_shutdown

Crashes were failing CHECKs, which expected the client to release all
evas WebView objects before calling ewk_shutdown. Client should fix that,
but Chromium shouldn't crash because of client's bug, so checks were
replaced with information in log.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14628

Reviewed by: g.czajkowski, msu.koo

Change-Id: I23b09de78f2536608621a1de07b7de8c956c1ab3
Signed-off-by: Tomasz Czekala <t.czekala@samsung.com>

[MM] Handling multiple instances of media players.

This patch introduces a way to manage the number of player instances,
which is needed to meet a policy or limitation imposed by platform.
For example, in TV target HawkP, only 1 player instance is allowed to run
by hardware limitation.

When active player instances exceeds the maximum number, the implementation
releases the oldest player instance.

Details for current implementation:
- Max player instances : TV - 1, Mobile - 5
- Choosing player to remove when full: FIFO (managed by |resumed_queue|).
- The purpose of each queue.
|init_queue| : Whenever a player is created, it's pushed in a init_queue.
Then the players are initialized one-by-one.
|resumed_queue| : Managing the activated(resumed) player id.
Push id by |OnResumeComplete| and pop by |OnSuspendCompleted|.
- When we try to play or seek a player, the implementation check the
resumed(activated) player and then suspend(release) player instance
if it exceeds the maximum instance number.

Reference : http://165.213.202.130/gerrit/#/c/72551/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14833
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14853

Reviewed by: msu.koo, sm.venugopal, sns.park

Change-Id: I31e2cbefdfa60222315b8743a73072c5009c20b2
Signed-off-by: sangdeug.kim <sangdeug.kim@samsung.com>

Do not trigger selection for when selection is forbidden or cancelled

If user is long pressing on a content with
-webkit-user-select: none, we should bail and not enter
selection neither show magnigier class or context menu.

Same (not enter selection mode) should happen when JS side
calls preventDefault on 'selectstart' event.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15148

Reviewed by: a.renevier, a1.gomes, g.czajkowski

Change-Id: Ice4e0a5b3c0ada166f3cbc9a2d48e24534f86d38
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>

[MM] Remove duplicated enums from MM codes.

This patch is to remove duplicated NetworkState and ReadyState
from codes and use already-existing blink::WebMediaPlayer::NetworkState
and blink::WebMediaPlayer::ReadyState.

Bug: http://168.219.209.56/jira/browse/TSAM-536
Reference: http://suprem.sec.samsung.net/gerrit/#/c/54032
Reference: http://suprem.sec.samsung.net/gerrit/#/c/56307

Reviewed by: sm.venugopal, sns.park

Change-Id: Ic81fb513f41d1e7d9a688b66f798cd3f827b47ea
Signed-off-by: msu.koo <msu.koo@samsung.com>

Filter out events jittery for single-touch gestures.

In the majority of cases when user taps on the screen, many touch move events
are generated from platform. These unexpected touch move events cause frequent
calling seek on media player.

It looks too sensitive compared with Android browser.

This patch dispatches first move event if distance from last pressed position
exceeds scroll threshold.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=13974

Reviewed by: a1.gomes, sns.park

Change-Id: Ia491beb54c1cf32df78d9af8b7e7a212c7f54e61
Signed-off-by: Sanghyup Lee <sh53.lee@samsung.com>

Changed translations of ContextMenu items names

Few translations need to be fixed to meet UI Guidelines.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15105
Reviewed by: Grzegorz Czajkowski, Min-Soo Koo

Reviewed by: a1.gomes

Change-Id: If0e6d8e456844ada6a13d56f0578e93c869b0757
Signed-off-by: Piotr Krysiewicz <p.krysiewicz@samsung.com>
(cherry picked from commit c5e7206d44749d9b150790ca8c02ce33357f9d00)

[MM][MSE][CAPI] Crash while playing more than one clip in youtube

During CAPI destruction, OnDemuxerDataAvailable call from demuxer will
lead to crash. In such scenario, just return the control. This patch also
solves the crash issue in demo.htm, playing webaudio/MSE/Webaudio/MSE

Original Patch: http://suprem.sec.samsung.net/gerrit/#/c/53523

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14835

Reviewed by: msu.koo, sm.venugopal, sns.park

Change-Id: I9c18941735b2550ec81876362f3d50f85f6ae5c2
Signed-off-by: k2.nagaraju <k2.nagaraju@samsung.com>

Delete unnecessary eglibc-2.18-5.1.armv7l.rpm

The file was added in [1] to support GCC 4.8 build, but now obsolete.

[1] http://165.213.202.130/gerrit/#/c/70019/

Reviewed by: a1.gomes, msu.koo

Change-Id: I0b5e0ab8688ff4d57e3309c73e89db3711c27dca
Signed-off-by: SeungSeop Park <sns.park@samsung.com>

Revert "Workaround for tizen's nss version issue"

This reverts commit 0f59aac6f36d809a46979b6c0f2c099c59c715b6.

Nss version in tizen 3.0 is 3.17. We are not relying on the old hack
to bypass nss version issue. Patch removes the legacies.

Reviewed by: a1.gomes, msu.koo

Change-Id: I38bd5adaf42171b5620dae79005667d24c5bff80
Signed-off-by: SeungSeop Park <sns.park@samsung.com>

fixup! Make build_tv.sh script usable again.

Symbolic link src/packaging which points to src/tizen_src/packaging
is generated during the build, which yields dirty status when 'git status'.

Patch removes 'src/packaging' symlink after the rpm build is done.

Reviewed by: a1.gomes, msu.koo

Change-Id: If1d2cff7f4ca62415fb6deba5bea051302053606
Signed-off-by: SeungSeop Park <sns.park@samsung.com>

Implement device display info and get device scale factor.

This patch implements basic display functionalities like display
resolution size, dip scale and rotation degrees.

Obtains the device scale factor (dsf) using device display info. The
display info computes this factor from dpi.

Reference gerrit IDs:
http://165.213.202.130/gerrit/#/c/83364/
http://165.213.202.130/gerrit/#/c/83651/
http://165.213.202.130/gerrit/#/c/85226/
http://165.213.202.130/gerrit/#/c/85236/
http://165.213.202.130/gerrit/#/c/85700/
http://165.213.202.130/gerrit/#/c/87076/

Together with: I8f6dbee2545283c3002591e4965f5c2f677fc6c9

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14881
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14877

Reviewed by: a1.gomes, prashant.n, sns.park

Change-Id: I22014fabf1e2ce2160be75dec4e8bdb8ecd76545
Signed-off-by: suneel <suneel.kota@samsung.com>

Enable ccache on gbs build system.

In order to enable ccache build option,
we should change it's PATH value to /var/lib/build/ccache/bin.
Because ccache path is moved to there in latest gbs build system.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15086

Change-Id: Icd70453aa96a85553a9e949aad3ee3d2d0128688
Signed-off-by: Jongsoo Yoon <join.yoon@samsung.com>

Changed the IP of mirror server in Tizen 3.0 Profile.

This patch is only valid in HQ.

Before : http://10.252.81.117/snapshots
After  : http://10.113.63.117/snapshots

Change-Id: Ic77bb2e9014cc532b7d5c50aac95a8f5ab7c9808
Signed-off-by: Jongsoo Yoon <join.yoon@samsung.com>

Re-implement didCreateScriptContext to RenderFrameObserverEfl.

DidCreateScriptContext and WillReleaseScriptContext are moved
to ContentRendererClient by (1),(2) in opensource.

DidCreateScriptContext and WillReleaseScriptContext are called
when calling open/close window by javascript.
Those functions must be called together.

In case of m47 chromium-efl, we do not call DidCreateScriptContext.
For this reason, renderer process is crashed when
javascript context is released from injected_bundle.

This patch has 2 modifications as below.
- Remove |WillReleaseScriptContext| from ContentRendererClientEfl.
- Move |didCreateScriptContext| to RenderFrameObserverEfl.

(1) https://codereview.chromium.org/1040403003
(2) https://codereview.chromium.org/185713008

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15036

Reviewed by: a.renevier, a1.gomes, djmix.kim, g.czajkowski, msu.koo

Change-Id: I008937710d08472a6cfb77a51b1cbf01ffacc221
Signed-off-by: DongJun Kim <djmix.kim@samsung.com>

Do not pass kMobileFlags for all TIZEN builds using ubrowser

kMobileFlags are defined in ubrowser/main.cc and are passed to chromium in case
ubrowser is launching with mobile param.

However, as |!desktop_ui| is also true for all TIZEN builds those flags are
unexpectedly set for them. As for OS_TIZEN_MOBILE they are just duplicated and
do not bring side effects whereas for OS_TIZEN_TV they are not expected.

This patch passes kMobileFlags in case "ubrowser --mobile" only.

Additionally, this patch gets rid of custom EFL command line
"use-mobile-viewport-style" as it can not be propagated to Renderer by default.
For OS_TIZEN_MOBILE |UseMobileViewportStyle| is set by default on src/ side [1].

This patch introduces a new EFL command line:
"ewk-enable-mobile-features-for-desktop" in order to turn on a bunch of
settings for which there is no command line switches. This allows desktop
"ubrowser --mobile" to have similar set of features that mobile build has.
Secondly, we stop abusing |UseMobileViewportStyle| in |ApplyCustomSettings|.

[1] http://165.213.202.130/gerrit/#/c/97109/1

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14816
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15031

Reviewed by: a1.gomes, sns.park

Change-Id: I7dd9f693f53abab0916e6113bbc15bfe635a3159
Signed-off-by: Grzegorz Czajkowski <g.czajkowski@samsung.com>

Implement ewk_favicon_database_icon_get()

Implement ewk_favicon_database_icon_get() by calling
EwkFaviconDatabase::GetIcon() (also added by this patch).
EwkFaviconDatabase holds an instance of EWebContext, needed
in the call chain.

Original beta/m42_2311_t patch:
- http://165.213.202.130/gerrit/#/c/85278/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14809

Reviewed by: d.waslicki, g.czajkowski, msu.koo

Change-Id: I64ee94547570758a45d9d4aa6c0c8556c8c8f258
Signed-off-by: Marcin Niesluchowski <m.niesluchow@samsung.com>

[MM][MSE] seek offset not synced to actual browser seek time

CAPI player_set_position as to be called with actual browser seek time
not with user seeked time. Otherwise CAPI will wait for
(|actual_browser_seek_time| - |seek_offset_|) milliseconds buffer.

Original Patch: http://suprem.sec.samsung.net/gerrit/#/c/53159/

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14835

Reviewed by: msu.koo, sm.venugopal, sns.park

Change-Id: I878697fdc98f259a02b9849bc47955464aec2799
Signed-off-by: k2.nagaraju <k2.nagaraju@samsung.com>

Add support for Copy/Paste operations for Image element

Although Clipboard operations for text and markups work well,
when image is long-pressed, copied (Copy Image) and pasted - nothing
happens.

This patch introduces copy/paste support of images. This allows
for pasting images to richly editable field and text fields
(URL otherwise).

This patch is a tizen_src part of the implementation (Webkit is here
http://165.213.202.130/gerrit/#/c/95868/), which uses image markup to
generate all necessary clipboard data.

You can check it by selecting "copy image" on html5 "badge" image
and then pasting it into editable area on the following adress:
http://html5demos.com/contenteditable

Original author: Lukasz Bialek <l.bialek@samsung.com>
Based on patch: http://165.213.202.130/gerrit/#/c/92243/

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14453

Together with: I11136d8352014f1acca144a0c4416c3bcf432e0c

Reviewed by: a1.gomes, djmix.kim, g.czajkowski

Change-Id: I11126d8352044f1acca044a0c4416e3bcf412e0c
Signed-off-by: Pawel Forysiuk <p.forysiuk@samsung.com>

Fixed wrong behaviors with HW back button.

1. If IME is shown, context menu and handlers shouldn't be hidden by H/W back
key. Currently context menu and handlers are cleared regardless of IME state
becuase |SelectionControllerEfl::HideHandleAndContextMenu()| is called in
|RWHVEfl::ImeCompositionRangeChanged| which is called when IME is shown or
hidden. This patch removes it.

2. HW back key deletes selected text. |IMContextEfl::OnPreeditChanged| isn't
returned propery when buffer is empty.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15025
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15060

Reviewed by: a1.gomes, djmix.kim, g.czajkowski

Change-Id: If85f59f0121cf36159e4b5d0ea9adda6a9df78d6
Signed-off-by: Sanghyup Lee <sh53.lee@samsung.com>

Make EwkFaviconDatabase per EWebContext

EwkFaviconDatabase implementation is unsafe and does not allow
ewk_favicon_database_* api implementation.

Associate EwkFaviconDatabase with EWebContext class.

Original beta/m42_2311_t patch:
- http://165.213.202.130/gerrit/#/c/85277/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14805

Reviewed by: a.renevier, d.waslicki, g.czajkowski

Change-Id: I2df25d877bd958a8f3a8a3f3298069c2f4e62557
Signed-off-by: Marcin Niesluchowski <m.niesluchow@samsung.com>

Change evas gl option to option bits.

[Problem] efl version >= 1.13.2 deprecates mem optimize
environment flags.

[Solution] Add efl mem optimize flags to option bit.
Environment variables for evas surface are not supported
from efl 1.13.2. Environment variables are supported only
for evas engine. With mem optimize option for evas surface,
intermediate buffers created by evas are generated on demand.
16MB 3D memory is reduced.
Original commit: http://165.213.202.130/gerrit/#/c/77299/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14879

Reviewed by: a1.gomes, d.waslicki, dhyuna.ko, hyuki.kim, prashant.n, sns.park

Change-Id: I1c5ca9dfdd1ee8c4cc7cf27f14831112b8d26c1d
Signed-off-by: Aron Kim <aron.kim@samsung.com>

Reimplement range and page scale functionality

minimumPageScaleFactor and maximumPageScaleFactor were removed from
blink::WebView by upstream. Those values were cached by
RenderViewObserverEfl::DidUpdateLayout() to:

1. Prevent user from setting a new scale which is out of the limits.
We don't need to worry about this. Blink will do that for us.

2. Get current scale limits.
Currently it's error prone. For different page (based on their
content width and auto fit calculated scale), call to
ewk_view_scale_range_get() may return different values.
If such functionality is really needed, it should be implemented
using smart callbacks.
Now, ewk_view_scale_range_get() returns default values from
content::WebPreferences.

This commit mostly gets rid of caching scale limits as they are
useless now.

Restore RenderViewObserverEfl::DidChangePageScaleFactor and cache
page scale once it was validated and successfully set by Blink.
This gives us a correct value on Browser side which is currently
using by engine.

Additionally, implemented ubrowser's slider range by using
ewk_view_scale_range_get().

Since the page scale is set asynchronously this patch fixes
utc_blink_ewk_view_scale_get_func.cpp as well.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12080

Reviewed by: a.renevier, a1.gomes, djmix.kim

Change-Id: I62668f2f0f17a59844022dc0df273576b8dfa4f5
Signed-off-by: Grzegorz Czajkowski <g.czajkowski@samsung.com>

Revert "Removal of appending mobile specific flags to TV"

This reverts commit c5c83cabfe40a882b1fcf6765243506640e0f414.

kUseMobileViewportStyle is not being passed to Renderer for Note4 resulting in
not collecting view port style which defines layout width (980px) for desktop
sites. Without it, desktop sites (google.com/wikipedia.org) have wrong layout.

Secondly, we shouldn't propagate deskotop's mobile flags to TIZEN builds at all.
Not just subset of them. If they are specific for some target they should be
controlled within command_line_efl.cc.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14816
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15031

Reviewed by: a1.gomes, sns.park

Change-Id: If383ae25aea687899949318e55798f5a60aa88bc
Signed-off-by: Grzegorz Czajkowski <g.czajkowski@samsung.com>

fixes crash in utc_blink_ewk_view_javascript_confirm_reply_func test

When calling ewk_view_javascript_confirm_reply, the close dialog
callback is executed. But if returning false from
ewk_view_javascript_confirm_callback_set, the close dialog callback is
also executed.

So, we need to return EINA_TRUE from confirmCallback; otherwise, the
close dialog callback is executed twice, which leads to a crash.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=6980

Reviewed by: a1.gomes, g.czajkowski

Change-Id: I7a7300161a52e5180ae52f82301d478d203a2cdd
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>

Fix issue in context creation for ewkunittests.

[Problem]
ewkunittests fails with error:
"evgl_api_ext_init() glGetString returned NULL!"

[Solution]
Save and Restore GLContext after using surfaceless extension.
Implemented in commit[1], enable in commit[2] for desktop.
commit[1]: http://165.213.202.130/gerrit/#/c/93813/
commit[2]: http://165.213.202.130/gerrit/#/c/96506/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14682

Reviewed by: a1.gomes, sns.park

Change-Id: I67b886b83be8d1af4a3f32cbfff392b3634f3cf1
Signed-off-by: venu.musham <venu.musham@samsung.com>

[MM][WebRTC] Fix crash on WebRTC connection with S/W codec

Since M43, GetSupportedProfiles also reflected to VideoDecoder.
This patch is to let gpu_info_collector_efl also collect
decoder accelerator informations.

Also let TizenVideoDecodeAccelerator report supported profiles
which required by GpuVideoDecodeAccelerator.

Bug: http://165.213.149.170/jira/browse/TSAM-594

Reviewed by: sm.venugopal, sns.park

Change-Id: I0df6fdaaa1f7405d095f85a7617e0fe0ca6a7ebb
Signed-off-by: msu.koo <msu.koo@samsung.com>

Exit selection mode upon BACK HW press

Hiding selection handles in no enough in order
to exit selection mode upon BACK HW button.
On renderer side, selection will still be active, and
the selection highlight will be still drawn.

Patch fixes this by requesting a "unselecting" text
on the renderer side.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15083

Reviewed by: a.renevier, djmix.kim

Change-Id: I430c9338f9895ee90a2ce8dc117b170af6c93221
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>

Change class members names according to chromium-efl coding style

Several _Ewk_Policy_Decision class members names are typed in camel case style
instead of using underscores

Reviewed by: a1.gomes, d.waslicki, djmix.kim

Change-Id: I19b7541d90aacddd3a863692db8fb4fd7cfdf25a
Signed-off-by: Piotr Krysiewicz <p.krysiewicz@samsung.com>

Add HW Back Key support for select lists and date/color pickers

Previously, select lists and data/color pickers did not react to HW
Back Key for platforms other than MOBILE. This patch adds necessary
callback and introduces expected action for ESC key on TV and
Desktop platforms.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14840

Reviewed by: a1.gomes, d.waslicki, djmix.kim, msu.koo

Change-Id: I2d11d5a8a87cb62a003c2c36cf04803b55a765ff
Signed-off-by: Lukasz Bialek <l.bialek@samsung.com>

fixup! Remove an erroneous scroll callback calls from ScrollDetector::OnChangeScrollOffset.

utc_blink_ewk_view_scroll_by_func.cpp was deleted, but not removed from
ewk-tests.gypi

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=13283
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14702

Reviewed by: a.renevier, a1.gomes, djmix.kim, sns.park

Change-Id: If83d681759576017a31e84752e9b70b76a833779

Add g.czajkowski into S_TRUNK_OWNERS file

Change-Id: I9c5cd01fdb7ff7f702806606e7e7bf6f76bfb967
Signed-off-by: Dongwoo Joshua Im <dw.im@samsung.com>

fixup! [MM][MSE][CAPI] Implement MSE with CAPI port.

CAPI returning internal errors through error callback function.
Updated error handling code to handle CAPI errors.

CAPI player state is updated based on buffer status callback.

Shared memory is wrapped to mediapacket. So, memcpy of input buffer is
reduced.

It fixs minor style violations, which is applied in beta/m42_2311_t.

Reference: http://165.213.202.130/gerrit/#/c/95263/

Reviewed by: msu.koo, sm.venugopal, sns.park

Change-Id: I9a44300f4ca76fe26e44f3f5a85edd0087cc5fb1
Signed-off-by: k2.nagaraju <k2.nagaraju@samsung.com>

Implement ewk_context_favicon_database_directory_set()

There is no implementation of ewk_context_favicon_database_directory_set()
api. This patch adds missing implementation. It will set directory path
to argument passed and filename to default one.

Original beta/m42_2311_t patch:
- http://165.213.202.130/gerrit/#/c/85276/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14792

Reviewed by: a.renevier, d.waslicki, djmix.kim, msu.koo, sns.park

Change-Id: Ic525ba7e0c9e39e8e91ab480cc5852f5d530a363
Signed-off-by: Marcin Niesluchowski <m.niesluchow@samsung.com>

Add private browsing, incognito, exit and fix style in ubrowser

Added private browsing, new incognito window and exit options and made
pointers style consistent.
This patch is based on the following patch
http://165.213.202.130/gerrit/#/c/87196/
made by Daniel Waślicki

Reviewed by: a.renevier, a1.gomes, d.waslicki, djmix.kim, g.czajkowski

Change-Id: I57904917670bb8f20b8426cdf0d9fda603c9c797
Signed-off-by: Tomasz Czekala <t.czekala@samsung.com>

Change favicon database's default path

Different users may have problem accessing default database path due to
lack of sufficient rights as current default database's path is same for
whole system regardless of security context.

Put default database path in user home directory making it user dependent.

Original beta/m42_2311_t patch:
- http://165.213.202.130/gerrit/#/c/85272/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14799

Reviewed by: a.renevier, a1.gomes, d.waslicki, m.roj, sns.park

Change-Id: Ie218d37b238575c65827871fbfe6a23da8ea4d4a
Signed-off-by: Marcin Niesluchowski <m.niesluchow@samsung.com>

Fix utc_blink_cb_popup_blocked

Patch executes Javascript strings with non-null user callbacks
with FRH::ExecuteJavaScriptWithUserGestureForTests rather than
FRH::ExecuteJavaScriptForTests.

Former allows window opening.

Patch also fixes the test utc_blink_cb_popup_blocked itself. First part
of the test relies on the popup opening setting being disabled, so
patch ensures it by explicitly set it to FALSE. Second part of the test,
it assumes that popup opening is enabled, so we set it explicitly to
TRUE. Last, after executing the tests, patch sets the setting value to
its original status (see changes in PostSetUp and PreTearDown).

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=13056

Reviewed by: a.renevier, djmix.kim

Change-Id: I2c2218406901735007435251e0549570b9d031bf
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>

fixup! [M47_2526] Chromium upversion to m47_2526 branch

Debug builds of ubrowser assert at startup with the
following backtrace:

#0 0x7f0b4052cfe9 base::debug::StackTrace::StackTrace()
#1 0x7f0b4056e8e3 logging::LogMessage::~LogMessage()
#2 0x7f0b4051c7c3 base::AtExitManager::RegisterTask()
#3 0x7f0b41507f77 ui::DeviceDataManager::CreateInstance()
#4 0x7f0b40502f0b ui::OzonePlatform::InitializeForUI()
#5 0x7f0b3ff5c934 EwkGlobalData::GetInstance()
#6 0x7f0b3ff29390 EWebContext::EWebContext()
#7 0x7f0b3ff29217 EWebContext::EWebContext()
#8 0x7f0b3ffd5013 Ewk_Context::Ewk_Context()
#9 0x7f0b3ffd4e55 Ewk_Context::Create()
#10 0x7f0b3ffd4cb6 Ewk_Context::DefaultContext()
#11 0x7f0b3ffdf188 ewk_context_default_get
#12 0x000000407e04 Browser::Browser()
#13 0x00000040c61b app_create()
#14 0x00000040c935 main
#15 0x7f0b3d874ec5 __libc_start_main
#16 0x000000407b19 <unknown>

This happens because OzonePlatform::InitializeForUI
needs to be called prior to BrowserMainRunner::Initialize (ok)
but past ContentMainRunner::Initialize (not ok).

ContentMainRunner::Initialize initializes exit_maneger at line 525.

Note that for ports like AURA, OzonePlatform::InitializeForUI
is called as part of BrowserMainRunner::Initialize itself with
the following (pseudo) backtrace:

-BrowserMainRunner::Initialize
-BrowserMainLoop::InitializeToolkit
-aura::Env::CreateInstance
-aura::Env::Init
-ui::OzonePlatform::InitializeForUI();

For EFL, it has to be called manually, to we place the call point
prior to calling BrowserMainRunner::Initialize.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14494

Reviewed by: a.renevier, sns.park

Change-Id: I68146a820e44fc21df0716ad7c6e90290616096d
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>

Implement ::OnFirstUserGestureObserved and ::HideRequestAutocompleteDialog

Patch implements AutofillClientEfl's OnFirstUserGestureObserved
and HideRequestAutocompleteDialog methods by minic'ing
chrome's implementation.

First method is based on the follow comment in WebViewImpl.cpp:

"On the first input event since page load, |notifier| instructs the
autofill client to unblock values of password input fields of any forms
on the page. There is a single input event, GestureTap, which can both
be the first event after page load, and cause a form submission. In that
case, the form submission happens before the autofill client is told
to unblock the password values, and so the password values are not
submitted. To avoid that, GestureTap is handled explicitly".

Second hides an autocomplete dialog when requested. This happens on
page loads while the autocomplete is shown, etc.

As a consequence of these two methods not being unimplemented,
EWK unit tests are much less verbose too.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14553

Reviewed by: a.renevier, djmix.kim, g.czajkowski, sns.park

Change-Id: I43e3b577435e240b52e67e1ef44112042b38f76a
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>

fixup! Implement ewk_settings_auto_fitting_set

chromium-efl has a mechanism to inform the embedding
app about a blocked window opening. It is the "popup,blocked"
smart signal.
In [1], we removed the EWK specific preference that used to
control window opening by javascript, in favor of chromium's
WebPreferences::javascript_can_open_windows_automatically.

Problem is that this changed the way chromium-efl blocks popups.
When javascript_can_open_windows_automatically is set, it gets
bubbled down to Blink, and popup blocking happens in Blink level
(see third_party/WebKit/Source/core/frame/LocalDOMWindow.cpp,
LocalDOMWindow::open() method).
This makes it impossible to inform the embedding app about a blocked
popup.

Additionally, [1] also started to cache a RenderView instance
(see ContentRendererClientEfl::RenderViewCreated) in order to
query for javascript_can_open_windows_automatically.
This is crash-prone because it caches the latest RenderView
instance created. If the for instance window associated to the
RenderView gets closed, ContentRendererClientEfl will hold a dangling
pointer, and crash upon accessing it.

Patch fixes this by reimplementing the way we control window
opening by javascript. Now
WebPreferences::javascript_can_open_windows_automatically (the chromium
preference) is always ON, so new window creation request messages
always reach the browser process. From there, we can control it
according to the EWK specific setting, and emit the
"popup,blocked" smart signal if needed.

The following EWK unit test were crashing due to this:
- utc_blink_ewk_settings_scripts_can_open_windows_set
Tests were also changed based on the default value of
javascript_can_open_windows_automatically_ewk, which is true.

[1] http://165.213.202.130/gerrit/#/c/92971/

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14640

Reviewed by: a.renevier, djmix.kim, g.czajkowski

Change-Id: I3c5e45961265d78c0832387586c4f2aa103f398a
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>

fixup! [MM][MSE][CAPI] Implement MSE with CAPI port.

Previously, Updating the seeking states was fulfilled asynchronously,
but this delayed state update can harm the state handling of seeking.

This patch is to fix updating the seeking state synchronously.

Original patch: http://suprem.sec.samsung.net/gerrit/#/c/52013/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14835

Reviewed by: sm.venugopal, sns.park

Change-Id: I48187edfdd162e772820eb77ab12fbffc7271b2b
Signed-off-by: umeshk.patel <umeshk.patel@samsung.com>

[MM][ME] HTMLMediaElement error should be set properly.

Proper error should be raised based on the error returned by CAPI
player.

Original Patch: http://suprem.sec.samsung.net/gerrit/#/c/51693/

Bug: http://168.219.209.56/jira/browse/TNEF-6215
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14835

Reviewed by: msu.koo, sm.venugopal, sns.park

Change-Id: I6d3ebefb03fd83f566f19657adced9989a5bb31f
Signed-off-by: sm.venugopal <sm.venugopal@samsung.com>

Fix memset on std::vector error in ubrowser initialization

In file ewk/ubrowser/main.cc in function main there was memset function
used on an AppData struct, which contains a std::vector member. Memset
on a std::vector can destroy its private data and lead to unexpected
behavior.

Fixed by removing memset and zeroing other members of AppData struct
in its constructor.

This error was reported by cppcheck static analysis tool.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15002

Reviewed by: a.renevier, a1.gomes, d.waslicki, sns.park

Change-Id: Ibaa89e4fa97e88f094f94beaac3318122c61b16b
Signed-off-by: Grzegorz Ludwikowski <g.ludwikowsk@samsung.com>

Fix uninitialized variable use in SelectionControllerEfl

In file chromium_impl/content/browser/selection/selection_controller_efl.cc
in member function SelectionControllerEfl::ShowContextMenu() variables
blinkX and blinkY are intialized conditionally depending on the result
of web_contents_.GetRenderWidgetHostView(). Later the variables are used
anyway, even if uninitialized. This causes garbage values to be passed
to WebContentsViewEfl::ShowContextMenu member function with no
indication of failure.

Fixed by conditionally calling WebContentsViewEfl::ShowContextMenu
only if web_contents_.GetRenderWidgetHostView() succeeds.

This error was reported by cppcheck static analysis tool.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15001

Reviewed by: a1.gomes, d.waslicki, sns.park

Change-Id: I36ca59bf56d5c9737a03b92a662c8b074797f7f9
Signed-off-by: Grzegorz Ludwikowski <g.ludwikowsk@samsung.com>

Enable "Copy link address" context menu option for Wayland

This features was disabled on Tizen 3.0 because Copy/Paste was
not implemented for Wayland. However, after
http://165.213.202.130/gerrit/#/c/91062/ was merged, these
guards can be removed now.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14650

Reviewed by: a.renevier, djmix.kim

Change-Id: I9a291cdbe10c36f2e88a4fb7d43bda5a1394d58e
Signed-off-by: p.pajak <p.pajak@samsung.com>

Define scale limits for "ubrowser --mobile"

OS_TIZEN_MOBILE and ANDROID builds set minimum scale to 0.25
to make auto fitting work [1]. The same should have been done
for ubrowser --mobile.

[1] http://165.213.202.130/gerrit/#/c/95666/

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14929

Reviewed by: a1.gomes, djmix.kim

Change-Id: I0f0da8797c5508bb5b3f8f1fa2ce34f23193d39d
Signed-off-by: Grzegorz Czajkowski <g.czajkowski@samsung.com>

Fix utc_blink_cb_magnifier_hide.cpp unittest

The test in case (see title) flips selection mode on and off
by calling EWebView::HandleTextSelection{Down,Up}, which bubbles
down to SelectionControllerEfl::TextSelection{Down,Up} respectively.

When SelectionControlleEfl::TextSelectionDown is executed it
needs to "emulate" a long press event, so that the selection
machinary works. Also, when ::TextSelectionUp is called, it relies
on ::long_mouse_press_ being true.

Bug http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=13056

Reviewed by: a.renevier, djmix.kim

Change-Id: I76353ba95047ae654826c97936b0369d8f809e7d
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>

Change type of path member of FaviconDatabasePrivate class

path member of FaviconDatabasePrivate type changed from
std::string to base::FilePath making it less error-prone.
Proper checks has been added for that type.

Original beta/m42_2311_t patches:
- http://165.213.202.130/gerrit/#/c/85274/
- http://165.213.202.130/gerrit/#/c/85275/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14785

Reviewed by: d.waslicki, m.roj, msu.koo

Change-Id: Iec556b76f6eaade557b07d55fc42651c3f0d7f93
Signed-off-by: Marcin Niesluchowski <m.niesluchow@samsung.com>

fixup! Disambiguate ShowContextMenu implementation.

In case one right clicks with say a bluetooth mouse or on
on desktop builds, context menu should be shown right way.

Patch fixes the broken assuption that
WebContentsViewDelegateEwk::ShowContextMenu should always enter
selection mode. This is only valid in case of touch-based event
source, i.e. on Mobile or ubrowser --mobile.

Patch also fixes utc_blink_cb_contextmenu_allowed.cpp.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14304

Reviewed by: a.renevier, a1.gomes, djmix.kim

Change-Id: Ib409bb5d9ec2492cf622c43eddd710eebb4aabfc
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>

Enable ContentSecurityPolicy

This patch removes flags of 'EWK_BRINGUP' related to content security policy
and matches enumeration value against the one from Blink.

Reference: http://165.213.202.130/gerrit/#/c/87100/

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14791

Reviewed by: a1.gomes, jh11.kwon, sns.park

Change-Id: I620a57118d0ac26416094b0db4d7c6c8aeef53ad
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>

Set 'Policy Decision' to 'Download' when resource mime type is not supported

In webkit2-efl we set Policy Decision to DOWNLOAD when response mime type
is not supported whereas in chromium-efl we do not do that. This causes,
among others, the EWK API ewk_policy_decision_type_get always returns
EWK_POLICY_DECISION_USE. This incorrect behaviour was noticed by browser team
when porting a reference browser form Tizen 2.4 to Tizen 3.0.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14844

Reviewed by: a.renevier, a1.gomes, d.waslicki, sns.park

Change-Id: I90c2d6a5ce82a102f19f25fc32ac038137f98b40
Signed-off-by: Piotr Krysiewicz <p.krysiewicz@samsung.com>

Change session cookie mode

Using RESTORED_SESSION_COOKIES mode suits better WRT cases.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14212

Original beta/m42 patch:
http://165.213.202.130/gerrit/#/c/86852/

Reviewed by: a.renevier, a1.gomes, d.waslicki, m.roj, sns.park

Change-Id: I1182100761c23b802e91921a4e4125d64b57fbec
Signed-off-by: Marcin Kędzierski <m.kedzierski@samsung.com>

Initialize ScopedRestoreNonOwnedEGLContext instance.

ScopedRestoreNonOwnedEGLContext initialization was commented
during M47 bringup task, there was crash due to uninitialized
shared glcontext.

Since the initialization is fixed in commit[1], this patch brings
back the commented code in M47 bringup.
[1] http://165.213.202.130/gerrit/#/c/94641/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14682

Reviewed by: a1.gomes, sns.park

Change-Id: Iba00908a964398b73184180f4f2903551c1ada53
Signed-off-by: venu.musham <venu.musham@samsung.com>

[EWK] "title,changed" callback invoked on back and forward actions

Chromium by default doesn't inform view delegates when a page is
reloaded as a result of back and forward actions (the title doesn't
change indeed). This patch modifies this behaviour to fulfill EWK API
requirements.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14206
Together with: I0c6d2110836b6122e51311e3b6f3cb56dcf3dd79

Original beta/m42 patch:
http://165.213.202.130/gerrit/#/c/85470/

Reviewed by: a.renevier, a1.gomes, djmix.kim, j.majnert, sns.park

Change-Id: I039d6ea43939c38ad1550f78475b20a12a8c5b08
Signed-off-by: Wojciech Bielawski <w.bielawski@samsung.com>

fixup! Fix ewk_view_visibility_set.

Remove unused functions after [1].

[1] http://165.213.202.130/gerrit/#/c/94980/

Reviewed by: a1.gomes, djmix.kim, sns.park

Change-Id: I36f5ddc09f9c305893fb9ffb702a3b92e8af503f
Signed-off-by: Sanghyup Lee <sh53.lee@samsung.com>

Clear contexts on EWebView exit

When user turns on a private browsing mode, we create a new context
and save an old one. If user wants to close a browser under these
circumstance (with private browsing turned on) we have two contexts open
where the default one is unused (default context should be removed
as the last one). In order to work properly we need to delete a temporarily
created context for a private browsing purpose and bring back the default
one.

Original beta/m42 patch:
http://165.213.202.130/gerrit/#/c/87523/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=13557
Reviewed by: a1.gomes, djmix.kim

Change-Id: I6dd1391f9bbbb800f9149a94bf296a4ae7642a22
Signed-off-by: Daniel Waślicki <d.waslicki@samsung.com>

Fixes utc_blink_ewk_view_orientation_send_func unittest

In order to fix utc_blink_ewk_view_orientation_send, we need to use the
new screen orientation API instead of window orientation.

Also, an orientation of -90 is transformed into a 270 orientation
by ewk. So, we need to take that into account

In addition, in order to make the test more thorough, we add a test to
check that an invalid orientation value isn't propagated.

Lastly, we remove the files orientation_test1.html,
orientation_test2.html and orientation_test3.html which were unused.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=6980

Reviewed by: a1.gomes, djmix.kim

Change-Id: Iae76f08151a753272877d9839c3660842f0b161b
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>

Implement "Copy link text" context menu option

According to Tizen3.0_mobile_Web_Browser_Browser_UI_Guideline, there should
be a "Copy link address" option when user opens the context menu for a link
by doing a long press. This patch implements this option.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14650

Reviewed by: a.renevier, a1.gomes, djmix.kim, sns.park

Change-Id: I2d51d5a8a87cb32a003c2c33cf04803b55a795ff
Signed-off-by: p.pajak <p.pajak@samsung.com>

Remove FaviconService class

FaviconService class is unnecessary proxy of FaviconDatabase singleton
object due to bad implementation.

Remove FaviconService class and use FaviconDatabase api directly. Following
public member functions of FaviconDatabase have been changed to open
database if not opened:
* GetFaviconURLForPageURL
* GetBitmapForPageURL
* GetBitmapForFaviconURL
* SetFaviconURLForPageURL
* SetBitmapForFaviconURL
* ExistsForPageURL
* ExistsForFaviconURL
* Clear

Original beta/m42_2311_t patch:
- http://165.213.202.130/gerrit/#/c/85273/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14820

Reviewed by: a.renevier, a1.gomes, d.waslicki, sns.park

Change-Id: I49a8eb761a26caa16d526fa377713ed6cbf22f68
Signed-off-by: Marcin Niesluchowski <m.niesluchow@samsung.com>

Use ExecuteJavaScriptForTests instead of ExecuteJavaScript

This ensures that Javascript can be executed (and also, fixes a crash in
unittest utc_blink_ewk_view_script_execute_func.cpp)

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14494

Reviewed by: a1.gomes, sns.park

Change-Id: Ifcdefbacbf796d0eac9455aa1d6f67c017e7058b

Fill in missing copyrights.

Fill in missing parts of the copyrights in the headers of source files.

Original beta/m42 patch: http://165.213.202.130/gerrit/#/c/94645/

Reviewed by: a.renevier, a1.gomes, djmix.kim, j.majnert

Change-Id: Id7360aee2df3a66740818c37bfc10aec99108339
Signed-off-by: Andrzej Badowski <a.badowski@samsung.com>

[MM] Utilize |WebMediaPlayerParams| while creating media engine instance

|WebMediaPlayerParams| is passed on as argument to be in sync with
upstream code.

Together with: I6e6e327c9a0ba04986f157582321de5ad01e32aa

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14649

Reviewed by: a1.gomes, sns.park

Change-Id: I6d93e70974f2ffa9f45925cbd3ddde1bcc4a1706
Signed-off-by: sm.venugopal <sm.venugopal@samsung.com>

Re-enable option for message pump optimization

It was disabled by [1] because of regression in touch/click behaviour
in WCS test pages like gesture.html and select_picker_test.html.

1.  Fix the implementation of timer delay code. The code previously tried to
micro-manage timer delays. The intention of those delays was to make code
more robust. Unfortunately it would lead to perpetually postponing one task
instead of spawning new ones. Drop the code adding arbitrary delays.

2.  Make "limit-memory-allocation-in-schedule-delayed-work" flag be enabled
by default again.

[1] http://165.213.202.130/gerrit/#/c/83920/

Original beta/m42 patch: http://165.213.202.130/gerrit/#/c/84369/

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14786

Reviewed by: a1.gomes, msu.koo, sns.park

Change-Id: I1be8b12e84d6a1678e6ab070cf79868076bae5c4
Signed-off-by: Pawel Forysiuk <p.forysiuk@samsung.com>

'Save image' and 'Save link' doesn't work on context menu

[Problem]  Nothing happened after invoke 'Save image' and 'Save link'
commands from context menu. No message, no file is saved.
[Cause]    Paths for downloads were set wrong on Tizen v3.0.
[Solution] Set proper paths for given platform.

From now on, 'libtzplatform-config' library will be used to get
proper patch on Tizen v3.0.

Original beta/m42 patch: http://165.213.202.130/gerrit/#/c/84031/

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=13984

Reviewed by: a.renevier, a1.gomes, sns.park

Change-Id: I00f9fcaff8e2f0786e9913a92d25e871f91a590b
Signed-off-by: p.niemirski <p.niemirski@samsung.com>

Show or hide the elm_naviframe when is called ewk_view_visibility_set()

If ewk_view_visibility_set() is called, white screen is shown in mobile.
Since background of elm_naviframe is white, white screen is appear.
When ewk_view_visiblity_set() is called,
elm_naviframe needs shown or hidden.

This patch brought in M42 (http://165.213.202.130/gerrit/#/c/88300/).

Reviewed by: a1.gomes, djmix.kim, msu.koo

Change-Id: I2ae719a8d563c7732a9ba3769e22f1780e8613fb
Signed-off-by: ByungJun Kim <bj1987.kim@samsung.com>

Fix path of system folder in spec file

[issue]
The folder '%TZ_SYS_RO_ICONS' and '%TZ_SYS_RO_PACKAGE' have been created
on root path like below.

>> root@localhost:/# ls /
>> %TZ_SYS_RO_ICONS    boot  home        media  proc  sbin  tmp
>> %TZ_SYS_RO_PACKAGE  dev   lib         mnt    root  srv   usr
>> bin                 etc   lost+found  opt    run   sys   var

[resolve]
To make the '%TZ_SYS_' definitions properly work,
a package 'libtzplatform-config' needs to be installed.

The package provides the definitions via following file.
>> /PATH/TO/BUILD-ROOTS/scratch.armv7l.0/etc/tizen-platform.conf

Note that the definitions are only valid from tizen v3.0,
supporting multi-user mode.

Also, unused definition '_dbusservicedir' and '_systemduserservicedir' are removed.

original patch: http://165.213.202.130/gerrit/#/c/85251/

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14788

Reviewed by: sns.park

Change-Id: Ia8e0dece9d4a7031c35447bfa4c19165436ae406
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>

Pass "enable-viewport" flag for "ubrowser --mobile"

Mobile build by default turns on WebPreferences::viewport_meta_enabled
This results in setting WebPreferences::viewport_enabled as well:

RenderViewHostImpl::ComputeWebkitPrefs() {
...
prefs.viewport_enabled =
command_line.HasSwitch(switches::kEnableViewport) ||
prefs.viewport_meta_enabled;
}

Since viewport_meta_enabled(true) is set on Renderer side via
ApplyCustomSettings() for ubrowser --mobile, earlier call to
ComputeWebkitPrefs() always sets viewport_enabled to false.

This patch passes "enable-viewport" flag (switches::kEnableViewport)
for ubrowser to make it similar to mobile builds.

This results in different path code while determining layout size
(settings->viewportEnabled() is widely checked on Blink side),
also disabling zooming after tap and showing disambiguation popup for
mobile sites.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14921

Reviewed by: a1.gomes, sns.park

Change-Id: I3be1b4381c627c485e0db5254b4d796c5cd454e1
Signed-off-by: Grzegorz Czajkowski <g.czajkowski@samsung.com>

Enable ewk_context_favicon_database_get() also for mobile

ewk_context_favicon_database_get() for mobile profile is not implemented.
Api exists but is defined only for tv profile.

Remove tv profile ifdefs making it work regardless build profile.

Original beta/m42_2311_t patch:
- http://165.213.202.130/gerrit/#/c/85271/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14784

Reviewed by: sns.park

Change-Id: Ie3931c5ca7a523162556882d36b654758034d32b
Signed-off-by: Marcin Niesluchowski <m.niesluchow@samsung.com>

Fixing wrong evas_object hierarchy in eweb_view.

Previously, using |evas_object_below_get| to get the parent webview
cause warning and required addition of some defensive codes.

To prevent from warning, we should use evas_object_above_get() as
evas_object_below_get() has been misused.
This patch refactored to get the parent webView
from "evas_object_above_get", which is guided by
(https://docs.enlightenment.org/stable/efl/group__Evas__Font__Group.html)

Before the fix, evas_object_ isn't part of the EFL/Elementary layout
tree. So we needed to get value of width and height from ecore_evas.
With this patch, evas_object_ is part of the elementary layout tree.
This defensive code is no longer needed.

Original m42 patch:
- http://165.213.202.130/gerrit/#/c/87501/ .

Change-Id: I677dbde037b798773552de8473a62fdbb8b344ed
Signed-off-by: ByungJun Kim <bj1987.kim@samsung.com>

Long press on empty fields should show context menu if needed

If one long-presses an empty input field, we enter selection mode:
magnifier is show, etc.
This is a bad UX, and does not match other mobile browsers.

Patch changes that to show context menu right way, with clipboard
options if applicable. This matches other mobile browsers: firefox
mobile, chrome and s-browser.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14910

Change-Id: I002f57a1224431a915740d87e7694db358153493
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>

Implement TitleWasSet() for EWK

On Chromium Upstream, NOTIFICATION_WEB_CONTENTS_DISCONNECTED
and NotificationService was deprecated([1], [2]) and reflected
to ours([3]). Because of this ewk_back_forward_list_item_title_get()
has a problem that return empty.

In this patch, TitleWasSet() is introduced to enable
|ewk_back_forward_list_item_title_get()| from Chromium's guideline([2]).

[1] http://code.google.com/p/chromium/issues/detail?id=170921
[2] https://codereview.chromium.org/1242953004/patch/1/10004
[3] http://165.213.202.130/gerrit/#/c/92076

Change-Id: I44b4fe8a05f219b3efc7217eb242643c9b61876a
Signed-off-by: sihoons.yang <sihoons.yang@samsung.com>

Do not zoom-to-focused-element when long pressing.

zoom-to-focused element should happen on tap.
Long press should not trigger it, as it gives us a bad experience.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14911

Change-Id: I2d20a6ba247330b7a92781042dffc0c205624edd
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>

Implement "Share link" context menu option for Tizen OS

According to Tizen3.0_mobile_Web_Browser_Browser_UI_Guideline, there should
be a "Share link" option when user opens the context menu for a link
by doing a long press. This patch implements this option for Tizen OS.

Note that the share option requires a share-panel application
which is currently absent on Tizen 3.0

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14650

Change-Id: I51e9a460a2fcb57d298f3521dfe40bcd1a4433a6
Signed-off-by: p.pajak <p.pajak@samsung.com>

Close javascript popup when EWebView is deleted.

If EWebView is deleted, it calls destructor of JavaScriptModalDialogEfl
but destructor does nothing.
Destructor needs to close JS popup.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14764

Change-Id: Ib66e6afd43a2665e07ba7588f5bbeeb63f73b187
Signed-off-by: ByungJun Kim <bj1987.kim@samsung.com>

Add popup callbacks for closing with Esc key

[Issue] http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14099
[Problem] Popups are not closed when Esc button is pressed
[Cause] Only Mobile branch supports callbacks for HW back button
[Solution] Add callbacks for Esc button and connect them with
           existing popup-closing code

Original beta/m42 patch:
http://165.213.202.130/gerrit/#/c/87930/

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14765

Change-Id: Ic7656ee4a034b95aa5ee71ac85c79711e57d15e7
Signed-off-by: Lukasz Bialek <l.bialek@samsung.com>

Refactored context menu controller

1. Fill _context_menu_listdata when is_text_selection_ is true
2. Prohibit out of array boundary indexing
3. removed redundant variable _appended_item_size

Original beta/m42 patch:
http://165.213.202.130/gerrit/#/c/85782/

Signed-off-by: Wojciech Bielawski <w.bielawski@samsung.com>
Change-Id: Ic53167c11ea06f73e589c3af399f3e431b53b530

Remove an erroneous scroll callback calls from ScrollDetector::OnChangeScrollOffset.

This fixes a scrolling issue/crash.

Since renderer process perform all scrolling operation,
we don't have to handle this events on browser side.
Furthermore browser process handles this event using
ScrollDetector object which emits EFL's smart callbacks,
but there are no events registered on this callback.

Original M42 patch: http://165.213.202.130/gerrit/#/c/82644/

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=13283
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14702

Change-Id: Id6fcf4120a0d396a3dbbbd6fb5c378666b0671b5
Signed-off-by: Sanghyup Lee <sh53.lee@samsung.com>

Wrongly computed string length in database initialization.

Compute proper string length and only if string is not null.

Original beta/m42_2311_t patch:
- http://165.213.202.130/gerrit/#/c/85780/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14777

Change-Id: I06a19c2f6e413709d7dbf8f5097ab1ee499b28c4
Signed-off-by: Wojciech Bielawski <w.bielawski@samsung.com>