platform/framework/web/chromium-efl.git
6 years ago[Tizen][M34-Merge] Fix screen size reporting.
Kamil Klimek [Tue, 25 Nov 2014 08:39:57 +0000 (09:39 +0100)]
[Tizen][M34-Merge] Fix screen size reporting.

The current EFL implementation of gfx::Screen interface is not entirely
correct. The main differences compared to Android and Aura ports are:
1. The screen size is reported in px, not in DIP.
2. Screen size is always reported in portrait mode.

As a direct result of the differences most mobile pages relying on CSS3
media queries will render incorrectly. They'll use style sheet version
designed for devices with larger screen devices that the device has.
One example of such page is http://slideshare.net.

This patch changes current gfx::Screen implementation to report proper
screen size values in DIP for both portrait and landscape orientations.

Original patch: http://165.213.202.130:8080/#/c/70582/
Original author: Piotr Tworek <p.tworek@samsung.com>
Original bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=6308

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=5594
Reviewed by: Antonio Gomes, Viatcheslav Ostapenko

Change-Id: I21a7baeaf0a9b05dab7d00d255cc611589e784bf
Signed-off-by: Kamil Klimek <k.klimek@partner.samsung.com>
6 years agoMark kangil.han as INACTIVE
Dongwoo Joshua Im [Mon, 8 Dec 2014 07:53:21 +0000 (16:53 +0900)]
Mark kangil.han as INACTIVE

Introduce 'INACTIVE' into the S_TRUNK_OWNERS,
and mark kangil.han as inactive.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=9643
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I3e01e9e028c06eb60e48e6e22476d39de622835a
Signed-off-by: Dongwoo Joshua Im <dw.im@samsung.com>
6 years agofixup! Refactor disambiguation popup
KANGYONG PARK [Fri, 5 Dec 2014 08:01:01 +0000 (17:01 +0900)]
fixup! Refactor disambiguation popup

Disable disambiguation popup in DTV.
It is unnecessary feature in DTV, and disambiguation
popup is not supported in DTV browser base on Webkit.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=9647
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I2814d80df2d96bc874bca105f1216d56a5bd0d30
Signed-off-by: KANGYONG PARK <kaka.park@samsung.com>
6 years ago[M34-Merge] NULL or empty user agent string was not handled properly.
Piotr Grad [Sat, 13 Sep 2014 06:02:24 +0000 (08:02 +0200)]
[M34-Merge] NULL or empty user agent string was not handled properly.

Used ContentClientEfl::GetUserAgent as it returns default user agent for tizen.
Removed EwebView::overridden_user_agent_ as ewk api should return current UA.
Removed invalid use of memory form UA getters.

The fix changes EWebView::SetUserAgent to only use
WebContents::SetUserAgentOverride API. This is what desktop chrome uses
for UA override purposes.

The patch changes the logic so that "Mobile" is always added to product
part of UA string when --use-mobile-user-agent is specified. No ifdefs
required. This way the feature can be controlled entirely from one
place, command_line_efl.cc. The flag is always specified by default for
OS_TIZEN targets, but not for desktop. On desktop the user can simply
pass it when needed, ex:
./out.x64/Debug/efl_webview_app --use-mobile-user-agent

This is a combination of 5 commits:

NULL or empty user agent string was not handled properly.
http://165.213.202.130:8080/#/c/69394/
NULL or empty user agent string was not handled properly.
http://165.213.202.130:8080/#/c/69395/
fixup! NULL or empty user agent string was not handled properly.
http://165.213.202.130:8080/#/c/69475/
by Piotr Grad, and

fixup! NULL or empty user agent string was not handled properly.
http://165.213.202.130:8080/#/c/70312/
Cleanup --use-mobile-user-agent flag handling logic.
http://165.213.202.130:8080/#/c/70313/
by Piotr Tworek.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8270
Reviewed by: Kamil Klimek, Laszlo Gombos

Change-Id: Iba6d229472afb5dd0e483f8888fea5c4659cab40
Signed-off-by: Michał Pakuła vel Rutka <m.pakula@samsung.com>
6 years agoSet allow_running_insecure_content to true for Tizen
Zhang-hoon Oh [Tue, 2 Dec 2014 10:39:39 +0000 (19:39 +0900)]
Set allow_running_insecure_content to true for Tizen

[Problem]
Any MSE video from http://youtube.com/tv/ is not played.

[Solution]
This patch sets WebKit Preference |allow_running_insecure_content|
to true (in Tizen's content browser client implementation)
because some content providers such as YouTube use plain http
requests to retrieve media data chunks while running in a https
page.

[Future Work]
This pref should be disabled once all the content providers
are no longer doing that.

[Reference]
Please note that, in the open source chromecast content embedder,
this WebKit Preference is set to true, also. Please refer to:
- https://codereview.chromium.org/223143003

[Test]
To test:
- Load http://youtube.com/tv/
- Play any video.
- Look for the JavaScript console log.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=9343
Reviewed by: Antonio Gomes, Min-Soo Koo, SeungSeop Park

Change-Id: Iea6385a53f1e6999991b32f3e8b38a9b8e0bb608
Signed-off-by: Zhang-hoon Oh <dennis.oh@samsung.com>
6 years ago[Hotfix][M40] Alternative to deprecated 'ViewMsg_OrientationChangeEvent'
Youngsoo Choi [Mon, 1 Dec 2014 06:32:45 +0000 (15:32 +0900)]
[Hotfix][M40] Alternative to deprecated 'ViewMsg_OrientationChangeEvent'

[Issue]
Note that, In M40, IPC message 'ViewMsg_OrientationChangeEvent' has
been deprecated, to use IPC message 'ViewMsg_Resize' instead.
IPC message 'ViewMsg_Resize' delivers current orientation information
and triggers orientation changed event on renderer side.

[Resolve]
To keep that way, followings are applied.

1) Deprecated 'ViewMsg_OrientationChangeEvent'
Sending IPC message 'ViewMsg_OrientationChangeEvent' or arbitrarily
triggering IPC message 'ViewMsg_Resize' is against M40 architecture.

2) Shipping orientation information
When device rotation is changed, the orientation information needs
to be set before IPC message 'ViewMsg_Resize' is sent.
The changes of 'mini_browser.c' guarantee of current orientation
information to be always contained within IPC message 'ViewMsg_Resize'.

3) New method 'SetOrientation' and 'GetOrientation'
Those two methods are needed to send the orientation information from
rotation callback to resized callback fired by platform.

4) EWK API issue
As of now, the name of EWK API 'ewk_view_orientation_send' is not
changed although it does not send orientation event to renderer,
for supporting EWK API compatibility.
This issue will be discussed further and the API needs to be refactored.

FIXME: http://107.108.218.239/bugzilla/show_bug.cgi?id=9514
The EWK API 'ewk_view_orientation_send' may need to be deprecated,
or can be changed to something like 'ewk_view_orientation_set' later
because in M40 there is no way to directly send orientation changed
event to renderer side any more, without IPC message 'ViewMsg_Resize'.

This patch meets following WCS test cases:
>> WCS-TC-97 (JS properties on launching time)
>> WCS-TC-98 (JS properties on rotation)

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=9254
Reviewed by: Antonio Gomes, Min-Soo Koo

Change-Id: If2b4e5af10998b4fecef48b2fe185fdaa66b16bf
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
6 years agoRevisit the owner list
Dongwoo Joshua Im [Mon, 1 Dec 2014 00:54:42 +0000 (09:54 +0900)]
Revisit the owner list

Revisit the owner list due to structural changes.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=9468
Reviewed by: Antonio Gomes, Kangil Han, SeungSeop Park, mohan reddy

Change-Id: Ia4e2a917fce5447bf21f70a46c104d27f2882b8f
Signed-off-by: Dongwoo Joshua Im <dw.im@samsung.com>
6 years agoAdd Samsung Electronics BSD LICENSE
Taeho Kim [Fri, 5 Dec 2014 02:08:02 +0000 (11:08 +0900)]
Add Samsung Electronics BSD LICENSE

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=9639
Reviewed by: Antonio Gomes, Laszlo Gombos, SeungSeop Park

Change-Id: Ia129ff04f9c15f0e1101f74cc02c0f3093910868
Signed-off-by: Taeho Kim <th81.kim@samsung.com>
6 years ago[Tizen][MM][Webrtc] Video Capture Port using CAPI
msu.koo [Wed, 12 Nov 2014 10:06:02 +0000 (15:36 +0530)]
[Tizen][MM][Webrtc] Video Capture Port using CAPI

This commit is to enable the video capture port on Tizen.

- Implemented VideoCaptureDeviceFactoryTizen.
- Implemented VideoCaptureDeviceTizen using Tizen CAPI.

Together with:I9c48580455d73821b11ae0b358b43544d24bdf1e

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=7942
Reviewed by: Laszlo Gombos, Min-Soo Koo, Viatcheslav Ostapenko

Change-Id: I6883a136e2ab253c31f0a0051bf108bfe598da8e
Signed-off-by: msu.koo <msu.koo@samsung.com>
6 years agoRemove build/patches/webrtc.diff
Antonio Gomes [Wed, 26 Nov 2014 20:38:01 +0000 (16:38 -0400)]
Remove build/patches/webrtc.diff

This is not needed after src/third_party/webrtc got fixed:

https://webrtc-codereview.appspot.com/27859004
"
Fix double backslashes in incoming_video_stream.cc
Originally uploaded in https://codereview.appspot.com/149160043/.
"

Reviewed by: Balazs Kelemen, SeungSeop Park
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=9215
Change-Id: I37ec2d3fba90d97262faf78b756b993e49ed67db

6 years agoAdd option --clean to sync_repos.sh
SeungSeop Park [Wed, 3 Dec 2014 01:24:12 +0000 (10:24 +0900)]
Add option --clean to sync_repos.sh

