platform/framework/web/chromium-efl.git
6 years agofixup! [M47_2526] Chromium upversion to m47_2526 branch
Youngcheol Kang [Wed, 28 Oct 2015 07:19:59 +0000 (16:19 +0900)]
fixup! [M47_2526] Chromium upversion to m47_2526 branch

Add arm64 support to ffmpeg

We need to regenerate this arm64 support patch in ffmpeg
until ffmpeg supports arm64 architecture in mainstream.

The following commands were executed within arm64 GBS wayland chroot:

[1] Enter chroot
$ gbs chroot --root ~/GBS-ROOT/local/BUILD-ROOTS/scratch.aarch64.0/

[2] Copy over src/third_party/ffmpeg to chroot (e.g.: /home/abuild)
$ cp -rf src/third_party/ffmpeg
~/GBS-ROOT/local/BUILD-ROOTS/scratch.aarch64.0/home/abuild/

[3] Execute the below comment in ffmpeg directory
$ cd ffmpeg
$ ./chromium/scripts/build_ffmpeg.py linux arm64
$ ./chromium/scripts/copy_config.sh
$ python chromium/scripts/generate_gyp.py

[4] Edit ffmpeg.gyp; add "asm_sources" to the build

[5] Generate the patch against a unchanged ffmpeg copy

This patch referred to http://165.213.202.130/gerrit/#/c/80261

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

Reviewed by: a1.gomes, sns.park

Change-Id: I238cb36c294e27d9dce3a163a7a41c474d5f2360
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
6 years ago[Odroid][WCS] No logs for touch events on gesture.html
Antonio Gomes [Tue, 13 Oct 2015 16:00:59 +0000 (12:00 -0400)]
[Odroid][WCS] No logs for touch events on gesture.html

After [1], touch events stopped being forcibly enabled
for chromium-efl. That was the correct thing to do,
given that there are Tizen profiles where touch does
not play at: TV, desktop.

When ubrowser is launched on Tizen (both TV and mobile),
it makes a call to ewk_view_touch_events_enabled_set(true).
Same happens when it is launched on desktop with "--mobile"
parameter.
In the call chain, EWebView::SetTouchEventsEnabled is called
and enables touch in WCVEfl and RWHVEfl level.

However, problem relies on the fact that the API layer status
of touch events gets disconnected to content layer level status.
At content layer level, touch events are only enabled
to Tizen mobile builds, as one can see from the snippet below at
tizen_src/ewk/efl_integration/command_line_efl.cc:

78 #if defined(OS_TIZEN_MOBILE)
79   p_command_line->AppendSwitchASCII(switches::kTouchEvents,
80                                     switches::kTouchEventsEnabled);

Patch fixes that inconsistency by hooking up content layer's
touch event status to ewk_view_touch_events_enabled_set call chain.

As a consequence, "ubrowser --mobile" on desktop also get
its touch events emulation fixed: double tap to zoom works,
as well as the sample sites [2], [3] and [4].

[1] http://165.213.202.130/gerrit/92823
[2] http://www.google.com/intl/en/about/careers/locations/
[3] https://patrickhlauke.github.io/touch/tests/event-listener.html
[4] WSC/gestures.html

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/90480/ , reviewed by
Janusz Majnert, arno renevier.

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

Reviewed by: a.renevier, sns.park

Change-Id: I9ac69a381505d6064b269453ca8d034d7ad4510d
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoDisambiguate ShowContextMenu implementation.
Antonio Gomes [Thu, 29 Oct 2015 20:10:24 +0000 (16:10 -0400)]
Disambiguate ShowContextMenu implementation.

In the regular chromium flow, there is a hook invoked when
a long press gesture happens. The terminology used in the
messages and methods involved is "show context menu".

However, upon a long press gesture, chromium-efl does not
necessarily show a context menu right way. It might, for instance,
enter selection mode if the long press gesture happens on a
text (non anchor) content, and only tries to show an actual
context menu when user lifts up his finger.
Alternatively, it might show a context menu right away
in case the gesture happens on an <image> or a link.

For both cases, the same chromium hook is called by chromium,
whereas the former performs a second pass on the chromium hook,
which is confusing. Our current implementation ends up like:

EWebView::ShowContextMenu() {
if (called by chromium)
do this;
else // called by EWK on a second pass
do that;
}

Problem is that this is a constant source of issues because
it requires control of state variables to handle each passes
on the method.

Patch makes that separation clear by disambiguate this logic:
- EWebView::HandleLongPressEvent calls SelectionController
- SelectionController calls out to EWebView::ShowContextMenu.

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

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

Change-Id: I2bf2a1b24e20d7f0f50e6881144cb3cf8cc62925
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agofixup! [M47_2526] Chromium upversion to m47_2526 branch
Arnaud Renevier [Thu, 29 Oct 2015 22:03:11 +0000 (15:03 -0700)]
fixup! [M47_2526] Chromium upversion to m47_2526 branch

Since DisplayPersistentNotification and ClosePersistentNotification are
NOTIMPLEMENTED(), GetDisplayedPersistentNotifications should be
NOTIMPLEMENTED() too.

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

Reviewed by: a1.gomes, sns.park

Change-Id: Ida2c3aacfdf0f55a91a74edb600ce33e6380f15f
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agoLarge handler works 'jumpy' on scrolling
Antonio Gomes [Mon, 19 Oct 2015 18:50:46 +0000 (14:50 -0400)]
Large handler works 'jumpy' on scrolling

Patch sets EFL/Tizen editing behavior deliberately matching
Android when "touch events" are enabled. This avoids the "jumpy"
cursor / large selection handle when we move it beyond the editable
field boundary.

This aggressive approach (of matching Android's editing behavior)
might change in the future, and we introduce a EFL/Tizen specific
editing behavior. But for now, this seems to do the job.

Note that TV or other profiles where touch is not enabled,
will not benefit from this change.

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/91213/ reviewed by
DONGJUN KiM, Janusz Majnert, arno renevier.

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

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

Change-Id: I3077b014a779fda535260644de80b0d122b89bb2
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agofixup! [M47_2526] Chromium upversion to m47_2526 branch
Arnaud Renevier [Wed, 28 Oct 2015 18:30:57 +0000 (11:30 -0700)]
fixup! [M47_2526] Chromium upversion to m47_2526 branch

Compute maximum scroll offset by substracting the visible content size
to the full content size.

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

Reviewed by: a1.gomes, sns.park

Change-Id: Ie1f9b935767c369b622c451802b7642d96065846
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agoDo not queue up ViewMsg_UpdateWebPreferences messages before RenderView gets created.
Antonio Gomes [Tue, 27 Oct 2015 18:38:31 +0000 (14:38 -0400)]
Do not queue up ViewMsg_UpdateWebPreferences messages before RenderView gets created.

In EWebView::UpdateWebKitPreferences there is some logic to
avoid losing IPC messages. Logic is "if RenderView is not created
yet, do not send an IPC message to it".
However this logic does not apply to ViewMsg_UpdateWebPreferences
any more.

For instance, if one calls EWebView::UpdateWebKitPreferences
before RenderView is created, this is what would happen:

1) RenderViewHost::UpdateWebkitPreferences is called and "store"
the WebPreference instance passed as parameter.
2) Since RenderView is not created yet, ViewMsg_UpdateWebPreferences
would be queued up to be sent delayed.

In RenderViewHostImpl, though, as part of the RenderView creation,
the web_preference instance that was "store" is passed. So, there is
not need to send a delayed message.

As the code is today, after RenderView creation, EWebView::RenderViewCreated
(the hook) is called, and the message queued in (2) is sent again.
Although it might sound warmless, real problem is that the delayed
message has different values than the last message "stored" in
RenderViewHost. This leads to bugs.

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

Reviewed by: a.renevier, j.majnert, sns.park

Change-Id: I870d5028e96e5b3bcc3c5f13325e8f1983296888
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoMake javascript prompt one line entry scrollable by default.
Pawel Forysiuk [Wed, 21 Oct 2015 15:22:42 +0000 (17:22 +0200)]
Make javascript prompt one line entry scrollable by default.

Otherwise when the text is longer than the entry itself
the letters are not visible. Enable entry scroll so the text
cursor and thus the letters currently appended are visible to the user.

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/91533/ reviewed by
Antonio Gomes, Janusz Majnert.

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

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

Change-Id: Ib8621ca60d96191e799c69f97ae7d6ec4d4c3c76
Signed-off-by: Pawel Forysiuk <p.forysiuk@samsung.com>
6 years agoUnite ewk_context_inspector_server and ewk_view_inspector_server
Arnaud Renevier [Mon, 5 Oct 2015 19:57:08 +0000 (12:57 -0700)]
Unite ewk_context_inspector_server and ewk_view_inspector_server

Right now, we have two different sets of api to start and stop the
inspector server: ewk_context_inspector_server_(start/stop) and
ewk_view_inspector_server_(start/stop).

In this patch, we avoid this duplication: calls to
ewk_view_inspector_server_ will internally be forwarded to EWebContext.

Reviewed by: sns.park

Change-Id: Ia8e6177cef962ae5addf37d541ae128aab2f8630
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agoReimplement copy constructor for NativeWebKeyboardEvent
Arnaud Renevier [Mon, 26 Oct 2015 22:16:25 +0000 (15:16 -0700)]
Reimplement copy constructor for NativeWebKeyboardEvent

In m47, it's not possible to type in an <input>. This is because
NativeWebKeyboardEvent is not created correctly.

We need to reenable NativeWebKeyboardEvent copy constructor. It was
somehow deleted sometime between m34 and m40. It was fine until now, but
introduces this bug in m47.

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

Reviewed by: a1.gomes, sns.park

Change-Id: I25f847e950507dd7403c7f6dd34db2ded1e2a52f
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agofixup! [M47_2526] Chromium upversion to m47_2526 branch
Arnaud Renevier [Mon, 26 Oct 2015 22:42:23 +0000 (15:42 -0700)]
fixup! [M47_2526] Chromium upversion to m47_2526 branch

Modify TextInputStateChanged according to
https://codereview.chromium.org/1249013005

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

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

Change-Id: Ifa45fbde6246c8db678178cf194b5eb29d1f6b75
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agoubrowser --mobile gets stuck on long press
Antonio Gomes [Wed, 14 Oct 2015 18:23:40 +0000 (14:23 -0400)]
ubrowser --mobile gets stuck on long press

When in text selection mode after long press'ing on a
content editable field, SelectionBoxEfl::IsInEditableField is
true regardless of the selection mode (caret or range).

If one long presses elsewhere, chromium-efl first resets
the current selection point (by sending 0,0 0x0 selection
points), and only then triggers long press gesture, context
menu, and text selection in the new event location.

SelectionControllerEfl has code that discards these zero'ed
out selection points, but it is placed too down in chain,
missing the case described above. This cases problem on the
below scenario:

Steps to reproduce the problem:
1) Load WCS/text_selection_content_editable.html on uBrowser.
2) Long press on the content editable (gray) area.
Browser should enter caret selection mode
Large handle gets shown once user lift up his finger
3) Press "Select" option on context menu
The wrapping word should get selected.
Browser enters range selection mode.
4) Long press a #text node elsewhere on the page,
e.g. "contents!".
The wrapping word should get selected.
5) Release mouse button.
6) Move mouse around around the page

Patch fixes that.

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/90691/ reviewed by
Janusz Majnert, arno renevier.

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

Reviewed by: a.renevier, sns.park

