Karol Furmaniak [Tue, 19 May 2015 07:24:32 +0000 (09:24 +0200)]
Do not forward shift key to prevent shift keydown event.
When upper case letter is entered there are two events
Shift + letter key. Do not forward shift key to
prevent shift keydown event.
WCS-TC_90
Test case:
Type "A" in text box.
Expected result:
keydown
keypress
compositionend
input
keyup
Actual result:
keydown
keydown
keypress
input
keyup
keyup
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12739
Reviewed by: Antonio Gomes, Hyunhak Kim, Piotr Grad, Piotr Tworek
Change-Id: Ic5945947466c85a064d7603d5bfab8cc5c0162a9
Signed-off-by: Karol Furmaniak <k.furmaniak@samsung.com>
Antonio Gomes [Fri, 22 May 2015 19:30:53 +0000 (12:30 -0700)]
Update our ozone snapshot to be m42-compatible.
The SHA was extracted for crosswalk-m42, which
builds with ozone enabled by default.
For reference, see https://github.com/crosswalk-project/crosswalk/blob/crosswalk-13/DEPS.xwalk#L22
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12490
Reviewed by: Hyunhak Kim, arno renevier
Change-Id: I0c850549b823f65cda1be9b492a7bd0926211042
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Piotr An Nguyen [Mon, 11 May 2015 16:21:08 +0000 (18:21 +0200)]
[M40 Merge] Removing file:// protocol handler specific for WRT.
This is no longer needed on Browser Process side after architectural changes.
Original gerrit: http://165.213.202.130/gerrit/#/c/79548/
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=10969
Reviewed by: Antonio Gomes, Daniel Waślicki, Hyunhak Kim, Michal Roj, Ming Jin, Piotr Tworek
Change-Id: Ie6e1d71396abbc33f019b6d5af56a38ffcc845ee
Signed-off-by: yh106.jung <yh106.jung@samsung.com>
Antonio Gomes [Thu, 21 May 2015 22:09:39 +0000 (15:09 -0700)]
[m42] Add arm64 support to ffmpeg
The following commands were executed within arm64 GBS wayland chroot:
$ # enter chroot
$ gbs chroot --root ~/GBS-ROOT-3.0-MOBILE-ARM64-WAYLAND/local/BUILD-ROOTS/scratch.aarch64.0/
$ # copy over src/third_party/ffmpeg to chroot (e.g.: /home/abuild)
$ cd ffmpeg
$ ./chromium/scripts/build_ffmpeg.py linux arm64
$ ./chromium/scripts/copy_config.sh
$ # edit chromium/scripts/generate_gyp.py; add "arm64" to support platforms.
$ python chromium/scripts/generate_gyp.py
$ edit ffmpeg.gyp; add "asm_sources" to the build
$ # generate the patch against a unchanged ffmpeg copy
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12490
Reviewed by: SeungSeop Park, arno renevier
Change-Id: I7aaab106225cf9a65d97977e9c24220d7ab863c5
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Youngsoo Choi [Thu, 7 May 2015 09:35:22 +0000 (18:35 +0900)]
[tizen v3.0] Upversion tts lib
On tizen v3.0, the latest tts lib does not support variable
'tts_speed_e' and 'tts_voice_type_e'.
So, each variables are defined as 'void' and 'int',
as it was in tizen v2.4.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12933
Reviewed by: Antonio Gomes, SeungSeop Park
Change-Id: I49bcd9caeeac377639d4d4ed4a61fbd6f42c11b7
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Michał Pakuła vel Rutka [Thu, 21 May 2015 11:04:27 +0000 (13:04 +0200)]
Fix EWK unit test build
One of EWK unit tests uses ecore_x_display_get, thus requires ecore-x as a
dependency. Ecore-x was added to ewk-test.gypi.
Reviewed by: Antonio Gomes, Piotr Tworek
Change-Id: Ie694dadff689707d286e52aefcaefff875f93657
Signed-off-by: Michał Pakuła vel Rutka <m.pakula@samsung.com>
Piotr Tworek [Wed, 20 May 2015 11:21:22 +0000 (13:21 +0200)]
Pull the plug in old source files from impl/ directory.
Not used for anything, just left for reference. There is no reason to
keep them around anymore.
Reviewed by: Antonio Gomes, Hyunhak Kim
Change-Id: I2db8d7ee4a8c5916cc873d21dfffb933b23b74fb
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Arnaud Renevier [Mon, 18 May 2015 22:09:22 +0000 (15:09 -0700)]
fixup! Remove Xlib dependency in chromium_impl/ui
Now that we don't reimplement OpenNewXDisplay, we don't need to depend
on Xlib.h anymore in x11_types_override.cc
Reviewed by: Antonio Gomes, Piotr Tworek, Youngha Jung
Change-Id: I3a21d394bff6cd2516e966320eea432c5a00dd66
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Karol Furmaniak [Wed, 13 May 2015 12:26:43 +0000 (14:26 +0200)]
JavaScript window.onfocus does not work.
[Problem]: JavaScript window.onfocus does not work.
[Solution]: Added call WasShown on focus in and
Blur on focus out.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=12713
Reviewed by: Hyunhak Kim, Piotr Tworek
Change-Id: I18b2d2387c4bea525c770e7fc484b7bda877ce6c
Signed-off-by: Karol Furmaniak <k.furmaniak@samsung.com>
Antonio Gomes [Tue, 19 May 2015 15:29:28 +0000 (11:29 -0400)]
Make egde effect effective
Now that Edge Effect is brought up (see [1]), patch
make it effective for scroll/pinch gestures.
What is does? It hides the overscroll indicator
when it is not needed.
[1] http://165.213.202.130/gerrit/#/c/76762/
:
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=13002
Reviewed by: Piotr Tworek, arno renevier
Change-Id: I98a80374d6b8424b9af80a862c5795b21fc28a00
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Tue, 19 May 2015 15:25:42 +0000 (11:25 -0400)]
fixup! [dev/m42][uBrowser] One point touch.
We can use the same method as multi touch callback for
up/down/move events.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=12716
Reviewed by: Piotr Tworek, arno renevier
Change-Id: Iae532cdf70068f4d4cd17c2ca874eb2e57b0b3d7
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Mon, 18 May 2015 18:28:10 +0000 (14:28 -0400)]
Remove "tilt zoom" and EWK motion code.
This is how pinch zoom workflow was:
0- EWebView::SubscribeMotionEvents (motion,zoom smart callback)
1- EWebView::HandleTouchEvents
(if > 2 touch points)
2- wkext_motion_tilt_start
3- MotionUI:InitializeTiltToZoom
(set up tilt callback, initialize sensor)
4- MotionUI::tiltCallback fires which calls
5- MotionUI::tiltToZoom which fires "motion,zoom" smart trigger)
6- EWebView::OnMotionZoom calls
7- RWHVEfl::makePinchZoom
8- RWHVEfl::HandleGesture
9- RWHVEfl::FilterMotion
10- wkext_motion_tilt_position_update
(...)
After http://165.213.202.130/gerrit/#/c/79685/ and
http://165.213.202.130/gerrit/#/c/79684/ pinch is
brought up, and works by directly passing Motion
(multi touch) events to chromium.
Now the follow is:
0- listen to EVAS_CALLBACK_MULTI_DOWN,MOVE,UP from RWHVEfl
1- RWHVEfl::On{Mouse,Pointer}{Down,Move,Up}
2- RWHVEfl::ProcessTouchEvents
(create motion events)
3- pass events down to chromium
The whole tilt zoom machinery can then be removed.
PS: Note that since http://165.213.202.130/gerrit/#/c/69834/
all MotionUI code path is disabled.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=13002
Reviewed by: Piotr Tworek, arno renevier
Change-Id: If15f48fd2ff199450fcdfab01e0f5f667176fe89
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Thu, 14 May 2015 20:18:38 +0000 (16:18 -0400)]
Remove GCC 4.6 support
All our supported platforms have GCC 4.8 or higher available.
That includes Tizen TV 2.2.1, 2.4 and 3.0, as well as Tizen
Mobile 2.4 and 3.0.
The last known Tizen platform that depends on GCC 4.6 is
Tizen 2.3, which is not supported by the current dev and
beta branches.
Additionally, since the m42 upversion and the Xwalk refactor
work switched over to m42, GCC 4.6 specific hacks were not
carried over to s-chromium and s-blink. So the code being
removed is non-functional anymore anyways.
The only file left over is gcc46_compat_wrappers/ecore_x_wrapper.h
which will be used to support wayland. I deliberately
left this file over to avoid merge conflicts with that work.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=12987
Reviewed by: Piotr Tworek, SeungSeop Park
Change-Id: Id3cd6ecff1a41a40f300d70d67e7b14bf7ac37ab
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Youngsoo Choi [Mon, 18 May 2015 12:21:33 +0000 (21:21 +0900)]
To generate proper out folder according to architecture and tizen version
1) The tizen v2.2.1 and tizen v3.0 tv builds share a same out folder,
named 'out.tv.arm'.
Those builds should have separated folders each other with tizen version.
2) Each arm64, x86_64, and ia32 build generates a folder 'out.{profile}.i386'.
Those builds should generate folders, having proper architecture name,
instead of i386.
This patch suggests the name of the folder as 'out.{tizen_version}.{target}.{architecture}'.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=13001
Reviewed by: Antonio Gomes, SeungSeop Park
Change-Id: I184e78cc339e6c0c44d612dfe6df848eec70fb7b
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Antonio Gomes [Thu, 14 May 2015 16:18:11 +0000 (12:18 -0400)]
bringup double tap to zoom.
Patch toggles on the existing blink patch according
to the touch status.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=12986
Reviewed by: Antonio Gomes, Hyunhak Kim, Piotr Tworek
Change-Id: Ia831980b2e1321001952952b69a9057bdb72f469
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Jongsoo Yoon [Mon, 18 May 2015 08:02:57 +0000 (17:02 +0900)]
[Beta m42] Change branch to beta/m42_2311_t for CT s-* repo sync
[Branch Info]
- chromium-efl : beta/m42_2311_t
- s-chromium : beta/m42_2311_t
- s-blink : beta/m42_2311_t
- s-webrtc : beta/m42_2311_t
- s-libjingle : beta/m42_2311_t
- s-skia : beta/m42_2311_t
- s-v8 : beta/m42_2311_t
Reviewed by: Piotr Tworek, SeungSeop Park
Change-Id: Ic9434b0f8248ec7adabb6b3eb053a2fa2ffbdb69
Signed-off-by: Jongsoo Yoon <join.yoon@samsung.com>
Antonio Gomes [Fri, 15 May 2015 20:42:49 +0000 (16:42 -0400)]
fixup! Do not use elm_config_preferred_engine_set anymore
Call elm_config_accel_preference_set on mini_browser creation
otherwise it fails at gl context initialization.
PS: This is similar to uBrowser.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=12675
Reviewed by: SeungSeop Park, arno renevier
Change-Id: I10447a1104e255948e5eaad6c9bf71dafade4bc3
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Piotr Tworek [Thu, 14 May 2015 09:32:28 +0000 (11:32 +0200)]
Improve renderer process crash handling in EWK.
Since the refactoring removed a lot of unnecessary inter dependencies
between EWK and content layers we no longer need to re-create
WebContents each time the renderer process has crashed. On UI process
side renderer process is only represented by the RenderWidgetHostView
class. This fix brings a big benefit of not loosing browsing history
whenever renderer process crashes. Such state is associated with
WebContents not RWHV.
This patch also includes additional bug fix which makes sure
LoadHTMLString function is not called directly from a callback informing
us about a crash. Calling any load method directly from the crash
callback triggers a DCHECK in mojo code when renderer process crashes
multiple times for the same WebContents instance.
Reviewed by: Antonio Gomes, SeungSeop Park, Youngha Jung
Change-Id: Id220b563bea90e6dedd0b55a26ef856fb1528069
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Piotr Szawdyński [Mon, 4 May 2015 15:28:43 +0000 (17:28 +0200)]
Added macro support OS(TIZEN), OS(TIZEN_MOBILE) and OS(TIZEN_TV) for WebKit.
1) This patch allows to replace statement #if defined(OS_TIZEN_MOBILE) with
statement #if OS(TIZEN_MOBILE). Usage of OS macro is required because
buildbot has presubmit check which dissalow to commit patch which contains
macro which matches to pattern OS_*.
2) Added dependency to library capi-system-system-settings.
Library 'capi-system-system-settings' is used by WebKit project for Tizen build
(WebKit/Source/platform/fonts/skia/FontCacheSkia.cpp). Currently OS(TIZEN) macro
was not working and after fixing macro issue project won't compile, so we
need to fix also dependency.
Reviewed by: Antonio Gomes, Piotr Szawdyński, SeungSeop Park, Tomasz Czekala
Change-Id: I36694bce7090a02e53982c3d936dc4a997377470
Signed-off-by: Piotr Szawdyński <p.szawdynski@samsung.com>
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Antonio Gomes [Thu, 14 May 2015 16:04:07 +0000 (12:04 -0400)]
fixup! [dev/m42][uBrowser] Wrong output on double tap.
Make double tap toggle ON/OFF tied to touch events status.
Change-Id: Ie54dc656efa159cc75be8881bf86c2223cad7d4e
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Arnaud Renevier [Tue, 12 May 2015 20:19:50 +0000 (13:19 -0700)]
Remove Xlib dependency in chromium_impl/ui
We remove Xlib dependency, or rely on ecore-x instead.
We stop implementing ScreenEfl::GetCursorScreenPoint and
ScreenEfl::GetWindowUnderCursor.
Those methods are only used with toolkit view, which is not enabled for
our port. Those methods are also left unimplemented in android or ios.
In ScreenEfl::GetPrimaryDisplay, we use ecore-x methods instead of X11
methods.
In MakeWebKeyboardEvent, we hardcode all windowsKeyCode results, as is
currently done in quicksilver. This method is fragile, and won't work
currently on all layout keyboards. But it will still give better
results than the current behavior (where most keys are discarded, and
result in a 0 keyCode).
Additionally, we also set event.modifiers, which also used to implement
KeyBoardEvent.location.
Reviewed by: Antonio Gomes, Piotr Tworek
Change-Id: I1763ed7e43b2f216916abf75be56d0ca9ba2a84c
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Arnaud Renevier [Wed, 29 Apr 2015 23:30:49 +0000 (16:30 -0700)]
Bringup quota management api
Call ewk_view_quota_permission_request_callback_set in ubrowser.
Requests will always be approved
In order to use this feature, browser must be built with
enable_hardcode_runtime_features set to 0, and launched with
--enable-experimental-web-platform-features
Fixes unittest to use new HTML5 quota management api
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=8472
Reviewed by: Antonio Gomes, Piotr Tworek, Youngha Jung
Change-Id: I4d0064426c1ea49c9f2759ad1f37bf86ac68fa7e
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Michał Obrembski [Tue, 5 May 2015 09:19:05 +0000 (11:19 +0200)]
[Rebase][dev/m42_2311] Autofill bringup
This commit adds missing Autofill support from m40.
Tested on org.tizen.browser on gmail registration page
(http://accounts.google.com/SignUp)
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=12547
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=12730
Reviewed by: Antonio Gomes, Piotr Tworek, SeungSeop Park
Change-Id: I63d2f633ab8c672695e8de4b8bd3f60515177c71
Signed-off-by: Michał Obrembski <m.obrembski@samsung.com>
Arnaud Renevier [Wed, 6 May 2015 22:00:02 +0000 (15:00 -0700)]
Do not use elm_config_preferred_engine_set anymore
As of elementary 1.13, elm_config_preferred_engine_set is deprecated.
Opengl is chosen by default, and using elm_config_preferred_engine_set
does nothing anymore.
Also, instead of setting ELM_ACCEL environment variable, we can now use
elm_config_accel_preference_set. With that information, elm will pick
the right engine automatically for us.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=12675
Reviewed by: Antonio Gomes, Hyunhak Kim, Piotr Tworek
Change-Id: Ib72afeb59a3d447879166bed632f4a902f5fc0ae
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Youngsoo Choi [Thu, 26 Feb 2015 11:07:40 +0000 (20:07 +0900)]
[tizen v3.0] Improving build time with optional rpmlint
The rpmlint, verifying syntax of spec file, has been
supported from tizen v3.0.
GBS build always runs rpmlint at the end of the build,
and it takes too much time.
To improve the build time for developers,
this patch provides a option for enabling or disabling rpmlint.
To enable the rpmlint, following command can be used.
>> ./build/build_tv.sh -P tztv_v3.0 --define '_rpmlint 1'
By default, the rpmlint is disabled.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12968
Change-Id: I687582583b8eb8b0747a7f1355f2e9d32075e120
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Youngsoo Choi [Wed, 13 May 2015 04:40:48 +0000 (13:40 +0900)]
fixup! Don't use forward delaration of Elm_Object_Item.
The 'Evas.h' is already included in 'Elementary.h'.
Original message:
The definition of the object is different in Elementary 1.13. Since this
is what Tizen 3.0 currently uses the code failed to compile. Instead of
using forward delclarations simply include Elementary.h header. This
should ensure the code builds no matter what version of EFL/Elementary
is being used.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=11876
Change-Id: I78185632e0d61fcbb25e20bddc0aaf6884565532
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Youngsoo Choi [Tue, 24 Feb 2015 12:35:48 +0000 (21:35 +0900)]
[tizen v3.0] Added repositories for local build of OBS
The OBS repositories are included in the file 'gbs.conf'.
Using newly added repository, developers can do OBS build locally.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12969
Change-Id: Id2776ece309c10ea0db5c33d04dfec23dca1766d
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Youngsoo Choi [Tue, 3 Mar 2015 10:11:04 +0000 (19:11 +0900)]
[tizen v3.0] changes of smack rule as per instruction from tizen v3.0
On Tizen v3.0, most of smack rules are removed.
So, current smack rules need to be changed,
as per the smack instruction from Tizen v3.0.
This changes are confirmed by Tizen platform team.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=11716
Reviewed by: Antonio Gomes, KC Park
Change-Id: Id01dbcfaf25dc9f145d832a51c946428c9ad270a
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Youngsoo Choi [Tue, 3 Feb 2015 05:29:40 +0000 (14:29 +0900)]
Added tizen v2.2.1 hawkp gbs repository
Until tizen v3.0 build is fully ready,
We need to support tizen v2.2.1 hawkp.
bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=11069
Reviewed by: Antonio Gomes, KC Park
Reviewed by: Antonio Gomes, KC Park, SeungSeop Park, Sungsik Han, commitbot
Change-Id: Id2912f55182ab295462ebda44cbb848a4bcb9ae5
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Antonio Gomes [Tue, 12 May 2015 19:21:23 +0000 (15:21 -0400)]
[dev/m42][uBrowser] Wrong output on double tap.
RWHVEfl makes use of chromium's GestureRecognizer class
in order to provide gestures support. GestureRecognizer
itself is abstract and gets concrete by its child,
GestureRecognizerImpl, which holds and encapsulates
a GestureProviderAura instance. Problem is that GestureProviderAura
has 'double tap' support disabled by default.
Patch provides the following solution:
- Specialize GestureRecognizerImpl into a class called
GestureRecognizerImplEfl. The later overrides and makes
::GetGestureProviderForConsumer public, granting access to
its GestureProviderAura instance.
- Override GestureRecognizer::Create method, and instantiate
a GestureRecognizerImplEfl object (instead of GestureRecognizerImplEfl).
Once accessible, calling GestureProviderAura::SetDoubleTapSupportForPlatformEnabled
fixes the problem.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=12717
Change-Id: I81d5a06b80a453acd5b58d68a38e55648822a8b3
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Youngsoo Choi [Mon, 11 May 2015 07:29:59 +0000 (16:29 +0900)]
[tizen v3.0] Use app_get_data_path (instead of app_get_data_directory)
Tizen v2.4 and v3.0 do not support the deprecated
API 'app_get_data_directory'.
The API 'app_get_data_path' is suggested to use instead.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12956
Change-Id: Id5784a19c6df8f4286cf66dd3ba14b35ac289f84
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Antonio Gomes [Tue, 12 May 2015 18:25:32 +0000 (14:25 -0400)]
[dev/m42][uBrowser] One point touch.
Up to now, single point gestures (including panning, fling,
clicking) were handled through a Mouse->to->Pointer
events conversion solution. It happens on the RWHVEfl level,
and benefits both EWK and content apps.
However, multi touch gestures (e.g. pinch zoom) were not functional.
Patch fixes that by start listening to EVAS_CALLBACK_MULTI_{DOWN,MOVE,UP}
events, and forward them to chromium's (Aura) the existing gesture
recognition machinery to get them handled.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=12716
Change-Id: I1662c611af8ceba5ba20398adbad9b9bb5ada844
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Tue, 12 May 2015 18:22:42 +0000 (14:22 -0400)]
[Clean up]Remove unused UpdateWebTouchEventAfterDispatch helper
SSIA.
Change-Id: I7e83b1fbb073a78c116aec5510ea81cc6f321fa6
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Arnaud Renevier [Tue, 5 May 2015 23:30:58 +0000 (16:30 -0700)]
Move text_encoding_map_efl outside of ewk public directory
In addition, we make TextEncodingMapEfl a singleton class.
And also, we optimize the search by using a hash set instead of a list.
Search time in now constant time.
Change-Id: I22d8830b2d04c6bd4d4d4ba56fadb42950479d59
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Arnaud Renevier [Wed, 6 May 2015 22:45:09 +0000 (15:45 -0700)]
fixup! Use system request url getter for AccessTokenStore
Remove unneeded code.
Change-Id: Ia6200bcb392c1a192ffb298b81c97bea15fa59d3
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Arnaud Renevier [Tue, 5 May 2015 21:19:11 +0000 (14:19 -0700)]
Remove tw_webview_delegate
Use WebViewDelegateEwk directly instead of
tizen_webview::WebViewDelegate
Additionally, this patch moves WebViewEvasEventHandler out of
tizen_webview namespace, where it doesn't belong.
So we can now get totally rid of tizen_webview
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=11619
Change-Id: Idd9fdc5a007921422c27affe5fd61652442b3d18
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Arnaud Renevier [Wed, 6 May 2015 22:55:37 +0000 (15:55 -0700)]
Fixes various compiler warnings
Fixes an unused variable and a suggested parentheses warning.
Also, WrtUrlParseBase is a virtual class. So, it needs a virtual
destructor.
Change-Id: I363167fd6dc49ef68886a274442f80718a0071da
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Umesh Kumar Patel [Thu, 16 Apr 2015 10:11:01 +0000 (15:41 +0530)]
[M40 Merge][MM] GStreamer 1.2 Multimedia Support
Following are the m40 patches merged
MM Common:
GST 1.2 Up version
http://165.213.202.130/gerrit/#/c/75366/
Video corruption issue with AppSink
http://165.213.202.130/gerrit/#/c/75610/
ME:
Replace playbin with uridecodebin and fimcconvert
http://165.213.202.130/gerrit/#/c/75740/
Buffered data indication is incorrect for Media Element.
http://165.213.202.130/gerrit/#/c/75819/
HTML5 Video loadstart / progress event TCT fails
http://165.213.202.130/gerrit/#/c/76213/
Use S/W codec for Media Element
http://165.213.202.130/gerrit/#/c/76681/
MSE:
S/W codec support for MSE
http://165.213.202.130/gerrit/#/c/75777/
Fix video corruption when video resolution is changed
http://165.213.202.130/gerrit/#/c/75828/
Youtube playback is not working with PIXMAP
http://165.213.202.130/gerrit/#/c/76137/
Enabling MSE for desktop
http://165.213.202.130/gerrit/#/c/76860/
WebRTC:
Enable H/W accelerated H.264 support for WebRTC on Tizen
http://165.213.202.130/gerrit/#/c/73796/
Change OMX control-rate from VBR to CBR
http://165.213.202.130/gerrit/#/c/74361/
OMX codec integration on m0 target
http://165.213.202.130/gerrit/#/c/75671/
Appsink implementation with fimcconvert
http://165.213.202.130/gerrit/#/c/75791/
Memory leak fixes
http://165.213.202.130/gerrit/#/c/76561/
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12536
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12777
Reviewed by: Antonio Gomes, Piotr Tworek, SeungSeop Park
Change-Id: Icc88734422b24f684a49a3f27bb324ba6370f8fc
Signed-off-by: Umesh Kumar Patel <umeshk.patel@samsung.com>
sonal.g1@samsung.com [Tue, 5 May 2015 07:50:01 +0000 (13:20 +0530)]
[M40-Merge][WebRTC] Fix audio crash fix by passing ecore Display open
There is some problem with ecore_x_display_get() in chromium-efl.
So replacing the call with XOpenDisplay() as M34 does to fix the
browser crash during webRTC audio enabled.
Original gerrit: http://165.213.202.130/gerrit/#/c/75331/
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=9217
Change-Id: I13fc3e56d56b0ffcf4d00c6c4295f56ee54b7909
Signed-off-by: sonal.g1@samsung.com <sonal.g1@samsung.com>
Suneel Kota [Fri, 24 Apr 2015 10:06:11 +0000 (15:36 +0530)]
Revert "[Workaround] Disable EFL version of GPUInfoCollector" This commit reverts the workaround to disable EFL version of GPUInfoCollector
BUG: http://107.108.218.239/bugzilla/show_bug.cgi?id=12551
This reverts commit
119b241911714e6f1de6daf25c750c0c553fbb04.
Reviewed by: Antonio Gomes, DaeHyun Ko, Siba Samal
Change-Id: Ia3f6da5dababf377c02844aff033ced013cd09f2
Signed-off-by: Suneel Kota <suneel.kota@samsung.com>
Taeho Kim [Thu, 7 May 2015 05:02:01 +0000 (14:02 +0900)]
Modify DEPS.xwalk configuration
Fetch dev/m42_2311 branch instead of exp/dev/m42_2311 from s-chromium
Change-Id: I8333ed69ea0677bb427ada365c71dcd99ac5eb94
Signed-off-by: Taeho Kim <th81.kim@samsung.com>
Arnaud Renevier [Wed, 6 May 2015 19:49:58 +0000 (12:49 -0700)]
Implement vibration in ubrowser
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=10491
Reviewed by: Antonio Gomes, Hyunhak Kim
Change-Id: I544499034703130f027edad6008ab9c0abc2ec55
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Arnaud Renevier [Wed, 6 May 2015 21:03:48 +0000 (14:03 -0700)]
Bringup ewk_view_history_get
ewk_view_history_get is guarded by EWK_BRINGUP.
But actually, it is implemented, and unittests work fine.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=12675
Change-Id: I12727434e747da3a8f304a82ce21408d63510171
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Arnaud Renevier [Wed, 6 May 2015 00:24:11 +0000 (17:24 -0700)]
fixup! Remove tw_webview
restore correct EWK_VIEW_IMPL_GET_OR_RETURN
Change-Id: Ic4521134ca159e2d619b7dfa0499bf048773f00c
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Arnaud Renevier [Mon, 4 May 2015 22:52:04 +0000 (15:52 -0700)]
Remove tw_webview
Use EWebView directly
We need to make EWebView destructor and constructor public so they can
be used from CreateWebViewAsEvasObject
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=11619
Change-Id: I90dc26c0d6ffec593d9abf30df3ea87634278516
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Arnaud Renevier [Fri, 24 Apr 2015 05:43:57 +0000 (22:43 -0700)]
Remove tw_web_context
Use Ewk_Context directly
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=11619
Change-Id: Ica1a38660a38fdc2ac778d25989a2a798553cb89
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Antonio Gomes [Mon, 4 May 2015 22:24:32 +0000 (18:24 -0400)]
Crash @WebContentsDelegateEfl::DidFinishLoad
When a new window is being opened, it might happen that
about:blank is loaded first, and the actual target URL load
is triggered later.
When that happens, two calls to WebContentsDelegateEfl::DidFinishLoad
are made during the load.
When the first call is made (associated to about:blank) no entry
is added to the NavigationController control, and GetVisibleEntry
returns NULL.
Our implementation wrongly assumed that calling this method would
always return non-null values, and an DCHECK was added.
Patch fixes that assumption by replacing the DCHECK by a null-check.
Note the the signature of the associated NavigationController method
also clearly states that it can return a NULL value:
https://goo.gl/RBiSEu
"
NOTE: This can be nullptr if no entry has committed!
"
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=12867
Change-Id: Ie2f2253c7ec14e15f0b67910023a59d96f71b6f6
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Arnaud Renevier [Thu, 23 Apr 2015 21:52:33 +0000 (14:52 -0700)]
WebViewEvasEventHandler does not rely on tizen_webview::WebView anymore
WebViewEvasEventHandler will use EWebView directly
This patch also renames SelectionControllerEfl::HideHandle to
SelectionControllerEfl::HideHandles (because there are two handles)
Change-Id: I47e5729c6918ec3188b837c93af7be693a073f80
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Arnaud Renevier [Wed, 15 Apr 2015 22:19:34 +0000 (15:19 -0700)]
Remove tw_settings and tw_touch_point
We use Ewk_Touch_Point instead of tw::touch_point
tw::Settings was the webview settings member of Ewk_Settings. It's the
object sent from browser process to render process. So this patch
renames it to WebViewSettings, and puts in common/ folder.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=11619
Change-Id: I62ff5383accb3065d67a3b22e2d3ac85ec7cce8c
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Arnaud Renevier [Thu, 23 Apr 2015 17:30:28 +0000 (10:30 -0700)]
Forward events to ewk
To match previous behaviour, events (focus, key and mouse) are forwarded
to ewk. If they are handled by the application, they are not forwarded
to chromium anymore.
motion events will be handled in a later patch
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=12031
Change-Id: I67d0bf3599e918353b04d8f753dc29b3fad13583
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Piotr Tworek [Thu, 30 Apr 2015 12:22:38 +0000 (14:22 +0200)]
uBrowser: Use slider elementary widget to change zoom level
This solution is much more user friendly compared to previous solution.
The user can now change page zoom level by simply dragging the slider
left, or right.
Change-Id: Iac769a47f001ae42d099bc1e670e3d3b7480583b
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Piotr Tworek [Thu, 30 Apr 2015 11:42:22 +0000 (13:42 +0200)]
Handle ui::TEXT_INPUT_TYPE_NONE in im_context_efl.cc
Without this the NOTREACHED() assert in default case can fail in certain
test scenarios.
Change-Id: Icbf9cefc8c9dc30f6f645b399c35ee1df6c8e6dd
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Piotr Tworek [Thu, 30 Apr 2015 11:39:42 +0000 (13:39 +0200)]
uBrowser: Don't rely on specific layout of EFL popups.
The popup handling code relied on the specific layout of elm popups to
find the root widget of the element. This is rather fragile and actually
stopped working on 2.4 images recently. Instead of finding popup
elementary widget manually simply store a pointer to the currently
opened popup in the WindowUI object. This should be much more reliable.
Change-Id: Iecfcdf1105d0f5cd61940b31bba7a7214798818d
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Piotr Tworek [Wed, 29 Apr 2015 14:10:38 +0000 (16:10 +0200)]
uBrowser: Add menu option to toggle EWK auto fitting setting.
This was requested by SRPOL QA. Apparently some of the tests they plan
to perform on the uBrowser require this feature. The feature is also known
as overview mode.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12737
Reviewed by: Hyunhak Kim, arno renevier
Change-Id: I141fbff5395bc0324b3b342efaf257cceeca2763
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Grzegorz Spryszynski [Tue, 28 Apr 2015 08:16:22 +0000 (10:16 +0200)]
Page rotation look weird
[Problem] In landscape mode pages looks inproper
(rotation and scale are invalid).
[Solution] Old rotation mechanism require matrixes
describing texture transformations. Now it is enougth
to use one matrix regardles of rotation.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=12721
Signed-off-by: Grzegorz Spryszynski <g.spryszynsk@samsung.com>
Change-Id: Ib8dd3472ed0b322d79308a9a96744354d566810a
Piotr Tworek [Wed, 29 Apr 2015 13:39:09 +0000 (15:39 +0200)]
uBrowser: Add support for EWK tracing feature.
This is one of the features that is currently available in mini_browser,
but is not exposed in uBrowser. This patch bridges the gap.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12849
Change-Id: I0bb293794cebaede96776839fc8093843851e097
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Piotr Tworek [Wed, 29 Apr 2015 13:10:14 +0000 (15:10 +0200)]
uBrowser: Refactor extra actions menu to use Elementary ctxpopup
Unfortunately the previous elementary menu object did not work correctly
on 2.4 targets. It was shown, but with an transparent background making
it difficult to use. This patch changes uBrowser to use elementary
ctxpopup widget as the basis of extra actions menu. This widget is
rendered properly all currently supported platforms.
While at this the patch also changes the menu options a bit. On the
refactoring branch stuff like zoom in/out does work on desktop so the patch
exposes it there. Since window resizing is also much more reliable, menu options
for resizing windows are not longer needed.
Change-Id: Ibf09267a81854b5e49f17fa79ba827275459246f
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Antonio Gomes [Wed, 29 Apr 2015 20:28:27 +0000 (16:28 -0400)]
fixup! Enable sound effect when tapping on a link
In order to closely match follow our adjustment logic, patch
performs a hit test "for tap" (instead of a point based one)
using the same "padding" as the original tap event.
That way, as long as touch adjustment picks up a link for clicking,
we will emit a click sound.
Change-Id: Ia481fdd5b7e02ff3f433480bd5e8c1de32dedc06
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Arnaud Renevier [Wed, 29 Apr 2015 18:25:06 +0000 (11:25 -0700)]
fixup! Refactor async window creation feature to avoid chromium modifications.
WebContentsEflDelegate needs a virtual destructor
Change-Id: I1f4594e61ce10c5bc9052f0acaf6a023188bbe7a
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Arnaud Renevier [Tue, 28 Apr 2015 20:37:11 +0000 (13:37 -0700)]
Bringup mime type override
ShouldOverrideMimeType works correctly. We just need to remove thing
bringup guards
Change-Id: Ia6e66d369c0fab3cbb8a375aa69df61d853d7ffb
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Arnaud Renevier [Tue, 28 Apr 2015 20:14:33 +0000 (13:14 -0700)]
Bringup ewk_view_not_found_error_page_load
We Send EwkFrameMsg_LoadNotFoundErrorPage and handle it in ewk instead
of doing that in chromium
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12675
Change-Id: I0803df5c0f6353266bdbd7d308421583518765e9
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
uzair.jaleel [Tue, 28 Apr 2015 07:45:47 +0000 (13:15 +0530)]
[M40 Merge] Fixed that webview is crushed on resizing viewport.
webView is crushed when creating/removing IME and rotating device.
It is caused by diffrence between the size of egl surface rendered
and glViewport arguments. The arguments were set by evas geometry size,
but it was different from the size of rendered surface.
M40 patch: http://165.213.202.130/gerrit/#/c/75627/
Bug: http://168.219.209.56/jira/browse/TNEF-190
Reviewed by: DaeHyun Ko, Siba Samal
Change-Id: I9ef7d67f3d05ce1744489dbd14f83fa46772046c
Signed-off-by: uzair.jaleel <uzair.jaleel@samsung.com>
Antonio Gomes [Tue, 28 Apr 2015 16:23:44 +0000 (09:23 -0700)]
[dev/m42] Wrong confirm sign on keyboard
RenderWidgetHostViewEfl needs to filter in and handle
message 'ViewHostMsg_TextInputInFormStateChanged' in order
to handle the 'Done' x 'Go' label on Tizen. Basically, when
VKB is shown and focus is in a widget within a <form> tag,
"Go" label should be used.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=12727
Change-Id: I5388be90f186e8c126f97c0f34ef30e319e32a9e
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Mon, 27 Apr 2015 02:28:28 +0000 (19:28 -0700)]
[dev/m42][uBrowser]Select picker doesn't work
In RenderFrameImpl::OnSelectPopupMenuItems, 'external_popup_menu'
is reset when an item is selected.
This is not how the Tizen browser is supposed to behave: due to
compatibility reasons with webkit browser (kiran), chromium-efl
only close the popup when user actually dismisses it (by either
pressing "DONE" or back button). Thus, the custom behavior
is implemented on RenderFrameObserverEfl::OnSelectPopupMenuItems.
Patch fixes our select implement by making use of a
hack: in order to have access to RenderFrameImpl::external_popup_menu_
it "defines" private as public before including the associated
header. This works because RenderFrameObserver (the base class) is
declared as 'friend' to RenderFrameImpl. Thus we get access to its
private members.
Associated EWK_BRINGUP macros are also removed.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=12720
Change-Id: I3f2384d79da5ea7e5a2d3d696527aa19cc7d700d
Antonio Gomes [Thu, 23 Apr 2015 18:31:33 +0000 (14:31 -0400)]
Make popup implementation also available on desktop builds.
At this point, there is nothing OS_TIZEN specific
in our popup (<select>) implementation anymore.
Patch makes it also available on desktop for the good.
Change-Id: Iaca6fb6d40865b6fc5f489a1304d1c23f2275abd
sonal.g1@samsung.com [Wed, 22 Apr 2015 16:30:58 +0000 (22:00 +0530)]
[Tizen][WebRTC][m42] Enable Camera for Odroid using CAPI
Camera was not enabled for Tizen 3.0(TV) in gyp and Video CAPI files.
Made appropriate changes for enabling it.
Reviewed by: SeungSeop Park, Venugopal S M
Change-Id: Ib83c5bbc5820e0b54373aa9680284b4d167ba56f
Signed-off-by: sonal.g1@samsung.com <sonal.g1@samsung.com>
Piotr Tworek [Tue, 28 Apr 2015 09:24:37 +0000 (11:24 +0200)]
fixup! Refactor async window creation feature to avoid chromium modifications.
Since we decided to avoid adding new fields to WebContents::CreateParams
we need to invoke WebContetntsImplEfl constructor directly and give it a
pointer to platform specific data (EWebView wrapping the WebContents)
pointer as an argument. Without it the newly created WebContents won't
know it's EWK parent.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12774
Change-Id: I67cbea770ef452af071c0ea2ee4227a814753466
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Arnaud Renevier [Mon, 27 Apr 2015 18:43:36 +0000 (11:43 -0700)]
fixup! Introduce WebContentsViewEflDelegate interface.
WebContentsViewEflDelegate needs a virtual destructor
Change-Id: I215ad06d356e4bfe96a2bd4f6cddb142e8791720
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Piotr Tworek [Mon, 27 Apr 2015 11:59:44 +0000 (13:59 +0200)]
Adjust EWK notifications implementation to dev/m42 changes.
During the weekend dev/m42 branch gained some more notification related
patches which broke EWK. This patch aims to make them compatible once
again.
Change-Id: I586f73747cbbf796884270ca10a5cc243c801628
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
yh106.jung [Wed, 22 Apr 2015 10:31:41 +0000 (19:31 +0900)]
Upgrade EFL version for desktop to 1.13.1
_Evas_Event_Key_Down provides keycode as member variable since EFL 1.10.
So we need to upgrade EFL version for desktop in order to use it on both
tizen targets and desktop.
Change-Id: I94dc850ed0e034a313c4eeee391df3a7f65041e9
Signed-off-by: yh106.jung <yh106.jung@samsung.com>
Piotr Tworek [Fri, 24 Apr 2015 13:27:31 +0000 (15:27 +0200)]
Refactor async window creation feature to avoid chromium modifications.
The goal is to competely get rid of chromium src patch and contain the
feature entirely in tizen_src. To do this we need to create our own
custom WebContentsDelegate class only used by EFL implementation of
WebContentsImpl. Additionally we can no longer use WebContents::Create
call to instantiate the class. Instead, a direct call to
WebContentsImplEfl constructor followed by Init call has to be used.
This makes EWK usage of content API a bit non standard, but allows us to
get by without any modifications to the upstream code.
Change-Id: I2f1397842e7194afc331514986d12b13705af71c
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Piotr Tworek [Fri, 24 Apr 2015 11:20:00 +0000 (13:20 +0200)]
Introduce WebContentsViewEflDelegate interface.
The idea is to avoid the need to modify upstrem WebContentsViewDelegate
interface. WebContentsViewEflDelagate is supposed to hold only extra EFL
specific extensions we'd otherwise have to put in the non Efl
counterpart of the interface. The downside is WebCotnentsViewEfl
implementation has two delegates now and care has to be taken to call
function from the correct one. Still it's probably preferable to do it
this way instead of patching chromium.
Change-Id: I6cdb0c966b11be33bf71ab3f45f172259cf063ae
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Arnaud Renevier [Thu, 16 Apr 2015 20:32:25 +0000 (13:32 -0700)]
[Rebase][dev/m42] Vibration changes
Vibration is not signaled by ipc messages anymore. Instead, mojo is
used to call methods of a VibrationManager object living in the browser
process.
This patch implements this interface. It also provides a virtual
interface device::VibrationProviderClient
On ewk side, things are simplified: instead of having
Ewk_Vibration_Client and VibrationProviderClient. There is only one
class: VibrationProviderClientEwk, which implements
device::VibrationProviderClient
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=10491
Change-Id: Ifef41e504cb0b9ea3dec8e7f9703a325c1fb3e7d
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Piotr Tworek [Thu, 23 Apr 2015 16:16:19 +0000 (18:16 +0200)]
Place usage of FrameHostMsg_LoadNotFoundErrorPage in EWK_BRINGUP ifdefs.
The feature requires additional patches in chromium that we don't want
to push to dev/m42_2311 until we're sure the same functionality can't be
implemented entirely in tizen_src.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=12675
Change-Id: I46e145fa4ed67b0384c0bc0db9284f2fdf5d6748
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Piotr Tworek [Thu, 23 Apr 2015 15:27:03 +0000 (17:27 +0200)]
Change async window creation code to simplify chromium patch.
The goal here is to minimize the change done to upstream chromium code.
The solution may have a hacky feeling to it (#define private protected),
but hopefully it should ease s-chromium maintenance burden.
Change-Id: I07d32643db4a9686027dfa6cfa93591e2c083aa1
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Piotr Tworek [Wed, 22 Apr 2015 15:24:23 +0000 (17:24 +0200)]
Adjust the code to dev/m42 platform notification changes
dev/m42_2311 branch contains some additional patches changing
PlatformNotificationData structure. Since the goal is to make
experimental/dev/master build on top dev/m42_2311 adjust our code
accordingly.
Together with: I55a0b1d3d919a1ebfa83855e9056d6253fa70fdf
Change-Id: I208a2d85e016fc661da18a4d7f5e018635de61e5
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Antonio Gomes [Wed, 22 Apr 2015 15:57:40 +0000 (08:57 -0700)]
Override content::CreateInProcessGpuThread for EFL port.
Patch makes use of the same override technique used by
RenderThemeChromiumDefault to override the theme method.
Basically, we "redefined" the method we want to override
and include the original source file containing its definition
(.cc or .cpp). Additionally, we exclude the original
implementation file from the build (in GYP).
Then we can have our own implementation of this method
from our efl specific method.
Together with: Iac24e6d38174d7dc8f0d1bb1c37bc8dac05d83c5
Change-Id: Ib7c2aeedbf76de576a25ad95d5437919fec09d3c
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Piotr Tworek [Wed, 22 Apr 2015 14:50:53 +0000 (16:50 +0200)]
Alias EwkMsgStart to ShellMsgStart
Since no content_shell code is being used in libchromium-ewk we can
safely reuse Shell IPC message ID for EWK specific messages. The goal
here is to minimize the amount of changes to upstream code.
Change-Id: I646018f8c9050f32d1d2851eedeedc347b08be38
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Arnaud Renevier [Wed, 15 Apr 2015 21:32:43 +0000 (14:32 -0700)]
Handle notification silent option
Add ewk api ewk_notification_silent_get
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=10487
Change-Id: Ia4ed78ab2ad82a6b976b2e391d2b9230afb4d440
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Piotr Tworek [Tue, 21 Apr 2015 09:50:20 +0000 (11:50 +0200)]
Reuse ui_events_helper.cc implementation in EFL port.
Normally the file is excluded from build when use_aura is set to 0.
Since we're interested in reusing it lets add it back.
Change-Id: Ib552f3bd2839deef07172ca0953164f6853032e6
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Arnaud Renevier [Tue, 14 Apr 2015 22:20:24 +0000 (15:20 -0700)]
Bring up notifications
Most of the api has been pushed to ContentBrowserClientEfl to an
interface PlatformNotificationService.
https://code.google.com/p/chromium/issues/detail?id=439950
We implement PlatformNotificationService with NotificationControllerEfl
Since embedder API for notifications change a lot on chromium side, we
guard notifications with a compile flag. That way, it will be easier to
disable notifications temporarily if needed next time we update
chromium.
NotificationControllerEfl::SetPermissions is removed because it wasn't
used.
http://107.108.218.239/bugzilla/show_bug.cgi?id=10487
Change-Id: If155b5d5b8c7fc99ebe0ef75a0b5388e849bd1dd
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Antonio Gomes [Mon, 20 Apr 2015 17:55:56 +0000 (13:55 -0400)]
fixup! fixup! [Rebase][dev/m41_2272] API changes in GestureRecognizer
In
4bae880d, patch adapted our obsolete touch handling
implementation of "acked async" touch events by replacing
::ProcessTouchEventOnAsyncAck by a "sync" variant (which is
not what was intended): ::AckSyncTouchEvent.
That caused the processing events' queue to enter in a state
where it got stuck, and no further events could get processed.
On debug builds, an assert was hit @GestureProviderAura::OnSyncTouchEventAck.
Patch fixes it by making a call to the proper "async" replacement
of ::ProcessTouchEventOnAsyncAck, namely ::AckSyncTouchEvent.
Upstream reference:
https://chromium.googlesource.com/chromium/src/+/
8879cfca3b19c304004e34e6dec9629752cd3%5E!/#F6
PS: It is worth mentioning that, later on the first argument of
::AckAsyncTouchEvent was removed in
https://chromium.googlesource.com/chromium/src/+/
b1663498ac1d59fc3baa085c155f84a1726ca87d%5E!/#F9
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=10493
Change-Id: I2f5a671c4ff2dd5184ce175f7bc4d54aef663f33
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Arnaud Renevier [Mon, 13 Apr 2015 22:32:06 +0000 (15:32 -0700)]
Fixes utc_blink_cb_form_submit.NonSubmit
Right now, utc_blink_cb_form_submit.NonSubmit ends with timeout. But the
expected value is Failure. This is erroneous.
Change-Id: I1544c73d6b1b1de168ed2d1cac98b3ace807213a
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Arnaud Renevier [Mon, 13 Apr 2015 21:34:04 +0000 (14:34 -0700)]
Fix form,submit callback
EwkHostMsg_FormSubmit is Send on the RenderFrame. So it's not handled in
WebContentsDelegate anymore. It is handled in
WebViewBrowserMessageFilter now.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=10490
Change-Id: Iafd138bbbb198f32481e68f312ec178e008846a8
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Arnaud Renevier [Fri, 17 Apr 2015 20:01:12 +0000 (13:01 -0700)]
Bringup devtools
DevToolsHttpHandler::Stop has been removed. Instead, cleanup is done in
destructor.
Change-Id: Ie059ad9735f95bea0cb848f3f11d91e19580c565
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Arnaud Renevier [Mon, 20 Apr 2015 16:20:12 +0000 (09:20 -0700)]
Remove unused forward class declarations
Many forward class declarations are actually not needed. This patch
cleans them up.
Change-Id: I880039161b8415d36568bda5d646b150dfce8227
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
yh106.jung [Thu, 16 Apr 2015 12:44:59 +0000 (21:44 +0900)]
Enable EWebView::ExitFullscreen()
ExitFullscreen() was moved from RVH to WCI on M42.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12028
Change-Id: I94ce408e41b64c8eadcc5978c2b6b5c03e0078b4
Signed-off-by: yh106.jung <yh106.jung@samsung.com>
hh4813.kim [Thu, 16 Apr 2015 10:29:36 +0000 (19:29 +0900)]
[EWK_REFACTOR] Remove SelectRange() in EWebView
To remove dependency with EWebView in SelectionControllerEfl,
modify to call WebContentsImpl::SelectRange() directly and remove
SelectRange() in EWebView.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12028
Change-Id: Ie7bf812b25740319a5e1ee503d6709f3b719eb3b
Signed-off-by: hh4813.kim <hh4813.kim@samsung.com>
hh4813.kim [Fri, 17 Apr 2015 10:27:55 +0000 (19:27 +0900)]
fixup! Remove tw_selection_controller and tw_context_menu_controller
Change-Id: I41d7a04f1f1d8ec4548f87dc2782cd6014b87a44
Signed-off-by: hh4813.kim <hh4813.kim@samsung.com>
Arnaud Renevier [Tue, 14 Apr 2015 18:57:24 +0000 (11:57 -0700)]
Remove tw_selection_controller and tw_context_menu_controller
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=11619
Change-Id: I8d8f68391143070ba0798130f55456b601cce4b2
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
hh4813.kim [Thu, 16 Apr 2015 11:12:32 +0000 (20:12 +0900)]
[EWK_REFACTOR] Move IsIMEShow() and GetIMERect() from EWebView to RWHV
To remove dependency with EWebView in SelectionControllerEfl,
move IsIMEShow() and GetIMERect() from EWebView to RWHV so that they can
call directly without EWebView.
Also, remove im_context() in RWHV since there's no caller any more.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12028
Change-Id: I22784d116f6e2176541c4bfd70484f7b65bb0673
Signed-off-by: hh4813.kim <hh4813.kim@samsung.com>
Antonio Gomes [Thu, 16 Apr 2015 15:00:07 +0000 (08:00 -0700)]
fixup! Update our fetch procedure to point to src/DEPS
Although repositories sync'ed via DEPS file are
being updated fine, our custom deps are not.
The reason is because we are rebasing them
against themselves, not against a remote branch.
Patch fixes it by adding 'origin/' to each custom
deps branch we sync.
As a drawback, it ties us to a remote named 'origin'
but it should not affect normal developers' workflow
negatively.
Change-Id: Ifeba7fc09155aa5f6eeb38d82bf9b504061b1802
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Piotr Tworek [Thu, 16 Apr 2015 14:01:26 +0000 (16:01 +0200)]
Remove some of the NOTIMPLEMENTED() logs we don't have any interest in.
1. We're not interested in implementing our own overscroll controller.
We already have support for overscroll effect implemented via
EdgeEffect object in RWHV_EFL.
2. The AcceleratedSurfaceInitialized function was added mostly for
android port. We have no use for it in EFL and it's not even pure
virtual so we don't have to provide an empty implementation.
This should make our app startup contain a little bit less spam.
Change-Id: I47321dd729eb98d12b1ab72b7cd98fcc90bb5176
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
yh106.jung [Thu, 16 Apr 2015 12:09:39 +0000 (21:09 +0900)]
Fix IME keyboard issue for url bar of mini browser
There are 2 issues:
1. When IME keyboard is hidden by the outside, focus of url bar is not
removed. So when user tries to tap on url bar, IME keyboard is not
shown.
For resolving it, we should remove focus of url bar when callback of
ECORE_IMF_INPUT_PANEL_STATE_HIDE is called without hide_imf_panel()
call.
2. Sometimes IME is closed before it is hidden. So We cannot move tool
box and remove focus of url bar.
Solution is that move tool box and remove focus when IME close callback
is called.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12566
Change-Id: I019ed98513f3308fe9d80b8696a5b3adaff059f5
Signed-off-by: yh106.jung <yh106.jung@samsung.com>
Piotr Tworek [Thu, 16 Apr 2015 08:57:37 +0000 (10:57 +0200)]
Remove conformant object fron WebContentsViewEfl.
Elementary conformat widget is supposed to account for space taken up by
virtual keyboard and indicator widgets. It's an important element, but
it should be instantiated only once for each elementary window. If it's
instantiated several times the app layout may add extra padding at
window edges more than once. Such extra unnecessary pading was the root
cause behind bug #12564.
Currently uBrowser, Reference borwser and mini_browser instantiate
conformant widgets on the application level. The only app that does not
do it is efl_webview_app. The app is not exactly a good example on how
EWK apps should be implemented. It seems the standard practise is to
inject conformant widget at the app level. Doing it in the engine may
not be obvious and can lead to a subtle bugs that are only visible for
some elm layouts.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12564
Change-Id: I768c350a7a246ea5e6d79864f4d7916c46096f05
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Piotr Tworek [Wed, 15 Apr 2015 16:37:02 +0000 (18:37 +0200)]
Make sure disambiguation popup parent is a elementary widget.
Elementary widgets require to have other elm widgets as their parents.
When this requirement is not met elementary focus chain is broken.
Since disambiguation popup uses elementary widgets we need to make sure
the parent we give it is also an elm widget.
This patch fixes a couple of EFL runtime errors seen on Tizen 2.4 mobile
target during ubrowser/efl_webview_app startup.
Change-Id: I36a9b9920f5ca61aedc7ef1ca1420a04cbc2f6f8
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Arnaud Renevier [Wed, 15 Apr 2015 19:37:46 +0000 (12:37 -0700)]
Refactor _Ewk_Security_Origin
_Ewk_Security_Origin constructor only allows GURL
Also, if port is unspecified, GetPort returns 0 instead of -1. This
fixes utc_blink_ewk_notification_security_origin_get unit test.
Change-Id: I4c1633a4ff75d6962afef222514b33be86b20bd4
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Arnaud Renevier [Tue, 14 Apr 2015 18:03:19 +0000 (11:03 -0700)]
Remove tw_callbacks.h
We can also remove chromium_glue and api_static_check now.
Change-Id: Id9d23111224cfcba34ada147f701b4bf51afa603
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Arnaud Renevier [Mon, 13 Apr 2015 18:19:57 +0000 (11:19 -0700)]
Remove tw_hit_test
There were two distinct classes: Ewk_Hit_Test and _Ewk_Hit_Test.
Ewk_Hit_Test was a subclass of tizen_webview::Hit_Test _Ewk_Hit_Test
owned an Ewk_Hit_Test and accessed it's properties.
This patch simplifies everything: there is now only _Ewk_Hit_Test, and
Ewk_Hit_Test is an alias to it.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=11619
Change-Id: I885c5c8350f306b44c2d8ed19ab1c9a2ff43894a
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
hh4813.kim [Thu, 16 Apr 2015 05:44:42 +0000 (14:44 +0900)]
[EWK_REFACTOR] Move IsLastAvailableTextEmpty() from EWebView to RWHV
To remove dependency with EWebView in ContextMenuControllerEfl and
SelectionControllerEfl, move IsLastAvailableTextEmpty() from EWebView
to RWHV so that they can call directly without EWebView
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12028
Change-Id: Iccf6a6e1f7706d9092732dc9b77cd8916e93bb61
Signed-off-by: hh4813.kim <hh4813.kim@samsung.com>