If this option is passed, sync_repos.sh will clean up all untracked files and
directories that are not listed in .gitigore file. However, if an untracked
diretory is managed by a different Git repository, it's not removed.
Note that the operation is applied for not only in s-* but chromium-efl repo.
The intended client for this option is, but not confined to, the buildbot.

WARNING: Be careful when using this option outside of buildbot context,
         since it can blow up your local changes.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=9584
Reviewed by: Antonio Gomes, Tomasz Olszak, Viatcheslav Ostapenko, mohan reddy

Change-Id: Ic2172925edd691aa3bbfc35fd035c10a0f4f2cea
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years ago[Tizen] Implement tizen specific battery status manager.
KANGYONG PARK [Wed, 26 Nov 2014 14:16:10 +0000 (23:16 +0900)]
[Tizen] Implement tizen specific battery status manager.

This is required to have working HTML5 battery API on tizen.
This commit implements to call battery status in device.
And the status will be updated in chromium, when it is changed in device.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8614
Reviewed by: Antonio Gomes, Min-Soo Koo, SeungSeop Park

Change-Id: Ie3130646107395abe7151977b3f05c6e78402dd3
Signed-off-by: KANGYONG PARK <kaka.park@samsung.com>
6 years agoRefactoring ewk_view_application_name_for_user_agent_get / set
Piotr Sieduszewski [Wed, 26 Nov 2014 18:02:47 +0000 (19:02 +0100)]
Refactoring ewk_view_application_name_for_user_agent_get / set

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=9327
Reviewed by: Antonio Gomes, Kamil Klimek

Change-Id: I55a8fe891c66d2cc0ee655dd07a71aefc5ecaff9
Signed-off-by: Piotr Sieduszewski <p.sieduszews@samsung.com>
6 years ago[Tizen] Implementation "zoom,finished"
KANGYONG PARK [Fri, 31 Oct 2014 02:40:04 +0000 (11:40 +0900)]
[Tizen] Implementation "zoom,finished"

Implementation "zoom,finished" callback.
It is related to pinchZoom.

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

The original author was p.sieduszews@samsung.com.
M34 patches are reorganized.
Reviewed by: Balazs Kelemen, SeungSeop Park

Change-Id: I446a481b7e4adc4eef312c28fd97072f6a33a27d
Signed-off-by: KANGYONG PARK <kaka.park@samsung.com>
6 years ago[M34-Merge] Implementation and TCs for ewk_settings_scripts_can_open_windows_* and...
Michal Poteralski [Thu, 6 Nov 2014 15:39:04 +0000 (16:39 +0100)]
[M34-Merge] Implementation and TCs for ewk_settings_scripts_can_open_windows_* and ewk_settings_scripts_window_open_*

Patch implements ewk_settings_scripts_can_open_windows_* and
and ewk_settings_scripts_window_open_* API based on webkit-efl implementation.

Implementation uses IPC so decision about opening window is made on
renderer side not browser side. I didn't add implementation in
ContentBrowserClientEfl::CanCreateWindow because this API is not called
on UI thread. It's called (by design) on IO thread and taking UI objects
(EWebView) outside UI thread is not safe. Also I didn't use
WebPreferences::javascript_can_open_windows_automatically because this
property is not used anywhere inside chromium code. Toggling this
variable has no effect.

Original commits:
http://165.213.202.130:8080/#/c/69592
http://165.213.202.130:8080/#/c/68833/
http://165.213.202.130:8080/#/c/69947/
(by Kamil Klimek <k.klimek@partner.samsung.com>)

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8248
Reviewed by: Kamil Klimek, SeungSeop Park

Change-Id: Ib0cbf7777c55562dfc30e8bb9ce6e847a3b40b96
Signed-off-by: Michal Poteralski <m.poteralski@samsung.com>
6 years agoImplement Chromium Theme for Tizen.
Viatcheslav Ostapenko [Wed, 26 Nov 2014 00:38:12 +0000 (19:38 -0500)]
Implement Chromium Theme for Tizen.

Original patch:
1) Patch for selectpicker(single type)
http://suprem.sec.samsung.net/gerrit/#/c/3801
2) Implement SelectPicker (multi type)
http://suprem.sec.samsung.net/gerrit/#/c/4007

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=4833
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: Id659a3d17a41e6d6cccd030c8a7f6de2402bd382
Signed-off-by: Viatcheslav Ostapenko <sl.ostapenko@samsung.com>
6 years agoDo not crash when using a non attached context
Arnaud Renevier [Wed, 26 Nov 2014 01:28:09 +0000 (17:28 -0800)]
Do not crash when using a non attached context

Since http://165.213.202.130:8080/#/c/69045/ , browser_context_ and
web_cache_manager_ are not set at EWebContext creation anymore. This
results in a crash if we call a method that uses those members just
after creation.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=9337
Reviewed by: Antonio Gomes, Viatcheslav Ostapenko

Change-Id: I10e49dc3a05d2a8fa9efdfab8319f1dd220f4ffa
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years ago[Tizen][M34-Merge] export v8 headers in devel package
Yi Shen [Tue, 2 Dec 2014 18:37:38 +0000 (10:37 -0800)]
[Tizen][M34-Merge] export v8 headers in devel package

This is necessary because WRT code is dependent on v8 headers.
Also added corresponding include path and CFLAGS to chromium-efl.pc.

Original gerrit: http://165.213.202.130:8080/#/c/69503/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=6096
Reviewed by: Antonio Gomes, Balazs Kelemen

Change-Id: Id370c8882619933399d38b8ff15c88426f10282c
Signed-off-by: Yi Shen <yi.shen@samsung.com>
6 years agoAvoiding vertex attribute copy from CPU to GPU for each frame.
m.nataraj [Sat, 15 Nov 2014 12:35:17 +0000 (18:05 +0530)]
Avoiding vertex attribute copy from CPU to GPU for each frame.

This is the migration patch from m34 beta to m40 dev branch. The patch
required modification in the texture coordinates as the contents were
rendered flipped both horizontally and vertically. This patch avoid
copying geometry data from cpu to gpu by creating vertex buffer objects.

Original commit: http://suprem.sec.samsung.net/gerrit/#/c/7063/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8766
Reviewed by: SeungSeop Park, Viatcheslav Ostapenko

Change-Id: I0e55e81797a649804e390243e6dce9d27d89c022
Signed-off-by: m.nataraj <m.nataraj@samsung.com>
6 years ago[M34-Merge] Implement 2 geolocation APIs
Antonio Gomes [Tue, 30 Sep 2014 07:52:49 +0000 (09:52 +0200)]
[M34-Merge] Implement 2 geolocation APIs

Patch implements missing Geolocation APIs needed by
the com.samsung.browser application:

* ewk/efl_integration/public/ewk_geolocation.cc/h:
* ewk/efl_integration/public/ewk_view.cc/h:

Add ewk_geolocation_permission_reply and ewk_view_geolocation_permission_callback_set APIs.

When user replies to a geolocation permission request,
the callback function set in ewk_view_geolocation_permission_callback_set
is called. The ewk_geolocation_permission_reply API is
for propagating the result about geolocation permission
from client apps (e.g. WRT and Browser) back to the
browser engine. In other words, application calls this
API after getting user action.

Also, patch marks ewk_geolocation_permission_request_set as
"deprecate" given that in the browser side, its use is replaced
by ewk_geolocation_permission_reply (being added by this patch).

As seen in the patch, part of the ewk_geolocation_permission_request_set
is moved down in the pipeline to _Ewk_Geolocation_Permission_Request
class (see below).

Note that the existing code in ewk/efl_integration/public/ewk_geolocation.cc
violates various coding style conventions. To keep
consistency within the same file, I choose to follow
what is already being used.

* impl/API/ewk_geolocation_private.cc:

Implement ::SetDecision (partially moved from ewk_geolocation_permission_request_set)
and ::Suspended methods.

* impl/API/ewk_geolocation_private.h:

Make it a proper class, with getters and setters,instead of a POD.

* impl/browser/geolocation/geolocation_permission_context_efl.cc

Fix geolocation permission callback. Old smart callback based geolocation.

Original work: 165.213.202.130:8080/#/c/70204/ (Kamil Klimek)
               165.213.202.130:8080/#/c/70053/ (Piotr Sieduszewski)

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8498
Reviewed by: Kamil Klimek, SeungSeop Park

Change-Id: I937e5e8713c7db095d9da901059100e3e25175b0
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoBring ewk_context_memory_sampler_* back.
Antonio Gomes [Tue, 2 Dec 2014 03:26:54 +0000 (19:26 -0800)]
Bring ewk_context_memory_sampler_* back.

These APIs were wrongly removed in [1]. In order to build
t-mobile-browser app out of the box for chromium-efl/beta,
chromium-efl/dev and webkit-efl, this patch brings it back.

They are only stub APIs.

Patch reverts [1] partially.

[1] http://165.213.202.130:8080/#/c/71214/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8497
Reviewed by: Kamil Klimek, SeungSeop Park

Change-Id: I7674f8d40187db95ecca25a2ea5cf4805c922235
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years ago[MM][WEBRTC] Allow User Media request for Camera/Mic, automatically for EFLWebViewApp.
kapil.kumar [Mon, 1 Dec 2014 10:07:04 +0000 (15:37 +0530)]
[MM][WEBRTC] Allow User Media request for Camera/Mic, automatically for EFLWebViewApp.

As EFLWebViewApp doesnt have any UI dialogs to ask user,
for allowing/denying media permission requested by webpage.
So its allowed by default in webview application.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=9477
Reviewed by: Antonio Gomes, Min-Soo Koo

Change-Id: I06d7ceb42fad8186da901e73cf45f61d21c9b134
Signed-off-by: kapil.kumar <kapil.kumar@samsung.com>
6 years ago[M34-Merge] Rename ewk_context_form_password_data_clear to ewk_context_form_password_...
Antonio Gomes [Mon, 6 Oct 2014 04:52:50 +0000 (13:52 +0900)]
[M34-Merge] Rename ewk_context_form_password_data_clear to ewk_context_form_password_data_delete_all