Change-Id: Ic5dc45c66f61a0b86096ec68fba1f2933b350478
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoImplement a smoother handle movement while dragging
Antonio Gomes [Fri, 18 Sep 2015 14:48:04 +0000 (10:48 -0400)]
Implement a smoother handle movement while dragging

When in text selection mode, this is how handle dragging
happens: whenever user touches a handle and starts finger
movement to drag it around, the handle position is not updated
immediately following users' finger. Instead, it is updated when
the new text selection takes place.

Although this seems logical, it ends up with bad visual result
to the user. It is common to see the handle jumping around the
page while being dragged, for instance.

As an illustrative example, when user drag a handle from (-) to (*)
in a straight line in the text below ...

a bit text(-) here
some more text here (~)
and a large(*) paragraph to finalize

... one will see that the handle "jumps" from (-) to
(~) and only then to (*). Basically, the dragging handle gets
disconnected from users' finger.

Patch changes that so that the handle always follows
users' finger.

In order to do so, it tracks which handles is being
dragged, and which is stationary. Additionally, it makes
use of the 'is_anchor_first' bit in order to properly
place the handles.

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/87892/ , reviewed by
Janusz Majnert, SeungSeop Park, arno renevier.

Together with: Id45d27ffd9db5a497ba984f5b16e0077c98bc20d

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

Reviewed by: a.renevier, j.majnert, sns.park

Change-Id: I9c16d452bab50cd7ade17a0cbf5b2abb5c4d0af4
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agofixup! [M47_2526] Chromium upversion to m47_2526 branch
Antonio Gomes [Tue, 27 Oct 2015 21:18:46 +0000 (17:18 -0400)]
fixup! [M47_2526] Chromium upversion to m47_2526 branch

RWHImpl::SetInputMethodActive was removed from upstream in
https://codereview.chromium.org/1234193002 .
It used to serve as a way to avoid some IPC messages to get
sent, and then get some performance gain.
Discussion in [1], [2] explained that the gain was not relevant
and all ports decided to drop it.

Efl should follow it too.

[1] https://code.google.com/p/chromium/issues/detail?id=474828
[2] https://code.google.com/p/chromium/issues/detail?id=296509

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

Reviewed by: a.renevier, sns.park

Change-Id: I9b8384206999326879f73f01b213fffb8acc1385
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agofixup! [M47_2526] Chromium upversion to m47_2526 branch
Antonio Gomes [Tue, 27 Oct 2015 19:46:18 +0000 (15:46 -0400)]
fixup! [M47_2526] Chromium upversion to m47_2526 branch

Use ScopedVector variable instead of converting it to vector<>.

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

Reviewed by: a.renevier, sns.park

Change-Id: I54ea9a23e4ba41a64024a36a672a9fdb73a27b72
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agofixup! [M47_2526] Chromium upversion to m47_2526 branch
Arnaud Renevier [Mon, 26 Oct 2015 20:17:43 +0000 (13:17 -0700)]
fixup! [M47_2526] Chromium upversion to m47_2526 branch

Implement PluginPlaceholderEfl::GetV8Handle

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

Reviewed by: a1.gomes, sns.park

Change-Id: Ieac280248a1fe43ace9a43f77364b476bca55e94
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agoMinor code cleanup in RenderWidgetHostViewEfl::GetTextureFromMailbox
Arnaud Renevier [Mon, 26 Oct 2015 18:39:07 +0000 (11:39 -0700)]
Minor code cleanup in RenderWidgetHostViewEfl::GetTextureFromMailbox

Minor code cleanup in RenderWidgetHostViewEfl::GetTextureFromMailbox: do
not create cc::CompositorFrameAck variable if not needed.

Reviewed by: a1.gomes, sns.park

Change-Id: I789a4176c027193c641a71d500ec79d7e9c282f7
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agofixup! [M47_2526] Chromium upversion to m47_2526 branch
Arnaud Renevier [Tue, 27 Oct 2015 00:01:44 +0000 (17:01 -0700)]
fixup! [M47_2526] Chromium upversion to m47_2526 branch

Remove NOTIFICATION_WEB_CONTENTS_TITLE_UPDATED support. It has simply
been removed in chromium upstream.
https://codereview.chromium.org/1242953004

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

Reviewed by: a1.gomes, sns.park

Change-Id: Ib37d057e3e7e929ea51e398414f0a08e0cd7ae5c
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agofixup! [M47_2526] Chromium upversion to m47_2526 branch
Arnaud Renevier [Tue, 27 Oct 2015 00:22:16 +0000 (17:22 -0700)]
fixup! [M47_2526] Chromium upversion to m47_2526 branch

Use hasNonEmptyLayoutSize instead of hasNonEmptyBoundingBox. See
https://codereview.chromium.org/1226243002.

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

Reviewed by: a1.gomes, sns.park

Change-Id: I28aabd097d70c7e0956b44642409916175c35f9c
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agofixup! [M47_2526] Chromium upversion to m47_2526 branch
Arnaud Renevier [Mon, 26 Oct 2015 20:29:38 +0000 (13:29 -0700)]
fixup! [M47_2526] Chromium upversion to m47_2526 branch

dump-render-tree switch has been removed. It has been replaced with
run-layout-test

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

Reviewed by: a1.gomes, sns.park

Change-Id: Iabeb757fdc732342abe777c0c296c29c9a9d89e6
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agodeclare OzonePlatformEfl inside cc file
Arnaud Renevier [Mon, 26 Oct 2015 18:34:13 +0000 (11:34 -0700)]
declare OzonePlatformEfl inside cc file

Other OzonePlatform implementation declare OzonePlatformImpl inside the
cc file, and do not expose that class in the header file.

This patch refactors OzonePlatformEfl, to match other ports.

Reviewed by: a1.gomes, sns.park

Change-Id: I0c0b7824d0d18ad01505b9e5d74d1e9466e2115c
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agofixup! [M47_2526] Chromium upversion to m47_2526 branch
Youngcheol Kang [Tue, 27 Oct 2015 03:56:35 +0000 (12:56 +0900)]
fixup! [M47_2526] Chromium upversion to m47_2526 branch

The below dependencies were removed in original patch.

- chrome_resources.gyp:packed_resources
- extensions.gyp:extensions_shell_and_test_pak

The build breaks were occurred because the locales directory wasn't
generated in out.{PROFILE} which related to the this changing.

This patch adds the two dependencies again, and makes the temporary
patch file for resolving the build problem.
This patch file will be removed by webview team.

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

Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
6 years ago[M47_2526] Chromium upversion to m47_2526 branch
Youngcheol Kang [Thu, 15 Oct 2015 06:34:17 +0000 (15:34 +0900)]
[M47_2526] Chromium upversion to m47_2526 branch

This patch deals with following upstream changes:

1. Folder and class name changes

- Enabled to use NSS intead of the OpenSSL
- Changed the path of the serialized_navigation_entry.h to the
  components/sessions/core from the components/sessions
- Changed the path of the UnicodeIcu.h to the wtf/text/icu from the
  wtf/unicode/icu
- Changed the path of the video_capture_device_factory.h to the
  media/capture/video from the media/video/capture
- Changed the path of the video_capture_device.h to the
  media/capture/video from the media/video/capture
- Changed the path of the WebScreenOrientationLockType.h to the
  third_party/WebKit/public/platform/modules/screen_orientation from the
  third_party/WebKit/public/platform
- Removed the chrome_resources.gyp:packed_resources and
  extensions.gyp:extensions_shell_and_test_pak dependencies in chrome

2. API changes

- Added the ClientNativePixmapFactory class in OzonePlatform
- Added the ClearCompositorFrame, TextInputStateChanged API in
  RenderWidgetHostViewBase class
- Added the CopyTexSubImage API in GLImage class
- Added the GetDisplayedPersistentNotifications API in
  PlatformNotificationService class
- Added the GetScreenColorProfile API in RenderWidgetHost class
- Added the GetOverlayManager, OpenClientNativePixmapDevice API in
  OzonePlatform class
- Added the GetV8Handle API in LoadablePluginPlaceholder class
- Added the HandleWebSocketConnection API in DevToolsHttpHandlerDelegate
  class
- Added the IsContextSecure API in AutofillClient class
- Added the opener_render_process_id, opener_render_frame_id parameter
  in CreateParams struct
- Added the SchedulePendingLossyWrites API in AutofillPrefStore class
- Changed the OneShotTimer and RepeatingTimer class in base class
- Changed the PermissionManager class which associated with a frame id
  instead of a view id
- Changed the ProxyConfigService class in net class
- Changed the VibrationManager class
- Changed the kEnableDelegatedRenderer switch to the
  kDisableDelegatedRenderer switch also added the kCompositeToMailbox
  switch
- Changed the AckAsyncTouchEvent and AckSyncTouchEvent API to the
  AckTouchEvent API in GestureRecognizer class
- Changed the GetLastActiveTime API to the GetLastActivityTime API in
  DevToolsTargetDescriptor class
- Changed the argument of AutofillWebDataService class
- Changed the argument of GpuChildThread class
- Changed the argument of InProcessGpuThread class
- Changed the argument of LoadablePluginPlaceholder class
- Changed the argument of PasswordGenerationAgent class
- Changed the argement of WebContentsImpl class
- Changed the argument of WebDatabaseService class
- Changed the argument of AddTable API in WebDatabaseService class
- Changed the argument of CancelableCallback API, GetImageTextureTarget
  API, OnSwapBuffersCompleted in ContextFactory class
- Changed the argument of CancelPermissionRequest, RequestPermission API
  in PermissionManager class
- Changed the argument of CanCreateWindow API in ContentBrowserClient
  class
- Changed the argument of Compositor API in ui class
- Changed the argument of DefaultBackend API in HttpCache class
- Changed the argument of DidAcceptSuggestion API in
  AutofillPopupDelegate class
- Changed the argument of EnableRecording API in TracingController class
- Changed the argument of EndsWith API, SplitString API in base class
- Changed the argument of EnterFullscreenModeForTab,
  ShouldCreateWebContents API in WebContentsDelegate class
- Changed the argument of RemoveLoginsCreatedBetween API in
  LoginDatabase class
- Changed the argument of ResetConfigService API in ProxyService class
- Changed the argument of SetProtocolHandler API in URLRequestJobFactory
  class
- Changed the argument of SetValue, SetValueSilently API in
  AutofillPrefStore class
- Changed the argument of set_http_user_agent_settings,
  set_transport_security_state, set_http_transaction_factory,
  set_job_factory API in URLRequestContext class
- Changed the argument of SwapDelegatedFrame API in
  DelegatedFrameHostClient class
- Changed the argument of UploadPasswordForm API in
  PasswordStoreConsumer class
- Changed the argument and codes of CreateNewWindow API in
  RenderViewHostDelegate class
- Changed the codes of IsIsolatedGuest API to the IsForGuestsOnly API in
  RenderProcessHost class
- Changed the return type of GetLastActiveTime API in WebContents class
- Removed the BindWebFrame API in LoadablePluginPlaceholder class
- Removed the CanSubscribeFrame, GetCompositingSurface API in
  RenderWidgetHostView class
- Removed the CreatePluginPlaceholder, HandleNavigation API in
  ContentRendererClient class
