SeungSeop Park [Tue, 19 Jan 2016 09:10:17 +0000 (18:10 +0900)]
fixup! [M48_2564] Chromium upversion to m48_2564 branch
Fix warnings of '-Wunused-variable'.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15496
Reviewed by: a1.gomes, msu.koo, sns.park
Change-Id: I6063531c4a9e67277765b3e2475156e8dec1f9a2
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
y0.kim [Thu, 14 Jan 2016 03:12:50 +0000 (12:12 +0900)]
fixup! [MM] Webaudio with CAPI port.
Fix compilation warnings in audio_decoder_capi.cc
>comparison between signed and unsigned integer expressions [-Wsign-compare]
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15560
Reviewed by: a1.gomes, msu.koo, sns.park
Change-Id: Ie22b4f062e13474e428c20b7ce713d006e1a7bcc
Signed-off-by: y0.kim <y0.kim@samsung.com>
Youngsoo Choi [Tue, 19 Jan 2016 04:11:25 +0000 (13:11 +0900)]
[tizen v3.0] Add api-version to meet syntax
The definition of property api-version is essential on tizen v3.0.
It can be used for API 'privileg_manager_get_mapped_privilege_list'.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15627
Reviewed by: a1.gomes, sns.park
Change-Id: Icf2f52495eb42ba1c0398e8eebfba070e27a951c
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
SeungSeop Park [Tue, 12 Jan 2016 11:41:32 +0000 (20:41 +0900)]
fixup! [M40 Merge][Tizen3.0][WRT] Refactor ewk_send_widget_info API
Patch fixes warning:
> 'EWebContext::inspector_server_' will be initialized after...
> 'double EWebContext::widget_scale_' [-Wreorder]
> 'EWebContext::m_pixmap' will be initialized after...
> 'double EWebContext::widget_scale_' [-Wreorder]
Original beta/m47 patches:
- http://165.213.202.130/gerrit/#/c/103997/
- http://165.213.202.130/gerrit/#/c/103199/
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15496
Reviewed by: msu.koo
Change-Id: I087826db0dab99611f3b77646f559f8e912a128d
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
Tomasz Czekala [Thu, 14 Jan 2016 10:25:29 +0000 (11:25 +0100)]
Add return value for SSLHostStateDelegateEfl::HasAllowException
bool SSLHostStateDelegateEfl::HasAllowException was not returning anything
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15496
Reviewed by: sns.park
Change-Id: Ifb8ac5d0c3b1d6e5559b1fc4dd2549bd91cb33aa
Signed-off-by: Tomasz Czekala <t.czekala@samsung.com>
y0.kim [Thu, 7 Jan 2016 06:35:58 +0000 (15:35 +0900)]
fixup! [wayland] Webcursor for wayland
Fix warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15496
Reviewed by: sns.park
Change-Id: I7fc56ea91c46ff39d7259f51d230d99e5dcf0e93
Signed-off-by: y0.kim <y0.kim@samsung.com>
SeungSeop Park [Mon, 18 Jan 2016 09:14:04 +0000 (18:14 +0900)]
Fix compilation warning in permission_manager_efl.cc
Patch returns a dummy value to suppress warning.
> warning: no return statement in function returning non-void [-Wreturn-type]
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15541
Reviewed by: msu.koo
Change-Id: I8f093960c853b9b1bc13573b175c81b27241337d
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
msu.koo [Mon, 11 Jan 2016 08:20:09 +0000 (17:20 +0900)]
[MM] Consolidate error handling routines of MediaPlayerEfl.
This patch is to consolidate the error handling routines
for ME/MSE implementations.
Bug: http://10.113.136.204/jira/browse/TSAM-806
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15564
Reviewed by: k2.nagaraju, msu.koo
Change-Id: If0b2fe034b5a0055233d938421ddb0626715aa9a
Signed-off-by: msu.koo <msu.koo@samsung.com>
Antonio Gomes [Fri, 15 Jan 2016 15:41:49 +0000 (11:41 -0400)]
Hide disambiguation popup on HW BACK keypress.
SSIA.
Original beta/m47 patch: http://165.213.202.130/gerrit/#/c/103852/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15609
Reviewed by: sns.park
Change-Id: I3aa284fc1d400c8d6906794a00214ad5fe3c8605
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
SeungSeop Park [Tue, 19 Jan 2016 02:13:32 +0000 (11:13 +0900)]
Fix "libpng warning: iCCP: known incorrect sRGB profile"
There are 32 warnings emitted during the build process.
The cause of warning is that "libpng-1.6 is more stringent about checking
ICC profiles than previos versions"[1].
All png files in chromium-efl are converted by running following command:
$ find . -type f -name "*.png" -exec convert {} -strip {} \;
[1] http://stackoverflow.com/questions/
22745076/libpng-warning-iccp-known-incorrect-srgb-profile
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15534
Reviewed by: sns.park
Change-Id: Ic17441d15648d8fa776646e70d934b8bde0fd430
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
msu.koo [Mon, 21 Dec 2015 11:01:41 +0000 (20:01 +0900)]
[MM] Fix random crashes and flickering while playing video.
Random crash and flickering happens because
- Lifecycle between BrowserMediaPlayerManagerEfl(BMPMEfl) and
MediaPlayerEfls(MPEfl) is not properly managed.
- Lifecycle of delivered media_packet is not properly managed so that
Renderer occasionally accesses media_packet which was already destroyed.
Bundle of patches fix theses issues by
- Introducing weak pointer when posting tasks not to make task outlive
than tasker.
- Making MPEfl's lifecycle dependent with BMPMEfl.
- Introducing IPC for destroying media_packet explicitly after Renderer
uses it.
- Introducing scoped_ptr for media_packet not to make leaks even task
is cancelled.
In detail, this patch consists of following beta/m47 patches:
1. http://165.213.202.130/gerrit/#/c/100528/ by msu.koo
Previously, crash happens on refresh while playing videos using HTML's
iframe tag, because |OnNewTbmBufferAvailable| is queued after RenderFrame
is deleted, so it refers already-deleted RenderFrame.
This patch includes 2 major changes.
- On Renderer-side,
This separates OnNewTbmBufferAvailable() from RenderFrame's lifecycle.
- On Browser-side,
This introduces BrowserMediaPacketManager which is Control IPC handler
outlives than BrowsermediaPlayerManager. This class can handle releasing
|media_packet| without affected by Player's lifecycle.
2. http://165.213.202.130/gerrit/#/c/100838/ by sangdeug.kim
This patch fixes crash during run |MediaPlayerEfl::DeliverMediaPacket| by
checking the destructing state.
3. http://165.213.202.130/gerrit/#/c/100839/ by msu.koo
Previously, random crash happens because player accesses manager
without considering the manager's lifecycle.
Currently, BrowserMediaPlayerManager request to delete Players
when it's deconstructed, which means that Player can outlive
than Manager.
This fix introduces WeakPtr and let Player access Manager
via WeakPtr for lifetime handling between Player and Manager.
4. http://165.213.202.130/gerrit/#/c/102313/ by msu.koo
"Improve MediaPlayerEfl lifecycle management."
Previously, MediaPlayerEfl outlives than BrowserMediaPlayerManagerEfl
so additional checking if Manager is alive before callback.
This patch make MediaPlayerEfl's lifetime dependant with BMPMEfl
and remove above checking routines.
Also remove IsPlayerDestructing and relates, which is no more required
after introducing WeakPtr for PostTask.
5. http://165.213.202.130/gerrit/#/c/102463/ by msu.koo
"Fix possible |media_packet| leaks when DeliverMediaPacket is cancelled."
Previously, DeliverMediaPacket task can be cancelled if queued after
MediaPlayerEfl is destroyed. But when DeliverMediaPacket is cancelled,
there is no handling to destroy the passed |media_packet| on it, so it
causes |media_packet| leaks.
This patch introduces MediaPacketProxy and scoped_ptr to fix it.
6. http://165.213.202.130/gerrit/#/c/103502/ by sangdeug.kim
After making MediaPlayerEfl's lifetime dependent with
BrowserMediaPlayerManager(BMPM), crash happens while destructing player.
Root cause is
- WeakPtr of BMPM is invalidated before Players are destroyed
by the destruction sequence of member variables.
- While destructing Players, Player accesses BMPM
via already-invalidated WeakPtr -> Crash happens.
This patch removes redundant WeakPtr of BMPM and related codes,
and let Player access Manager with raw pointer.
This patch also ensure destroy all players before BMPM is destroyed.
7. http://165.213.202.130/gerrit/#/c/103039/ by msu.koo
Previous implementation implements Proxy Class for media_packet,
but it can be replaced with Chromium's scoped_ptr.
Together with: Iecc266fee83ae009285554dce4581d6c7e9bf545
Bug: http://10.113.136.204/jira/browse/TSAM-751
Bug: http://10.113.136.204/jira/browse/TSAM-836
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15296
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15410
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15425
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15439
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15445
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15458
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15502
Reviewed by: msu.koo, sm.venugopal, sns.park
Change-Id: I59747d19e18cdcb8708d6ba91c49bc2eb7fad66b
Signed-off-by: msu.koo <msu.koo@samsung.com>
Signed-off-by: sangdeug.kim <sangdeug.kim@samsung.com>
y0.kim [Thu, 14 Jan 2016 04:35:35 +0000 (13:35 +0900)]
fixup! [M44_2403] Chromium upversion to m44_2403 branch
Fix compilation warning in ewk_geolocation_private.cc
> '_Ewk_Geolocation_Permission_Request::callback_' will be initialized after [-Wreorder]
> '_Ewk_Security_Origin* _Ewk_Geolocation_Permission_Request::origin_' [-Wreorder]
> when initialized here [-Wreorder]
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15496
Reviewed by: sns.park
Change-Id: Ied47e7e95a1aa13e9926bd6f907ee9c762dea685
Signed-off-by: y0.kim <y0.kim@samsung.com>
sm.venugopal [Wed, 13 Jan 2016 09:31:15 +0000 (15:01 +0530)]
[MM] Fix compiler warnings in tizen_video_decode_accelerator
|release| was used to giveup the object held by scoped pointer. Unused
return from |release| is passed to |ignore_result| to remove warning.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15533
Reviewed by: msu.koo, sns.park
Change-Id: I155c25784511d5ad77a72fc1966bc68c1d4ecf63
Signed-off-by: sm.venugopal <sm.venugopal@samsung.com>
SeungSeop Park [Mon, 18 Jan 2016 08:53:37 +0000 (17:53 +0900)]
Replace '#warning' directive with '#pragma message'
If turn on '-Werror' option, '#warning' will be treated as an error,
which however cannot be suppressed even with "-Wno-error=cpp" option.
As a workaround, patch instead uses "#pragma message" which is information
only and is neither a warning nor an error.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15612
Reviewed by: sns.park
Change-Id: Icfbe90bcd9d9817307704189755d413a808fcc22
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
SeungSeop Park [Fri, 15 Jan 2016 04:13:49 +0000 (13:13 +0900)]
Ignore warnings in clipboard_helper_efl.cc
There are several warnings of -Wunused-variable and -Wunused-function.
These are originated from temporary code under defined(WAYLAND_BRINGUP) guard.
Rather than fixing the temporary code, patch just ignores the warnings.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15535
Reviewed by: sns.park
Change-Id: Ief83a2628d119154cd4f2b81adadc00636811987
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
SeungSeop Park [Mon, 18 Jan 2016 10:15:23 +0000 (19:15 +0900)]
Make PermissionPopup's destructor as virtual.
There are warnings in permission_popup_manager.cc:
> ... deleting object of abstract class type 'PermissionPopup' which has \
> non-virtual destructor will cause undefined behaviour \
> [-Werror=delete-non-virtual-dtor]
Since PermissionPopup class is used as base class of several classes, e.g,
- GeolocationPermissionPopup,
- NotificationPermissionPopup,
- NotificationPermissionPopup,
its destructor needs to be virtual.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15621
Reviewed by: sns.park
Change-Id: I634825ac2a8b2c3b5a9164569242fb09d67d96b7
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
j-hyun.lee [Fri, 15 Jan 2016 05:25:00 +0000 (14:25 +0900)]
fixup! [M47_2526] Chromium upversion to m47_2526 branch
Fix compiler warning in plugin_placeholder_efl.cc
blink::WebPlugin* PluginPlaceholderEfl::CreatePlugin() was not returning
anything. Set return value as nullptr.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15544
Reviewed by: sns.park
Change-Id: Ic8c9e5570686a3f8f6549fdbd3e07b24460beb93
Signed-off-by: j-hyun.lee <j-hyun.lee@samsung.com>
Antonio Gomes [Wed, 16 Dec 2015 19:35:02 +0000 (15:35 -0400)]
Disambiguation popup should send tap gestures not mouse events
When disambiguation popup is shown, and user clicks on its
content, RWHVEfl synthesizes and sends mouse down/up events
to the renderer.
Mouse events are different from a regular touch input, and various
touch-oriented features are not triggered:
- zoom to focused editable node;
- insert large handle on tap.
Patch switches from the disambiguation popup handling logic
from synthesized mouse events to tap gestures.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15359
Reviewed by: a1.gomes, g.czajkowski
Change-Id: I291e107634c4cc56c8512705bcf05ecbcfd81f19
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Fri, 8 Jan 2016 17:19:20 +0000 (13:19 -0400)]
Fix ewk_cookie_manager_persistent_storage_set when called before a WebView
Patch indirectly creates requestcontext, resourceContextEfl
and cookiemanager instances as part of EWebContext ctor.
These objects end up getting created as part of the
EWebView::InitializeContents routine anyways. The reason on why it
is manually triggered here is because there are ewk_cookie_manager
APIs that can be called before a webview instance is created. In such
circumstances, APIs fail to execute, including
ewk_cookie_manager_persistent_storage_set.
Also, patch sets file:// as a cookieable protocol
for WRT purposes.
Original beta/m47 patch: http://165.213.202.130/gerrit/#/c/102798/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15514
Reviewed by: djmix.kim
Change-Id: Ieffcbf1058237e032486aa3c715dbee1282304c9
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Mon, 4 Jan 2016 20:31:54 +0000 (16:31 -0400)]
Templatize EWebView callback classes.
Patch templatizes the following classes in order to remove duplication:
- EwkViewPlainTextGetCallback
- MHTMLCallbackDetails
- QuotaPermissionRequestCallback
- WebViewGeolocationPermissionCallback
- WebViewUserMediaPermissionCallback
- NotificationPermissionCallback
- WebViewUnfocusAllowCallback
They all share the same logic.
Original beta/m47 patches:
- http://165.213.202.130/gerrit/#/c/102267/
- http://165.213.202.130/gerrit/#/c/102906/ (by Grzegorz Czajkowski)
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15494
Reviewed by: djmix.kim, g.czajkowski
Change-Id: I6b8e97b05bed2fe30c4bfe236e9a48748108945c
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Tue, 15 Dec 2015 18:59:25 +0000 (14:59 -0400)]
Decouple context menu from selection controller class.
Chromium-efl enters selection mode by a long press gesture.
Such event is only triggered when touch events are enabled.
The upstream counterpart of our SelectionControllerEfl
class is TouchSelectionController (see
ui/touch_selection/touch_selection_controller.cc).
As the name implies, its logic is also tied to touch events.
Patch decouples our context menu logic from the
SelectionControllerEfl class, tying it up to touch
event status. This allows context menus to be shown on
TV and desktop builds.
Original beta/m47 patches:
- http://165.213.202.130/gerrit/#/c/99552/
- http://165.213.202.130/gerrit/#/c/100348/
- http://165.213.202.130/gerrit/#/c/100324/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15220
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15393
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15414
Reviewed by: g.czajkowski
Change-Id: I949afb612d3fb0806d95c865d34d51101014734b
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Thu, 7 Jan 2016 19:42:35 +0000 (15:42 -0400)]
Improve ewk_cookie_manager_persistent_storage_set signature
Patch replaces 'bool' parameters part of the call chain of
ewk_cookie_manager_persistent_storage_set.
There is an existing EWK enum we can use instead.
Original beta/m47 patch: http://165.213.202.130/gerrit/#/c/102797/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15514
Reviewed by: djmix.kim, g.czajkowski
Change-Id: I5320a523231392338bf894554ded4a240c7ee3fa
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Wed, 30 Dec 2015 17:45:54 +0000 (13:45 -0400)]
Enforce proper ewk_view creation in response to "create,window".
When a new webview is created in response to a request from the
engine, the BrowserContext instance of the originating WebContents
is shared with the newly created WebContents object.
In other words, both the WebContents instance that triggered
the new window creation, and the WebContents instance derived
from that request, will share the same BrowserContext instance.
This is an implementation detail that can be seen at
WebContentsImplEfl::HandleNewWebContentsCreate.
Hence, if as part of the WebView creation, the embedding APP
passes in a Ewk_Context instance that wraps a different BrowserContext
instance than the one the originating WebContents holds,
undefined behavior can be seen, including hard crashes.
Below is a snippet code that illustrates the scenario:
(..)
evas_object_smart_callback_add(web_view_, "create,window",
&OnNewWindowRequest, this);
(..)
void OnNewWindowRequest(void *data, Evas_Object*, void* out_view) {
(..)
EvasObject* new_web_view = ewk_view_add_with_context(GetEvas(), ewk_context_new());
*static_cast<Evas_Object**>(out_view) = new_web_view;
(..)
}
The new Ewk_Context object created and passed in as parameter to
ewk_view_add_with_context wraps a different instance of BrowserContext
than the one the new WebContents object will actually hold.
In order to prevent that, patch documents the proper usage of this API,
and adds a CHECK clause aiming at catching any misuse of it.
Original beta/m47 patch: http://165.213.202.130/gerrit/#/c/101739/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15459
Reviewed by: djmix.kim, g.czajkowski
Change-Id: I51260370eef965c7f1c1d63cde82c82fadc8fbe1
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Wed, 30 Dec 2015 14:28:05 +0000 (10:28 -0400)]
Remove leftover code from EWebView
Patch removes internal classes/methods no longer needed,
namely:
- EWebView::SelectClosestWord
- AsyncRequestHitTestPrivate (declaration only)
- class WebViewAsyncRequestHitTestDataInternalCallback
Original beta/m47 patches:
- http://165.213.202.130/gerrit/#/c/102076/
- http://165.213.202.130/gerrit/#/c/102075/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15459
Reviewed by: djmix.kim, g.czajkowski
Change-Id: Icf84b4f921ecd943f01e747b0de9e3e05c7ca5b0
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Tue, 29 Dec 2015 15:21:33 +0000 (11:21 -0400)]
Defer implementation of HTTPCustomHeaderXXX methods to EWebContext.
Ewk_Context struct should be a proxy to EWebContext.
It works this way for most of its methods but not all.
Patch fixes this.
Additionally, some char* signatures are converted to string&.
Original beta/m47 patch: http://165.213.202.130/gerrit/#/c/101544/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15459
Reviewed by: djmix.kim, g.czajkowski
Change-Id: I9719cd5ffccb8659b779fd4a99f41d33c32c0f1f
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Mon, 28 Dec 2015 22:30:24 +0000 (18:30 -0400)]
Crash @CookieManager::DeleteSessionCookiesOnIOThread
The way Tizen browser is creating WebView instances
is through ewk_view_add_with_context(bleh, ewk_context_new()).
In other words, it passes in a new context instance for
every view created.
However, given the way chromium-efl (specially EWK layer)
is implemented, it is possible that the instance of EWebContext
passed in is not going to be used by the WebContents object
associated to the receiver WebView.
Fixing this might require a larger rework of some of these assumptions.
Patch emergency basis fixes a related crash due to a null pointer
being deferenced. The same pointer is null-checked in all other
occurrences of this class.
Original beta/m47 patch: http://165.213.202.130/gerrit/#/c/101375/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15459
Reviewed by: djmix.kim, g.czajkowski
Change-Id: I72ad30ff660eddbb698339f496142553b5538d8a
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Fri, 18 Dec 2015 19:49:56 +0000 (15:49 -0400)]
Long press on links does not trigger context menu in some circumstances
In the current SelectionControllEfl, In order to confirm if a long
press gesture is ongoing, one just needed to check "long_mouse_press_".
However, when long press happens on a link or image, we set
long_mouse_press_ to false in ::HandleLongPressEvent, although
finger is still down. This is important in the current selection
controller logic.
To compensate this, patch adds an extra check asking from the
engine if a finger is still touching down.
Original beta/m47 patch: http://165.213.202.130/gerrit/#/c/100309/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15407
Reviewed by: djmix.kim, g.czajkowski
Change-Id: I19b664907d79ec4d18e782396e22b3c6c335d2b5
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
msu.koo [Mon, 11 Jan 2016 05:34:02 +0000 (14:34 +0900)]
[MM] Introduce a switch for zero-copy support from multimedia
This patch is to introduce a switch to support zero-copy
when delivering video frames. This switch can be used to test
multimedia features without affected by renderer.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14015
Reviewed by: sns.park
Change-Id: I3d7232c6338b2fd789eba29765ddd3285bd89fe7
Signed-off-by: msu.koo <msu.koo@samsung.com>
SeungSeop Park [Mon, 4 Jan 2016 05:46:28 +0000 (14:46 +0900)]
[MM] Remove dependency to ffmpeg library in mobile/tv build
Patch set |media_use_ffmpeg| to 0 if |building_for_tizen| is 1.
If media_use_ffmpeg=0, third_party/ffmpeg would be excluded from the build.
Hence, the ffmpeg patch under build/patches/ can be also removed.
Together with: I107068ac5ebfbf2c4da4c7e923747a8db9f0e57b
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14620
Reviewed by: a1.gomes, sm.venugopal, sns.park
Change-Id: Iecbb6086a3b1b7de869ac8c7c26b4344b2b82f1f
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
SeungSeop Park [Mon, 4 Jan 2016 04:36:55 +0000 (13:36 +0900)]
[MM] Disable speech recognition and remove dependency to flac library
Tizen does not require speech recognition feature.
And it's desirable to minimize third-party library dependency to avoid
unnecessary maintenance overhead.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15477
Reviewed by: a1.gomes, sm.venugopal, sns.park
Change-Id: I6b867dac31734eb92425fada12cf99d445028dc1
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
sm.venugopal [Tue, 29 Dec 2015 05:37:23 +0000 (11:07 +0530)]
[MM] Remove unused gstreamer port.
1) CAPI support is added for ME / MSE / WebAudio. So, removed
gstreamer port. On desktop, chromium's pipeline will be used.
2) Tizen MM flag is enabled only for Tizen3.0.
3) Tizen MM files are compiled based on "tizen_multimedia_support"
flag.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15468
Reviewed by: a1.gomes, sns.park
Change-Id: I1a397265f920c74c953b83c46bfe8f89eeae6dfc
Signed-off-by: sm.venugopal <sm.venugopal@samsung.com>
sangdeug.kim [Tue, 22 Dec 2015 06:52:36 +0000 (15:52 +0900)]
fixup! [MM] Handling multiple instances of media players.
Patch removes workaround code which was added to avoid |player_unprepare|
error due to platform issue.
* beta/m47 cherry-pick: http://165.213.202.130/gerrit/#/c/100693
On SetVolume call, after setting the player volume, the volume was not getting
stored in MediaPlayerBridgeCAPI. Because of this the initial volume value (0,
which gets stored during CreatePlayer), was being set and audio playback was
not happening unless we set the volume again. Updated the code to store volume
in MediaPlayerBridgeCAPI whenever SetVolume call is made.
* beta/m47 cherry-pick: http://165.213.202.130/gerrit/#/c/102232
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14833
Reviewed by: k2.nagaraju, sm.venugopal, sns.park
Change-Id: Ifc322aa1ad19080d1cd62b9c061e8324df7f7b76
Signed-off-by: sangdeug.kim <sangdeug.kim@samsung.com>
Signed-off-by: mck.giri <mck.giri@samsung.com>
y0.kim [Tue, 5 Jan 2016 08:10:56 +0000 (17:10 +0900)]
fixup! [MM] Webaudio with CAPI port.
Fix warning 'media::AudioDecoderCapi::pcm_output_' will be initialized after
'player_s* media::AudioDecoderCapi::player_' [-Wreorder]
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15496
Reviewed by: sns.park
Change-Id: If7fc3cd01e6374f8d65546d5039fe2b316dea3c5
Signed-off-by: y0.kim <y0.kim@samsung.com>
msu.koo [Wed, 16 Dec 2015 05:43:15 +0000 (14:43 +0900)]
fixup! [MM] Fix flickering when playing video.
Previously, indexing is required to find the proper |media_packet|
from |tbm_surface| by manager.
But from the previous patch, it has been guaranteed that every |tbm_surface|
will be released, so managing |tbm_surface| itself is just a overhead.
This fixup removes manager,
and let BrowserMediaPlayer directly release the matched |media_packet|
by also passing |media_packet_h| via TbmBufferHandle structure.
Reviewed by: sm.venugopal, sns.park
Change-Id: I0cd888b45ed9a4a64c4ba901c5d2e008de6d4f9a
Signed-off-by: msu.koo <msu.koo@samsung.com>
SeungSeop Park [Fri, 8 Jan 2016 06:35:59 +0000 (15:35 +0900)]
Fix compilation warnings in libvpx (-Wunused-function)
Currently, there are 27 warnings saying:
> ../../third_party/libvpx_new/source/libvpx/vp9/encoder/vp9_denoiser.h:63:12:\
> warning: 'total_adj_strong_thresh' defined but not used [-Wunused-function]
It's because |total_adj_strong_thresh| is defined as static function in .h file.
So warning will appear in any file that includes the header but does not use
the function.
The suggested solution is to declare the static function as inline.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15496
Reviewed by: sns.park
Change-Id: Ic2173c0c3560256df1319d4344988618c71ec9a2
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
msu.koo [Tue, 22 Dec 2015 04:57:39 +0000 (13:57 +0900)]
fixup! [MM] Consolidated routines for video frame delivering.
This fixup removes redundant member variables from previous patch.
Reviewed by: sm.venugopal, sns.park
Change-Id: I1eacf74c73444e46fda16b62adf462aa83d9e952
Signed-off-by: msu.koo <msu.koo@samsung.com>
Youngcheol Kang [Tue, 5 Jan 2016 07:20:48 +0000 (16:20 +0900)]
Removed the use_system_minizip flag
The gyp flag |use_system_minizip| no longer exists in upstream
chromium, hence can be safely removed.
As result, we don't need |use_system_minizip| flag any more.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15486
Reviewed by: a1.gomes, sns.park
Change-Id: Iaffefe5006d883d523c0b682db20f3a646f0b093
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
Youngcheol Kang [Mon, 4 Jan 2016 07:12:07 +0000 (16:12 +0900)]
Removed the speex dependency
The gyp flag |use_system_speex| no longer exists in upstream chromium,
hence can be safely removed.
As result, we don't need speex package any more.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15479
Reviewed by: a1.gomes, sns.park
Change-Id: I2f5ae8528a1bd263dbc23f25ad2fbfd7ae89fd2b
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
Antonio Gomes [Thu, 17 Dec 2015 15:30:12 +0000 (07:30 -0800)]
Use system icu in Tizen builds.
After coordinating with Platform team, icu version was
bumped from 51 to 54, which is the base version of icu
bundled with chromium (third_party/icu).
It turns out that icu-54 is not enough to build chromium
because it lacks TimeZone::detectHostTimeZone API used in
content/browser/time_zone_monitor.cc.
After some more investigation, it was figured out that icu bundled
with chromium is manually patched (see third_party/icu/patches).
It was request to the platform team to apply two patches then
as part of package build:
- third_party/icu/patches/tzdetect{2}.patch
It is now possible to build chromium m47 with system icu.
This saves about 1.5Mb in libchromium-efl.so size (stripped).
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12893
Reviewed by: sns.park
Change-Id: I90ce970b444c7865d7769a219949a699deb9b893
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Tomasz Czekala [Mon, 28 Dec 2015 11:00:23 +0000 (12:00 +0100)]
Revert "Filter out events jittery for single-touch gestures."
This reverts commit
f7f8b960f56cbd66560b7590e24983beb9e63509.
The reverted commit was removing the events which are required for
smoothness. The gesture_detector.cc module handles extra events
received. We just need to pass them to it.
This is a port of the following patch:
http://165.213.202.130/gerrit/#/c/100452/
Reviewed by: a1.gomes, djmix.kim, prashant.n
Change-Id: Iadc6aa67c7178f3b600c2fcf0aa6c7b0907b7a33
Signed-off-by: Tomasz Czekala <t.czekala@samsung.com>
Tomasz Czekala [Mon, 28 Dec 2015 08:50:31 +0000 (09:50 +0100)]
Fix wrong computation of scroll velocity.
Currently timestamp for touch event (ui::TouchEvent) is using
function ecore_time_get() which gives time in seconds. This makes
difficult to distinguish touch events occurring in sequence as time
comes almost same in seconds. This further breaks gesture event
handing as timestamp resolution is too low.
This patch adds current time in microseconds which is high enough
to distinguish two consecutive touch events distinctly.
This improves touch smoothness in all gestures quite a lot.
This patch sends the timestamp of event actually occurred in platform.
When browser process is busy, system may not be able to deliver the
events in time, so we may get events delayed. Actual events occurred
in platform are equi-spaced in time.
This solves the incorrect fling occurring in small scrolls due to
wrong velocity computation. This also solves now the smoothness with
all gestures and removes jitters from fling events.
This is a port of the following patches:
http://165.213.202.130/gerrit/#/c/100329/
http://165.213.202.130/gerrit/#/c/100453/
Reviewed by: a1.gomes, djmix.kim, prashant.n
Change-Id: I6669f167deb61dc004856985785daf62b39b8f61
Signed-off-by: Tomasz Czekala <t.czekala@samsung.com>
Tomasz Czekala [Thu, 24 Dec 2015 11:17:12 +0000 (12:17 +0100)]
Revert "Change velocity tracker strategy to INT1"
This reverts commit
603fa30d109aa0a36dcf8bfb334e44c69f545dc8.
Reviewed by: a1.gomes, djmix.kim, g.czajkowski, prashant.n
Change-Id: If90df856273a089466b0ff16329e512734c40361
Signed-off-by: Tomasz Czekala <t.czekala@samsung.com>
Youngcheol Kang [Mon, 28 Dec 2015 11:33:21 +0000 (20:33 +0900)]
Updated the gbs.conf by seperating base and main repositories
The snapshot was divided into base and main repositories.
By the request of platform team, the gbs.conf should be updated.
(http://165.213.149.170/confluence/pages/viewpage.action?pageId=
45394582)
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15463
Change-Id: I290f707942de2348001959be8ea0f84726da6f24
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
msu.koo [Fri, 18 Dec 2015 01:38:28 +0000 (10:38 +0900)]
[MM] Remove dead codes: TIZEN_VIDEO_HOLE
TIZEN_VIDEO_HOLE was used on Tizen 2.4 for HAWK-P,
but it's not used anymore on Tizen 3.0
Bug: http://165.213.149.170/jira/browse/TNEF-6526
Reviewed by: sns.park
Change-Id: Ie14b530195f58b590da6c9a5f2f0db1aad0fc73e
Signed-off-by: msu.koo <msu.koo@samsung.com>
msu.koo [Fri, 18 Dec 2015 01:29:23 +0000 (10:29 +0900)]
[MM] Remove dead codes: USE_LEGACY_PLAYER_CAPI
USE_LEGACY_PLAYER_CAPI was used for tizen 2.4 on HAWK-P
but it's not used anymore on Tizen 3.0
Bug: http://165.213.149.170/jira/browse/TNEF-6526
This patch includes patchs from beta branch:
8afad0c [MM] Remove dead codes: USE_LEGACY_PLAYER_CAPI
3b13a6e fixup! [MM] Remove dead codes: USE_LEGACY_PLAYER_CAPI
Reviewed by: sns.park
Change-Id: I1b30dadfc428051e6816beb3c250e6af16db86bd
Signed-off-by: msu.koo <msu.koo@samsung.com>
karthick.m1 [Wed, 16 Dec 2015 16:23:49 +0000 (21:53 +0530)]
[MM][MSE] Fix crash when clicking another clip while video playing in youtube
Crash happens, because media packet comes after player destroyed.
This patch introduces WeaKPointer not to make task outlive player.
Reference: http://165.213.202.130/gerrit/94565/
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14678
Reviewed by: sm.venugopal, sns.park
Change-Id: I9c666dcf2bc2bb44a601e15a5395548b695701cf
Signed-off-by: karthick.m1 <karthick.m1@samsung.com>
Aron Kim [Wed, 16 Dec 2015 02:40:14 +0000 (11:40 +0900)]
Add gles20 package dependency
After DDK team's asked to apply this change in order to avoid breakage
due to replacing mesa with coregl.
Reviewed by: Antonio Gomes, DaeHyun Ko, venu musham
Change-Id: Ia1e2ee62251bcaaee5a908ed32114bf725af82e1
Signed-off-by: Aron Kim <aron.kim@samsung.com>
puru [Wed, 16 Dec 2015 11:53:07 +0000 (17:23 +0530)]
[MM][MSE] EOS Handling and multiple clip playback is not working
When Video underflow occurs Pause() is called and meanwhile if EOS for Video
is pushed, player could not be put to play state without manual intervention.
So added check for audio and video EOS.
Maxium player instances for TV is 1 and mobile is 5. In case of TV when you
play one MSE clip and press second clips to play, black screen comes and
browser hangs. Same behaviour is noticed in mobile after playing 5 clips.
So after destroying the player |OnSuspendCompleted| should be called,
otherwise browser media player manager considers player still exists,
so the playback does not starts.
Bug: http://165.213.149.170/jira/browse/TSAM-548
Reviewed by: sm.venugopal, sns.park
Change-Id: I971f2463b3e0d3add79db86f64c4c0304f27b56a
Signed-off-by: puru <puru.dharma@samsung.com>
SeungSeop Park [Mon, 21 Dec 2015 05:54:29 +0000 (14:54 +0900)]
[MM] Up-version to M48 and re-enable Tizen MM support
Update MM code as per upstream changes.
- VideoCaptureDevice::Client::OnError() has an additional argument.
Upstream CL: https://codereview.chromium.org/
1418263006
- Include path changed for skcanvas_video_renderer.h
- Method renamed: media::SetDecryptorReadyCB => media::SetCdmReadyCB
- Header renamed: media/base/decryptor.h => media/base/cdm_context.h
- Method removed: media::{Audio|Video}DecoderConfig::extra_data_size().
=> Cherry-picked beta/m47 patch: http://165.213.202.130/gerrit/#/c/96258
- Argument changes for WebmediaPlayer::setSinkId()
- Replaced scoped_refptr<media::GpuVideoAcceleratorFactories>&
-> media::GpuVideoAcceleratorFactories*
as per Upstream CL: https://codereview.chromium.org/
1369843002
- Removed duplicated definition IPC::ParamTraits<media::SubsampleEntry>
from chromium-efl. It's moved to content/common/gpu/gpu_messages.h (M48)
from content/common/media/media_player_messages_android.h (M47).
Finally, patch reverts the temporary codes made by M48 bringup patch
(http://165.213.202.130/gerrit/#/c/99857/):
- Turn on |tizen_multimedia_support| flag again
- Remove a temp patch file.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15381
Reviewed by: msu.koo, sm.venugopal, sns.park
Change-Id: I78be30dc896724ca1c45d5fb7122f3ef850b25be
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
Sanghyup Lee [Mon, 7 Dec 2015 22:28:29 +0000 (07:28 +0900)]
Default behavior for HW back key should be implemented in web engine.
Currently browser and WRT gets HWBackKey and use it to:
1. If it is full screen mode, exit it.
2. If there is text selection, clear it.
3. WRT: execute script to send HWBackKey to web app.
Browser: Back navigation.
Currently it is implemented in applications.
This patch moves common implements from application to web engine.
|WebViewEvasEventHandler::HandleEvent_HWBack| is added to handle HWBack event.
On mobile this function is called by eext package, on the other profile it is
called by EVAS_CALLBACK_KEY_UP.
"hwback,unhandled" signal is added which is called if event isn't handled in engine.
The next patch will fix ubrowser back navigation with this signal.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15225
Reviewed by: a1.gomes, djmix.kim, g.czajkowski
Change-Id: I87beaf8c3c8a79b883c0de658077df98504c8b38
Signed-off-by: Sanghyup Lee <sh53.lee@samsung.com>
Grzegorz Czajkowski [Mon, 14 Dec 2015 15:29:57 +0000 (16:29 +0100)]
Do not show selection ctx when selection range is offscreen
Currently, regardless of selection visibilty (start, end) we show selection ctx.
This patch changes ShowHandleAndContextMenuIfRequired() method in a way that
at least one handler visibilty (selection start or end) is required to show
ctx menu.
This fixes WCS TC 10.
Inspired by http://165.213.202.130/gerrit/#/c/98864/.
Additionally, remove ::IsAnyHandleOnScreen because with composited updates the
text selection contains such information.
Original beta/m47 patch:
- http://165.213.202.130/gerrit/#/c/99275/
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12055
Reviewed by: a1.gomes, djmix.kim, g.czajkowski
Change-Id: I56d2ce545aeecec8f4e4bf9a5be22180eb98596c
Signed-off-by: Grzegorz Czajkowski <g.czajkowski@samsung.com>
Sanghyup Lee [Wed, 16 Dec 2015 04:47:23 +0000 (13:47 +0900)]
Fix popup size for input pickers.
Current popup covers only webview, so browser buttons are available
even if input picker is shown.
This patch changes popup size from webview size to top widget size to
prevent browser buttons clicked if input picker is shown.
Original beta/m47 patch: http://165.213.202.130/gerrit/#/c/99608/
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15150
Reviewed by: a1.gomes, djmix.kim, g.czajkowski
Change-Id: I6ce011a400fe7248d1e2ec2d5e829606448b9c9f
Signed-off-by: Sanghyup Lee <sh53.lee@samsung.com>
Sanghyup Lee [Wed, 16 Dec 2015 01:46:59 +0000 (10:46 +0900)]
Fix page visibility when user changes browser tab.
When we change browser tab, page visibility should be sent to renderer.
EWebView::HandleShow|Hide is called by Evas_Smart_Class callbacks.
This patch add calling EWebView::Show|Hide in these functions.
Original beta/m47 patch:
- http://165.213.202.130/gerrit/#/c/99575/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15074
Reviewed by: a1.gomes, djmix.kim
Change-Id: Iab5fe40dc67551323a8e182f5606cd5baf731455
Signed-off-by: Sanghyup Lee <sh53.lee@samsung.com>
Youngcheol Kang [Thu, 17 Dec 2015 00:40:12 +0000 (09:40 +0900)]
[M48_2564] Chromium upversion to m48_2564 branch
This patch deals with following upstream changes:
1. Folder and class name changes
- Changed the name of the inertial_sensor_consts.h to the
device_sensors_consts.h in content/browser/device_sensors/
- Changed the namespace of the GLImageEflPixmap to the gl from the gfx
- Changed the path of the strong_binding.h to the mojo/public/cpp
/bindings from third_party/mojo/src/mojo/public/cpp/bindings
- Removed the third_party/WebKit/public/web/WebNodeList.h
- Removed the wtf/FastMalloc.h
2. API changes
- Added the ConfirmSaveCreditCardLocally, ConfirmSaveCreditCardToCloud,
LoadRiskData API in AutofillClient class
- Added the GetBackgroundSyncController API in BrowserContext class
- Added the LockCompositingSurface, UnlockCompositingSurface API in
RenderWidgetHostViewBase class
- Added the RequestPermissions API in PermissionManager class
- Changed the argument of CancelPermissionRequest, RequestPermission API
in PermissionManager class
- Changed the argument of CopyFromCompositingSurfaceHasResultForVideo,
CopyFromCompositingSurfaceFinishedForVideo, CopyFromCompositingSurface
FinishedForVideo, CopyFromCompositingSurfaceToVideoFrame API
in DelegatedFrameHostClient class
- Changed the argument of CreateNewWindow API in RenderViewHostImpl
class
- Changed the argument of GetNavigationErrorStrings API in
ContentRendererClient class
- Changed the argument of IsFullscreenForCurrentTab API in
RenderWidgetHostDelegate class
- Changed the argument of Init API in PersonalDataManager class
- Changed the argument of net::HttpCache class
- Changed the argument of OnUnmaskVerificationResult API in
AutofillClient class
- Changed the argument of ReturnSubscriberTexture API in
DelegatedFrameHostClient class
- Changed the argument of Run API in Callback class
- Changed the argument of setPageScaleFactor API in WebView class
- Changed the argument of ScalePoint API in gfx::PointF class
- Changed the argument of ShouldCreateWebContents API in
WebContentsDelegate class
- Changed the argument of ViewHostMsg_StartContentIntent IPC message in
view_messages.h
- Changed the name of EnableRecording API to the StartTracing in
TracingController class
- Changed the name of DisableRecording API to the StopTracing in
TracingController class
- Changed the type of GetPasswordManager API in PasswordManagerClient
class
- Changed the sync_point(uint32) variable to the
sync_token(gpu::SyncToken) in gpu namespace
- Removed the ConfirmSaveCreditCard API in AutofillClient class
- Removed the WillUseTexImage, DidUseTexImage, WillModifyTexImage,
DidModifyTexImage API in GLImage class
3. Need to change API (TODO issues)
- The multimedia function has been changed. Temporary disabling the
'tizen_multimedia_support' flag
(http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15381)
- The autofill support function has been changed. Temporary disabling
the 'tizen_autofill_support' flag
(http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15382)
- The gold of system had a build problem. Temporary using the gold of
prebuilt in chromium-efl instead of the system
(http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15383)
- The CreateViewForWidget API in WebContentsView class has been changed.
Temporary disabling the codes
(http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15384)
- The ExecuteEditCommand API in RenderWidgetHostImpl class has been
changed. Temporary disabling the codes
(http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15385)
- The GetNavigationErrorStrings API in ContentRendererClient class has
been changed. Temporary disabling the codes
(http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15386)
- The RequestPermission API in PermissionManager class has been changed.
Temporary disabling the codes
(http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15387)
- The argument of the ViewHostMsg_StartContentIntent IPC message has
been changed. Need to implement the codes
(http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15388)
- The sync_point variable was removed in GLFrameData class. Temporary
disabling the codes
(http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15389)
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, also the browser operates in
all target
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15310
Change-Id: I0de1cea6b793c8e542fa8359f642b21d7fcb6db5
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
msu.koo [Fri, 11 Dec 2015 07:03:28 +0000 (16:03 +0900)]
[MM] Consolidated routines for video frame delivering.
Previously, both MSE and ME take the same approach
for video frame delivering routines, but codes are duplicated and diverged.
This patch is to consolidate the routines into MediaPlayerEfl,
then let derives(MediaPlayerBridgeCapi, MediaSourcePlayerCapi) use it.
Bug: http://165.213.149.170/jira/browse/TNEF-6573
Reviewed by: sm.venugopal, sns.park
Change-Id: Ie7c18a38daa7d2e32f1ab1361c11286c04bd0faa
Signed-off-by: msu.koo <msu.koo@samsung.com>
Pawel Niemirski [Mon, 14 Dec 2015 16:41:11 +0000 (17:41 +0100)]
Consider webview position while deciding on context menu visibility
Currently, the position of webview is not considered while
deciding on context menu visibility.
This fixes it, and indirectly fixes WCS TC 21.
Original beta/m47 patch:
- http://165.213.202.130/gerrit/#/c/99334/
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15137
Reviewed by: a1.gomes
Change-Id: I70b0e911bcdf70ef92a16d62f2ab9fc54cbd0977
Signed-off-by: Pawel Niemirski <p.niemirski@samsung.com>
Grzegorz Czajkowski [Fri, 11 Dec 2015 08:44:36 +0000 (09:44 +0100)]
fixup! Reimplement range and page scale functionality
Since chromium-efl sets scale asynchronously after [1] ewk_view_scale_get is
likely to fail after immediately call to ewk_view_scale_set.
This patch adds this note to the doxygen documentation for ewk_view_scale_set.
Additionally, fix ewk_view_scale_set coding style.
[1] http://165.213.202.130/gerrit/95891
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=12080
Reviewed by: a1.gomes, sns.park
Change-Id: Ie00a64dc619d047f6c3f94c4afae24931ed82bae
Signed-off-by: Grzegorz Czajkowski <g.czajkowski@samsung.com>
Tomasz Czekala [Fri, 11 Dec 2015 13:51:42 +0000 (14:51 +0100)]
Fix for black screen in new window for web apps
Currently, we see black screen for the new window that is opened
using window.open() in web apps. physical_backing_size(view size
in pixels) was not sent in the resize message to the renderer.
The same is now sent in the resize message.
Original beta/m47 patch: http://165.213.202.130/gerrit/#/c/98852
Bug: http://165.213.149.170/jira/browse/TSAM-613
Reviewed by: a.renevier, djmix.kim
Change-Id: Iafcd2a64eed7e29af2e4b212b8601b5df4bfa442
Signed-off-by: Tomasz Czekala <t.czekala@samsung.com>
Chandan Padhi [Mon, 7 Dec 2015 13:01:32 +0000 (18:31 +0530)]
Fix for crash on webview resize for a new page opened in the same tab.
Currently, segmentation fault occurs on webview resize when we load a
new page on an already existing tab. In this scenario, a new RWHVEfl
is created first and then the previous RWHVEfl is destroyed. However,
the same WebContentsViewEfl(therefore the same native view) is used
for the new RWHVEfl as well.
Two resize callbacks(one for each RWHVEfl) are added to the same
native view in order of their creation.
In destructor of the previous RWHVEfl, evas_object_event_callback_del()
deletes the most recently added callback(the one for the new RWHVEfl).
When resize occurs for the new RWHVEfl, the callback for the previous
RWHVEfl(already deleted) is invoked that results in crash.
To fix this issue, we now use evas_object_event_callback_del_full()
instead of evas_object_event_callback_del().
evas_object_event_callback_del_full() deletes the callback for the
corresponding RWHVEfl only.
Bug: http://suprem.sec.samsung.net/jira/browse/CBEFL-806
Reviewed by: sm.venugopal, sns.park
Change-Id: I36e4538c12b1847bedd8263fe1d18e3815ee0e3d
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
msu.koo [Tue, 15 Dec 2015 04:27:29 +0000 (13:27 +0900)]
fixup! [MM] Fix flickering when playing video.
Previous patch relied mostly on CleanUp() for releasing
used tbm_surface. But it has issues like
- It causes that many number of tbm_surfaces are pended for releasing,
because compositor sometimes does not signal to release.
This causes locking on capi-player because capi-player has constraints
on output tbm_surface counts.
- CleanUp() is cost operation.
This fixup introduces Closure to bound the lifecycle of
tbm_surface with VideoFrame by following the Chromium's VideoFrame approach.
(So no more hack is required on GLImage for releasing tbm_surface.)
When Chromium invalidates the VideoFrame, tbm_surface will also be released.
For this approach, this fixup introduces IPC message for releasing tbm_surface,
so any process/thread can request to release tbm_surface.
(On previous approach, Browser-Process can only request releasing tbm_surface).
By taking this approach, it's guaranteed that every tbm_handles are
requested to be released, so CleanUp is removed.
Bug: http://165.213.149.170/jira/browse/TSAM-676
Bug: http://165.213.149.170/jira/browse/TSAM-662
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15255
Bug: http://suprem.sec.samsung.net/jira/browse/CBEFL-835
Reviewed by: msu.koo, sm.venugopal, sns.park
Change-Id: I9c622396402ec34e0aa77e28bff7bf3e937f83eb
Signed-off-by: msu.koo <msu.koo@samsung.com>
msu.koo [Wed, 9 Dec 2015 10:18:47 +0000 (19:18 +0900)]
[MM] Fix flickering when playing video.
Previously, Player deletes |tbm_surface| when next frame is delivered.
But occationally, it causes frame drops and crashs
because |tbm_surface| is released before the gpu does not finish to handle it.
This patch introduces |MediaPacketManager|
to hold media_packets which wraps tbm_surface and CleanUp regularly.
Also supplying APIs to let Renderer explicitly release the |tbm_surface|s
by indexing the |media_packet|s using |tbm_surface_h| as the key.
(Renderer only knows tbm_surface.)
CleanUp is introduced to prevent media_packet(tbm_surface) leaks
even Renderer does not request to release.
Theoritically CleanUp should do nothing because Renderer invoke
Release() for all |tbm_surface|s afer rendering so that
implementation is focused on optimizing Release() rather than CleanUp().
Bug: http://165.213.149.170/jira/browse/TSAM-653
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15255
Reviewed by: sns.park
Change-Id: I9a1e3b0710da0072b3459818690d16e5c43d6fd3
Signed-off-by: msu.koo <msu.koo@samsung.com>
sm.venugopal [Wed, 9 Dec 2015 07:50:09 +0000 (13:20 +0530)]
[MM][MSE]CAPI code refactor
Removed unused code. Re-arranged code to improve readablity.
Bug: http://165.213.149.170/jira/browse/TSAM-663
Reviewed by: msu.koo, sm.venugopal
Change-Id: I5d481488d5f9ba1398d2d5058b2866cdd5b80f1c
Signed-off-by: sm.venugopal <sm.venugopal@samsung.com>
Piotr Ganicz [Thu, 10 Dec 2015 14:44:03 +0000 (15:44 +0100)]
Refactoring code with creating OK button on popup.
The issue is that we has CreateOKButton() function in Layout class,
but in the code there is not any usage. In every function with
name started with the "show" word (e.g. showDatePopup) there is
created "OK" button the same way, without using special function
for it. It creates a redundancy of code.
Reviewed by: a1.gomes, g.czajkowski
Change-Id: Ic7a98c43de3dcdc4106187b9e2624d12d9f42cd9
Signed-off-by: Piotr Ganicz <p.ganicz@samsung.com>
Piotr Krysiewicz [Wed, 9 Dec 2015 13:19:17 +0000 (14:19 +0100)]
Clear selection rectangle on tap gesture
On branch m42 the selection rectangle is cleared when on tap gesture is handled.
This does not happen on m47.
Because of the above WCS#41 fails when JavaScript preventDefault() Event Method
is used to cancel mouse down event. Context menu is being hidden but the
selection rectangle is still visible. This patch makes m47 to works as m42.
Original beta/m47 patch:
- http://165.213.202.130/gerrit/#/c/98569/
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15140
Reviewed by: a1.gomes
Change-Id: I71a6ffe0b0645d4e91811b2eb87ff65f9169d13b
Signed-off-by: Piotr Krysiewicz <p.krysiewicz@samsung.com>
Piotr Ganicz [Thu, 10 Dec 2015 13:33:49 +0000 (14:33 +0100)]
Align the content of popups (date,month,week,time,colorpicker)
This patch is to set a proper position of swallow element on the
popup on TV platform. The solution for this issue is to send a signal
from C++ code to the edje file (control.edc) with the information
of the current platform. Depending on the platform the proper code
will be running in edc file in part "elm.swallow.datetime".
In colorpicker top_padding element was displayed as a white RECT,
due to that fact we should change the type of it to SPACER
(it is transparent).
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14841
Reviewed by: a1.gomes, g.czajkowski
Change-Id: I1a787979581cf218fdf8d8d1d630a17a620296ab
Signed-off-by: Piotr Ganicz <p.ganicz@samsung.com>
Antonio Gomes [Wed, 9 Dec 2015 21:44:02 +0000 (17:44 -0400)]
fixup! Do not zoom-to-focused-element when long pressing.
There is another spot in the code that calls
::ScrollFocusedEditableNodeIntoRect and triggers
zoom-to-focused-element.
It is trigger on target builds, when dpad shows up.
Patch fixes it.
Original beta/m47 patch:
- http://165.213.202.130/gerrit/#/c/98597/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14911
Reviewed by: g.czajkowski
Change-Id: I7c1cfc6f8d13f8842848950a32d7a3ea78804580
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Thu, 10 Dec 2015 18:19:20 +0000 (14:19 -0400)]
Set enable-prefer-compositing-to-lcd-text on ubrowser (desktop).
Our text selection implementation is based on updates that
come as part of the compositing updates (cc/ land).
One bit of this update is the ui::SelectionBound::visible property,
which holds the visibility status of a given selection bound.
For instance, this takes into account when a selection happens
on an scrollable box (e.g. <div>) or a scrollable inner frame
(e.g. <iframe>), and it gets scrolled out of the view.
The visible status is determinated by checking if the given
point is hit-test'able considering the bounds of the clipping
parent layers.
However, if a layer is not promoted as composited by a reason,
it is not taken into account on the upwards frame traversal.
Scrollable iframes get their promotion to a composited layer
controlled by a setting: preferCompositingToLCDTextEnabled()
(see CompositingReasonFinder.cpp).
This setting gets set on RenderViewImpl::Initialize and bubbled
down to blink::WebView.
Its value is controlled by:
1) command line switches kDisablePreferCompositingToLCDText and
kEnablePreferCompositingToLCDText
2) the device_scale_factor (see DeviceScaleEnsuresTextQuality).
On desktop, where the device_scale_factor is 1.0, preferCompositingToLCDTextEnabled
gets set to FALSE. Due to that, scrollable inner frames do not
get promoted to composited layers, and all the visibility set
logic of selection bounds (described above) does not work properly -
it is always TRUE.
In order to make ubrowser test what TIZEN runs, patch passes
'enable-prefer-compositing-to-lcd-text' switch at launch time.
This makes clipping work for selection bounds that go offscreen.
Original beta/m47 patch:
- http://165.213.202.130/gerrit/#/c/98862/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14720
Reviewed by: g.czajkowski
Change-Id: I3587e53c1e357107d03b9b32d9870d91de791a7e
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Thu, 10 Dec 2015 20:13:31 +0000 (16:13 -0400)]
Offscreen handles are visible
Patch changes the selection controller logic a bit to
take SelectionBound::visible bit into account before
show/hide a "input" handle.
This fixes WCS TC 15, and various real world websites.
Original beta/m47 patch:
- http://165.213.202.130/gerrit/98864
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14720
Reviewed by: g.czajkowski
Change-Id: I104c0b38f583c7b0467fe224c100052e1b8fe98a
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Wed, 9 Dec 2015 20:47:45 +0000 (16:47 -0400)]
Dismiss disambiguation popup when touch outside of it.
When disambiguation popup gets shown, it is only possible
to dismiss it when clicking on the zoomed area. This differs
from s-browser, where clicking outside the zoomed dismiss it.
Patch implements the same for chromium-efl.
Original beta/m47 patch:
- http://165.213.202.130/gerrit/98592
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15238
Reviewed by: g.czajkowski
Change-Id: Ieff0a1ce0d4784d30036c2ec02bdc5ca6c7ddc9b
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Piotr Pruski [Wed, 25 Nov 2015 08:54:48 +0000 (09:54 +0100)]
[EWK] "policy,decision,navigation" callback invoked to m47_2526
On Chromium Upstrean HandleNavigation was deprecated ([1]) and reflected
to ours ([2]). Because of this callback for "policy,navigation,decide"
was not called normally. This patch modifies this behaviour to fulfill
EWK API requirements.
[1] http://code.google.com/p/chromium/issues/detail?id=325351
[2] http://165.213.202.130/gerrit/#/c/90805/17
Together with: I922aeeb194e9c4a207716657c1b5d8b9663ed20b
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14732
Reviewed by: a.renevier, a1.gomes, djmix.kim
Change-Id: I3a01cd41861d0913b36797c5ceb21e10fcfa32a9
Signed-off-by: Piotr Pruski <p.pruski@samsung.com>
Grzegorz Czajkowski [Thu, 10 Dec 2015 13:25:57 +0000 (14:25 +0100)]
Save traces.json in $HOME for TIZEN
ubrowser app allows tracing events which are saved to
/opt/usr/media/traces.json for TIZEN. Since ubrowser can not be run on root
saving to this path fils due to lack of permission for non root user.
This patch saves traces.json in root of home directory.
Original beta/m47 patch:
- http://165.213.202.130/gerrit/#/c/98830/
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15248
Reviewed by: a1.gomes
Change-Id: Ie0cdf091518e12351d9495541e3858d4c77c9b71
Signed-off-by: Grzegorz Czajkowski <g.czajkowski@samsung.com>
Pawel Niemirski [Thu, 22 Oct 2015 12:33:33 +0000 (14:33 +0200)]
Implementation of DownloadManagerDelegateEfl class
The patch implements DownloadManagerDelegateEfl::DetermineDownloadTarget,
which allows to use Chromium downloads.
We are checking whether the external download manager is available,
in order for it to be considered, before using Chromium fallback
download code.
Right now, PathService class is used to determine where downloads
should be saved on given platform based on mime type.
Original beta/m47 patch:
- http://web.sec.samsung.net/gerrit/#/c/96713/
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14817
Reviewed by: a1.gomes
Change-Id: Ia1362cad11044dab4de4e1ee7eb908ebeee3c995
Signed-off-by: Pawel Niemirski <p.niemirski@samsung.com>
(cherry picked from commit
d0bde5a4fdf6f0591bf194478e1b7f3b5997bd00)
Pawel Niemirski [Fri, 11 Dec 2015 10:43:23 +0000 (11:43 +0100)]
Add WebViewBrowserMessageFilter in RenderViewCreated
The renderer process is created for each domain. Communication
between the browser process and renderer processes is done
by ChannelProxy. In order to initialize communication
the RenderProcessHost inserts WebViewBrowserMessageFilter object
into the channel.
Adding WebViewBrowserMessageFilter object to ChannelProxy by
RendererProcessHost is called too early because RWHV was not created
yet (while calling RenderProcessWillLaunch). This breaks
communication.
To make sure that RenderWidgetHostView is already created,
when adding WebViewBrowserMessageFilter to the channel, this patch
moves it to WebContentsDelegateEfl::RenderViewCreated.
The issue happens ONLY after domain change, because the first instance
Instance of RWHV is created by EWebView::Init(), but every consequent
is created after EWebView::SetURL().
Original beta/m47 patch:
- http://165.213.202.130/gerrit/#/c/99073/
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15100
Reviewed by: a1.gomes
Change-Id: I058e6dc4666cb913f7256fd613528e165cbd1352
Signed-off-by: Pawel Niemirski <p.niemirski@samsung.com>
SeungSeop Park [Fri, 11 Dec 2015 07:08:13 +0000 (16:08 +0900)]
Revert "fixup! Make build_tv.sh script usable again."
This reverts commit
fa1b0b3713341f8123a0e147ce48387f68d5a690.
Reverting due to problem in bot.
Change-Id: Ifd57cac9022af2167d6f892eaac7d6711cf16cd8
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
msu.koo [Tue, 10 Nov 2015 09:41:29 +0000 (18:41 +0900)]
[MM] Cleanup dead codes(pixmap support) on multimedia
Pixmap was introduced for Tizen 2.4 mobile,
but Pixmap support was deprecated from Tizen 3.0.
(because Tizen 3.0 moves to wayland backend from X.)
Bug: http://165.213.149.170/jira/browse/TSAM-558
Reviewed by: sm.venugopal, sns.park
Change-Id: I4a5a10d15105a764c8fcc63d340ecfcd657c3975
Signed-off-by: msu.koo <msu.koo@samsung.com>
venu.musham [Fri, 4 Dec 2015 06:28:55 +0000 (11:58 +0530)]
Upgrade EFL version for desktop to 1.16.0.
EFL version is updated to 1.16+ on mobile and TV.
- With EFL 1.16, recreating surface on resize is not
needed, commit[1] revert's implementation of
recreating evasgl surface on resize.
- Desktop version is using EFL 1.13, whitescreen issue
is observed with commit[1].
commit[1]: http://165.213.202.130/gerrit/#/c/97324/
Update EFL version for desktop to 1.16.0.
Reviewed by: a1.gomes, sns.park
Change-Id: I482af687ffe06aa2805968aaaf30e25312411f1d
Signed-off-by: venu.musham <venu.musham@samsung.com>
vm.arjun [Fri, 27 Nov 2015 14:15:10 +0000 (19:45 +0530)]
[MM][CAPI] Copy the YUV data from TBM surface by each plane
Now copying YUV plane to shared memory uses correct start point of
each plane's pointer using libyuv function.
Before this change, there was segmentation fault because it was trying to
access complete YUV size where plane[0] was allocated of only Y size
Reference: http://165.213.202.130/gerrit/#/c/88539/
Reviewed by: msu.koo, sm.venugopal, sns.park
Change-Id: Id8b4e7ae3b267cb3b5d103a6ecb4d63ae6fc8d9a
Signed-off-by: ws29.jung <ws29.jung@samsung.com>
Signed-off-by: vm.arjun <vm.arjun@samsung.com>
Andrzej Badowski [Tue, 8 Dec 2015 12:56:24 +0000 (13:56 +0100)]
[EFL] Implement orientation_lock and orientation_unlock API
This is a very basic implementation of orientation lock/unlock API:
http://www.w3.org/TR/screen-orientation/, based on current Chromium Android
implementation. Android's version has been stripped from Android specific
Start/StopAccurateListening methods, which will be provided in next patch.
This patch deprecates EWK API's ewk_view_orientation_lock_callback_set.
Original beta/m42_2311_t patch:
- http://165.213.202.130/gerrit/#/c/80821/
Original patch was partially passed to dev/master. This is supplement.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14887
Together with: Ia91c2dc181fd9c08f0027c1a50e681c349e20294
Reviewed by: a1.gomes, djmix.kim, msu.koo
Change-Id: I1617916634ee03b54c80124a724eb4e24d01e427
Signed-off-by: Andrzej Badowski <a.badowski@samsung.com>
uzair.jaleel [Thu, 9 Jul 2015 11:09:35 +0000 (16:39 +0530)]
[MM][ME][MSE] Implemented to pass over media_packet to renderer.
This patch is to enable MediaPacket on ME/MSE for Tizen 3.0.
Currently actual composition happens inside Browser Process,
(to be exact, GPU thread lives in Browser Process)
so passing the raw video frames via SharedMemory can be avoid.
This patch is to pass over the frame memory address to renderer process
to let GPU thread directly access Tizen video buffers.(aka, tbm_surface)
Bug: http://165.213.149.170/jira/browse/TSAM-560
Reference: http://165.213.202.130/gerrit/#/c/87210/
Reviewed by: msu.koo, sm.venugopal, sns.park
Change-Id: I1abf25be1c346fbf1ec046d1fd5307745756ca2f
Signed-off-by: uzair.jaleel <uzair.jaleel@samsung.com>
Signed-off-by: msu.koo <msu.koo@samsung.com>
Signed-off-by: ws29.jung <ws29.jung@samsung.com>
Sanghyup Lee [Tue, 8 Dec 2015 13:42:39 +0000 (22:42 +0900)]
When selection is cleared, context menu also should be cleared.
When |ewk_view_text_selection_clear| is called, context menu is still shown
although selection was already cleared.
This patch clear context menu in |EWebView::ClearSelection|.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15223
Reviewed by: a1.gomes, sns.park
Change-Id: Ib07dcf92777f49206bd899fa14934cf27e4c1364
Signed-off-by: Sanghyup Lee <sh53.lee@samsung.com>
msu.koo [Wed, 9 Dec 2015 01:53:49 +0000 (10:53 +0900)]
fixup! [MM] Handling multiple instances of media players.
This is to fix the wrong file-mode changes of previous patch.
File mode is reverted from 100755 to 100644.
Reviewed by: msu.koo, sm.venugopal, sns.park
Change-Id: I86072782c55de96d2d2d25df1a2001f5e3bfe096
Signed-off-by: msu.koo <msu.koo@samsung.com>
Tomasz Czekala [Tue, 8 Dec 2015 11:34:32 +0000 (12:34 +0100)]
Fix crash when calling ewk_shutdown
Crashes were failing CHECKs, which expected the client to release all
evas WebView objects before calling ewk_shutdown. Client should fix that,
but Chromium shouldn't crash because of client's bug, so checks were
replaced with information in log.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14628
Reviewed by: g.czajkowski, msu.koo
Change-Id: I23b09de78f2536608621a1de07b7de8c956c1ab3
Signed-off-by: Tomasz Czekala <t.czekala@samsung.com>
sangdeug.kim [Tue, 8 Dec 2015 06:11:18 +0000 (15:11 +0900)]
[MM] Handling multiple instances of media players.
This patch introduces a way to manage the number of player instances,
which is needed to meet a policy or limitation imposed by platform.
For example, in TV target HawkP, only 1 player instance is allowed to run
by hardware limitation.
When active player instances exceeds the maximum number, the implementation
releases the oldest player instance.
Details for current implementation:
- Max player instances : TV - 1, Mobile - 5
- Choosing player to remove when full: FIFO (managed by |resumed_queue|).
- The purpose of each queue.
|init_queue| : Whenever a player is created, it's pushed in a init_queue.
Then the players are initialized one-by-one.
|resumed_queue| : Managing the activated(resumed) player id.
Push id by |OnResumeComplete| and pop by |OnSuspendCompleted|.
- When we try to play or seek a player, the implementation check the
resumed(activated) player and then suspend(release) player instance
if it exceeds the maximum instance number.
Reference : http://165.213.202.130/gerrit/#/c/72551/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14833
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14853
Reviewed by: msu.koo, sm.venugopal, sns.park
Change-Id: I31e2cbefdfa60222315b8743a73072c5009c20b2
Signed-off-by: sangdeug.kim <sangdeug.kim@samsung.com>
Antonio Gomes [Fri, 4 Dec 2015 14:44:01 +0000 (10:44 -0400)]
Do not trigger selection for when selection is forbidden or cancelled
If user is long pressing on a content with
-webkit-user-select: none, we should bail and not enter
selection neither show magnigier class or context menu.
Same (not enter selection mode) should happen when JS side
calls preventDefault on 'selectstart' event.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15148
Reviewed by: a.renevier, a1.gomes, g.czajkowski
Change-Id: Ice4e0a5b3c0ada166f3cbc9a2d48e24534f86d38
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
msu.koo [Thu, 22 Oct 2015 07:02:13 +0000 (16:02 +0900)]
[MM] Remove duplicated enums from MM codes.
This patch is to remove duplicated NetworkState and ReadyState
from codes and use already-existing blink::WebMediaPlayer::NetworkState
and blink::WebMediaPlayer::ReadyState.
Bug: http://168.219.209.56/jira/browse/TSAM-536
Reference: http://suprem.sec.samsung.net/gerrit/#/c/54032
Reference: http://suprem.sec.samsung.net/gerrit/#/c/56307
Reviewed by: sm.venugopal, sns.park
Change-Id: Ic81fb513f41d1e7d9a688b66f798cd3f827b47ea
Signed-off-by: msu.koo <msu.koo@samsung.com>
Sanghyup Lee [Fri, 4 Dec 2015 02:04:50 +0000 (11:04 +0900)]
Filter out events jittery for single-touch gestures.
In the majority of cases when user taps on the screen, many touch move events
are generated from platform. These unexpected touch move events cause frequent
calling seek on media player.
It looks too sensitive compared with Android browser.
This patch dispatches first move event if distance from last pressed position
exceeds scroll threshold.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=13974
Reviewed by: a1.gomes, sns.park
Change-Id: Ia491beb54c1cf32df78d9af8b7e7a212c7f54e61
Signed-off-by: Sanghyup Lee <sh53.lee@samsung.com>
Piotr Krysiewicz [Wed, 2 Dec 2015 17:03:11 +0000 (18:03 +0100)]
Changed translations of ContextMenu items names
Few translations need to be fixed to meet UI Guidelines.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15105
Reviewed by: Grzegorz Czajkowski, Min-Soo Koo
Reviewed by: a1.gomes
Change-Id: If0e6d8e456844ada6a13d56f0578e93c869b0757
Signed-off-by: Piotr Krysiewicz <p.krysiewicz@samsung.com>
(cherry picked from commit
c5e7206d44749d9b150790ca8c02ce33357f9d00)
k2.nagaraju [Tue, 20 Oct 2015 03:48:41 +0000 (09:18 +0530)]
[MM][MSE][CAPI] Crash while playing more than one clip in youtube
During CAPI destruction, OnDemuxerDataAvailable call from demuxer will
lead to crash. In such scenario, just return the control. This patch also
solves the crash issue in demo.htm, playing webaudio/MSE/Webaudio/MSE
Original Patch: http://suprem.sec.samsung.net/gerrit/#/c/53523
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14835
Reviewed by: msu.koo, sm.venugopal, sns.park
Change-Id: I9c18941735b2550ec81876362f3d50f85f6ae5c2
Signed-off-by: k2.nagaraju <k2.nagaraju@samsung.com>
SeungSeop Park [Wed, 2 Dec 2015 00:02:43 +0000 (09:02 +0900)]
Delete unnecessary eglibc-2.18-5.1.armv7l.rpm
The file was added in [1] to support GCC 4.8 build, but now obsolete.
[1] http://165.213.202.130/gerrit/#/c/70019/
Reviewed by: a1.gomes, msu.koo
Change-Id: I0b5e0ab8688ff4d57e3309c73e89db3711c27dca
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
SeungSeop Park [Tue, 1 Dec 2015 23:47:35 +0000 (08:47 +0900)]
Revert "Workaround for tizen's nss version issue"
This reverts commit
0f59aac6f36d809a46979b6c0f2c099c59c715b6.
Nss version in tizen 3.0 is 3.17. We are not relying on the old hack
to bypass nss version issue. Patch removes the legacies.
Reviewed by: a1.gomes, msu.koo
Change-Id: I38bd5adaf42171b5620dae79005667d24c5bff80
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
SeungSeop Park [Tue, 1 Dec 2015 07:57:18 +0000 (16:57 +0900)]
fixup! Make build_tv.sh script usable again.
Symbolic link src/packaging which points to src/tizen_src/packaging
is generated during the build, which yields dirty status when 'git status'.
Patch removes 'src/packaging' symlink after the rpm build is done.
Reviewed by: a1.gomes, msu.koo
Change-Id: If1d2cff7f4ca62415fb6deba5bea051302053606
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
suneel [Tue, 17 Nov 2015 10:08:49 +0000 (15:38 +0530)]
Implement device display info and get device scale factor.
This patch implements basic display functionalities like display
resolution size, dip scale and rotation degrees.
Obtains the device scale factor (dsf) using device display info. The
display info computes this factor from dpi.
Reference gerrit IDs:
http://165.213.202.130/gerrit/#/c/83364/
http://165.213.202.130/gerrit/#/c/83651/
http://165.213.202.130/gerrit/#/c/85226/
http://165.213.202.130/gerrit/#/c/85236/
http://165.213.202.130/gerrit/#/c/85700/
http://165.213.202.130/gerrit/#/c/87076/
Together with: I8f6dbee2545283c3002591e4965f5c2f677fc6c9
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14881
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14877
Reviewed by: a1.gomes, prashant.n, sns.park
Change-Id: I22014fabf1e2ce2160be75dec4e8bdb8ecd76545
Signed-off-by: suneel <suneel.kota@samsung.com>
Jongsoo Yoon [Thu, 3 Dec 2015 07:16:06 +0000 (16:16 +0900)]
Enable ccache on gbs build system.
In order to enable ccache build option,
we should change it's PATH value to /var/lib/build/ccache/bin.
Because ccache path is moved to there in latest gbs build system.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15086
Change-Id: Icd70453aa96a85553a9e949aad3ee3d2d0128688
Signed-off-by: Jongsoo Yoon <join.yoon@samsung.com>
Jongsoo Yoon [Fri, 4 Dec 2015 01:22:19 +0000 (10:22 +0900)]
Changed the IP of mirror server in Tizen 3.0 Profile.
This patch is only valid in HQ.
Before : http://10.252.81.117/snapshots
After : http://10.113.63.117/snapshots
Change-Id: Ic77bb2e9014cc532b7d5c50aac95a8f5ab7c9808
Signed-off-by: Jongsoo Yoon <join.yoon@samsung.com>
DongJun Kim [Wed, 2 Dec 2015 00:43:44 +0000 (09:43 +0900)]
Re-implement didCreateScriptContext to RenderFrameObserverEfl.
DidCreateScriptContext and WillReleaseScriptContext are moved
to ContentRendererClient by (1),(2) in opensource.
DidCreateScriptContext and WillReleaseScriptContext are called
when calling open/close window by javascript.
Those functions must be called together.
In case of m47 chromium-efl, we do not call DidCreateScriptContext.
For this reason, renderer process is crashed when
javascript context is released from injected_bundle.
This patch has 2 modifications as below.
- Remove |WillReleaseScriptContext| from ContentRendererClientEfl.
- Move |didCreateScriptContext| to RenderFrameObserverEfl.
(1) https://codereview.chromium.org/
1040403003
(2) https://codereview.chromium.org/
185713008
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15036
Reviewed by: a.renevier, a1.gomes, djmix.kim, g.czajkowski, msu.koo
Change-Id: I008937710d08472a6cfb77a51b1cbf01ffacc221
Signed-off-by: DongJun Kim <djmix.kim@samsung.com>
Grzegorz Czajkowski [Tue, 1 Dec 2015 10:22:55 +0000 (11:22 +0100)]
Do not pass kMobileFlags for all TIZEN builds using ubrowser
kMobileFlags are defined in ubrowser/main.cc and are passed to chromium in case
ubrowser is launching with mobile param.
However, as |!desktop_ui| is also true for all TIZEN builds those flags are
unexpectedly set for them. As for OS_TIZEN_MOBILE they are just duplicated and
do not bring side effects whereas for OS_TIZEN_TV they are not expected.
This patch passes kMobileFlags in case "ubrowser --mobile" only.
Additionally, this patch gets rid of custom EFL command line
"use-mobile-viewport-style" as it can not be propagated to Renderer by default.
For OS_TIZEN_MOBILE |UseMobileViewportStyle| is set by default on src/ side [1].
This patch introduces a new EFL command line:
"ewk-enable-mobile-features-for-desktop" in order to turn on a bunch of
settings for which there is no command line switches. This allows desktop
"ubrowser --mobile" to have similar set of features that mobile build has.
Secondly, we stop abusing |UseMobileViewportStyle| in |ApplyCustomSettings|.
[1] http://165.213.202.130/gerrit/#/c/97109/1
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14816
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15031
Reviewed by: a1.gomes, sns.park
Change-Id: I7dd9f693f53abab0916e6113bbc15bfe635a3159
Signed-off-by: Grzegorz Czajkowski <g.czajkowski@samsung.com>
Marcin Niesluchowski [Mon, 17 Aug 2015 11:48:41 +0000 (13:48 +0200)]
Implement ewk_favicon_database_icon_get()
Implement ewk_favicon_database_icon_get() by calling
EwkFaviconDatabase::GetIcon() (also added by this patch).
EwkFaviconDatabase holds an instance of EWebContext, needed
in the call chain.
Original beta/m42_2311_t patch:
- http://165.213.202.130/gerrit/#/c/85278/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14809
Reviewed by: d.waslicki, g.czajkowski, msu.koo
Change-Id: I64ee94547570758a45d9d4aa6c0c8556c8c8f258
Signed-off-by: Marcin Niesluchowski <m.niesluchow@samsung.com>
k2.nagaraju [Fri, 16 Oct 2015 08:00:27 +0000 (13:30 +0530)]
[MM][MSE] seek offset not synced to actual browser seek time
CAPI player_set_position as to be called with actual browser seek time
not with user seeked time. Otherwise CAPI will wait for
(|actual_browser_seek_time| - |seek_offset_|) milliseconds buffer.
Original Patch: http://suprem.sec.samsung.net/gerrit/#/c/53159/
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14835
Reviewed by: msu.koo, sm.venugopal, sns.park
Change-Id: I878697fdc98f259a02b9849bc47955464aec2799
Signed-off-by: k2.nagaraju <k2.nagaraju@samsung.com>
Lukasz Bialek [Tue, 27 Oct 2015 13:15:20 +0000 (14:15 +0100)]
Add support for Copy/Paste operations for Image element
Although Clipboard operations for text and markups work well,
when image is long-pressed, copied (Copy Image) and pasted - nothing
happens.
This patch introduces copy/paste support of images. This allows
for pasting images to richly editable field and text fields
(URL otherwise).
This patch is a tizen_src part of the implementation (Webkit is here
http://165.213.202.130/gerrit/#/c/95868/), which uses image markup to
generate all necessary clipboard data.
You can check it by selecting "copy image" on html5 "badge" image
and then pasting it into editable area on the following adress:
http://html5demos.com/contenteditable
Original author: Lukasz Bialek <l.bialek@samsung.com>
Based on patch: http://165.213.202.130/gerrit/#/c/92243/
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14453
Together with: I11136d8352014f1acca144a0c4416c3bcf432e0c
Reviewed by: a1.gomes, djmix.kim, g.czajkowski
Change-Id: I11126d8352044f1acca044a0c4416e3bcf412e0c
Signed-off-by: Pawel Forysiuk <p.forysiuk@samsung.com>
Sanghyup Lee [Wed, 2 Dec 2015 22:25:58 +0000 (07:25 +0900)]
Fixed wrong behaviors with HW back button.
1. If IME is shown, context menu and handlers shouldn't be hidden by H/W back
key. Currently context menu and handlers are cleared regardless of IME state
becuase |SelectionControllerEfl::HideHandleAndContextMenu()| is called in
|RWHVEfl::ImeCompositionRangeChanged| which is called when IME is shown or
hidden. This patch removes it.
2. HW back key deletes selected text. |IMContextEfl::OnPreeditChanged| isn't
returned propery when buffer is empty.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15025
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15060
Reviewed by: a1.gomes, djmix.kim, g.czajkowski
Change-Id: If85f59f0121cf36159e4b5d0ea9adda6a9df78d6
Signed-off-by: Sanghyup Lee <sh53.lee@samsung.com>