API was changed in webkit-efl, and this is a chromium-efl catch up.

This is also reflected in the browser side, as per commit
deab57b0 (by junghwan.kang).

Original work: http://165.213.202.130:8080/#/c/70359/ (join.yoon)

Bug : http://107.108.218.239/bugzilla/show_bug.cgi?id=9464
Reviewed by: Antonio Gomes, Kamil Klimek, SeungSeop Park

Change-Id: I88184023f4f87e64d8b1f30aad171eaa232b6eca
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years ago[M34-Merge] Rename ewk_context_form_candidate_data_clear() to ewk_context_form_candid...
Antonio Gomes [Fri, 26 Sep 2014 11:56:27 +0000 (13:56 +0200)]
[M34-Merge] Rename ewk_context_form_candidate_data_clear() to ewk_context_form_candidate_data_delete_all()

API was changed in webkit-efl, and this is a chromium-efl catch up.

This is also reflected in the browser side, as per commit
deab57b0 (by junghwan.kang).

Original work: http://165.213.202.130:8080/#/c/70014/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8496
Reviewed by: Kamil Klimek, SeungSeop Park

Change-Id: Id70ef37703d888da2290a136e481cc9dd87a3a25
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years ago[M34-Merge] Support for incognito mode implementation
Tomasz Czekala [Fri, 28 Nov 2014 10:00:57 +0000 (11:00 +0100)]
[M34-Merge] Support for incognito mode implementation

In chromium incognito mode works in different manner than in WebKit. In
WebKit we could toggle incognito mode for view, in chromium we must
create view in incognito mode and this option can't be toggled.

This is a merge of the following patch:
http://165.213.202.130:8080/#/c/70246/
Ieb8b70dcd73f7d589de91c4805476a20d8fc5cf5
by Kamil Klimek

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8249
Reviewed by: Kamil Klimek, SeungSeop Park

Change-Id: Ice68463efb572327074c49f78094fb258a37177d
Signed-off-by: Tomasz Czekala <t.czekala@partner.samsung.com>
6 years ago[M34-Merge] Implementation of "geolocation,valid" smart callback
chaoren.li [Tue, 21 Oct 2014 03:08:06 +0000 (11:08 +0800)]
[M34-Merge] Implementation of "geolocation,valid" smart callback

Patch implements "geolocation,valid" smart callback.
This allows the browser to grant a given webview
permission to start off the geolocation path before
GeolocationPermissionRequest object is created.

This is code migration from m34 beta to DEV.
Original patch : http://suprem.sec.samsung.net/gerrit/#/c/6962/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=7871
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I26ffb8ebf54b3ea4eb7cdfb6f7c38912ee5e8d2f
Signed-off-by: chaoren.li <chaoren.li@samsung.com>
6 years agoCall Blur on focus out.
Robert Jadczak [Thu, 27 Nov 2014 11:40:41 +0000 (12:40 +0100)]
Call Blur on focus out.

Added Blur method call when handling focus out
event to inform renderer that focus is lost.

It fixes js_focus.html test.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8942
Reviewed by: Antonio Gomes, Kamil Klimek

Change-Id: I1a4de43adf99fb13b0cd521cc3ed19f95927fbce
Signed-off-by: Robert Jadczak <r.jadczak@samsung.com>
6 years agoAllow local git clone in sync_repos.sh
SeungSeop Park [Mon, 24 Nov 2014 12:47:20 +0000 (21:47 +0900)]
Allow local git clone in sync_repos.sh

When cloning a repo from local machine, git is smart enough to
use hardlinking to save disk space. However, even if we clone
chromium-efl repo locally, we could not exploit this great feature
since sync_repos.sh will clone or update required repos from the
remote repositories hard coded in the script. As a result, we will
consume bigger disk space and more time unnecessarily.

This patch is to overcome the above mentioned limitation.
If chromium-efl is cloned from local directory, all the sub-repos
will be sync'd from the local directory. And for developer's
convenience the libraries compiled from jhbuild are cloned as well.

How to test:
$ git clone /path/to/chromium-efl chromium-efl-test
$ cd chromium-efl-test
$ ./sync_repo.sh
Reviewed by: Antonio Gomes, Min-Soo Koo, Tomasz Olszak

Change-Id: I753e8df2b9506f3d3d4149f4c4abccb51a7fc7c5
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years ago[M34-Merge] Implementation of ewk_view_scroll_* API.
Michał Pakuła vel Rutka [Thu, 3 Jul 2014 13:02:08 +0000 (15:02 +0200)]
[M34-Merge] Implementation of ewk_view_scroll_* API.

When some changes happend to ScrollView, RenderWidgetHostView received
only information about maximum available scroll offset. There was missing
information about current scroll offset. There was no possibility to get
current offset from browser process. I have added signal which transfers
information about current scroll offset (vertical and horizontal) whenever
it changes.

Added scroll,* smart callback calls after scroll position change. In
addition when scroll,* smart callback occured value returned by
RenderWidgetHost::GetLastScrollPosition was invalid, because value of
GetLastScrollPosition was updated after repaint event occured not after
scroll event. This change was required to return proper value after
scroll,* smart callback call.

RenderWidgetHostView object may not be created on EFL API call.
Thus every time rwhv() method is used (in eweb_view) it must be check
against nullness.

Scroll size is a content size reduced by view port size, so previous
implementation ("Fix Scroll bar does not work properly in Email app")
was wrong. That's why I'm removing it. Test improved in the way it checks
whether API returns non-zero values. Additionally more assertions have
been added to test API with negative values.

When ewk_view_scroll_pos_get is executed just after ewk_view_scroll_set
without running main loop the get function shall return new value even if
engine didn't scroll the page yet

This is a combination of five commits:
Added signal for providing current scroll information.
by Daniel Waslicki,

Change implementation details of scroll position related APIs
http://165.213.202.130:8080/#/c/68588/
by Kamil Klimek, and

ewk_view_scroll_* APIs can cause segmentation fault
http://165.213.202.130:8080/#/c/71201/
Fixup! Fix Scroll bar does not work properly in Email app
http://165.213.202.130:8080/#/c/71568/
ewk_view_scroll_pos_* functions shall behave like synchronous
http://165.213.202.130:8080/#/c/71205/
by Wojciech Bielawski
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I09af9ac17b9f0c0ce53fc832f2fa8b0f884097f0
Signed-off-by: Michał Pakuła vel Rutka <m.pakula@samsung.com>
6 years agoEnable accelerated 2d canvas feature
ajay berwal [Mon, 24 Nov 2014 06:24:10 +0000 (11:54 +0530)]
Enable accelerated 2d canvas feature

Accelerated 2d canvas feature is gpu blacklisted in m40. So enabling
it by adding switches::kForceAccelerated2dCanvas.

BUG: http://107.108.218.239/bugzilla/show_bug.cgi?id=8788
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I911f7323f12302441a4777e1c6874a676d914436
Signed-off-by: ajay berwal <ajay.berwal@samsung.com>
6 years ago[Tizen][M34-Merge] Fix IME Event Handling
Youngsoo Choi [Thu, 16 Oct 2014 20:34:26 +0000 (13:34 -0700)]
[Tizen][M34-Merge] Fix IME Event Handling

IME virtual keyboard does not properly work on TV.
Even though following two lines are applied, non-numeric character input,
including Korean and alphabet, does not work on TV.

>> if (pressed && modifiers & blink::WebInputEvent::AltKey)
>>   webKitEvent.isSystemKey = true;

There are OnCommit and OnPreeditChanged Ecore callbacks for key events.
The OnCommit is called by numeric character input,
and the OnPreeditChanged is called by non-numeric character input.
The problem is that OnPreeditChanged is not called when
there is a non-numeric character input.
Another one is that IPC message has been commented out on impl side
and removed on renderer side, so browser process couldn't
receive result of processed input event from renderer process.

The following two patches resolve above issues.

>> Simplify and fix current IME context handling logic
>> Fix IME events handling

[Merged Patches]
==========================================================
- Support for multiple commit events.
Original author: Marcin Krakowiak <m.krakowiak@samsung.com>
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=6123

- Composed text disappear after return key.
Origianl author: Karol Furmaniak <k.furmaniak@samsung.com>
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=6215

- Simplify and fix current IME context handling logic
Origianl author: Piotr Tworek <p.tworek@samsung.com>
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=5835

- fixup! Wrong korean keycodes in IME.
Original author: Karol Furmaniak <k.furmaniak@samsung.com>
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=5411

- Fixed handling of input panel state change
Original author: Kamil Klimek <k.klimek@partner.samsung.com>

- Fix IME events handling.
Original author: Karol Furmaniak <k.furmaniak@samsung.com>
Issue: CBBROWSER-248, CBBROWSER-249
==========================================================

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=6715
Reviewed by: SeungSeop Park, Viatcheslav Ostapenko, Youngbok Yoon

Change-Id: I876a3c6da9f19dfb6dea08e2d53b59ccd3bd3c8b
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
6 years agoImplement JavaScriptDialogManagerEfl::RunBeforeUnloadDialog
Marcin Krakowiak [Mon, 17 Nov 2014 15:43:03 +0000 (16:43 +0100)]
Implement JavaScriptDialogManagerEfl::RunBeforeUnloadDialog

This commit removes EWK_BRINGUP build guards related to this
features. For that, it partly reverts commit 5cf129faa.

In order to avoid src/ changes, it also adds an enum Type to
JavaScriptModalDialogEfl to handle ALERT, CONFIRM, NAVIGATION
and PROMPT.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8778
Reviewed by: Antonio Gomes, Laszlo Gombos, SeungSeop Park

Change-Id: Id03636f130b27abb0a159156eabd70f8d6466ff2
Signed-off-by: Marcin Krakowiak <m.krakowiak@samsung.com>
Signed-off-by: Daniel Grad <d.grad@samsung.com>
6 years agoPass cookie information to Policy Decision struct
Kamil Lysik [Tue, 25 Nov 2014 08:38:09 +0000 (09:38 +0100)]
Pass cookie information to Policy Decision struct