- Removed the Destory API in GLContext class
- Removed the IsPublicSuffixMatch API in PasswordForm class
- Removed the kAutofillAuxiliaryProfilesEnabled in namespace of pref
- Removed the LinkClicked API in AutofillClient class
- Removed the OnCanThrottleRequest API in NetworkDelegateImpl class
- Removed the opener parameter in CreateParams struct
- Removed the password_autocomplete_set in PasswordForm struct
- Removed the ReleaseTexImage API in GLImage class
- Removed the SchemeIsSecure API in GURL class
- Removed the surface_id API in RenderWidgetHostImpl class
- Removed the TextInputTypeChanged API in RenderWidgetHostViewBase class
- Removed the IsContextLost API in ContextProvider class. We can uses
  the GetGraphicsResetStatusKHR API instead of that
- Removed the MessageLoopProxy class. We can uses the
  base::ThreadTaskRunnerHandle, base::SingleThreadTaskRunner class
  instead of that
- Removed the StartsWithASCII API. We can uses the base::StartsWith API
  instead of that
- Removed the renderer/plugins/shadow_dom_plugin_placeholder.cc | h and
  ewk/efl_integration/renderer/plugins/shadow_dom_plugin_placeholder.cc
  | h files
-

3. Need to change API (TODO issues)

- The ffmpeg patch on ARM64bit should be made. Temporary creating the
  build/patches/m47_ffmpeg_temp_arm64bit.diff file to disabling the
  codes (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14507)
- The hasNonEmptyBoundingBox API in was removed in WebNode class.
  Temporary disabling the codes
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14509)
- The kEnableViewportMeta switch was removed in switches class.
  Temporary disabling the codes
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14510)
- The maximumScrollOffset API was removed in WebLocalFrameImpl class.
  Temporary disabling the codes
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14511)
- The NOTIFICATION_WEB_CONTENTS_TITLE_UPDATED was removed in enum of
  NotificationType. Temporary disabling the codes
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14512)
- The SetInputMethodActive API was removed in RenderWidgetHostImpl
  class. Temporary disabling the codes
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14513)
- The SetTextZoomFactor API was removed in RenderViewHost class.
  Temporary disabling the codes
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14514)
- The code implementation about GetDisplayedPersistentNotifications API
  in PlatformNotificationService class is necessary
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14515)
- The code implementation about GetV8Handle API in
  LoadablePluginPlaceholder class is necessary
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14516)
- The code implementation about IsContextSecure API in AutofillClient
  class is necessary
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14517)
- The code implementation about TextInputStateChanged API in
  RenderWidgetHostViewBase class is necessary
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14518)
- The multimedia function has been changed. Temporary disabling the
  'tizen_multimedia_support' flag
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14519)
- The libffmpegsumo.so is not generated in the out.TARGET/Release after
  building the project. Temporary disabling the codes
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14520)
- The API of system/device.h cannot use in
  battery_status_manager_tizen.cc. Temporary disabling the codes
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14521)
- The build error happens when using the best_matches variable.
  Temporary disabling the codes
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14523)
- The build error happens when using the Restore API in
  NavigationController class. Temporary disabling the codes
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14524)
- The CanSubscribeFrame was removed in DelegatedFrameHost class.
  Temporary disabling the codes
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14548)
- The IsDelegatedRendererEnabled API was removed in content class.
  Temporary disabling the code
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14572)
- The switch of kDumpRenderTree was removed in swiches class. Temporary
  disabling the codes
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14573)

4. Verification

- This patch supports to the desktop and mobile(Tizen 3.0 32/64bit) and
  tv(Tizen 3.0 32bit) profiles
- There is no build problem in all target

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

Change-Id: I97d4de70b2582825cee84cdf663035032664173f
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
6 years agoRestore dev branch S_TRUNK_OWNERS
SeungSeop Park [Wed, 21 Oct 2015 01:04:35 +0000 (10:04 +0900)]
Restore dev branch S_TRUNK_OWNERS

The file was mistakenly overwritten with beta branch's when we force
update dev branch with beta/m42. So, restore the file from previous
dev/master branch.
In addition, removes invalid owners - p.tworek and k.klimek.

Change-Id: I8f18e07fd390fe732370070f6c880df255a46faa
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
6 years agoWindow creation mis-setup Evas_Object hierarchy
Antonio Gomes [Fri, 9 Oct 2015 22:20:52 +0000 (18:20 -0400)]
Window creation mis-setup Evas_Object hierarchy

When a new window is created originated from window.open
or target=_blank, we have to follow EWK procedure
of creating windows: a smart signal is emitted, and the embedding
app can allow or deny the creation.
In that process, RWHVEfl is created when
WebContentsView::CreateViewForWidget is called from
WebContentsImplEfl::HandleNewWebContentsCreate.

RWHVEfl owns SelectionControllEfl. At the time of its
creation, we have to ensure that the EFL widgets are
properly set up. Patch adds a hook to allow that.

In regular window creation, for example, when user clicks
a new window/tab button, the regular chromium flow is taken,
instead of this special codepath.

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/90112/ , reviewed by
Janusz Majnert, arno renevier.

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

Reviewed by: a.renevier

Change-Id: If250990897051af40fee36224bdc278521bb6d9a
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoSelectionController::OnMouse{Up,Move,Down} have broken signatures
Antonio Gomes [Fri, 18 Sep 2015 15:11:17 +0000 (11:11 -0400)]
SelectionController::OnMouse{Up,Move,Down} have broken signatures

In all three methods, names are misleading.
Patch does the following:

- OnMouseDown -> HandleDragBeginNotification
- OnMoveMove -> HandleDragUpdateNotification
- OnMouseUp -> HandleDragEndNotification

Additionally, patch removes an unused parameter
in all three methods.

Original beta/m42 patch:
- http://165.213.202.130/gerrit/87893 , reviewed by
Hyunhak Kim, arno renevier.

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

Reviewed by: a.renevier, j.majnert, sns.park

Change-Id: Iea48026c242bdde6f65c0782369f30d5dadc233a
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoUnify SelectionControllerEfl::ShowHandleAndContextMenuIfRequired overloads
Antonio Gomes [Fri, 18 Sep 2015 15:53:50 +0000 (11:53 -0400)]
Unify SelectionControllerEfl::ShowHandleAndContextMenuIfRequired overloads

SelectionControllerEfl::ShowHandleAndContextMenuIfRequired has
two overloads, where one calls the other.

Details
1) overload_1 is called from many places;
2) overload_2 is only called by overload_1
3) overload_1 passes some unused parameters to overload_2.

Patch unifies both methods.

Original m42 patch:
- http://165.213.202.130/gerrit/#/c/87894/ , reviewed by
DONGJUN KiM, arno renevier.

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

Reviewed by: a.renevier

Change-Id: Ie2a77199a3689489c39483805c13a57a151b0382
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agofixup! Make text selection feature available on desktop builds.
Antonio Gomes [Wed, 23 Sep 2015 20:05:49 +0000 (16:05 -0400)]
fixup! Make text selection feature available on desktop builds.

"theme_path" was getting a corrupted value as a result of
the snippet below:

const char* theme_path = selection_handles_edj.AsUTF8Unsafe().c_str();

My theory is that the temporary shadow std::string variable
resulting of the call to AsUTF8Unsafe was getting unpredictably
deleted, and the c_str pointer held in "theme_path" gets its
content corrupted.

As theme_path is passed as a path to edje_xxx APIs, this was
causing selection handles to randomly fail to show.
Bug reproduces on both desktop and mobile builds.

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/88401/ , reviewed by
DONGJUN KiM, arno renevier.

Reviewed by: a.renevier

Change-Id: I34024d7341d29de6c1ad778747fb348c240e998c
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agofix ewk_view_user_agent_set
Arnaud Renevier [Fri, 25 Sep 2015 19:46:41 +0000 (12:46 -0700)]
fix ewk_view_user_agent_set

Right now, even when setting user agent, this change is not seen by web
pages. In order for the user agent to be considered, we need to load url
with override_user_agent param.

In order to simplify the setting of this flag, this patch also refactor
EWebView GetURL/SetURL. Now they get/return GURL instead of strings

Also, this patch prevents webpage from reloading when a load is in
progress.

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

Reviewed by: a1.gomes, j.majnert

Change-Id: Ie170885f10927cdda7afd9d5352e6a24944a2d29
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agoImplement unregisterProtocolHandler
Arnaud Renevier [Sat, 19 Sep 2015 00:20:41 +0000 (17:20 -0700)]
Implement unregisterProtocolHandler

Now, unregisterProtocolHandler is a method of upstream
WebContentsDelegate.

The patch also fixes the protocal handler unittests. It doesn't assume
that resource files will be in /opt/resources anymore

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

Reviewed by: a1.gomes, j.majnert

Change-Id: I34738f0bf08bddb4e467a9daf4f290810eedf9ef
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agofixes utc_blink_ewk_policy_decision_frame_get unittest
Arnaud Renevier [Wed, 23 Sep 2015 21:32:49 +0000 (14:32 -0700)]
fixes utc_blink_ewk_policy_decision_frame_get unittest

in order for utc_blink_ewk_policy_decision_frame_get to work, we need to
set ewk_settings_scripts_can_open_windows_get to true before trying to
open the popup.

Reviewed by: a1.gomes, sns.park

Change-Id: I8e4d674f93278b2df7d685e55fe63af888e8e925
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agofixes utc_blink_ewk_policy_decision_response_status_code_get_func unittest
Arnaud Renevier [Wed, 23 Sep 2015 21:59:23 +0000 (14:59 -0700)]
fixes utc_blink_ewk_policy_decision_response_status_code_get_func unittest

url in the test should't redirect. But http://www.enlightenment.org
redirects to https://www.enlightenment.org/
This patch fixes that

Reviewed by: a1.gomes, sns.park

Change-Id: I24f6d93d0385dda5481b011efe01f50ce2bd6961
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agofix geolocation unittests
Arnaud Renevier [Fri, 18 Sep 2015 22:12:50 +0000 (15:12 -0700)]
fix geolocation unittests

In the current tests, we rely on geolocation callbacks to be dispatched
before document load signal. But that's not necesseraly the case.
Instead, we will use timeouts.

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

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

Change-Id: I4434d3fab580c396c91c4c1159c94427e809c80b
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agofix contextmenu unittests
Arnaud Renevier [Fri, 18 Sep 2015 22:56:00 +0000 (15:56 -0700)]
fix contextmenu unittests

In the unittestts, we try to emulate mouse click with calling .select on
textarea. But in the recent version of chromium, this won't work
anymore. Instead, we use evas_event_feed_mouse_* methods

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

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

Change-Id: Ie22c7bfdf4a7f1390d7dcff35e21648852518932
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years ago[MM][M42 Merge] Fix to add initialization of voice list instance
yournim.lee [Tue, 15 Sep 2015 04:47:02 +0000 (13:47 +0900)]
[MM][M42 Merge] Fix to add initialization of voice list instance

This patch is merged with m42 trunk but didn't migrate to m44.
The step to initialize the voice list is added when TTS starts.

This patch is not necessary for the normal usage.
But when tct-speech-w3c-tests is processed,
the below 6 cases are failed without this.

1. SpeechSynthesisVoice_default_attribute
2. SpeechSynthesisVoice_default_attribute
3. SpeechSynthesisVoice_lang_attribute
4. SpeechSynthesisVoice_localService_attribute
5. SpeechSynthesisVoice_name_attribute
6. SpeechSynthesisVoice_voiceURI_attribute

Original gerrit post : http://165.213.202.130/gerrit/#/c/83931/

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

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