ContentRendererClientEfl::HandleNavigation didn't pass cookie information
to Navigation Policy struct. This causes ewk_policy_decision_cookie_get
work invalid.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8247
Reviewed by: Kamil Klimek, SeungSeop Park

Change-Id: Ibc5869cae55a4f5dc8f661eff3bba915557b4b4a
Signed-off-by: Kamil Lysik <k.lysik@samsung.com>
6 years agoImproper displaying of favicon.
Jongsoo Yoon [Wed, 29 Oct 2014 07:20:00 +0000 (16:20 +0900)]
Improper displaying of favicon.

Favicon was created without alpha channel so that transparent
backgorund was premultipled to black.

Origin patch(M34): http://suprem.sec.samsung.net/gerrit/#/c/7010/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8200
Reviewed by: Antonio Gomes, Piotr Tworek, SeungSeop Park, Youngbok Yoon

Change-Id: I977cada182c2d2c2866f83ef49f43d1ba8dee828
Signed-off-by: Jongsoo Yoon <join.yoon@samsung.com>
6 years agoEnable ewk_policy_decision API. Add httpMethod to Policy Decision object.
Kamil Lysik [Thu, 14 Aug 2014 12:05:59 +0000 (14:05 +0200)]
Enable ewk_policy_decision API. Add httpMethod to Policy Decision object.

Current this API is unusable due to missing callback.
With callback enabled it is possible to use this API.
Moreover, there is missing policyDecision object.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8247
Reviewed by: Antonio Gomes, SeungSeop Park, Viatcheslav Ostapenko

Change-Id: Id3516e67beb8adc520eefd0fe2cc24ca4c5e9e51
Signed-off-by: Kamil Lysik <k.lysik@samsung.com>
6 years ago[WRT] Export ewk_view_mode_set()
Yi Shen [Tue, 25 Nov 2014 00:33:08 +0000 (16:33 -0800)]
[WRT] Export ewk_view_mode_set()

This is necessary because WRT code needs this ewk API.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=9256
Reviewed by: Antonio Gomes, Laszlo Gombos

Change-Id: I7facd35eba80e76fc8102cb64f853d56948e5605
Signed-off-by: Yi Shen <yi.shen@samsung.com>
6 years agoDeprecate TIZEN_ENGINE_SUPPORT define
SeungSeop Park [Tue, 25 Nov 2014 07:08:39 +0000 (16:08 +0900)]
Deprecate TIZEN_ENGINE_SUPPORT define

From now on, USE_EFL (or TIZEN_MULTIMEDIA_SUPPORT) should be used instead.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=9216
Reviewed by: Antonio Gomes, Kamil Klimek

Change-Id: Ie5ab4ea5e41218a443a229befb4e9e3f5bb94a0a
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years ago[Tizen][M34-Merge] No "This web page is not available" message
Grzegorz S [Mon, 17 Nov 2014 14:04:38 +0000 (15:04 +0100)]
[Tizen][M34-Merge] No "This web page is not available" message

Page with error doesn't appear because there was no
GetNavigationErrorStrings function implementation in
ContentRendererClientEfl.
Solution is to add GetNavigationErrorStrings and generate error page message.

Merged commit: http://165.213.202.130:8080/#/c/70428/12
Reviewed by: SeungSeop Park, Viatcheslav Ostapenko

Change-Id: I02bf0f2c30086be3af3620ed30b60804298328f7
Signed-off-by: Michał Obrembski <m.obrembski@samsung.com>
6 years ago[M34-Merge] Browser silently exits in some cases.
sihoons.yang [Mon, 13 Oct 2014 07:27:30 +0000 (16:27 +0900)]
[M34-Merge] Browser silently exits in some cases.

When a view is created using ewk_view_add_with_session_data and both data and
length values passed equal 0, function returns NULL which causes browser
to exit silently on application start.
This patch fixes the bug by adding a fallback which creates a view but
without taking into account session data.

Original patch: http://165.213.202.130:8080/#/c/70731/

Reviewed by: Antonio Gomes, Balazs Kelemen
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8530
Change-Id: I9a835cc5c29ddd70f0c8745f6e0427fb991577a7
Signed-off-by: Michał Pakuła vel Rutka <m.pakula@samsung.com>
6 years agofixup! Add Kamil klimek as an owner
mohan [Mon, 24 Nov 2014 13:36:29 +0000 (19:06 +0530)]
fixup! Add Kamil klimek as an owner

Corrected the email-id of the  Kamil klimek as k.klimek@partner.samsung.com
instead of k.klimek@samsung.com as per Janusz Majnert recommendation

Change-Id: Ic97661be6e60efb8b285edcbfa6e31784ef96213
Signed-off-by: mohan <mohan.reddy@samsung.com>
6 years ago[Tizen][M34-Merge] Implement favicon database get API
zhishun.zhou [Thu, 16 Oct 2014 10:17:50 +0000 (18:17 +0800)]
[Tizen][M34-Merge] Implement favicon database get API

This patch is needed to support ewk_context_favicon_database_get.
This is code migration from m34 beta to DEV,
Original patch info: http://suprem.sec.samsung.net/gerrit/#/c/6865/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=7636
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: Ida3a261efa38ea4b636eb1abe65da64f107fc244
Signed-off-by: zhishun.zhou <zhishun.zhou@samsung.com>
6 years ago[Tizen][M34-Merge] Port locale packages from WebKit & get locale string for error...
Nguen Van Canh [Tue, 18 Nov 2014 09:39:17 +0000 (10:39 +0100)]
[Tizen][M34-Merge] Port locale packages from WebKit & get locale string for error page content

This commit adds translation files to ewk/po_tizen
and adds proper changes into chromium-efl.spec to
include transtion files during creation of RPM.

Merged commit: http://165.213.202.130:8080/#/c/69776/9

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=9194
Reviewed by: SeungSeop Park, Viatcheslav Ostapenko

Change-Id: Id9440a66b97c86c249739844fb6d3f8f044c50dd
Signed-off-by: Michał Obrembski <m.obrembski@samsung.com>
6 years agoAdd new gyp/c++ define, 'use_efl'/'USE_EFL', to deprecate TIZEN_ENGINE_SUPPORT
SeungSeop Park [Mon, 24 Nov 2014 06:32:03 +0000 (15:32 +0900)]
Add new gyp/c++ define, 'use_efl'/'USE_EFL', to deprecate TIZEN_ENGINE_SUPPORT

Patch is a partial work for our plan to deperecate TIZEN_ENGINE_SUPPORT.
See bug site for detailed discussion.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=9216
Reviewed by: Kangil Han, SeungSeop Park

Change-Id: I739682bc50db1c37131b726298950190001d9380
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years agoAdd Minsoo Koo as an owner
Dongwoo Joshua Im [Mon, 24 Nov 2014 10:30:02 +0000 (19:30 +0900)]
Add Minsoo Koo as an owner

Add Minsoo Koo as an owner of chromium-efl directory

Change-Id: Idcc82c275ca7d69d2c455274d8ddd391b4d9a999
Signed-off-by: Dongwoo Joshua Im <dw.im@samsung.com>
6 years agoAdd Kamil klimek as an owner
Dongwoo Joshua Im [Mon, 24 Nov 2014 10:28:21 +0000 (19:28 +0900)]
Add Kamil klimek as an owner

Add Kamil klimek as an owner of chromium-efl directory

Change-Id: If6463e64bb39c1dc34c43f831ac46638c93aae70
Signed-off-by: Dongwoo Joshua Im <dw.im@samsung.com>
6 years ago[MM][WebAudio] Upmerge WebAudio code to dev/master
Chandan Padhi [Wed, 29 Oct 2014 13:49:21 +0000 (19:19 +0530)]
[MM][WebAudio] Upmerge WebAudio code to dev/master

1) This patch is for upmerge of WebAudio code from m34 beta/ to dev/master.
   It involves changes only on the impl/ side.
2) It defines the following gyp variables
   a) 'tizen_multimedia_support' --> top level MM gyp variable
3) It also defines top level preprocessor flag(TIZEN_MULTIMEDIA_SUPPORT) for all
   MM modules.

On top of this patch, other MM modules(ME/MSE) and WebRTC too can upmerge
their code from m34 to m40.

Original patches(M34):
http://165.213.202.130:8080/#/c/71266/
http://165.213.202.130:8080/#/c/71386/

Together-with: Ied59b4f61c906e03634084f2490913ab7f1b1abd

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=7941
Reviewed by: Hyungsun Ryu, Laszlo Gombos, SeungSeop Park, Vinod Keshav

Change-Id: I6d9c536a6b40d572102b3149ff0001e68c8de781
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
Signed-off-by: varun kumar pathak <varun.pathak@samsung.com>
6 years ago[M34-Merge] ewk_view_scale_get function shall behave like synchronous one
Wojciech Bielawski [Wed, 29 Oct 2014 16:49:20 +0000 (17:49 +0100)]
[M34-Merge] ewk_view_scale_get function shall behave like synchronous one

Although the ewk_view_scale_set is asynchronous, the TCT compliance tests
from webkit-efl EWK implementation require the getter to return the value
set by the setter even if the two functions were called immediately
after each other.
This change makes the setter modify the proxy value,
thereby subsequent calls of the getter will return proper value,
even if it hasn't been applied in the UI process.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=7679
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8260
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I34d1e77a97a0d7395538a6300b9430531f6934a7
Signed-off-by: Wojciech Bielawski <w.bielawski@samsung.com>
6 years ago[M34-Merge] Implement ewk_cookie_manager_accept_policy_async_get
Tomasz Czekala [Fri, 14 Nov 2014 08:40:10 +0000 (09:40 +0100)]
[M34-Merge] Implement ewk_cookie_manager_accept_policy_async_get

This patch refactors ewk_cookie_manager_async_accept_policy_get to
ewk_cookie_manager_accept_policy_async_get, replaces callback to a new
one and fixes some coding style.