Change-Id: If6981f80cd77a289c67ba5ba4d50dcef818b1603
Signed-off-by: yournim.lee <yournim.lee@samsung.com>
6 years agoFix 'select' context menu option.
Antonio Gomes [Wed, 16 Sep 2015 18:32:04 +0000 (14:32 -0400)]
Fix 'select' context menu option.

After [1] is fixed, whenever ones does a long press
gesture on a link content, a context menu shows up, whereas
one of the menu options is "select".
Naturally, once picked up, the link text should be selected,
and selection handles and context menu should appear.

Problem is that text gets selected (on the renderer side),
but there is synchronization problem on the browser side
which causes handles and context menu not being shown.
Problem is the flow:
1) user selects menu item
2) this calls ContextMenuController:ShowSelectionHandleAndContextMenu
3) This sends a IPC message to the renderer to select the content..
4) .. and tries to show the context menu and handles right after.

Calling (4) right way (3) is wrong because (3) is async.

Patch fixes this by changing (4): it now only flags
SelectionControllerEfl that a renderer text selection
change is expected.
Once the selection notification change gets to the browser
(RWHVEfl::OnSwapCompositorFrame), context menu and handles
are shown as expected.

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

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/87568/ , reviewed by
Janusz Majnert, arno renevier.

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

Reviewed by: hh4813.kim

Change-Id: I057bc74c3edaa141e7ed6fc4edab7c72638d93d2
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agofixup! [M44_2403] Chromium upversion to m44_2403 branch
Arnaud Renevier [Wed, 16 Sep 2015 23:53:16 +0000 (16:53 -0700)]
fixup! [M44_2403] Chromium upversion to m44_2403 branch

Enable Gesture detection again.

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

Reviewed by: a1.gomes, sns.park

Change-Id: I3f468e6599a0c674178d4d6bafa1494e3147c6b4
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agofixup! [M44_2403] Chromium upversion to m44_2403 branch
Arnaud Renevier [Thu, 17 Sep 2015 01:04:42 +0000 (18:04 -0700)]
fixup! [M44_2403] Chromium upversion to m44_2403 branch

Use StopChildProcess instead of base::KillProcess to kill a renderer
process.

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

Reviewed by: a1.gomes, sns.park

Change-Id: Icfdb4a657cf63d82a90b2de1796aa8069d7d7e41
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agofixup! [M44_2403] Chromium upversion to m44_2403 branch
Arnaud Renevier [Thu, 17 Sep 2015 00:39:21 +0000 (17:39 -0700)]
fixup! [M44_2403] Chromium upversion to m44_2403 branch

Implement CreateReflector. Since beta m44, Reflector cannot be
instancied directly. We create a FakeReflector class, as is done in
InProcessContextFactory.

Bug: https://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14038

Reviewed by: a1.gomes, sns.park

Change-Id: I560c4eb7d1ea945220f3a673068425662fd9ed83
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agoContext menu only shows up once
Antonio Gomes [Mon, 14 Sep 2015 21:07:15 +0000 (17:07 -0400)]
Context menu only shows up once

When one long-presses links or images, a context menu
is to be shown for further user action. In our current
implementation, this was being done by making use of EFL
'genlist' widget.

As per the docs [1], 'genlist' is a widget aimed at being
used for sophisticated list content, with support to submenus,
runtime item modifications, etc.
Problem is that there seems to be a bug in EFL (affecting
even desktop), making the context menu to show always
an empty list for the second time it is invoked.

As a quick solution for the problem, patch replaces 'genlist'
by 'list'.

[1] https://docs.enlightenment.org/auto/elementary/group__Genlist.html

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/87251/ , reviewed by
Janusz Majnert, arno renevier.

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

Reviewed by: a.renevier

Change-Id: I46efa39175d9ec9467db185cddd8347787a58607
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoMake context menu work on links and images.
Antonio Gomes [Thu, 10 Sep 2015 14:05:00 +0000 (10:05 -0400)]
Make context menu work on links and images.

Context menu were not being shown in response to long
press gestures on links and images.

These are the two cases context menu are shown:

1) user does a long press on selectable text and input fields:
context menu is only shown when user lift its finger up.
While finger is still down, user sees a magnifier glass with
zoomed in content, and can select different content as long
as it has not lifted up its finger yet.

2) user does a long press on a link and image:
context menu is shown right after the long press event gets triggered.

However, (2) was not triggering any context menu creation.
Problem was that EWebView::ShowContextMenu was calling
SelectionControllerEfl::HandleLongPress, and it was simply
calling a LOG for links and images cases.

Patch adds return value to SelectionControllerEfl::HandleLongPress to
work as an indicator that the context menu should be shown immediatly
or if it should be shown when users' finger is lifted up.

Original beta/m42 patches:
- http://165.213.202.130/gerrit/#/c/86940/ , reviewed by
Janusz Majnert, arno renevier.
- http://165.213.202.130/gerrit/#/c/86943/ , reviewed by
Jaesik Chang, arno renevier.
- http://165.213.202.130/gerrit/#/c/87569/ , reviewed by
Janusz Majnert, arno renevier.

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

Reviewed by: a.renevier, sns.park

Change-Id: I6f9873582c1b3eccd1e5f084b177538b59093502
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoFix Disambiguation popup event handling.
Antonio Gomes [Thu, 3 Sep 2015 18:55:10 +0000 (14:55 -0400)]
Fix Disambiguation popup event handling.

Add the needed interfaces in RWHVEfl so that the clicking
events on the disambiguation popup get propagated down to
the engine.

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/86481/ , reviewed by
Janusz Majnert, arno renevier.

Reviewed by: a.renevier

Change-Id: I961b4d8e558268f3c94f66c73473ecf358ead521
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoallow to build xwalk on m44
Arnaud Renevier [Tue, 15 Sep 2015 00:25:02 +0000 (17:25 -0700)]
allow to build xwalk on m44

Update build patch with latest version form chromium xwalk

In spec file: depends on dbus-glib. Modify dump.json path which has
changed in gyp. And update package versions

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

Reviewed by: a1.gomes, sns.park

Change-Id: Ic4fc5707b67605309d91fbdc3394f1287a467b4a
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agofixup! [M44_2403] Chromium upversion to m44_2403 branch
puru [Tue, 15 Sep 2015 08:56:58 +0000 (14:26 +0530)]
fixup! [M44_2403] Chromium upversion to m44_2403 branch

Build errors occurs when enabling MM feature, hence fixing them.

1. The arguments for ScopedPtrHashMap changed from <Key, value> to
<Key, ScopedPtr> updated the same.

2. VideoFrameCompositor constructor now accepts three arguments,
included compositor_task_runner as first argument.

3. UpdateCurrentFrame API was missing, hence added

4. Implementation for API UpdateCurrentFrame is changed,
and painting is done using PaintFrameUsingOldRenderingPath API

5. Implementation of OnIncomingCapturedVideoFrame API has changed,
such that we don't need to pass format, updated accordingly.

6. API ReserveOutputBuffer returns a scoped_ptr,
hence changed type for variable buffer_.

7. MessageLoopProxy is deprecated. Code should prefer to depend on
TaskRunner for passing task runners around.
scoped_refptr<base::MessageLoopProxy> -> scoped_refptr
<base::SingleThreadTaskRunner>

8. Removing some interfaces as not used anymore.

Verified ME, MSE, Webaudio working on Note4 64bit.

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

Reviewed by: SeungSeop Park, Venugopal S M

Change-Id: I0e12ada6b85b1f91168a2e1d8fcd7541214f129a
Signed-off-by: puru <puru.dharma@samsung.com>
6 years agoRemove unused SelectionControllerEfl::IsShowingMagnifier.
Antonio Gomes [Fri, 28 Aug 2015 00:37:34 +0000 (20:37 -0400)]
Remove unused SelectionControllerEfl::IsShowingMagnifier.

SSIA.

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/86029/ , reviewed by
Janusz Majnert, arno renevier.

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

Reviewed by: Janusz Majnert

Change-Id: Icb13b23c477a49224d40b8bf22de03e2afab570e
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoRemove unused param from SelectionControllerEfl::UpdateSelectionDataAndShow
Antonio Gomes [Thu, 20 Aug 2015 16:13:04 +0000 (12:13 -0400)]
Remove unused param from SelectionControllerEfl::UpdateSelectionDataAndShow

SSIA.

Patch, also removes an unneeded 'else' after a 'return'.

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/86030/ , reviewed by
Janusz Majnert, arno renevier.

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

Reviewed by: Janusz Majnert

Change-Id: Ie863b4edda43db3bcbc3256a69bd9648a2299fbf
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoRemove now obsolete hit-test based mechanism
Antonio Gomes [Fri, 28 Aug 2015 19:35:19 +0000 (15:35 -0400)]
Remove now obsolete hit-test based mechanism

Our text selection implemented has aligned to chromium's
more, and started using composited frame swap based
messaging.

Additionally, it has moved away from the previous hit
test based implementation, where messages were sent
back and forth in order to figure the context of a gesture.

It now makes use of chromium's FrameHostMsg_ContextMenu
message to get context menu / long press gesture information,
and customize existing did-handle-tap like hooks for
its needs.

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/86094/ , reviewed by
Janusz Majnert, arno renevier

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

Reviewed by: arno renevier, Janusz Majnert, SeungSeop Park

Change-Id: Ibf76529ffe3fd96de7c29ae1e6a4a5fd7dfc471b
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoRemove Handle{LongPress,Tap}Gesture declaration
Antonio Gomes [Wed, 19 Aug 2015 17:32:40 +0000 (13:32 -0400)]
Remove Handle{LongPress,Tap}Gesture declaration

These are likely left over of a past commit.

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/85430/ , reviewed by
Janusz Majnert, arno renevier.

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

Reviewed by: Janusz Majnert

Change-Id: I09025426f03964ea9d8fafcf0aab45ae3345164d
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoMove away from AsyncHitTest messages to handle tap
Antonio Gomes [Fri, 28 Aug 2015 18:28:47 +0000 (14:28 -0400)]
Move away from AsyncHitTest messages to handle tap

We have an existing mechanism to post-handle tap events.
This mechanism lives in render_view_observer_efl.cc
and is used by the renderer to inform the browser to
play sound in response to clicking a link.
In order to have the context information, an extra hit
test is performed on the renderer side, and browser gets
notified by a message.

In order to eliminate the last client of the hit test
ping-pong messaging in the text selection code, patch
extends the mechanism described above to provided the
information needed by text selection (read is_content_editable),
in addition to the current link information.

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/86093/ , reviewed by
Janusz Majnert, Arnaud Renevier.

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

Reviewed by: Janusz Majnert

Change-Id: I0b5d7f0b0d457bd89e911a2768ce0c480f331f9b
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoMove away from AsyncHitTest messages to handle long press.
Antonio Gomes [Wed, 26 Aug 2015 23:11:05 +0000 (19:11 -0400)]
Move away from AsyncHitTest messages to handle long press.

As another step to move our text selection implementation
away from hit test messages, patch makes use of the existing
FrameHostMsg_ContextMenu message in order to update the
context menu data. Flow:

0- (..)
1- RenderFrameImpl::showContextMenu
2-       <<IPC>>
3- RenderFrameHostImpl::OnContextMenu
4- WebContentsViewDelegateEwk::ShowContextMenu
5- EWebView::ShowContextMenu
6- SelectionControllerEfl::HandleLongPressEvent
7- (..)

Note that EWebView::ShowContextMenu *needs* to get called twiced
when long press is being handled: First, when long press event
processing itself happens. This is called as part of the chain
that handles message FrameHostMsg_ContextMenu (see stable above).
The second call to EWebView::ShowContextMenu used to happen
when user lifts its finger, and context menu should actually
get shown.

Additionally, patch moves in advance the moment when
long_mouse_press_ is set to true: instead of setting it
in SelectionControllerEfl::HandleLongEventPrivate, it is
now set in ::HandleGesture. That allows EWebView::ShowContextMenu
to check for it reliably instead of checking for ::IsShowingMagnifier
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=6663

Original beta/m42 patches:
- http://165.213.202.130/gerrit/#/c/85895/ , reviewed by
DONGJUN KiM, Janusz Majnert, Arnaud Renevier.
- http://165.213.202.130/gerrit/#/c/86028/ , reviewed by
Janusz Majnert, Arnaud Renevier.

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

Reviewed by: Janusz Majnert

Change-Id: I3eabb73f56d1003cff738c817e42ba8eae4e14c6
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years ago[tizen v2.4 tv] Changed gbs.conf to avoid build crash
Jongsoo Yoon [Tue, 8 Sep 2015 07:59:41 +0000 (16:59 +0900)]
[tizen v2.4 tv] Changed gbs.conf to avoid build crash

The latest tizen_v2.4 product repository is unstable.
It is better to stay at tztv-2.4-main2016-hawk-p_20150905.1 version
until the latest will be stable.

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

Change-Id: I59ccb01542323551b7fd9ba7277aa97c6e9034a8
Signed-off-by: Jongsoo Yoon <join.yoon@samsung.com>
6 years agoReplaced EXPORT_API from EAPI
Jongsoo Yoon [Thu, 3 Sep 2015 07:15:48 +0000 (16:15 +0900)]
Replaced EXPORT_API from EAPI

Now the build breaks was occurred due to the update of EFL.
According to the EFL team, we cannot use the EAPI, because
the EAPI is internal API in EFL.
So, we should use the EXPORT_API in tizen.h instead of the EAPI.

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

Reviewed by: SeungSeop Park, Antonio Gomes

Change-Id: Ic4b9c52fdad6c12d97c96c5f120ddd3ed509f9cf
Signed-off-by: Jongsoo Yoon <join.yoon@samsung.com>
6 years agoRemove ununsed ContextMenuType parameter of various methods.
Antonio Gomes [Fri, 28 Aug 2015 00:25:12 +0000 (20:25 -0400)]
Remove ununsed ContextMenuType parameter of various methods.

WebContentsViewDelegateEwk::ShowContextMenu calls
EWebView::ShowContextMenu which calls ContextMenuController
ctor. ContextMenuType is passed as a parameter through
them, but is not used.

Patch removes it.

Origin beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/86027/ , reviewed by
Laszlo Gombos, Arnaud Renevier.

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

Reviewed by: Janusz Majnert

Change-Id: Idfff4e1fd961e8cfc68a478a20bbb23fed4c0c18
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoMove SelectionMagnifierEfl away from Ecore_Animator
Antonio Gomes [Tue, 25 Aug 2015 19:12:45 +0000 (15:12 -0400)]
Move SelectionMagnifierEfl away from Ecore_Animator

In order to initiate a text selection, user has to
press and hold its finger on a selectable content.
While user has not lifted, browser keeps selecting
the content underneath user's finger. In other words,
the selected content might change while user moves
his finger around.

The way this functionality is implemented today is
extremely inefficient though: once text selection starts
an Ecore_Animator based timeout fires at every mainloop run.
In the animation callback, the following is performed:
- a snapshot is requested to fill the magnifier glass canvas
(see SelectionMagnifierEfl::UpdateLocation)
- there is a request to select the content underneath
user's finger position
(see SelectionControllerEfl::HandleLongPressMoveEvent).

That happened continously at a high rate no matter if the
finger position had changed on not.

Patch fixed this by replacing the expensive timer based
implementation by an one based on finger movement.

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/85786/ , reviewed by
DONGJUN KiM, Janusz Majnert and Arnaud Renevier.

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

Reviewed by: Janusz Majnert

Change-Id: Iffa0374a49ee912274b61e97880f7e0a5f87c9b2
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoIntegrate Composited Text selection to chromium-efl
Antonio Gomes [Wed, 19 Aug 2015 17:34:34 +0000 (13:34 -0400)]
Integrate Composited Text selection to chromium-efl

The previous way of passing text selection data from
the renderer to browser was by making use of the message
ViewHostMsg_SelectionBoundsChanged.
However, upstream Aura and Android ports moved away from using
it in favor of using "Composited selection" updates instead.

Basically, when WebRuntimeFeatures::isCompositedSelectionUpdateEnabled
is enabled, the text selection data are a RenderFrame member,
available to the browser through listening to message
ViewHostMsg_SwapCompositorFrame.
In practice, RWHV{port}::SwapCompositorFrame method collects
the selection data, and handle it accordingly.

Apart from being more aligned to other upstream ports,
that integration allows us to remove some tweaks done
in common src/ code.

Additionally, the selection_data provided by the compositor
data, contains information about the visibility of a selection
anchor. Thus, SelectionControllerEfl::visibility_rect can be dropped.

Original beta/m42 patches:
- http://165.213.202.130/gerrit/#/c/85431/ , reviewed by
Janusz Majnert and Arnaud Renevier;
- http://165.213.202.130/gerrit/#/c/85784/ , reviewed by
Janusz Majnert and Arnaud Renevier;
Patch also reverts http://165.213.202.130/gerrit/#/c/84598/ ,
which did not work, in any case.

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

Reviewed by: arno renevier, Antonio Gomes

Change-Id: I6522c6ae1d58d81908c63446a33de6cb6fad9a97
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoAlign the SelectionControllEfl selects text to TouchSelectionController
Antonio Gomes [Wed, 19 Aug 2015 15:34:32 +0000 (11:34 -0400)]
Align the SelectionControllEfl selects text to TouchSelectionController

TouchSelectionController is the upstream class that handles
touch based text selection logic for Aura and Android.
We are on our way towards being able to use it, but
we still need to eliminate some hit-testing ping-pong
messages that SelectionControllerEfl relies on in order
to figure out the text selection context.

In the meanwhile, patch aligns our SelectionControllerEfl
logic with regards to how text selection is changed
upon user movements of selection handles.

Previously, every time the handle is dragged WCI::SelectionRange
was called. That required us to have control of which
handle comes first (see SelectionDataEfl::is_anchor_first_).

Patch changes that by doing the following:
- On mouse down, we set the selection range.
- On mouse move, we just adjust the selection 'extent' accordingly.

For reference, this is the logic in TouchSelectionController, as
one can see [1] and [2].

[1] http://bit.ly/1K6NmYj
[2] http://bit.ly/1UTzAve

Original beta/m42 patches:
- http://165.213.202.130/gerrit/#/c/85429/ , reviewed by
Janusz Majnert, Arnaud Renevier.
- http://165.213.202.130/gerrit/#/c/85705/ , reviewed by
Janusz Majnert, Arnaud Renevier.

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

Reviewed by: Janusz Majnert, arno renevier

Change-Id: Ifba5ed44d335f25b15bc1975d3f54c2f9243dce6
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoSimplify SelectionControllerEfl::OnMouseMove
Antonio Gomes [Wed, 19 Aug 2015 15:18:45 +0000 (11:18 -0400)]
Simplify SelectionControllerEfl::OnMouseMove

By passing the SelectionHandleEfl object as a parameter
instead of its type, we can greatly simplify this method.

For instance, patch factors out the magnifier positioning
logic out of the switch, now that it is common for all 3
types of handles.

Additionally, now two cases in the 'switch' statement can be
colapsed (ones that handle HANDLE_TYPE_LEFT and RIGHT).

Last, patch also moves the declaration of the
local variables within the scope they are needed.

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/85428/ , reviewed by
Janusz Majnert, Arnaud Renevier.

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

Reviewed by: Janusz Majnert

Change-Id: Ie1b8a105e155e54e33e267d1ba70243a57234bd5
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoDefer to SelectionControllerEfl more gesture handling.
Antonio Gomes [Fri, 14 Aug 2015 16:15:29 +0000 (12:15 -0400)]
Defer to SelectionControllerEfl more gesture handling.

There is no funcionality change, but similarly to [1]
patch frees RWHVEfl from text selection specific gesture
handling. Logic now is centrilized SelectionControllerEfl.

This makes an eventual switch to chromium's composited
text selection more smooth from a RWHVEfl perspective.

The following class members and related logic were moved:
- should_restore_selection_menu_
- selection_acked_on_tap_
- single_tap_performed_
- was_scrolled_

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

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/85200/ , reviewed by
Janusz Majnert, Arnaud Renevier.

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

Reviewed by: arno renevier, Janusz Majnert, SeungSeop Park, David Michael Barr

Change-Id: I24115b21d4901ce1cb9070ac9f68a21c54843ead
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years ago[tizen v2.4 tv] change repository from base_2016_backup to base_2016
Youngsoo Choi [Fri, 4 Sep 2015 02:34:30 +0000 (11:34 +0900)]
[tizen v2.4 tv] change repository from base_2016_backup to base_2016

The repository base_2016_backup has been replaced with base_2016.

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

Change-Id: I0b722af18c7f9dc3b64556b3c69d88c37b4c291c
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
6 years agoRemove SelectionControllerEfl::IsCaretSelection and bogus logic
Antonio Gomes [Fri, 7 Aug 2015 17:20:24 +0000 (13:20 -0400)]
Remove SelectionControllerEfl::IsCaretSelection and bogus logic

SelectionControllerEfl::caret_selection_ is meaningless
class member currently. It is supposed to store the state
of either text selection is in "caret mode" or not. But this
is done in SelectionBoxEfl::GetCaretSelectionStatus.

In fact, ::caret_selection_ is set to 'true' at the ctor,
then set to true again in ::HandleLongPressEventPrivate.
Any associated if that checks for it is bogus since it is always
true. That includes its getter SelectionControllerEfl::IsCaretSelection.

Patch removes the associated code, and also code paths
that were unreachable before.

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/84781/ reviewed by
Arnaud Renevier and Janusz Majnert

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

Reviewed by: Janusz Majnert, arno renevier, SeungSeop Park

Change-Id: Iadbcc11f0800ceb4490a2b0ce37ee079dd45ebbb
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoSimplify SelectionControllerEfl::OnMouseDown
Antonio Gomes [Wed, 19 Aug 2015 14:42:28 +0000 (10:42 -0400)]
Simplify SelectionControllerEfl::OnMouseDown

By passing the SelectionHandleEfl object as a parameter
instead of its type, we can greatly simplify this method.

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/85427/ , reviewed by
Janusz Majnert and Arnaud Renevier.

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

Reviewed by: arno renevier, Janusz Majnert

Change-Id: I39617552aca386078d8ed110add788be650c5bdd
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoDefer to SelectionControllerEfl some gesture handling.
Antonio Gomes [Wed, 12 Aug 2015 20:14:26 +0000 (16:14 -0400)]
Defer to SelectionControllerEfl some gesture handling.

RWHVEfl class hosts most of our touch handling logic.
However some events/gestures are specific to text selection
and could be handled there.

Patch does that by adding HandleGesture{Begin,End} methods
to SelectionControllerEfl, as well as move the class variable
'restore_showing_large_handle_on_gesture_end_' there too.

That has some advanteges:
- it cleans up RWHVEfl
- It makes SelectionControllerEfl mode self-contained
- It make a future/possible replacement of SelectionControllerEfl
easier.