This patch is a squash of the following patches:
http://165.213.202.130:8080/#/c/70271/
by Piotr Sieduszewski

http://165.213.202.130:8080/#/c/70954/
by Wojciech Bielawski

http://165.213.202.130:8080/#/c/69727/
by Kamil Klimek

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8807
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: Id94ba3b349c4eb12e05e1ad4457edc192b2bdd0c
Signed-off-by: Tomasz Czekala <t.czekala@partner.samsung.com>
6 years agoPort orientation and tilt to tizen 2.3
Arnaud Renevier [Tue, 7 Oct 2014 23:19:06 +0000 (16:19 -0700)]
Port orientation and tilt to tizen 2.3

We're using the patches that exist for branch m34:
 http://165.213.202.130:8080/#/c/69111/
 http://165.213.202.130:8080/#/c/68985/
 http://165.213.202.130:8080/#/c/68983/
 http://165.213.202.130:8080/#/c/72090/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=6511
Reviewed by: Antonio Gomes, Balazs Kelemen, Viatcheslav Ostapenko

Change-Id: I27002bb7887e1966ae53e1f796f787facbfe5d55
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years ago[M34-Merge] Enable cache model setting
Kamil Lysik [Thu, 20 Nov 2014 14:59:52 +0000 (15:59 +0100)]
[M34-Merge] Enable cache model setting

EWK_BRINGUP flag is now disabled for EWebContext::SetCacheModel.
This blocks setting any other cache model.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8239
Reviewed by: Antonio Gomes, Viatcheslav Ostapenko

Change-Id: I3592e81f3d3f9a78b6715a28ce16690f7f9b4128
Signed-off-by: Kamil Lysik <k.lysik@samsung.com>
6 years ago[Dev] built-in linker (ld) can not handle chromium-efl
Antonio Gomes [Wed, 19 Nov 2014 14:24:46 +0000 (06:24 -0800)]
[Dev] built-in linker (ld) can not handle chromium-efl

Use a prebuilt gold linker binary instead of the default ld linker
installed by binutils-2.24-6.3.armv7l.rpm in the chroot.

Using 'gold' allows us to create smaller binaries for chromium-efl.so
(e.g. from 74Mb to 71Mb in my local machine). Besides, the linking step
also consumes less RAM memory, and is faster than with ld.bfd.
As a result, we can link with no "Memory exhausted" error in both
release and debug builds, and not depend on using Wl,--no-keep-memory.

So patch removes --no-keep-memory for both release and debug TV/mobile
builds as it is not supported by gold linker.

linux_use_bundled_gold and linux_use_bundled_binutils were explicitly
set to 0 to ensure we use our own linker.

NOTE: this patch differs from [1] (reverted CL) as following: it links
libchromium-efl.so with gold, and while others shared libraries and executable
are still linked with the ld.bfd.
In [1], both libchromium-efl.so and efl_webprocess were linked with gold,
which were problematic on TV.

[1] http://165.213.202.130:8080/#/c/72105/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=6629
Reviewed by: Antonio Gomes, SeungSeop Park, Viatcheslav Ostapenko

Change-Id: I2c5c3823b63d7dab418c01a67d3bcd5e13b19ab0
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years ago[M40_2214] Chromium upversion to m40_2214 branch
SeungSeop Park [Thu, 20 Nov 2014 05:14:45 +0000 (14:14 +0900)]
[M40_2214] Chromium upversion to m40_2214 branch

* Notes for this rebase:
1. gyp variable 'enable_printing' changed in m40_2214
   : enable_printing -> enable_basic_printing, enable_print_preview

2. Changed include path for gesture_configuration.h
   : ui/events/gestures/ -> ui/events/gesture_detection/

3. OnRequestPlatformNotificationPermission does not exist
   : The API removed from content/public/browser/content_browser_client.h.
     So, removed it from our code since it was dummy anyway.

4. ShowDesktopNotification() : arguments changed

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=9170
Reviewed by: Kangil Han, SeungSeop Park, Viatcheslav Ostapenko

Change-Id: I9ecef10f24a48422fa5e44d5862d6935f161aa0a
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years ago[M40_2214] Auto-generate LASTCHANGE file
SeungSeop Park [Thu, 20 Nov 2014 07:52:37 +0000 (16:52 +0900)]
[M40_2214] Auto-generate LASTCHANGE file

Patch fixes following build error:
> ninja: error: '../../src/build/util/LASTCHANGE', needed by XXX

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=9170
Reviewed by: Kangil Han, SeungSeop Park, Viatcheslav Ostapenko

Change-Id: I9ab298dc5cddbb9e09792fa93b4ffc20cd95b08b
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years ago[M40_2214] StartPageForVectorCanvas() removed from printing/metafile.h
SeungSeop Park [Thu, 20 Nov 2014 10:22:38 +0000 (19:22 +0900)]
[M40_2214] StartPageForVectorCanvas() removed from printing/metafile.h

API only exists in printing/pdf_metafile_skia.h (PdfMetafileSkia class).

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=9170
Reviewed by: Antonio Gomes, Laszlo Gombos, Viatcheslav Ostapenko

Change-Id: Ifb76cfa4466cf4696356e7c1d525d711f6f980f3
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years ago[Tizen][M34-Merge] Implement Ewk_Settings: cache builder & default text encoding...
zhishun.zhou [Mon, 20 Oct 2014 08:17:40 +0000 (16:17 +0800)]
[Tizen][M34-Merge] Implement Ewk_Settings: cache builder & default text encoding name

This patch implement two Ewk_Settings: set cache builder enable
and set default text encoding name.
This is code migration from m34 beta to DEV,
Original patch info:
http://suprem.sec.samsung.net/gerrit/#/c/6533/3 and
http://165.213.202.130:8080/#/c/68824/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=7799
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I4c344a8ee508804939e9219f736ec5e4a901a703
Signed-off-by: zhishun.zhou <zhishun.zhou@samsung.com>
6 years agoImplement ewk_context_form_password_data_clear
Yi Shen [Wed, 19 Nov 2014 21:14:33 +0000 (13:14 -0800)]
Implement ewk_context_form_password_data_clear

Original Bug: http://suprem.sec.samsung.net/jira/browse/CBWEBVIEW-320
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=7566

unittest: utc_blink_ewk_context_form_password_data_clear_func.cpp
Reviewed by: Antonio Gomes, Balazs Kelemen

Change-Id: I0ca9edc924a275ada6c3c8542c96a41e20939305
Signed-off-by: Yi Shen <yi.shen@samsung.com>
6 years agoDon't return frame mailbox until next frame comes.
Viatcheslav Ostapenko [Wed, 19 Nov 2014 21:05:04 +0000 (16:05 -0500)]
Don't return frame mailbox until next frame comes.

MailboxOutputSurface expects, that client holds at least one pending mailbox texture.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8886
Reviewed by: Antonio Gomes, Balazs Kelemen

Change-Id: Idc360f44a392db20f4e6195ade2d7ca8d9c1b316
Signed-off-by: Viatcheslav Ostapenko <sl.ostapenko@samsung.com>
6 years ago[M34-Merge] Implementation of ewk_context_preferred_languages_set API and correspondi...
Lukasz Krok [Tue, 26 Aug 2014 09:04:56 +0000 (11:04 +0200)]
[M34-Merge] Implementation of ewk_context_preferred_languages_set API and corresponding unit test cases.

The API role is to allow Accept-language request header substitution.

This commit is based on two previous commits:
http://165.213.202.130:8080/#/c/68879/
http://165.213.202.130:8080/#/c/68880/
made by Lukasz Krok <l.krok@samsung.com>

There are also added minor changes to fit to latest chromium sources.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=4952
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8241
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I81431dd927cac6a172760f2f17160c93c09773c5
Signed-off-by: Dariusz Frankiewicz <d.frankiewic@samsung.com>
6 years agoRevert "[Tizen][Dev] built-in linker (ld) can not handle chromium-efl"
SeungSeop Park [Tue, 18 Nov 2014 11:44:10 +0000 (20:44 +0900)]
Revert "[Tizen][Dev] built-in linker (ld) can not handle chromium-efl"

This reverts commit 72dbab00224102c43c4c6452344f39f5d283f427.
It's due to Problem in TV run:
  - efl_webview and  mini_browser can not open any websit today

Log prompt efl_webprocess cannot execute. When comparing the normal
and abnormal of efl_webprocess (via readelf -h efl_webprocess),
ELF Header Flags are as follows:
  - before:  0x5000000, Version5 EABI
  - current: 0x5000202, has entry point, Version5 EABI, <unknown>.

Change-Id: I75f548ea58e99484fade7fe3b68918497597bf70
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years agoBringup error domain in Ewk_Error
zhaoze [Thu, 6 Nov 2014 22:38:21 +0000 (17:38 -0500)]
Bringup error domain in Ewk_Error

Fill out the domain value in Ewk_Error.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8592
Reviewed by: Balazs Kelemen, SeungSeop Park

Change-Id: I37ae2424e32844eb7b93b39680b2d6f259690e68
Signed-off-by: zhaoze <zhaoze.zhou@partner.samsung.com>
6 years agoReplace supplement.gypi with a more reliable solution
Balazs Kelemen [Sat, 15 Nov 2014 01:11:56 +0000 (20:11 -0500)]
Replace supplement.gypi with a more reliable solution

Adapt the chromium-efl build to the new gyp_trunk.py wrapper.
Import and apply it conditionally so we can land both patches without
temporarily breaking the build. Condition can be removed later in a fixup.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=6661
Reviewed by: SeungSeop Park, Viatcheslav Ostapenko

Change-Id: Ic6de4b64acf70d63060581fcd10620f1f5b0634b
Signed-off-by: Balazs Kelemen <b.kelemen@samsung.com>
6 years agoAdd --single-branch parameter to git clone in sync_repos.sh
Viatcheslav Ostapenko [Sat, 15 Nov 2014 04:47:21 +0000 (23:47 -0500)]
Add --single-branch parameter to git clone in sync_repos.sh