Origin beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/85092/ , reviewed by
Janusz Majnert and Arnaud Renevier.

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

Reviewed by: arno renevier, Jaesik Chang, Janusz Majnert, SeungSeop Park

Change-Id: I09749ba363205a389d1790b92b412daa1d5e6bed
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoMove SelectionControllerEfl ownership to RWHVEfl
Antonio Gomes [Wed, 12 Aug 2015 18:24:11 +0000 (14:24 -0400)]
Move SelectionControllerEfl ownership to RWHVEfl

Patch moves ownership of this object from WebContentsViewEfl
to RWHVEfl.

This is because SelectionControllerEfl is essentially an RWHVEfl
object. Additionally, analogy classes like RWHVAndroid and RWHVAura
also own their respective TouchSelectionController instances.

Origin beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/85091/ , reviewed by
Janusz Majnert and Arnaud Renevier.

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

Reviewed by: arno renevier, Janusz Majnert

Change-Id: I84ed0ed9bc7535b8f2ebb003b7eeb9cdb1a3e951
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoDo not enter "caret" text selection mode for non editable content
Antonio Gomes [Mon, 10 Aug 2015 15:54:06 +0000 (11:54 -0400)]
Do not enter "caret" text selection mode for non editable content

Our SelectionControllerEfl implementation consists in a
state machine. States are:

-not in selection mode
-selection mode (two handles on regular page content / text)
-editable selection mode (two handles on editable content)
-caret selection mode (one handle/cursor on editable content)

In summary, we need to sync our own state machine with chromium's.

When PRESS_SHOW event is handled, it is checked if
underneath content is "editable". If it is, selection enters
in either "editable selection" mode or "caret mode".
Otherwise, we should not enter neither of such modes
Patch fixed this.

In practice, it fixes the following problem:

When context menu is to be shown, its location is calculated based
on the handle(s) location. There are two code paths.
- when both handles are shown (i.e. non caret selection mode).
- when only the large handle is shown (i.e. carent selection mode).

In case of (1), context menu is placed either close to the "start"
handle or "end" handle, in that order, depending of their
visibility.
In case of (2), only the "start" handle is checked, as it is
supposed to only when handle being shown.

If we were wrong in caret mode (code path (2) above), the wrong
code path would be used to calculate the position, and context menu
would get misplaced.

Original beta/m42 patch:
- http://165.213.202.130/gerrit/#/c/84851/, reviewed by Hyunhak Kim
and Arnaurd Renevier

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

Reviewed by: arno renevier, Janusz Majnert, SeungSeop Park

Change-Id: I09f37fbf2d226d3a1feaef6aff5c3211188246db
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agofixup! [M44_2403] Chromium upversion to m44_2403 branch
Arnaud Renevier [Wed, 26 Aug 2015 22:07:29 +0000 (15:07 -0700)]
fixup! [M44_2403] Chromium upversion to m44_2403 branch

Enable permissions. Since m44, Permissions related methods have been
refactored out of ContentBrowserClient.

This patch creates a PermissionManagerEfl interfaces that subclasses
PermissionManager.

The callback has changed too. Instead of accepting a boolean, it accepts
a PermissionStatus which can be DENIED, GRANTED or ASK. We only return
DENIED or GRANTED, but need to update callback signatures.

In order to do so, we also refactor Ewk_Suspendable_Object. Callback is
no longer a direct member of the class. That way, subclasses
(_Ewk_Geolocation_Permission_Request and
_Ewk_Certificate_Policy_Decision) can have different callback
signatures.

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

Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I24b08137cf27c4ada57e92305e03957dd3b0681d
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agoBuildRequires ail when building crosswalk
Arnaud Renevier [Fri, 28 Aug 2015 22:49:30 +0000 (15:49 -0700)]
BuildRequires ail when building crosswalk

crosswalk now depends on ail

Reviewed by: SeungSeop Park, Antonio Gomes

Change-Id: Ice9727feeeaf638c8c50cdc22b00887efebff0c9
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agofixup! [M44_2403] Chromium upversion to m44_2403 branch
Arnaud Renevier [Tue, 25 Aug 2015 22:26:34 +0000 (15:26 -0700)]
fixup! [M44_2403] Chromium upversion to m44_2403 branch

update to new DevToolsHttpHandler signature

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

Reviewed by: SeungSeop Park, Antonio Gomes

Change-Id: I3c906a4749983edbef540ef3a385e0a00ee668e7
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agofixup! [M44_2403] Chromium upversion to m44_2403 branch
Arnaud Renevier [Mon, 24 Aug 2015 23:19:34 +0000 (16:19 -0700)]
fixup! [M44_2403] Chromium upversion to m44_2403 branch

For now, when typing in an input box, if autofill is enabled, browser
will crash. It is because pref_service of PersonalDataManager has not
been initialized.

This patch fixes the crash by initializing it. It safe for the moment to
pass a NULL account tracker. No server profile is registered in the
database.  So account tracker will not be used in PersonalDataManager.

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

Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I2fc70a6658273942fd1b2cadf77c1afaa32e9a04
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agoApplied the python acceleration for generating gyp files
Youngcheol Kang [Mon, 24 Aug 2015 01:50:51 +0000 (10:50 +0900)]
Applied the python acceleration for generating gyp files

The previous way of python acceleration used the below build option.
(--extra-packs python-base-x86-arm,python-x86-arm,python-xml-x86-arm)

However, the python-cross package is supported instead of above
packages in tizen 3.0 profile by platform team.
(https://review.tizen.org/gerrit/#/c/46462/)

Using the python-cross package, the speed of gyp generation gets faster
than before. (10~20min -> 1~2min)

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

Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I8f82f24a55f2f4e4c1bf3765b1b0455d50a45a8d
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
6 years agofixup! [M44_2403] Chromium upversion to m44_2403 branch
Arnaud Renevier [Tue, 18 Aug 2015 01:03:00 +0000 (18:03 -0700)]
fixup! [M44_2403] Chromium upversion to m44_2403 branch

This patch fixes a crash when processing touch events. This happens when
running ubrowser with --mobile on the desktop, and clicking on a
document that doesn't have touch event handlers.

For example, on google.com with ubrowser --mobile, clicking somewhere,
and then moving the mouse (without first releasing it) results in a
crash.

Since m44, we don't use ShouldForwardTouchEvent, and and we always
forward the touch events to the renderer. But this means that we
shouldn't call AckSyncTouchEvent. Instead, ProcessAckedTouchEvent will
be called.

Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I278fd83f59b74a764a3fa8ad683dfec55e4951d0
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agoallow content_shell to run in sandbox mode
Arnaud Renevier [Fri, 14 Aug 2015 21:06:54 +0000 (14:06 -0700)]
allow content_shell to run in sandbox mode

Calling elm_init/ecore_init and other efl init methods is not needed in
renderer process. Also, this prevents the process to be sandboxed
(because those methods call pthread_create, and for some reason, it is
not possible to enter sandbox mode afterwards).

So this patch do not calls those methods in a zygote process.

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

Reviewed by: Antonio Gomes, Laszlo Gombos, SeungSeop Park

Change-Id: I15bf9d8f09952cfdb217ce3b3d4dc6e53bc7aacd
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
6 years agofixup! Bringup Text selection
Antonio Gomes [Tue, 11 Aug 2015 18:37:08 +0000 (14:37 -0400)]
fixup! Bringup Text selection

Patch removes RWHVEfl::SelectionRootBoundsChanged now that
ViewHostMsg_SelectionRootBoundsChanged and
ViewHostMsg_SelectionBounds_Params are condensed.

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

Together with: Ie0756433b5a5301000b23d97fb72ebf837caf4e8

Reviewed by: arno renevier, Balazs Kelemen, Hyunhak Kim, Antonio Gomes

Change-Id: Ic2878c9f6d09b11f89f79b18e443500fc63d1d6a
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years ago[M44_2403] Add arm64 support to ffmpeg
Youngcheol Kang [Tue, 11 Aug 2015 11:04:57 +0000 (20:04 +0900)]
[M44_2403] Add arm64 support to ffmpeg

We need to regenerate this arm64 support patch in ffmpeg
until ffmpeg supports arm64 architecture in mainstream.

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

This patch referred to http://165.213.202.130/gerrit/#/c/80261

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

Reviewed by: Antonio Gomes, joonhwan kwon

Change-Id: Ic7e6b1a8ab9b3976f0cf3fc3eafb2a7d9202708a
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
6 years agoRemove dependencies of tizen v2.2.1
Youngsoo Choi [Fri, 7 Aug 2015 03:43:50 +0000 (12:43 +0900)]
Remove dependencies of tizen v2.2.1

The tizen v2.2.1 is not supported anymore.

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

Reviewed by: SeungSeop Park, Antonio Gomes

Change-Id: I3072ff0cf6825caaf8cc487061df9ee0c3aae7d9
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
6 years ago[dev/m42][uBrowser] No selection handler on long text
Antonio Gomes [Thu, 6 Aug 2015 22:05:21 +0000 (18:05 -0400)]
[dev/m42][uBrowser] No selection handler on long text

Our text selection and context menu code has some logic
to _not_ show the context menu when it calculates handles
are not positioned properly. See for example, some of cases
returning 'false' in ContextMenuControllerEfl::ShowContextMenu.

This has two problems:
1) It only takes into account the "start" handle, and no
matter if the "end" handle were onscreen, it was ignored.
2) It was not only not showing the context menu.
By returning "false" in ContextMenuControllerEfl::ShowContextMenu,
selection was actually canceled, leaving SelectionController
and the renderer process selection code out of sync.

In s-browser (verified on S6) context menu is not shown
if both handles are offscreen and the selected content itself
is also offscreen.

Patch makes us behavior similarly to s-browser by.
implementing the first patch of the behavior described:
if no handles are on screen (i.e. both are scrolled away from
viewport), do not show the context menu.

Another big different from the previous behavior is that
we do not leave selection mode when handles are offscreen.
Instead, we do not show context menu only. Once handles
are onscreen again, context menu is shown just fine.

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

Reviewed by: SeungSeop Park

Change-Id: Ic39d9497f2fa4400096bd8c8356183b0cab760d6
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoMake context menu available on desktop builds.
Antonio Gomes [Thu, 6 Aug 2015 22:05:49 +0000 (18:05 -0400)]
Make context menu available on desktop builds.

By setting the "copypaste" which is Tizen specific,
context menu fails to show.
Patch makes this workaround (as per the comment describing
it) OS Tizen specific.

Reviewed by: SeungSeop Park

Change-Id: Ie3342b1e50ca3e1d0f41bc6a4796a0e11b709428
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years ago[M44_2403] Chromium upversion to m44_2403 branch
Youngcheol Kang [Wed, 29 Jul 2015 06:51:54 +0000 (15:51 +0900)]
[M44_2403] Chromium upversion to m44_2403 branch

This patch cherry-picked from below patches of dev/master branch,

[Origin Patch]