Faster initial clone of repositories.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8173
Reviewed by: Antonio Gomes, Balazs Kelemen

Change-Id: I8553ca353d1faf9c7a5ebafa609b9d2a17215d94
Signed-off-by: Viatcheslav Ostapenko <sl.ostapenko@samsung.com>
6 years agoSet default strategy of not remembering certificate decision.
Michal Poteralski [Mon, 17 Nov 2014 09:30:47 +0000 (10:30 +0100)]
Set default strategy of not remembering certificate decision.

SSLHostStateDelegate may be left unimplemented to use a default strategy of not
remembering certificate decisions at all. As long as the default strategy is
used the GetSSLHostStateDelegate left unimplemented, thus
BrowserContextEfl::GetSSLHostStateDelegate() method should return NULL.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8884
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I3e826a8b5116abdf2f5a9a3b8bbe590526f89008
Signed-off-by: Michal Poteralski <m.poteralski@samsung.com>
6 years ago[Tizen][MM][Webrtc] GetUserMedia Support
“kapil.kumar” [Wed, 12 Nov 2014 09:21:57 +0000 (14:51 +0530)]
[Tizen][MM][Webrtc] GetUserMedia Support

Added support require for m40 Upmerge.
-Added the extra argument for status result in callback
-Implement checkAccess callback, as we dont support extenstions
so return default as true. So it will ask user for media access everytime.
-Added device id for TV, as TV has only one primary camera.
-Move all changes under TIZEN_MULTIMEDIA_SUPPORT flag, for easy upmerging of
GetUserMedia.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=7942
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: Ic1ff44d9ceb23b29bb41e6318821602dfe1bf25f
Signed-off-by: “kapil.kumar” <kapil.kumar@samsung.com>
Signed-off-by: kapil.kumar <kapil.kumar@samsung.com>
6 years agoRemove S_TRUNK_OWNERS files which were added by mistake.
Dongwoo Joshua Im [Mon, 17 Nov 2014 07:01:18 +0000 (16:01 +0900)]
Remove S_TRUNK_OWNERS files which were added by mistake.

Remove S_TRUNK_OWNERS files which were added during migrating
chromium-ewk into chromium-efl on beta branch.
Reviewed by: Kangil Han, SeungSeop Park

Change-Id: I7bd887a33ccf4774073dee7c22130181f191b9b4
Signed-off-by: Dongwoo Joshua Im <dw.im@samsung.com>
6 years ago[M34-Merge] Mechanism for deactivation of MessagePumpForUIEfl timers added
Michal Poteralski [Tue, 4 Nov 2014 10:29:07 +0000 (11:29 +0100)]
[M34-Merge] Mechanism for deactivation of MessagePumpForUIEfl timers added

The patch implements a way to deactivate and delete MessagePumpForUIEfls
timers which are still active (no triggered) despite the fact that
application has been closed.
If the timers are NOT deactivated after the application close
then the timers will try call MessagePumpForUIEfl::TimerCallback
function which actually does NOT exist because MessagePumpForUIEfl has
been destructed.

Original commit: http://165.213.202.130:8080/#/c/71335/
(by Wojciech Kosowicz <w.kosowicz@samsung.com>)

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8500
Reviewed by: Antonio Gomes, Balazs Kelemen, Piotr Tworek, SeungSeop Park

Change-Id: Id993b36dd15d8fe15900cf5e0b544cc36643ff30
Signed-off-by: Michal Poteralski <m.poteralski@samsung.com>
6 years agofixup! [Tizen][Dev] built-in linker (ld) can not handle chromium-efl
SeungSeop Park [Mon, 17 Nov 2014 07:08:07 +0000 (16:08 +0900)]
fixup! [Tizen][Dev] built-in linker (ld) can not handle chromium-efl

Fixes the desktop build break.
Reviewed by: Kangil Han, SeungSeop Park

Change-Id: Ieb013fabc621a0a3afc50ab79e9775fa11b5711b
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years ago[Tizen][Dev] built-in linker (ld) can not handle chromium-efl
Antonio Gomes [Thu, 13 Nov 2014 19:34:23 +0000 (11:34 -0800)]
[Tizen][Dev] built-in linker (ld) can not handle chromium-efl

Use a prebuilt linker gold binary instead of the default ld
installed by binutils-2.24-6.3.armv7l.rpm.
linux_use_bundled_gold and inux_use_bundled_binutils were explicitly
set to 0 to ensure we use our own linker.

Using 'gold' allows us to create smaller binaries for chromium-efl.so,
e.g. from 74Mb to 71Mb in my local machine.
As a result, we can link with no "Memory exhausted" error in both
release and debug builds, and not depend on using Wl,--no-keep-memory.

Patch intentionally leaves --no-keep-memory set for debug builds
because of bug [1], [TIZEN][Dev] GBD rpmbuild error with m39 build.

[1] http://107.108.218.239/bugzilla/show_bug.cgi?id=6457

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=6629
Reviewed by: SeungSeop Park, Viatcheslav Ostapenko

Change-Id: I1e1a0224bf157f10c506112b980fe4355a3e2efc
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoBack-forward list unit tests added.
Tomasz Czekala [Thu, 13 Nov 2014 13:25:11 +0000 (14:25 +0100)]
Back-forward list unit tests added.

This commits adds unit tests for following functions from EWK API:
1.  ewk_view_back_forward_list_get
2.  ewk_back_forward_list_item_url_get
3.  ewk_back_forward_list_item_unref
4.  ewk_back_forward_list_item_title_get
5.  ewk_back_forward_list_item_ref
6.  ewk_back_forward_list_item_original_url_get
7.  ewk_back_forward_list_previous_item_get
8.  ewk_back_forward_list_next_item_get
9.  ewk_back_forward_list_n_forward_items_copy
10. ewk_back_forward_list_n_back_items_copy
11. ewk_back_forward_list_current_item_get
12. ewk_back_forward_list_count
13. ewk_back_forward_list_item_at_index_get

This patch ports following patch:
I8ecaa82514cc2129a1ac25b0ec2cd63c8ad0989f
 d2c3228b8ca472595f3c00caa8dfa0c3d839352d
made by Filip Piechocki

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8237
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: Iba6a1fb044ea8a7416db455b71b815b33b5dddf6
Signed-off-by: Tomasz Czekala <t.czekala@partner.samsung.com>
6 years agouse ExecuteJavaScriptForTests instead of ExecuteJavaScript
Arnaud Renevier [Wed, 5 Nov 2014 22:03:51 +0000 (14:03 -0800)]
use ExecuteJavaScriptForTests instead of ExecuteJavaScript

Full screen test fail because the full screen opening is blocked.
Actually, a fullscreen can only be opened if triggered by a gesture.

In chromium m34, executeJavascript used to act like if the script had
been executed by a gesture. But in recent chromium, there are two
different calls: executeJavascript and executeJavascriptForTests.

Since we don't want to change ewk api, we use executeJavascriptForTests
when a script is executed. This will mimick m34 behaviour

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8558
Reviewed by: Antonio Gomes, Balazs Kelemen

Change-Id: I8e5025b826f7462aabdefa2a299f16222dba3f25
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years ago[M34-Merge] Add CSS view-mode media feature support (chromium-efl)
Zhang-hoon Oh [Tue, 28 Oct 2014 09:20:40 +0000 (18:20 +0900)]
[M34-Merge] Add CSS view-mode media feature support (chromium-efl)

To achieve Web App compatibility for Blink on Tizen, CSS
"view-mode" media feature support should be resurrected.
There is a strong requirement from WRT developers.
This patch is the "chromium-efl" side of a two-sided
patch.

"s-blink" side change-id: I38ce341531cba1069f407d41ef8a5f90f3cd0c71

About M34 Patches:
* M34 patches are reorganized.
* Commits:
** http://suprem.sec.samsung.net/gerrit/#/c/1047/
** http://suprem.sec.samsung.net/gerrit/#/c/1906/
** http://suprem.sec.samsung.net/gerrit/#/c/1907/
* Original Author:
** The original author was kaka.park@samsung.com.