- [M43_2357] Chromium upversion to m43_2357 branch
  (http://165.213.202.130/gerrit/#/c/79126/)
- fixup! [M43_2357] Chromium upversion to m43_2357 branch
  (http://165.213.202.130/gerrit/#/c/80912/)
- Always forward touch events to the TouchEventQueue
  (http://165.213.202.130/gerrit/#/c/80137/)
- [M44_2403] Chromium upversion to m44_2403 branch
  (http://165.213.202.130/gerrit/#/c/81624/)
- [M44_2403][EWK_BRINGUP] modify to call WebContents::ClosePage()
  (http://165.213.202.130/gerrit/#/c/82170/)
- [M44_2403][EWK_BRINGUP] modify to call RenderViewHostImpl::Blur() directly
  (http://165.213.202.130/gerrit/#/c/82168/)

This patch deals with following upstream changes:

1. Folder and class name changes

- Changed the name of the LayoutThemeChromium* to the LayoutTheme*
- Changed the name of the devtools_target.h to the
  devtools_target_descriptor.h. Also the namespace and path changed to
  the components/devtools_discovery from the content/public/browser
- Changed the path of the content_renderer to the relative path from the
  absolute path
- Changed the path of the skcanvas_video_renderer.h to the media/blink
  from the media/filters
- Changed the path of the video_capture_types.h to the media/base from
  the media/video/capture
- Changed the namespace and path of the devtools_http_handler.h to the
  components/devtools_http_handler from the content/public/browser
  Because of this the namespace of the DevToolsDelegateEfl class in
  impl/ changed to the devtools_http_handler from the content
- Changed the namespace and path of the sqlite_persistent_cookie_store.h
  to the net/extras/sqlite from the content/browser/net

2. API changes

- Added the CreateGPUTimingClient API in the gfx::GLContext class
- Added the CreatePlugin API in LoadablePluginPlaceholder class
- Added the DidHostRunInsecureContent, HostRanInsecureContent API in
  SSLHostStateDelegate class
- Added the GetPermissionManager API in the BrowserContext class
- Added the GetFrontendResource, GetPageThumbnailData API in
  DevToolsHttpHandlerDelegate class
- Added the GetImageTextureTarget, GetTaskGraphRunner API in
  ContextFactory class
- Changed the argument of Compositor API in ui class
- Changed the argument of CreateNewWindow API in Shell class
- Changed the argument of CreateOutputSurface API in ContextFactory
  class
- Changed the argument of DownloadImage API in WebContentsImpl class
- Changed the argument of DidStartLoading API in RenderViewImpl class
- Changed the argument of EnterFullscreenModeForTab API in
  WebContentsDelegate class
- Changed the argument of InProcessGpuThread in content class
- Changed the argument of ReadbackYUV API in ReadbackYUVImpl class
- Changed the argument of RemoveReflector API in ContextFactory class
- Changed the argument of SQLitePersistentCookieStore API in net class
- Changed the argement of ReportValueChanged, SetValue, RemoveValue API
  in PersistentPrefStore class
- Changed the return type of CreateReflector API in ContextFactory class
- Changed the ExecuteJavaScriptForTests API to the
  ExecuteJavaScriptWithUserGestureForTests API in the RenderFrameHost
  class
- Changed the GetLastActivityTime API to the GetLastActiveTime API in
  the DevToolsTargetDescriptor class. Also the return type was changed
  to the base::Time from the base::TimeTicks
- Changed the OnUnmaskVerificationResult, GetRapporService,
  PropagateAutofillPredictions, IsAutocompleteSavingEnabled API in
  AutofillClient class
- Changed the PermissionType class to enum class
- Removed the Blur API in RenderWidgetHostView class
- Removed the BundlesFrontendResources, GetDebugFrontendDir API in
  DevToolsHttpHandlerDelegate class
- Removed the CreateMediaSourceErrorEvent API in MediaLog Class
- Removed the CreateNewTarget, EnumerateTargets, GetPageThumbnailData
  API in DevToolsManagerDelegate class
- Removed the DidCreateScriptContext API in ContentRendererClient class
- Removed the GetCompositorMessageLoop API in ContextFactory class
- Removed the IsAutocompleteSavingEnabled, DetectAccountCreationForms
  API in AutofillClient class
- Removed the OrientationLock API in ScreenOrientationDelegate class
- Removed the OrientationLock, OrientationUnlock class in
  WebContentsDelegate class
- Removed the ShouldForwardTouchEvent API in RenderWidgetHostImpl class
- Removed the WebContentsDestroyed API and added the ResetDialogState
  API in JavaScriptDialogManager class

3. Need to change API (TODO issues)

- The icu function has been changed. Temporary using the icu of the
  chromium instead of the system.
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=13290)
- The multimedia function has been changed. Temporary disabling the
  'tizen_multimedia_support' flag
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14037)
- The Reflector class has been changed. Temporary disabling the codes
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14038)
- The KillProcessAPI was removed in base class. Temporary disabling the
  codes (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14039)
- Some switch was removed in the cc::switches class. Temporary disabling
  the codes (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14040)
- The GestureConfiguration class has been changed. Temporary disabling
  the 'gesture_detection' setting
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14041)
- The PersonalDataManager class has been changed. Temporary disabling
  the codes (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14042)
- The RequestPermission class has been changed. Temporary disabling the
  codes (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14043)
- The DevToolsHttpHandler class has been changed. Temporary disabling
  the codes (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14044)
- The g_app_info_launch_default_for_uri API was undefined in desktop
  build (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14045)
- The GPU function has been changed. Temporary disabling the
  'tizen_disable_gpu_thread' flag
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14046)
- The build breaks in libGLESv2.gypi. For the temporary solution the
  patch file into the /src/tizen_src/build/patches
  (http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14048)

4. Verification

- This patch supports to the mobile and tv target on Tizen 2.4 and 3.0
- There is no build breaks in all target
- The browser (efl_webview_app, mini_browser, ubrowser) is working in
  all target

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

Change-Id: Ic9f5cf9bde8340030ac8a296d6c324f35c547f3c
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
6 years agoFixing warnings of conflicted gcc switches for tizen v2.4 product tv
Youngsoo Choi [Tue, 4 Aug 2015 09:19:50 +0000 (18:19 +0900)]
Fixing warnings of conflicted gcc switches for tizen v2.4 product tv

[issue]
There are tons of warnings like below, during build for tizen v2.4 tv.
>> warning: switch -mcpu=cortex-a15.cortex-a7 conflicts with
>> -march=armv7-a switch

[cause]
The tizen v2.4 platform provides a switch '-mcpu=cortex-a15.cortex-a7'.
Simultaneously, some of chromium gyp try to insert a switch '-march=armv7-a'.

Current gcc version does not support a combination of the switches
'-mcpu=cortex-a15.cortex-a7' and '-march=armv7-a'.

[resolve]
The options '-mcpu' and '-march' are used to schedule things appropriately
for particular chip.

>> -march=cpu-type
>> Generate instructions for the machine type cpu-type.
>> The choices for cpu-type are the same as for -mcpu.
>> Moreover, specifying -march=cpu-type implies -mcpu=cpu-type.

According to above gnu docs, march implies mcpu,
so one of them can be chosen alone.

This patch utilizes platform switch '-mcpu=cortex-a15.cortex-a7',
instead of chromium switch '-march=armv7-a' to avoid the confliction,
and another platform switch '-mtune=cortex-a15.cortex-a7' too.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14098
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I933a78c8dda63809e5aa7bf82c67c0a0cc83891f
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
6 years agofixup! Bringup Text selection.
Youngcheol Kang [Wed, 5 Aug 2015 10:10:11 +0000 (19:10 +0900)]
fixup! Bringup Text selection.

Disabled the CompositedSelectionUpdate by disable-blink-features

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

Change-Id: Iea324e315b808581279bb6375f4d973f6d1fa4d3
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
6 years agoShare more configuration options between desktop and Tizen builds
Youngcheol Kang [Tue, 4 Aug 2015 05:05:28 +0000 (14:05 +0900)]
Share more configuration options between desktop and Tizen builds

This patch cherry-picked from the dev/master by Laszlo Gombos
Origin Patch : http://165.213.202.130/gerrit/#/c/83779

Sharing more options makes the desktop build closer
to Tizen specific builds and enablers better simulation
of Tizen environment on desktop.

This change also disables alsa, gconf, kerberos for
desktop builds.

Bug: None.

Change-Id: I7364cf703b8f20c1692a274de7d8c0a84c8fae0e
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
6 years agoIntroduce version-aware Tizen macros
Antonio Gomes [Tue, 4 Aug 2015 17:32:33 +0000 (13:32 -0400)]
Introduce version-aware Tizen macros

Previously there were #if defined(TIZEN_V_XXX) guards
over the code in order to condition compilation to specific
Tizen versions.
Although it works, it is not elegant and other parts of
the source base do not use similar practice.

Instead, expanding macros are used to condition code
to a specific compiler version or alike
(see GCC_VERSION in third_party/WebKit/Source/wtf/Compiler.h).

Patch implements and deploys a set of macros to improve the code:
- TIZEN_VERSION_EQ(major,minor,patch)
- TIZEN_VERSION_AT_LEAST(major,minor,patch)

The old TIZEN_V_XXX macros are still available
to be used by our sample apps (ubrowser, mini_browser,
and efl_webview_app) through tizen_version_defines_for_apps.gypi

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=7198
Reviewed by: SeungSeop Park, arno renevier

Change-Id: Id1df977540edfa95b2eb509db4743539157c3922
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years ago[MM] Enabling MM features for Tizen v2.4 on TV
Umesh Kumar Patel [Fri, 31 Jul 2015 06:44:50 +0000 (12:14 +0530)]
[MM] Enabling MM features for Tizen v2.4 on TV

This patch contains following change list:
1. Using software path for MM, until pixmap is enabled.
2. Using GST Port for ME, as MediaPacket is not supported.
3. Changes to use legacy capi for HawkP 2.4
4. Moved related changes tizen_multimedia_use_capi_for_me
   to media_efl.gypi
5. Removed ASM related obsolete files
6. Other dependency and build error changes

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14072
Reviewed by: SeungSeop Park, Sharath Kamath

Change-Id: I1af848361483ed6d0dae0101621db02ec9884bbc
Signed-off-by: Umesh Kumar Patel <umeshk.patel@samsung.com>
6 years agoImplement ssl_host_state_delegate
DongJun Kim [Thu, 30 Jul 2015 11:23:58 +0000 (20:23 +0900)]
Implement ssl_host_state_delegate

Certificate popup have always showing up,
because certificate decisions are not remembering on "blink" side.

"ssl_host_state_delegate" is for managing decisions from user,
but it was not implemented on latest code.
We need to add it for resolving this issue.
Reviewed by: Hyunhak Kim, Janusz Majnert

Change-Id: I9d11b7414816431047c91a6c87cd83d93a9db192
Signed-off-by: DongJun Kim <djmix.kim@samsung.com>
6 years ago[MM] Proper use of tts_get_speed_range()
ws29.jung [Wed, 22 Jul 2015 11:12:33 +0000 (20:12 +0900)]
[MM] Proper use of tts_get_speed_range()

We should pass non-null parameters to |tts_get_speed_range|.
Otherwise, invalid parameter error would be raised by TTS engine.

In addition to fixing above, patch does followings as well.
- Update the logic of converting from float rate to int speed used by
  TTS engine. According to platform team, the rate-to-speed mapping varies
  depending on specific TTS engine on use, but current setting is roughly
  as follows:
    'tts_speed' | rate
    -----------------
    Min.        | 0.6
    Normal      | 1.0
    Max.        | 2.0
  Linear interpolation method is adopted to get |tts_speed| from
  an arbitrary |rate|.
- Make the private method that did the conversion as a local free function.
- Drop legacy 2.3 code path, which seems no longer valid.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=13980
Reviewed by: SeungSeop Park, Sharath Kamath

Change-Id: Ic3964561b5fc8bf62d6d3e1f67c55669db978039
Signed-off-by: ws29.jung <ws29.jung@samsung.com>
6 years agoFix crash in WebContentsViewEfl::UpdateDragDest function
Youngcheol Kang [Tue, 4 Aug 2015 04:56:38 +0000 (13:56 +0900)]
Fix crash in WebContentsViewEfl::UpdateDragDest function

This patch cherry-picked from the dev/master by Hyunhak Kim
Origin Patch : http://165.213.202.130/gerrit/#/c/81251/

Due to patch on upsteam(https://codereview.chromium.org/950223006/#ps150001),
UpdateDragDest function is called too early before RenderWidgetHostView is created.
So, modify to check RenderWidgetHostView value in UpdateDragDest() and call it when
creating View.

Change-Id: I8d376259e60471847537e2a5caabc3150ee885e5
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
6 years ago[mobile][tizen v3.0] changes of sensor lib header
Youngcheol Kang [Tue, 4 Aug 2015 04:47:48 +0000 (13:47 +0900)]
[mobile][tizen v3.0] changes of sensor lib header

This patch cherry-picked from the dev/master by Youngsoo Choi
Origin Patch : http://165.213.202.130/gerrit/#/c/77837/

The header name of sensor lib has been changed
from sensor/sensors.h to sensor/sensor.h.

In addition, tizen v2.2.2 which is used only for tv doesn't
need to support the sensor feature.

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

Change-Id: I565461557a788b43dbbc2f6f9709b46b01946eab
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
6 years agoMark b.kelemen, l.gombos, sl.ostapenko as inactive.
Youngcheol Kang [Tue, 4 Aug 2015 05:01:53 +0000 (14:01 +0900)]
Mark b.kelemen, l.gombos, sl.ostapenko as inactive.

This patch cherry-picked from the dev/master by Laszlo Gombos
Origin Patch : http://165.213.202.130/gerrit/#/c/83366

This means they still qualified and authorized owners but
at the moment they not activly involved with the project.

Also sorted the inactive section.

Change-Id: If0928a883bb6c98ef6f3d3109e9d73acaaf7da08
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
6 years agoReplacing deprecated 'efl-assist' with 'efl-extension'
Youngsoo Choi [Mon, 3 Aug 2015 07:27:08 +0000 (16:27 +0900)]
Replacing deprecated 'efl-assist' with 'efl-extension'

The related APIs are changed to use 'efl-extension'.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14077
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I30ca226d679abc8ac3ab0f37520445bf934d32c0
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
6 years agoAdded option to uBrowser to enable web inspector server.
Wojciech Bielawski [Fri, 31 Jul 2015 08:40:37 +0000 (10:40 +0200)]
Added option to uBrowser to enable web inspector server.

Sometimes it's usefull to have a web inspector running on mobile/TV.
This patch provides new "context menu" option for uBrowser to start
web inspector server.
Reviewed by: Antonio Gomes, Daniel Waślicki

Change-Id: I83613375a83a1f6361a4c9a9a1a2ef10731d299d
Signed-off-by: Wojciech Bielawski <w.bielawski@samsung.com>
6 years agoDisable v8 command line options tweaking v8 garbage collector.
Pawel Forysiuk [Fri, 31 Jul 2015 09:58:34 +0000 (11:58 +0200)]
Disable v8 command line options tweaking v8 garbage collector.

They add some performance penalty and should probably be enabled
selectively only on very memory constrained devices.
Reviewed by: Antonio Gomes, kavitha devara

Change-Id: Id9f6e676fea9077b8a709577dd8e316051fa5c2a
Signed-off-by: Pawel Forysiuk <p.forysiuk@samsung.com>
6 years agoInitial support for tizen v2.4 product tv
Youngsoo Choi [Wed, 22 Jul 2015 01:12:35 +0000 (10:12 +0900)]
Initial support for tizen v2.4 product tv

1) Providing python acceleration
   It makes generating gyp extremely faster than before.

2) Solving python-base issue with supplementary repository
   This issue has been observed on tizen v2.2.1 as well.
   The following patch of python-base causes errors.
   >> subprocess-can-raise-OSError-EPIPE-when-communicatin.patch

3) Supporting prebuilt gold for tizen v2.4 tv
   The chromium-efl built by our own prebuilt gold can not be executed
   on target, so gold linker is newly built based on current platform.

4) Fixing not founding EGL header error
   Following error was happened.
   >> fatal error: EGL/egl.h: No such file or directory

5) Applying changed API usage of capi-appfw-application
   Tizen v2.4 and v3.0 tv support the changed API usage as mobile does.

6) Disabling multi-media feature
   Some of undefined multi-media variables are being used on x11.
   Enabling the feature will be handled in other patch set.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14033
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: If510789d3e8990867ddd3618e072c43bbba0ab37
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
6 years agoRemove gcc46_compat_wrappers default include path
Antonio Gomes [Wed, 29 Jul 2015 17:15:13 +0000 (13:15 -0400)]
Remove gcc46_compat_wrappers default include path

This directory does not exist anymore, since we
remove GCC 4.6 support.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12987
Reviewed by: Antonio Gomes, SeungSeop Park

Change-Id: I1acc42eb14e59900840c19bf5f9f735472f2ec16
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agoUnify LOCALE_DIR resolve logic for desktop and GBS builds
Antonio Gomes [Wed, 29 Jul 2015 21:59:22 +0000 (17:59 -0400)]
Unify LOCALE_DIR resolve logic for desktop and GBS builds

Patch uses similar logic then EDJE_RESOURCES_DIR,
and eliminates more #if in our more.
Reviewed by: Janusz Majnert, SeungSeop Park

Change-Id: I45844c0a67cecf6c1801712aa4c4d85fa2b0c6c4
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agofixup! Make text selection feature available on desktop builds.
Antonio Gomes [Wed, 29 Jul 2015 20:20:22 +0000 (16:20 -0400)]
fixup! Make text selection feature available on desktop builds.

The original patch [1] had one problem:

No matter if one makes a debug or release build,
the value passed to GYP was the same, e.g.:

"$ build_desktop.sh --debug"

... was going to pass "-Dedje_dir=out.x64/Debug" to ninja,
regardless of the build configuration (Release or Debug), as show below.

$ grep -nHR edje_dir out.x64 --include=*.ninja
/out.x64/Debug/obj/content/content_common.ninja:
  '-DEDJE_DIR="/path>/out.x64/Debug/resources"'
/out.x64/Release/obj/content/content_common.ninja:
  '-DEDJE_DIR="/path>/out.x64/Debug/resources"'

See "Debug" in both ninja files.

To fix that, patch passes only the generic "out" directory to GYP,
and yields to GYP the responsibility to identify the build configuration.

"out" will allow us to use the logic to unify more codepaths,
including LOCALE_DIR.

[1] http://165.213.202.130/gerrit/#/c/84154/
Reviewed by: Janusz Majnert, SeungSeop Park

Change-Id: I6504ddb14ba4ee770701d61e26b28a108be68777
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years ago[Chromedriver] Enable chromedriver build
Kamil Nowac [Wed, 15 Jul 2015 10:50:59 +0000 (12:50 +0200)]
[Chromedriver] Enable chromedriver build

Patch enables chromedriver build with --chromedriver flag
and fixes some major gypi problems.
Reviewed by: Antonio Gomes, Janusz Majnert

Change-Id: Ia51f2fb134867ae144ab76417848b01f8fde0741
Signed-off-by: Kamil Nowac <k.nowac@samsung.com>
6 years ago[EWKAPI] Implementation features of tizen extensible api
Sebastian Stolinski [Wed, 29 Jul 2015 06:57:43 +0000 (08:57 +0200)]
[EWKAPI] Implementation features of tizen extensible api

Patch adds functions ewk_context_tizen_extensible_api_string_set
and ewk_context_tizen_extensible_api_string_getewk_object_unref

This API is requested by WRT team. WRT used this api since tizen 2.4

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

Reviewed by: DONGJUN KiM, Janusz Majnert, Karol Furmaniak
Signed-off-by: Sebastian Stolinski <s.stolinski@samsung.com>
Change-Id: Ib5ccf0146b6a2126d4abf12d866fd3ab86192269

6 years agoFix for black screen issue on tizen 2.4 product TV
Chandan Padhi [Wed, 29 Jul 2015 12:11:40 +0000 (17:41 +0530)]
Fix for black screen issue on tizen 2.4 product TV

Currently, glFlush gives black screen on tizen 2.4 product tv
similar to that in Note4 and XU3.
This patch uses glFinish instead of glFlush in SwapBuffers.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14021
Reviewed by: Antonio Gomes, DaeHyun Ko, Siba Samal, venu musham

Change-Id: I09e55ff7aaff0eb8407c28524fbc03c9c506e211
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
6 years agoRevert "Set same 'Source' name for chromium-efl-libs.spec and crosswalk-bin.spec"
hh4813.kim [Fri, 24 Jul 2015 04:43:41 +0000 (13:43 +0900)]
Revert "Set same 'Source' name for chromium-efl-libs.spec and crosswalk-bin.spec"

This reverts commit 40e0e4088732224c3c812860bf065d8bd72c17e1.

Due to SR build failed on OBS, revert the commit.

Comments: build fail : chromium-efl-libs
[  229s] + rm -rf chromium-efl-libs-42.2311.0.11
[  229s] + /bin/gzip -dc /home/abuild/rpmbuild/SOURCES/chromium-efl-42.2311.0.11.tar.gz
[  229s] + /bin/tar -xf -
[  301s] + STATUS=0
[  301s] + [ 0 -ne 0 ]
[  301s] + cd chromium-efl-libs-42.2311.0.11
[  301s] /var/tmp/rpm-tmp.ROzt89: line 32: cd: chromium-efl-libs-42.2311.0.11: No such file or directory

Git project: [platform/framework/web/chromium-efl]
Tag: submit/tizen_mobile/20150721.055717
Reviewed by: Antonio Gomes, DONGJUN KiM

Change-Id: Ia500c33d83a37791f3a5c5a63c4d0dd14f381e84
Signed-off-by: hh4813.kim <hh4813.kim@samsung.com>
6 years agoMake text selection feature available on desktop builds.
Antonio Gomes [Fri, 24 Jul 2015 20:31:36 +0000 (16:31 -0400)]
Make text selection feature available on desktop builds.

In GBS builds, we pass edje_dir value used to specify EDJ
resources' location as a GYP parameter (see -Dedje_dir in
chromium-efl.spec, for instance).

In order to have text selection on desktop builds, patch
does the same from build/common.sh.
Also, it makes GBS and desktop builds to take the same
codepath when resolving the respective edj path, eliminating
Reviewed by: Janusz Majnert, SeungSeop Park

Change-Id: Ia00bef1690f97ea77ff763718e6b36b6534fd9d0
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
6 years agofixup! Add command line switches for memory optimization patches recently merged.
Pawel Forysiuk [Mon, 27 Jul 2015 08:33:58 +0000 (10:33 +0200)]
fixup! Add command line switches for memory optimization patches recently merged.

Patch disables "--grow-semi-space-by-n-pages" and
"--custom-semi-space-growth=0" switches from V8's js-flags.
It seems to introduce performance regression in sunspider 1.0.2 test.

Note that [1] only enables the switch. Feature itself was worked on in [2].

[1] http://165.213.202.130/gerrit/#/c/83144/
[2] http://165.213.202.130/gerrit/#/c/81306/
Reviewed by: Antonio Gomes, kavitha devara

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