Spec.: http://www.w3.org/TR/view-mode/
     (http://www.w3.org/TR/2012/REC-view-mode-20120619/)

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=7803
Reviewed by: Balazs Kelemen, SeungSeop Park, Yi Shen
Reviewed by: Balazs Kelemen, SeungSeop Park

Change-Id: Ic4fcdecd58d493f190bab4fb6c268c9b2f41f845
Signed-off-by: Zhang-hoon Oh <dennis.oh@samsung.com>
6 years agoRefactor disambiguation popup
Dongjun Kim [Sun, 2 Nov 2014 09:53:00 +0000 (18:53 +0900)]
Refactor disambiguation popup

Previous implementation of disambiguation popup was buggy. It was
scaling already scaled image. It was scaling without keeping aspect
ratio. It caused lot of bugs with clicked position and rendering.
To tests the new implementation on desktop please make sure to add
--touch-events flag to efl_webview_app startup parameters.

Original patch : http://165.213.202.130:8080/#/c/70671
Original Author : Kamil Klimek <k.klimek@partner.samsung.com>

Together with : I5e0cf66df8922f8d4523b950bbcc999a7b90b402
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8405
Reviewed by: Kamil Klimek, Piotr Tworek, SeungSeop Park, Viatcheslav Ostapenko

Change-Id: I1563a1cc2f722866ae9be0aa5f73440fcb8d5d25
Signed-off-by: Dongjun Kim <djmix.kim@samsung.com>
Signed-off-by: hh4813.kim <hh4813.kim@samsung.com>
6 years agoBack-forward list implementation.
Tomasz Czekala [Wed, 12 Nov 2014 12:41:27 +0000 (13:41 +0100)]
Back-forward list implementation.

Following EWK API functions were implemented in this commit:
1.  ewk_view_back_forward_list_get
2.  ewk_back_forward_list_item_url_get
3.  ewk_back_forward_list_item_unref
4.  ewk_back_forward_list_item_title_get
5.  ewk_back_forward_list_item_ref
6.  ewk_back_forward_list_item_original_url_get
7.  ewk_back_forward_list_previous_item_get
8.  ewk_back_forward_list_next_item_get
9.  ewk_back_forward_list_n_forward_items_copy
10. ewk_back_forward_list_n_back_items_copy
11. ewk_back_forward_list_current_item_get
12. ewk_back_forward_list_count
13. ewk_back_forward_list_item_at_index_get
These functions are a C wrappers on a C++ implementation in chromium-efl

This commit is a merge of 3 patches made by Filip Piechocki:
Ibc45f932d7bfc98837bd330fcf9c2c4f886410d2
 f4fdf69e5c40b8e3b0dc78dd3cfa13bf97561aec
I2fc1871a9f34a9f5fa7038e10e666767b9a90e5c
 df2b2cd674e289ebd00a5f7415e7163ae7e24eca
Ia153812ecb5ba4942be89e7f483e83704f26cbc2
 50aa4c9308f63bfdde5dea042eb7b1cd98d33c3c

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8237
Reviewed by: SeungSeop Park, Viatcheslav Ostapenko

Change-Id: I9b70d36e7bfae57c9ad4afec0434c383b3f2ecc8
Signed-off-by: Tomasz Czekala <t.czekala@partner.samsung.com>
6 years ago[M40] Enabling ViewHostMsg_TextInputStateChanged
Youngsoo Choi [Wed, 12 Nov 2014 12:13:11 +0000 (21:13 +0900)]
[M40] Enabling ViewHostMsg_TextInputStateChanged

ViewHostMsg_TextInputStateChanged needs to be enabled
to update text input state.

Together with: I1dc9519a08e6c17593b961755fc33fc125bdeac7

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8818
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: Ia6f60691dcc316f6b064e5efcd421fded4e95e10
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
6 years agoFixes utc_blink_ewk_autofill_profile_id_get.POS_TEST
Arnaud Renevier [Wed, 5 Nov 2014 23:33:08 +0000 (15:33 -0800)]
Fixes utc_blink_ewk_autofill_profile_id_get.POS_TEST

utc_blink_ewk_autofill_profile_id_get.POS_TEST fails because it tests
that ewk_autofill_profile_id_get returns 0. This test made sense when
each test file was executed in a different profile. But now that all
tests are executed in a single executable, multiple profiles may already
have been created once this test is run.

We remove totally the file because ewk_autofill_profile_id_get is
already used in some other unittests
(utc_blink_ewk_context_form_autofill_profile_utils.*)

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8561
Reviewed by: Antonio Gomes, Viatcheslav Ostapenko

Change-Id: I15a59d4064e81fc30fd082b35ea3f479d63550be
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agoAdd ewk_settings_default_text_encoding_name_get implementation.
Michal Poteralski [Wed, 5 Nov 2014 09:46:20 +0000 (10:46 +0100)]
Add ewk_settings_default_text_encoding_name_get implementation.

ewk_settings_default_text_encoding_name_get function is NOT implemented.
This patch provides implementation of the function.

Original commit: http://165.213.202.130:8080/#/c/68831/
(by Piotr Sieduszewski <p.sieduszews@samsung.com>)

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8248
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I7abb589a1cbfd8df3f6004bfdd7559d884c0b0d5
Signed-off-by: Michal Poteralski <m.poteralski@samsung.com>
6 years agoDeploy scripts are not awasome enough
Balazs Kelemen [Thu, 6 Nov 2014 22:56:42 +0000 (17:56 -0500)]
Deploy scripts are not awasome enough

Some fixes to make it work out of box and restructured the script
to be more straightforward. Now it is an executable script (instead
of an environment one).

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=6900
Reviewed by: Antonio Gomes, Viatcheslav Ostapenko

Change-Id: Ia70104ed0f77e67133760c655e5c508d4dd972dc
Signed-off-by: Balazs Kelemen <b.kelemen@samsung.com>
6 years agoFixes: form,submit callback is not called
Arnaud Renevier [Tue, 11 Nov 2014 19:14:40 +0000 (11:14 -0800)]
Fixes: form,submit callback is not called

As of now, WillSubmitForm is not called once a form has been submitted.
This is because WillSubmitForm first argument has changed in m39. It is
now a blink::WebLocalFrame* instead of a blink::WebFrame*

So, this patch changes the argument back. And to prevent that problem from
happening in the future, it also sets the override identifier for that
function.
Reviewed by: Antonio Gomes, Balazs Kelemen

Change-Id: I485c82934748fa1a19a66ad21ef9cb29b774a817
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agoFixes crash in content::URLRequestContextGetterEfl::GetURLRequestContext()
Arnaud Renevier [Sat, 25 Oct 2014 00:56:04 +0000 (17:56 -0700)]
Fixes crash in content::URLRequestContextGetterEfl::GetURLRequestContext()

The crash happens because URLRequestContextGetterEfl outlives web context.

Since ResourceContextEfl is deleted in the IO thread, it outlives the web
context. So this patch also set it's getter_ member to in ~BrowserContextEfl.
That way, the URLRequestContextGetter doesn't outlive the web context.

Additionally, DCHECK(BrowserThread::DeleteSoon(BrowserThread::IO, FROM_HERE,
resource_context_)); is a noop in non debug build. So BrowserThread::DeleteSoon
wasn't actually called in release mode.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8063
Reviewed by: Antonio Gomes, Balazs Kelemen, Yi Shen

Change-Id: Icb35b71fb014824f4f93c3b930723e8796dbefbf
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agoScreen is upside down
KANGYONG PARK [Mon, 10 Nov 2014 02:40:48 +0000 (11:40 +0900)]
Screen is upside down

After changing the screen orientation to landscape, view is upside down.
This patch fixes vertex attributes regarding orientation angle (90/270).

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8608
Reviewed by: Antonio Gomes, Viatcheslav Ostapenko

Change-Id: Ib9dfc354d782c851ab26bb3cc1891042e4e1971d
Signed-off-by: KANGYONG PARK <kaka.park@samsung.com>
6 years ago[tizen][M34-Merge] Use setScrollOffset instead of setScrollPosition() API
qi1988.yang [Wed, 29 Oct 2014 07:11:21 +0000 (15:11 +0800)]
[tizen][M34-Merge] Use setScrollOffset instead of setScrollPosition() API

setScrollPosition is deprecated. Modify OnSetScroll in render_view_observer_efl.cc

Relate to patch(M34):http://suprem.sec.samsung.net/gerrit/#/c/966/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8197
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I12b050445be7f383262d018a1783309bd930e814
Signed-off-by: qi1988.yang <qi1988.yang@samsung.com>
6 years ago[M40] Fix black screen issue
Prashant Nevase [Sat, 8 Nov 2014 14:47:34 +0000 (20:17 +0530)]
[M40] Fix black screen issue

For transport surface we were creating TextureImageTransportSurface
which used to call swap buffers. In following patch
https://codereview.chromium.org/615503004
TextureImageTransportSurface is removed as it was deprecated. The
alternative mechanism is composite-to-mailbox.

In this patch, OnSwapCompositorFrame is implemented to support
composite-to-mailbox.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8577
Reviewed by: Kangil Han, SeungSeop Park

Change-Id: I25d2fd787044d111243ec6f630526d67f3bb00c8
Signed-off-by: Prashant Nevase <prashant.n@samsung.com>
6 years ago[M40] Switch to trunk's webrtc and libjingle_source_talk repo.
SeungSeop Park [Mon, 3 Nov 2014 01:51:39 +0000 (10:51 +0900)]
[M40] Switch to trunk's webrtc and libjingle_source_talk repo.

Updated .gclient and sync_repo.sh to get newly added trunk repositories
instead of upstream's: s-webrtc.git and s-libjingle_source_talk.git.

In addition, to deal with any future repo additions, patch adds an
additional step to check the existing repo url in target path against
the desired repo url and remove the path if the two urls do not match.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8410
Reviewed by: Kangil Han, Piotr Tworek, SeungSeop Park

Change-Id: I8b9fd9499a3e1433c43bd4685e9821b60331a5d3
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years ago[M40] Chromium upversion to m40_2202 branch
SeungSeop Park [Thu, 30 Oct 2014 23:59:52 +0000 (08:59 +0900)]
[M40] Chromium upversion to m40_2202 branch

* Note for M40 rebase:
- The gyp target name components.gyp:sessions in M39 is now
  components.gyp:sessions_content,
- Clipboard must now have a derived class

Contributor: arno renevier (a.renevier@samsung.com)

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8410
Reviewed by: Antonio Gomes, Kangil Han

Change-Id: I46b0dbb58903b7e421f66bee6e048af215d797cc
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years agofixup! [M40] Force define DL_ARM_NEON_OPTIONAL to bypass mobile/tv build break in...
SeungSeop Park [Mon, 10 Nov 2014 04:52:39 +0000 (13:52 +0900)]
fixup! [M40] Force define DL_ARM_NEON_OPTIONAL to bypass mobile/tv build break in M40.

Patch fixes TV build.
Reviewed by: Antonio Gomes, Kangil Han, SeungSeop Park

Change-Id: I9fbbe47a1a3c611115f3ad06b2ea6780fdb6342c
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years agoRevert "[M34-Merge] Add CSS view-mode media feature support (chromium-efl)"
SeungSeop Park [Mon, 10 Nov 2014 01:16:50 +0000 (10:16 +0900)]
Revert "[M34-Merge] Add CSS view-mode media feature support (chromium-efl)"

This reverts commit 0d5f6191b7e10c582dfc7fdf8a27269dda3a25ff.
Reverting the patch to unblock M40 switching because this change is
accompanied by a s-blink change which is however not yet available
in M40 branch.
TODO: Re-land this in M40.
Reviewed by: Antonio Gomes, Kangil Han, Laszlo Gombos

Change-Id: I551c8837c00ff0dd5b065f75808ed5ba2fea539e
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years ago[M34-Merge] Add CSS view-mode media feature support (chromium-efl)
Zhang-hoon Oh [Tue, 28 Oct 2014 09:20:40 +0000 (18:20 +0900)]
[M34-Merge] Add CSS view-mode media feature support (chromium-efl)

To achieve Web App compatibility for Blink on Tizen, CSS
"view-mode" media feature support should be resurrected.
There is a strong requirement from WRT developers.
This patch is the "chromium-efl" side of a two-sided
patch.

"s-blink" side change-id: I814898d47f18136eed10f05857c16ed10e34cb27

About M34 Patches:
* M34 patches are reorganized.
* Commits:
** http://suprem.sec.samsung.net/gerrit/#/c/1047/
** http://suprem.sec.samsung.net/gerrit/#/c/1906/
** http://suprem.sec.samsung.net/gerrit/#/c/1907/
* Original Author:
** The original author was kaka.park@samsung.com.

Spec.: http://www.w3.org/TR/view-mode/
     (http://www.w3.org/TR/2012/REC-view-mode-20120619/)

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=7803
Reviewed by: Balazs Kelemen, SeungSeop Park, Yi Shen

Change-Id: I49b5333fb1e4b759fccca8c54ebc6fdd034fa155
Signed-off-by: Zhang-hoon Oh <dennis.oh@samsung.com>
6 years agoAdded --reset option to sync_repo.sh
Tomasz Olszak [Mon, 3 Nov 2014 09:04:58 +0000 (10:04 +0100)]
Added --reset option to sync_repo.sh

This option allows to delete local changes and ensure that e.g. daily builds
are based on clean sources. Also this allows to fix the state of repositories
inside src which where modified by some unintentional process.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8173
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: Ib95b6c4fcd2f58e2e1626f3ca9f9dfc6ecb71e73
Signed-off-by: Tomasz Olszak <t.olszak@samsung.com>
6 years agoRemove duplicated include paths in gyp step
SeungSeop Park [Wed, 5 Nov 2014 13:03:45 +0000 (22:03 +0900)]
Remove duplicated include paths in gyp step

Currently there are a lot of duplicated include paths in generated
ninja file, which may slow down the build process.
It's because during the gyp step we call pkg-config command multiple
times, one call per required package, and then concatenate the outputs
to get the final compile/link flags without removing any duplications.

Patch tries to mitigate this problem. The prepared solution is to
call a pkg-config command with multiple dependent packages at a time.
In addition, this patch removes code duplications of package name from
.gyp file, which would make less error-prone in on-going developement.

Original Patch: http://165.213.202.130:8080/#/c/71859/ (reworked for dev/)
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8182
Reviewed by: Balazs Kelemen, Kangil Han

Change-Id: I02ca8c04da0903fcd2c92988656db3d265d3f631
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years ago[M40] Hot-fix link error of "undefined reference to `WebRtc_GetCPUFeaturesARM'"
SeungSeop Park [Mon, 3 Nov 2014 09:10:53 +0000 (18:10 +0900)]
[M40] Hot-fix link error of "undefined reference to `WebRtc_GetCPUFeaturesARM'"

Patch hot-fixes it by providing the function definition for tizen mobile&tv.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8410
Reviewed by: Antonio Gomes, Kangil Han, Piotr Tworek, SeungSeop Park, Tomasz Olszak

Change-Id: I55f061ca2ed063802e1caf0945a8f49b1730952e
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years ago[M40] Force define DL_ARM_NEON_OPTIONAL to bypass mobile/tv build break in M40.
SeungSeop Park [Mon, 3 Nov 2014 06:12:36 +0000 (15:12 +0900)]
[M40] Force define DL_ARM_NEON_OPTIONAL to bypass mobile/tv build break in M40.

In M40, mobile build failed due to undefined omxSP_FFTInv_CCSToR_F32_vfp
in src/third_party/openmax_dl/dl/sp/api/omxSP.h:2558. The seleciton of the
API is affected by compilation flag DL_ARM_NEON_OPTIONAL.
This patch defines DL_ARM_NEON_OPTIONAL by force for Tizen mobile/tv build.

Note that a commit "2cc824e fixup! Fix Android build with profiling=1"
pushed to s-chromium by the trunk merge team seems to resolve the issue
for Android, but not enough for Tizen.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8410
Reviewed by: Antonio Gomes, Kangil Han, Piotr Tworek, Tomasz Olszak, mohan reddy

Change-Id: If2f79782246a984b02264ffa21608040c25e695c
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years agofixup! [M34-Merged] Assuring correct pointer initializations. (impl folder)
SeungSeop Park [Wed, 29 Oct 2014 11:38:55 +0000 (20:38 +0900)]
fixup! [M34-Merged] Assuring correct pointer initializations. (impl folder)

This patch does additionally in RWHV_efl.cc:
- fixes the order of member initializers to follow the order in .h
- gives default 0 for current_orientation_
- gives default true for is_hw_accelerated_ in member initialization list
- toggle local static variable scale_factor_initializer_ once initialized
Reviewed by: Antonio Gomes, Balazs Kelemen, Piotr Tworek

Change-Id: I63b6d171d87c68d0a9211e2c5a7c3f02b57e5ef3
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years agofixup! [Tizen] Switch chromium-efl / mobile development to GCC 4.8
Michal Poteralski [Tue, 4 Nov 2014 10:57:01 +0000 (11:57 +0100)]
fixup! [Tizen] Switch chromium-efl / mobile development to GCC 4.8

Currently user is NOT in root mode during deploy_libc_to_target.sh script
execution. It cause that as long as 'sdb root on' is NOT invoked the files
copy does by script cannot be performed.

The patch provides that user will be by default in root mode during script
execution.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8503
Reviewed by: Antonio Gomes, Piotr Tworek, SeungSeop Park, Tomasz Olszak

Change-Id: Id487d83eee337676dca2b260d4b046b416e9d892
Signed-off-by: Michal Poteralski <m.poteralski@samsung.com>
6 years agoRemove unused style guide from readme file
Balazs Kelemen [Wed, 29 Oct 2014 19:35:45 +0000 (15:35 -0400)]
Remove unused style guide from readme file

Most of the code is already following chromium style guide as is so let's
remove this vestige from the past.

Bug: none, this is just an internal documentation change
Reviewed by: Antonio Gomes, SeungSeop Park, Zhang-hoon Oh

Change-Id: Ic79461f3f48395e8dac5025cf35dbabb8800f856
Signed-off-by: Balazs Kelemen <b.kelemen@samsung.com>
6 years agofixup! Build ewk with gyp instead of cmake Reviewed by: Balazs Kelemen, SeungSeop...
Kamil Klimek [Mon, 3 Nov 2014 09:45:34 +0000 (10:45 +0100)]
fixup! Build ewk with gyp instead of cmake Reviewed by: Balazs Kelemen, SeungSeop Park

Change-Id: I60229e13e61a68df807592f5aad56b366d91e05c
Signed-off-by: Kamil Klimek <k.klimek@partner.samsung.com>
6 years ago[M40] Replace OVERRIDE by override within impl/.
Antonio Gomes [Sun, 2 Nov 2014 14:30:46 +0000 (06:30 -0800)]
[M40] Replace OVERRIDE by override within impl/.

In M40, base/compiler_specific.h removed OVERRIDE definition.
It means compilers that do not support 'override' can not
be used to build chromium out of the box anymore.

Patch replaces OVERRIDE by override in impl. The following
scripts were used.

$ find impl -name "*.h" -print0 | xargs -0 sed -i 's/OVERRIDE/override/g'
$ find impl -name "*.cc" -print0 | xargs -0 sed -i 's/OVERRIDE/override/g'

Based on the original work of SeungSeop Park <sns.park@samsung.com>
in http://165.213.202.130:8080/#/c/71676 .

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8410
Reviewed by: Kangil Han, SeungSeop Park, arno renevier, mohan reddy

Change-Id: Ieddb11caaf084aa920ad9f6691eb001fa835aaf1
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years ago[M40] Replace FINAL by final.
SeungSeop Park [Mon, 3 Nov 2014 01:49:26 +0000 (10:49 +0900)]
[M40] Replace FINAL by final.

In M40, base/compiler_specific.h removed FINAL definition.
It means compilers that do not support 'final' can not
build used to build chromium out of the box anymore.
So, patch replaces FINAL with final.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=8410
Reviewed by: Antonio Gomes, Kangil Han

Change-Id: I5d52196d157956a5a7ec8b7c3e5f628101afb4aa
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years agoCommonize boilerplate - ewk
Taeho Kim [Wed, 29 Oct 2014 09:34:20 +0000 (18:34 +0900)]
Commonize boilerplate - ewk

This patch commonizes the boilerplate, and copyright.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=7963
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I537f01bc31e309b43350b14ed2df336d4674d9de
Signed-off-by: Taeho Kim <th81.kim@samsung.com>
6 years agoCommonize boilerplate - impl
Taeho Kim [Wed, 29 Oct 2014 13:01:34 +0000 (22:01 +0900)]
Commonize boilerplate - impl

This patch commonizes the boilerplate, and copyright.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=7963
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: Ib87ccaf5e1e76aefccb35dae75ff1babb3eca064
Signed-off-by: Taeho Kim <th81.kim@samsung.com>
6 years agofixup! [Tizen] Re-factor WebContents usage by EWK API implementation
zhaoze [Fri, 24 Oct 2014 18:01:54 +0000 (14:01 -0400)]
fixup! [Tizen] Re-factor WebContents usage by EWK API implementation

Rename member variables in class InputPicker, to keep
the variable name consistent with chromium code
style.
Reviewed by: Antonio Gomes, Balazs Kelemen, Kamil Klimek

Change-Id: I6a2783ab69bb3b08e9afcb072519a3d9202c95d7
Signed-off-by: zhaoze <zhaoze.zhou@partner.samsung.com>