Chandan Padhi [Mon, 6 Aug 2018 10:12:36 +0000 (15:42 +0530)]
fixup! [M63 Dev][EFL] Integrate EvasGL Delegated Renderer
This commit removes the left over files.
Change-Id: I7c0a2c6951eafbbcaa96d925c3fc1d58f3358fee
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
Chandan Padhi [Fri, 3 Aug 2018 12:48:23 +0000 (18:18 +0530)]
[M67 Dev][Tizen] Redirect UncheckedMalloc to malloc for Tizen
On Tizen, BufferManager::DoBufferData tries to call free() to
deallocate a memory that was allocated by tc_malloc in
UncheckedMalloc(), which caused crash for most web pages.
To fix it, we redirect UncheckedMalloc to malloc for Tizen.
Seems like we do not use tcmalloc for chromium-efl in a consistent
manner. We should either fix it or disable tcmalloc.
Reference:
http://165.213.202.130/gerrit/#/c/71875/
http://165.213.202.130/gerrit/#/c/72475/
Change-Id: Iacc6776736c193d6fa66707ff28048d42267ac12
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
Youngsoo Choi [Mon, 6 Aug 2018 01:45:37 +0000 (18:45 -0700)]
[M67 Dev][Tizen] Remove GYP dependencies in tizen_src
This removes GYP dependencies in tizen_src.
Change-Id: Iedd7c5e6f1665f601e04100a4c0d46ee728bcc58
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Youngsoo Choi [Fri, 2 Mar 2018 11:37:05 +0000 (17:07 +0530)]
[M67 Dev][Tizen] Integrate GN and set up build environment
This cherry-picks following patches to support Git Build System(GBS)
and generates ninja for Tizen targets.
Commit id
5f3edd710 [Refactoring] GN Improvement
Commit id
01b6a00b9 Replace the new repo path in gbs.conf for TV
Commit id
b75b104a5 Changed the build repository for tizen 4.0 in gbs.conf
Commit id
03a1d8bed [A&O] Support build script and gbs.conf for tizen 4.0 kantm
Commit id
047b94f8f [A&O] Support of tizen 4.0 mirror profiles
Commit id
ea3ce6588 [A&O] Remove target profiles from tizen 4.0
Commit id
737826fd8 [A&O] Add build script for unified profile
Commit id
06ec8ccdc [Platform Integration] Support building for x86_64 by GN
Commit id
6ab2e11ed [Build] Support to KantM Product TV target.
Commit id
ee659fe58 fixup! [M56_2924] Chromium upversion to m56_2924 branch
<Partial cherry-pick. Taken changes w.r.t. to GBS only>
Commit id
ba3fbbcc3 [A&O] Separate out folder as per architecture
Commit id
85228a474 fixup! [A&O] Separate out folder as per architecture
Commit id
3ff2cce91 fixup! [M56_2924] Switching to GN build
<Partial cherry-pick of tizen_src/build/toolchain/tizen/BUILD.gn>
Commit id
9b43ac32c [M63_3239][Bring Up] Remove crosswalk related codes and files
Commit id
1c6f09561 [M63_3239][Bring Up] Remove the apply_patches.sh calls
Commit id
4bca0b3ef [M63][Build] Support optional builds for tizen versions
Commit id
b125e1e7d fixup! [M63][Build] Support optional builds for tizen versions
Commit id
35449da2d fixup! [M63_3239] Switching to GN build
Commit id
ec7160231 Fix warning errors in gn generation
Change-Id: I80ef5bb2e5fa4370698ac4f6503b1678cb67ffeb
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
Chandan Padhi [Thu, 2 Aug 2018 11:06:36 +0000 (16:36 +0530)]
fixup! [M63 Dev][EFL] Integrate EvasGL Delegated Renderer
This commit removes NOTIMPLEMENTED() logs that are seen during
rendering of every frame which may affect KPI performance.
Change-Id: Ic5c9240592d9cf919b68b12a40031c550b7f5406
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
Youngsoo Choi [Wed, 8 Mar 2017 04:31:53 +0000 (13:31 +0900)]
[M67 Dev][Tizen] Use system ninja
The chromium-efl has used prebuilt ninja in build.
This removes the prebuilt ninja and makes chromium-efl use system ninja.
Change-Id: I9af66d0843d639248ecd77da983bf6b683a4ed12
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Slava Barinov [Tue, 11 Apr 2017 08:26:05 +0000 (11:26 +0300)]
[M67 Dev][Tizen] Use the x86_64 GN for all architectures
Enabling x86_64 GN instead of native ones dramatically speeds up ninja files
generation for arm/arm64 platforms.
This GN is built without tcmalloc and supports ASan builds.
Also, the GN is upgraded to support newly added functions
such as 'not_needed' and so on.
Reference:
https://review.tizen.org/gerrit/#/c/134218/
Change-Id: Idc9db604362eb464833fd7b0c34cf68aa7ed4ac0
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Chandan Padhi [Fri, 27 Jul 2018 06:26:07 +0000 (23:26 -0700)]
[SlimDownCode] Remove unused libjpeg in //third_party/
chromium-efl uses libjpeg_turbo and not libjpeg.
Therefore, removing it from the source tree.
Change-Id: I597a583fd09ab3c5e148db711f5d767017190daf
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
Chandan Padhi [Fri, 27 Jul 2018 08:55:36 +0000 (01:55 -0700)]
[M67 Dev][EFL] Fix crashes at webview launch
This commit fixes the crashes during webview launch.
It also fixes the white screen observed later.
Basic page rendering works fine on desktop.
Reference: https://review.tizen.org/gerrit/#/c/171308/
Change-Id: I5825876d8294ebfed387ee4b09da3aeb13e58954
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
Chandan Padhi [Fri, 27 Jul 2018 08:31:57 +0000 (01:31 -0700)]
[M67 Dev][EFL] Format chromium-efl according to Chromium style
This commit formats chromium-efl code as per Chromium style
by using "git cl format".
Change-Id: I349fb24c9a08ec4d21915326b19ea5226417a8c4
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
Chandan Padhi [Wed, 25 Jul 2018 13:25:34 +0000 (06:25 -0700)]
[M67 Dev][EFL] Fix linker errors
This resolves linker errors to enable EFL port.
Change-Id: I8c6affe1bf430e478c874df9fe6fb1ebcf56a169
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
Chandan Padhi [Tue, 24 Jul 2018 08:08:29 +0000 (01:08 -0700)]
[M63 Dev][EFL] Integrate EvasGL Delegated Renderer
This patch brings up the rendering path for EFL port.
Reference:
https://review.tizen.org/gerrit/#/c/105131/
https://review.tizen.org/gerrit/#/c/171539/
Change-Id: I15364a5ac77e2ea6f012f7edc5a96b8aac7b84bb
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
Chandan Padhi [Tue, 24 Jul 2018 13:11:13 +0000 (06:11 -0700)]
[M67 Dev][EFL] Fix compiler errors (2/2)
This resolves compiler errors to enable EFL port.
Change-Id: I9de1665ce3d04761ffa45d170d99fa892554f8bd
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
Youngsoo Choi [Tue, 24 Jul 2018 06:48:21 +0000 (06:48 +0000)]
[M67 Dev][EFL] Fix compiler errors (1/2)
This resolves compiler errors to enable EFL port.
Reference: https://review.tizen.org/gerrit/#/c/169959/
Change-Id: I634566226533aea17f774ffff45cdb7d2236c57b
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Chandan Padhi [Tue, 24 Jul 2018 09:42:23 +0000 (15:12 +0530)]
[SlimDownCode] Remove unused libpng in //third_party/
chromium-efl uses system libpng and not libpng in //third_party/.
Also, presence of libpng in //third_party/ results in linker errors.
Therefore, removing it from the source tree.
Change-Id: Ib9c5f3d5ebc3e987137dcb256313182cfa2cdc17
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
Chandan Padhi [Wed, 25 Jul 2018 08:41:44 +0000 (14:11 +0530)]
fixup! [Rom Size] Use system libpng
This commit adds the missing dependency of libpng
to fix related desktop build errors.
Change-Id: I18a33f8be25c60b383d8e54599f13d3b8cfe7e08
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
Youngsoo Choi [Tue, 17 Jul 2018 01:23:19 +0000 (01:23 +0000)]
[Rom Size] Use system libpng
The libchromium-ewk.so size can be reduced by using system libpng.
Note that this adds libpng 1.6 which ubuntu doesn't have
so that the library can be used for desktop build by jhbuild.
Change-Id: Id92c55f172c0a9591e19e0838e942aec6527489f
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Youngsoo Choi [Mon, 16 Jul 2018 07:06:25 +0000 (07:06 +0000)]
[Rom Size] Remove dependencies of chrome, crashpad, and extension
This removes unused component dependencies to reduced binary size.
Removed Dependency List:
- chrome
- crashpad
- extension
Change-Id: Ic11a3f3bc999c91af00b21e8916d5d60ad42b4d7
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Antonio Gomes [Tue, 7 Apr 2015 06:43:26 +0000 (02:43 -0400)]
[M67 Dev][EFL] Allow chromium to use EFL instead of Aura
This commit allows chromium code to use EFL as the main UI toolkit
instead of Aura. The actual implementation of EFL port of chromium
is done in an external module. This commit only exposes some of the
needed generic code paths that are normally disabled for non Aura or
MacOSX ports.
Reference:
5e22fe4d0bb137a1b73530205f7cdc07a31643c9
Change-Id: I1f327fb9e31616879cf0448735f1adbe82e476bf
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Arnaud Renevier [Fri, 15 May 2015 19:02:40 +0000 (12:02 -0700)]
[M67 Dev][EFL] Define WebCursor::GetPlatformCursor for efl
It will be necessary to define WebCursor::GetPlatformCursor in order
to build ozone.
Reference:
6bf5529a7a033a692f0ced0d25761f839114395c
Change-Id: Ic681ccd587e0a5ea274b07ed0fa9283b2abd2e2e
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Youngsoo Choi [Sat, 25 Feb 2017 05:06:32 +0000 (14:06 +0900)]
[M67 Dev][EFL] Set up build environment
The gn build is supported by default:
$ ./tizen_src/build/build_desktop.sh
The followings are included in this:
[Build] Use GN by default instead of GYP
https://review.tizen.org/gerrit/#/c/116494/
[Build] Move tizen_src/packaging/ directory to top
https://review.tizen.org/gerrit/#/c/116944/
[A&O] Revise replace_gn_files.py to support multiple replacements
https://review.tizen.org/gerrit/#/c/128811/
fixup! [A&O] Revise replace_gn_files.py to support multiple replacements
https://review.tizen.org/gerrit/#/c/129273/
Upgrade version of efl, elementary
https://review.tizen.org/gerrit/#/c/161190/
Remove the apply_patches.sh calls
https://review.tizen.org/gerrit/#/c/171551/
Change-Id: Id2e9c444434ed7ba5b6557920d111fc802364d71
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Youngsoo Choi [Fri, 9 Feb 2018 14:53:16 +0000 (20:23 +0530)]
[M67 Dev][EFL] Integrate GN configuration
The upstream chromium hasn't supported EFL port.
So, this enables GN configurations for that.
Temporary command:
$ ./tizen_src/build/build_desktop.sh --use-gn
Reference:
https://review.tizen.org/gerrit/#/c/169727/
https://review.tizen.org/gerrit/#/c/174607/
https://review.tizen.org/gerrit/#/c/178028/
Change-Id: Ie426852c63a4ebcbfea482297ef2915c4a3b6edf
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Suchit Agrawal [Tue, 31 Jan 2017 10:08:46 +0000 (10:08 +0000)]
Revert "Enabled the use_system_re2 flag"
This reverts commit
e9eaac2bf49757f4d57624cf6b4d153e3e059a1f.
Change-Id: I93171ed4d4cd4ade05d978eb10659afe9b5a714e
Signed-off-by: Suchit Agrawal <a.suchit@samsung.com>
Chandan Padhi [Thu, 22 Sep 2016 09:25:17 +0000 (14:55 +0530)]
Remove temporary patch for gpu_control_list.cc
std::find() in gpu_control_list.cc caused build errors for
mobile and tv profiles. This temporary patch was added to
avoid the build error. The build error has been fixed in [1].
[1] http://165.213.202.130/gerrit/#/c/139091
Bug: http://suprem.sec.samsung.net/jira/browse/TWF-2058
Reviewed by: dhyuna.ko, mohan.reddy, sm.venugopal, sns.park
Change-Id: I531a62ba3c2caab984ab6964ef3186927ff79799
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
Chandan Padhi [Fri, 21 Oct 2016 13:26:44 +0000 (18:56 +0530)]
Fix for browser crash on loading HTTPS sites
CTVerifier and CTPolicyEnforcer were not set for URLRequestContext.
This resulted in failure of CHECKs in SSLClientSocketImpl ctor.
Bug: http://suprem.sec.samsung.net/jira/browse/TWF-2310
Reviewed by: g.czajkowski, sns.park
Change-Id: I21498162e2c34a0cc1c5aeb0c1771b0185bb189d
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
yh106.jung [Tue, 27 Sep 2016 02:22:20 +0000 (11:22 +0900)]
Remove aw_pref_store.h|cc from build sources
|InMemoryPrefStore| is being used as simple preference store instead of
|AwPrefStore|. So we can remove aw_pref_store.h|cc from build sources.
Bug: http://suprem.sec.samsung.net/jira/browse/TWF-2071
Reviewed by: g.czajkowski, sns.park
Change-Id: I813dce7678feeb3d8e859549b595b5d1ea3d3edb
Signed-off-by: yh106.jung <yh106.jung@samsung.com>
yh106.jung [Tue, 27 Sep 2016 05:28:03 +0000 (14:28 +0900)]
Move the function, FilesSelectedInChooser, from RVH to RFH
FilesSelectedInChooser is moved from RenderViewHost to RenderFrameHost
by upstream codes. So this patch updates about it for EWK side.
Bug: http://suprem.sec.samsung.net/jira/browse/TWF-2072
Reviewed by: g.czajkowski, sns.park
Change-Id: Icf6481e2fc08e09f59d56b668df807a9b66157d4
Signed-off-by: yh106.jung <yh106.jung@samsung.com>
Youngcheol Kang [Thu, 22 Sep 2016 10:17:20 +0000 (19:17 +0900)]
fixup! Added TM1 mirror server profile and updated mirror server ip address.
For connecting the mirror server, we need to authorization about that.
This patch adds the authorization information in gbs.conf file.
Change-Id: I8c7b2aa982f4ecadfeaaeaaaf029c356b0aafa43
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
Suchit Agrawal [Thu, 22 Sep 2016 04:24:37 +0000 (04:24 +0000)]
Added TM1 mirror server profile and updated mirror server ip address.
Changed mirror server IP from 10.113.63.117 to 165.213.149.200.
Added TM1 mirror server profile in gbs.conf file.
Change-Id: I6f5ec74bf9d4b1eb65e6609faa07fe02aff5b39c
Signed-off-by: Suchit Agrawal <a.suchit@samsung.com>
Prashanth R K Shandilya [Tue, 5 Jul 2016 07:13:35 +0000 (12:43 +0530)]
[M53_2785] Chromium upversion to m53_2785 branch.
This patch deals with the following changes:
1. Folder and class name changes
- Added the leveldatabase.gyp:leveldatabase dependency in chromium-ewk
target
- Added the components.gyp:network_session_configurator_switches
dependency in chromium-ewk target
- Added the codes for enabling the 'use_external_popup_menu' flag
- Changed the make_scoped_ptr to base::WrapUnique
- Changed the path of "v8.gyp" to "v8/src" from the "v8/tools/gyp"
- Changed the path of "gpu_channel_manager.h" to the
"gpu/ipc/service" from the "content/common/gpu"
- Changed the blink::WebNotificationPermission class to the
blink::mojom::PermissionStatus class
- Changed the cc::OnscreenDisplayClient class to the cc::Display class
- Changed the content::RenderProcessObserver class to the
content::RenderThreadObserver class
- Changed the gfx::GLContext class to the gl::GLContext class
- Changed the ui::SelectionBound class to the gfx::SelectionBound class
- Changed the type of ContextProviderCommandBuffer class
- Changed the type of base::WaitableEvent class
- Changed the namespaces of GLHelper to display_compositor::GLHelper
- Removed the s-libjingle_source_talk.git in DEPS.efl
- REmoved the chromedriver dependency in chromium-efl
- Removed the libpci dependency in chromium-efl
- Removed the kEnableLinkDisambiguationPopup switch
2. API changes
- Added the CreateMediaRequestContext function in BrowserContext class
- Added the CreateMediaRequestContextForStoragePartition function in
BrowserContext class
- Added the CreateZoomLevelDelegate function in BrowserContext class
- Added the DelegatedFrameHostGetGpuMemoryBufferClientId function in
RenderWidgetHostViewEfl class
- Added the DelegatedFrameHostGetGutterColor function in
RenderWidgetHostViewEfl class
- Added the GetFramebufferCopyTextureFormat function in
cc::OutputSurface class
- Added the SetBeginFrameSource function in
RenderWidgetHostViewEfl class
- Added the IsWindowUnderCursor function in gfx::Screen class
- Added the OnDestruct function in content::RenderViewObserver class
- Added the OnLoadedRectUpdate function in
plugins::LoadablePluginPlaceholder class
- Changed the GetRequestContext function to the GetURLRequestContext
function in net::URLRequestContext class
- Changed the enum of LOAD_TYPE_BROWSER_INITIATED_HTTP_POST to
LOAD_TYPE_HTTP_POST in content::NavigationController class
- Changed the DeleteAllCreatedBetweenForHostAsync function to the
DeleteAllCreatedBetweenWithPredicateAsync function in
net::CookieStore class
- Changed the arguments of cc::OutputSurface class
- Changed the arguments of SwapBuffers function in
MailboxOutputSurfaceEfl class
- Changed the arguments of OnSwapBuffersCompleted function in
cc::OutputSurface class
- Changed the arguments of OnSwapCompositorFrame function in
RenderWidgetHostViewEfl class
- Changed the arguments of DispatchSyntheticTouchEvent function in
RenderWidgetHostViewEfl class
- Changed the arguments of CopyOutputCallback function in
RenderWidgetHostViewEfl class
- Changed the arguments of IsFullscreenForCurrentTab function in
WebContentsImpl class
- Changed the arguments of DragTargetDrop function in
RenderViewHostImpl class
- Changed the arguments of GpuChildThread function in
content::GpuChildThread class
- Changed the arguments of Vibrate function in VibrationManager class
- Changed the arguments of Cancel function in VibrationManager class
- Changed the arguments of DownloadStarting function in
ResourceDispatcherHostDelegate class
- Changed the arguments of RunFileChooser function in
WebContentsDelegate class
- Changed the arguments of InProcessGpuThread function in
content::GpuChildThread class
- Removed the GetNativeViewId function in RenderWidgetHostViewEfl class
- Removed the GetScreenColorProfile function in RenderWidgetHostViewEfl
class
- Removed the GetWindowUnderCursor function in gfx::Screen class
- Removed the setUseMobileViewportStyle function in blink::WebSettings
class
- Removed the CreateZoomLevelDelegate function in BrowserContext class
- Removed the GetRequestContext function in BrowserContext class
- Removed the GetMediaRequestContext function in BrowserContext class
- Removed the GetMediaRequestContextForRenderProcess function in
BrowserContext class
- Removed the GetMediaRequestContextForStoragePartition function in
BrowserContext class
- Removed the CreateAccessTokenStore function in ContentBrowserClient
class
- Removed the OverrideSystemLocationProvider function in
ContentBrowserClient class
- Removed the enum type of cc::SelectionBoundType class
3. Need to change API (TODO issues)
- The multimedia function has been changed. Temporary disabling the
'tizen_multimedia_support' and 'use_gstreamer_fft' flag
(http://suprem.sec.samsung.net/jira/browse/TWF-2060)
- The VideoCaptureDevice class has been changed. Temporary disabling the
files which related to VideoCaptureDevice class
(http://suprem.sec.samsung.net/jira/browse/TWF-2061)
- The BatteryStatusManager class has been changed. Temporary disabling
the files which related to BatteryStatusManager class
(http://suprem.sec.samsung.net/jira/browse/TWF-2062)
- The AwPrefStore class has been removed. Temporary disabling the codes
which related to AwPrefStore class
(http://suprem.sec.samsung.net/jira/browse/TWF-2071)
- The FilesSelectedInChooser function has been changed. Temporary
disabling the codes which related to FilesSelectedInChooser function
(http://suprem.sec.samsung.net/jira/browse/TWF-2072)
- The codes which related to "std::find()" makes the build problem in
mobile and tv profiles. Temporary disabling the codes
(http://suprem.sec.samsung.net/jira/browse/TWF-2058)
- The ComputeZoomAreaAndScaleFactor function has been changed. Temporary
disabling the codes which related to ComputeZoomAreaAndScaleFactor
(http://suprem.sec.samsung.net/jira/browse/TWF-2057)
- The codes which related to gzip_string (tools/grit/grit/node/include.py)
makes the build problem. Temporary disabling the codes
(http://suprem.sec.samsung.net/jira/browse/TWF-2059)
- APIs related geolocation were extracted from ContentBrowserClient to
content::GeolocationDelegate. So we should implement it
(http://suprem.sec.samsung.net/jira/browse/TWF-2084)
- The SelectClosestWord function has been changed. Temporary disabling
the codes which related to SelectClosestWord function
(http://suprem.sec.samsung.net/jira/browse/TWF-2122)
4. Verification
- This patch supports to the desktop and mobile(Tizen 3.0 32bit TM1) and
tv(Tizen 3.0 32bit Odroid) profiles
- There is no build problem in all target.
- Basic rendering is fine on desktop and mobile.
Bug: http://suprem.sec.samsung.net/jira/browse/TWF-1995
Change-Id: Ife6029cd18d471402b789cf4abf09f6eae185da7
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
Signed-off-by: Suchit Agrawal <a.suchit@samsung.com>
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
Gajendra N [Wed, 18 May 2016 12:49:14 +0000 (18:19 +0530)]
[M51_2704] Chromium upversion to m51_2704 branch.
This patch deals with the following changes:
1. Change in parameters of RunJavaScriptDialog() & RunBeforeUnloadDialog() functions
of JavaScriptDialogManagerEfl.
2. WebGraphicsContext3DCommandBufferImpl constructor takes additional parameters
gfx::PreferIntegratedGpu, share_resources & automatic_flushes. Also surface_id (int)
is changed to SurfaceHandle.
3. Use gpu::gles2::ContextCreationAttribHelper in place of blink::WebGraphicsContext3D
::Attributes and initialize their members accordingly.
4. GpuChannelHost is moved from 'content' to 'gpu' namespace.
5. Context type BROWSER_COMPOSITOR_ONSCREEN_CONTEXT renamed to
DISPLAY_COMPOSITOR_ONSCREEN_CONTEXT.
6. CommandBufferProxyImpl is moved from 'content' to 'gpu' namespace.
7. MovePluginWindows() of RenderWidgetHostView is removed is upstream.
8. ViewHostMsg_TextInputState_Params is renamed to TextInputState.
9. switches::kTabCaptureDownscaleQuality & switches::kTabCaptureUpscaleQuality are
removed, hence SCALER_QUALITY is not dependant on these switches.
10. Change in parameter types for HandleNewWindowRequest() & HandleNewWebContentsCreate()
functions due to scoped_refptr changes.
11. webplugin_delegate_impl.h is removed in upstream, hence remove related files from efl.
12. GpuChildThreadEfl constructor takes addtional parameter gpu::GpuPreferences.
13. Use switches::IsRunLayoutTestSwitchPresent() in place of HasSwitch(switches::kRunLayoutTest)
as switches::kRunLayoutTest is no longer available.
14. media/ozone/media_ozone_platform.h is removed, hence remove related files from efl.
15. PermissionStatus is moved from content to blink::mojom:: namespace.
16. Include base/macros.h for DISALLOW_COPY_AND_ASSIGN macro.
17. Initialize BrowserContext with DIR_USER_DATA path before creation of RequestContext.
18. Overloaded CreateRequestContext() is replaced with CreateRequestContextForStoragePartition().
19. CookieStore & CookieMonster are no more ReferenceCounted hence use scoped_ptr and std::move
during assignment and parameter passing.
20. NavigationController::ReloadIgnoringCache() is renamed to ReloadBypassingCache().
21. RenderProcessObserver::WebKitInitialized() is removed in upstream.
22. blink::WebFrameContentDumper::dumpFrameTreeAsText() is renamed to dumpWebViewAsText() which
accepts WebView object instead of WebFrame.
23. setBaseBackgroundColor() is moved to WebFrameWidget.
24. 'network_delegate' member is removed from NetworkSessionParams.
25. Change in path for headers : gl_helper.h, command_buffer_proxy_impl.h, resize_lock.h,
gpu_messages.h, gpu_host_messages.h, delegated_frame_host.h & permission_status.mojom.h
26. Version change from 50.2661 to 51.2704 in build and packaging related files.
AddExtraPluginDir() is removed from PluginListi class. The code under EWebContext::
AddExtraPluginDir() is disabled under EWK_BRINUP flag. This will be fixed by WebView team.
JIRA issue raised at http://suprem.sec.samsung.net/jira/browse/TWF-1132
Bug : http://suprem.sec.samsung.net/jira/browse/TWF-1060
Change-Id: I588d9c25c051dc72d76b7e6aa82d4a0f15ff01a2
Signed-off-by: Suchit Agrawal <a.suchit@samsung.com>
Chandan Padhi [Wed, 20 Apr 2016 09:29:58 +0000 (14:59 +0530)]
[M50_2661] Delegated Rendering using Surfaces and Surface Layer.
Delegated rendering was implemented with DelegatedRendererLayer.
DelegatedRendererLayer has been removed in M50 and Surfaces and
Surface Layer have been enabled by default. Due to this, rendering
failed and black screen was observed. With this commit, we adapt
delegated rendering accordingly.
This commit also removes dead code under TIZEN_DISABLE_GPU_THREAD.
Together with: Idd982f5731c82eb1fa689cc631643f157f67c957
Bug: http://suprem.sec.samsung.net/jira/browse/TWF-893
Reviewed by: dhyuna.ko, sm.venugopal, sns.park, venu.musham
Change-Id: Ic181ab6fe0bb691cc18e167c77169753c3f83b3c
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
Karthick M [Wed, 30 Mar 2016 12:37:37 +0000 (18:07 +0530)]
fixup! [MM] Webaudio with CAPI port.
Chromium upstream removed audio_decoder_android.h [1], which is however
needed in audio_decoder_efl.h. This CL addresses build issue due to the
missing header file.
[1] https://codereview.chromium.org/
1565623002
Bug: http://suprem.sec.samsung.net/jira/browse/TWF-610
Reviewed by: k2.nagaraju, msu.koo, sm.venugopal, sns.park
Change-Id: I916e7589f1dccec08b0590a9ff12b70c7c209dae
Signed-off-by: Karthick M <karthick.m1@samsung.com>
Gajendra N [Fri, 22 Apr 2016 06:20:24 +0000 (11:50 +0530)]
[M50_2661] Chromium upversion to m50_2661 branch.
This patch deals with the following changes:
1. Removing libvpx.diff patch from build/patches folder, as it is no longer needed.
2. context_factory_efl.cc file will be removed in another patch. To avoid build
break, part of the code is commented under EWK_BRINGUP flag (TWF-893).
3. gfx::Screen::GetNativeScreen() is renamed to GetScreen().
4. RenderWidgetHostImpl::ImeSetComposition() expects additional parameter gfx::Range.
5. delegated_frame_{provider, resource_collection}.h headers are removed.
6. FrameTree::ForEach() is removed, replace with a loop iterating over
FrameTree::Nodes().
7. RenderProcessHostImpl::ResumeRequestsForView() is replaced with
ResourceDispatcherHostImpl::ResumeBlockedRequestsForRouteFromUI().
8. ViewMsg_Resize is renamed to ResizeParams.
9. Set proper dependency path for 'mojo_cpp_bindings'.
10. gpu_messages.h is renamed to gpu_host_messages.h.
11. content::GpuProcess() constructor takes a parameter base::ThreadPriority.
12. |rtc_use_h264| is set to 0, without which link fails due to missing symbol.
13. rjsmin.py is located under third_party/WebKit/Source/devtools/scripts
directory, hence set proper import path in make-file-arrays.py.
14. LayoutTheme::theme is already defined in Blink, which calls nativeTheme()
on a condition, hence define nativeTheme() in RenderThemeChromiumTizen.cpp.
15. ResourceBundle::GetNativeImageNamed() takes only one parameter.
16. ui/gfx/screen_type_delegate.h is removed and SetScreenInstance() takes
only one parameter of type ui::Screen.
17. Add 'ozone_common' target dependency required by ozone_efl.
18. GetEGLSurfaceProperties() is removed and GLSurfaceEGL uses default configuration
attributes, which are same as it was returned by SurfaceFactoryEfl.
19. LoadAccessTokensCallbackType & AccessTokenSet are renamed to
LoadAccessTokensCallback & AccessTokenMap respectively, in AccessTokenStore.
20. "PERMISSION_STATUS_" prefix is trimmed on members of PermissionStatenum members.
21. Changes in prototype of DisplayNotification() & DisplayPersistentNotification()
functions of NotificationControllerEfl.
22. user_gesture parameter of PermissionManager::RequestPermission{s}() is removed.
23. base/prefs/ is moved to components/prefs/ folder.
24. net/base/net_util.h is removed.
25. GetCookieMonster() is removed, replace with GetCookieStore() at necessary
places.
26. MemoryPressureController::GetInstance() is moved to MemoryPressureControllerImpl.
27. Refactor EWebView::Suspend() & Resume() based on changes with respect to
ResourceDispatcherHost.
28. WebContents::ExitFullScreen() takes a bool parameter 'will_cause_resize'.
29. Change in first parameter type from void* to size* for HttpResponseHeaders::
EnumerateHeaderLines()
30 Use blink::WebStringToGURL() to instantiate GURL object from WebString.
31. typedef PlatformCanvas is removed from skia, hence use SkCanvas directly.
32. WebFrame::contentHasText() is removed and reimplemented as WebFrameContentDumper::
dumpFrameTreeAsText().
33. Cast CookieStore to CookieMonster to call SetPersistentCookies() and
SetCookieableSchemes() APIs.
34. Version change from 49.2623 to 50.2661 in build and packaging related files.
Debug build is not working because libchromium-ewk.so size is exceeding 2 GB during linking
time. Temporarily disabling debug build with 'nodebug 1' to avoid the error.
JIRA issue raised at http://suprem.sec.samsung.net/jira/browse/TWF-891
With M50 rebase, we face black screen issue due to some changes in Delegated Rendering,
which will be fixed separately.
JIRA issue raised at http://suprem.sec.samsung.net/jira/browse/TWF-893
'blink_platform' requires utrie2.h which is not supported by system icu yet. To avoid build
break, temporarily using Chromium's third_party/icu by setting 'use_system_icu=0'.
JIRA issue raised at http://suprem.sec.samsung.net/jira/browse/TWF-967
Bug : http://suprem.sec.samsung.net/jira/browse/TWF-728
Change-Id: I8994f376a599ede9045c730cb1c1ce340311deb1
Signed-off-by: Gajendra N <gajendra.n@samsung.com>
Youngcheol Kang [Mon, 22 Feb 2016 11:05:07 +0000 (20:05 +0900)]
Support the TM1 target in mobile profile
This patch cherry-picked from beta/m47 branch.
(http://165.213.202.130/gerrit/#/c/96218/)
We decided to support only TM1, not emulator profile in dev branch,
so the patch which related to emulator was removed in this patch.
[Build Command]
$ ./tizen_src/build/build_mobile.sh -P tzmb_v3.0_TM1 -A armv7l
Bug: http://165.213.149.170/jira/browse/TWF-170
Reviewed by: g.czajkowski, jh11.kwon, sns.park
Change-Id: Ie9b679155555c686ee0c943dc882841b9573ba67
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
Gajendra N [Thu, 25 Feb 2016 13:02:15 +0000 (18:32 +0530)]
[M49_2623] Chromium upversion to m49_2623 branch.
This patch deals with following changes:
1. Set gcc_version gyp variable.
2. base/basictypes.h is removed in upstream.
Replace (u)int{8,16,32,64} with (u)int{8,16,32,64}_t.
3. SingleThreadTaskGraphRunner is now part of upstream available under
cc namespace.
4. scoped_ptr .Pass() is deprecated, use std::move instead.
5. Use GenerateSyncToken() to generate a new SyncToken.
6. ScopedVector<> to std::vector<scoped_ptr<>> conversion is not possible.
Replace when there is mismatch in parameter passing or assignment.
7. COMPILE_ASSERT macro is removed from base/macros.h.
Use static_assert instead.
8. WebPageSerializer has been removed from blink.
Commented part of code that calls it.
9. Changes in number and type of arguments for many API's.
10. Indentation fixes by 'git cl format' for the files that are modified in
this commit only.
Bug: http://165.213.149.170/jira/browse/TWF-345
Change-Id: Ice87e1b73430ba3d269f5632b73c9f14e8fef60b
Signed-off-by: Gajendra N <gajendra.n@samsung.com>
Youngsoo Choi [Mon, 22 Feb 2016 04:09:59 +0000 (13:09 +0900)]
[tizen] Disable a cflag '-Wno-format' for global option change
This patch (http://165.213.202.130/gerrit/#/c/109103/) was cherry-picked
from the m47/beta.
The build option '-Wall', '-Wformat, and '-Wformat-security' have been used as
tizen global options. Recently, tizen platform team decided to remove a option
'-Wformat' which is already included in '-Wall'.
Due to the chnage, following build break happens from chromium-efl build.
>> cc1: error: -Wformat-security ignored without -Wformat [-Werror=format-security]
The build break is caused by a cflag '-Wno-format', defined in 'build/common.gypi'.
So, this CL tries to ignore the cflag for tizen port.
Bug: http://10.113.136.204/jira/i#browse/TWF-338
Change-Id: I2e381eef38083bd50f67a24fcd3439650b95fccd
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
Grzegorz Czajkowski [Thu, 17 Dec 2015 09:49:02 +0000 (10:49 +0100)]
fixup! Implement ewk_settings_auto_fitting_set
Auto fitting (aka Page Overview) changes the scale factor to adjust the content
width to be equal the device width. This feature is by default turned on for
mobile builds and what's more both Reference Browser and ubrowser enable it
in runtime, optionally adding the user possibility to switch it off via UI.
Disabling this feature in runtime results in not reseting the scale factor,
usually from ~0.4f to 1.0f. This causes bad user experience as the scale is
small but user is able to scroll the (empty) content.
Android exposes WebSettings::setLoadWithOverviewMode(boolean overview) [1] which
in Blink resets the scale when auto fit gets disabled. See
adjustForAndroidWebViewQuirks() for details.
This commit sets setLoadWithOverviewMode along with
setShrinksViewportContentToFit when ewk_settings_auto_fitting_set is called.
Not all utc_blink_ewk_settings_scripts_* tests has been fixed. This patch
fixes them, change comments and test names accordingly to their behaviour.
This is squash of two commits from beta/m47 branch:
- http://web.sec.samsung.net/gerrit/#/c/99946/
Reviewed by: sns.park
Change-Id: I00745a5b1de2252c9259fe9d5c21062a40f62c6d
Author: Grzegorz Czajkowski<g.czajkowski@samsung.com>
- http://web.sec.samsung.net/gerrit/#/c/102256/
Reviewed by: sns.park
Change-Id: I576a465d8ac5702e446df95d210707158c5af36c
Author: Marcin Niesluchowski<m.niesluchow@samsung.com>
[1] http://developer.android.com/reference/android/webkit/WebSettings.html
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15298
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15197
Reviewed by: sns.park
Change-Id: I00745a5b1de2252c9259fe9d5c21062a40f62c6d
Signed-off-by: Grzegorz Czajkowski <g.czajkowski@samsung.com>
Chandan Padhi [Thu, 7 Jan 2016 10:39:54 +0000 (16:09 +0530)]
[M48] Enable sync point related code for delegated renderer.
In M48, sync_point has been replaced by gpu::SyncToken in
GLFrameData class.
This commit adapts the same on chromium-efl side as well.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15389
Reviewed by: dhyuna.ko, g.czajkowski, sns.park, venu.musham
Change-Id: Id7b86e28e88fe17794dbcb195827624f4da8216d
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
Youngcheol Kang [Wed, 27 Jan 2016 04:46:28 +0000 (13:46 +0900)]
Enabled the use_system_re2 flag
Currently we use the re2 library on the internal chromium
(third_party/re2). This patch set the 'use_system_re2' flag
to use re2 library on system instead of internal chromium.
+--------------------+----------+----------+
| Size | Before | After |
+--------------------+----------+----------+
| chromium-ewk.so |
28390640 |
28305872 |
+--------------------+----------+----------+
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15517
Reviewed by: sns.park
Change-Id: I7ceb1112732efe5abdca924a95304dd2cc1e3f9d
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
SeungSeop Park [Mon, 1 Feb 2016 10:26:10 +0000 (19:26 +0900)]
Revert "Build with chromium libusb"
This reverts commit
6f9a252072c9303b827a78c8b09c3bb13917e88e.
Now Tizen 3.0 has libusb 1.0.19, which is newer than chromium's 1.0.17.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15683
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=13185
Reviewed by: sns.park
Change-Id: If4bce3695fe8564edebe975cfdd427b672d07322
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
KangYong Park [Mon, 25 Jan 2016 08:07:20 +0000 (17:07 +0900)]
fixup! [M48_2564] Chromium upversion to m48_2564 branch
The arguments of |ContentRendererClient::GetNavigationErrorStrings|
changed in upstream (https://codereview.chromium.org/
1406303002).
This CL adapts the code accordingly.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15386
Reviewed by: a1.gomes, hyuki.kim, sns.park
Change-Id: I5ceee5ef9c9a3dbea8344e6ae097e394768a7474
Signed-off-by: KangYong Park <kaka.park@samsung.com>
y0.kim [Thu, 14 Jan 2016 05:47:15 +0000 (14:47 +0900)]
fixup! Rename AutofillManagerDelegateEfl to AutofillClientEfl.
Fixes warning:
> no return statement in function returning non-void [-Wreturn-type]
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15496
Reviewed by: sns.park
Change-Id: I444e7fab4f2bb419f5195b657efb1423e4444d61
Signed-off-by: y0.kim <y0.kim@samsung.com>
Antonio Gomes [Thu, 28 Jan 2016 17:41:25 +0000 (13:41 -0400)]
Remove two bogus asserts in GpuChildThreadEfl::OnInitialize.
GpuChildThreadEfl's main goal is to override GpuChannelManager's
shared context and mailbox manager by the ones created by Efl
shared evas gl context.
In order to do that, GpuChildThreadEfl overrides GpuChildThread
and reimplements its OnInitialize method.
OnInitialize first calls GpuChilfThread::OnInitialize, where
gpu_channel_manager_ is instantiated, performs the overrides.
Problem is that by the time GpuChildThread::OnInitialize is executed
and GpuChannelManager is instantiated, its ctor also instantiates
share_group_ and mailbox_manager_ class member variables.
Hence, both DCHECKs (being removed by this patch) fail.
Patch removes them because they check for logically invalid
conditions.
Original beta/m47 patch: http://165.213.202.130/gerrit/#/c/105899/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15731
Reviewed by: g.czajkowski, sns.park
Change-Id: Ib6d8927f189fdb3595d7da528adebf60d0f2d562
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Thu, 28 Jan 2016 16:45:29 +0000 (12:45 -0400)]
fixup! Unify ContentMainDelegateEfl and WebProcessContentMainDelegateEfl
ui::ResourceBundle::InitSharedInstanceWithPakPath is now called by
ContentMainDelegateEfl for both browser and renderer processes.
No need to explicit call it in EwkGlobalData::GetInstance anymore.
It fixes the following ASSERT failure:
#0 0x7fcf2f7795bf [0128/115917:FATAL:resource_bundle.cc(602)]
Check failed: g_shared_instance_ == NULL. ResourceBundle initialized twice
#0 0x7f4d3b6fd5bf base::debug::StackTrace::StackTrace()
#1 0x7f4d3b73eeb9 logging::LogMessage::~LogMessage()
#2 0x7f4d3e3b6697 ui::ResourceBundle::InitSharedInstance()
#3 0x7f4d3e3b4258 ui::ResourceBundle::InitSharedInstanceWithPakPath()
#4 0x7f4d3b12945b EwkGlobalData::GetInstance()
#5 0x7f4d3b0f8d6d EWebContext::EWebContext()
#6 0x7f4d3b0f8be1 EWebContext::EWebContext()
#7 0x7f4d3b19fefb Ewk_Context::Ewk_Context()
#8 0x7f4d3b19fd3d Ewk_Context::Create()
#9 0x7f4d3b19fc02 Ewk_Context::DefaultContext()
#10 0x7f4d3b1a9ebe ewk_context_default_get
#11 0x000000407f74 Browser::Browser()
#12 0x00000040c85f app_create()
#13 0x00000040cb7e main
#14 0x7f4d38a5fec5 __libc_start_main
#15 0x000000407c89 <unknown>
Original beta/m47 patch: http://165.213.202.130/gerrit/#/c/105894/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14350
Reviewed by: g.czajkowski
Change-Id: I7b1c9c1ba509a521c0b7c16ca534bb3579ff7032
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Wed, 27 Jan 2016 05:16:12 +0000 (01:16 -0400)]
Retire EwkViewMsg_DidFailLoadWithError
Chromium has two messages renderer used to notify browser of a load fail:
FrameHostMsg_DidFailLoadWithError and
FrameHostMsg_DidCommitProvisionalLoad.
Although they are both sent to browser, their hooks are not actually
used by chromium-efl. Instead chromium-efl has a custom message for
its own use: EwkViewMsg_DidFailLoadWithError.
Patch removes EwkViewMsg_DidFailLoadWithError and its plumbing logic,
and use the hooks provided by the former two messages.
Previous stack:
- RenderFrameImpl::didFailLoad
- RenderViewObserverEfl::DidFailLoad
- (IPC - EwkViewMsg_DidFailLoadWithError)
- WebViewBrowserMessageFilterPrivate::OnReceivedDidFailLoadWithError
- EWebView::InvokeLoadError
The stack is:
- RenderFrameImpl::didFailLoad
- (IPC - FrameHostMsg_DidFailLoadWithError)
- RenderFrameHostImpl::OnMessageReceived
- RenderFrameHostImpl::OnDidFailLoadWithError
- (..)
- WebContentsImpl::DidFailLoadWithError
- WebContentsDelegateEfl::DidFailLoad
- EWebView::InvokeLoadError
Original beta/m47 patch: http://165.213.202.130/gerrit/#/c/105526/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15697
Reviewed by: g.czajkowski
Change-Id: I680fe89af89bf98181cf02a47e585743f185a994
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Mon, 25 Jan 2016 18:52:50 +0000 (14:52 -0400)]
Reimplement EWebContext::NotifyLowMemory method.
MemoryPurger class is a fork of the obsolete files
chrome/browser/memory_purger.cc/h, removed in [1].
In [2], it is advised for callers of this API to use
RenderThreadImpl::OnMemoryPressure instead.
Patch provides an implementation of EWebContext::NotifyLowMemory
based on MemoryPressureController class that calls out to
MemoryPressureListener and down to RenderThreadImpl::OnMemoryPressure.
ewk/efl_integration/memory_purger.cc/h are also removed, as well
as message EwkViewMsg_PurgeMemory and its associated handler
RenderProcessObserverEfl::OnPurgeMemory.
Experiments with this API shows that it frees up to 40Mb of RAM
when browsing facebook.com.
[1] https://codereview.chromium.org/
192573010
[2] https://code.google.com/p/chromium/issues/detail?id=350455#c13
Original beta/m47 patch: http://165.213.202.130/gerrit/#/c/105433/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=4822
Reviewed by: g.czajkowski
Change-Id: I437046a02b4486422dfd2913b1c2b51e405e8e2c
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
SeungSeop Park [Thu, 21 Jan 2016 01:14:18 +0000 (10:14 +0900)]
fixup! Enforce "treat warning as error" policy for chromium-ewk
Although officially CT (and buildbot) support only Tizen 3.0,
there are unofficial activities targeted for old Tizen version.
So, let's restrict the warning/error policy only to Tizen 3.0 for now.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15612
Reviewed by: g.czajkowski, sm.venugopal
Change-Id: Ieac5a206a8579be1875df35d4a0dae28f7c1a505
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
SangHyup Lee [Tue, 26 Jan 2016 07:04:47 +0000 (16:04 +0900)]
Fix crash in |SelectionControllerEfl::ClearSelectionViaEWebView|.
If webview evas object deleted by application, destructor of RWHVEfl will be
called. And when RWHVEfl deleted, destructor of SelectionControllerEfl also
will be called because RWHVEfl has a scoped_ptr<SelectionControllerEfl>.
SelectionController try to clear all selection by call |ClearSelectionViaEWebView|
in the destructor. In this case, we should check RWHV and WebContents because
these can be deleted already.
Original beta/m47 patch: http://165.213.202.130/gerrit/#/c/105349
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15677
Reviewed by: a1.gomes, sns.park
Change-Id: I4f0cb35b2d68a62408be81c4300e93741f3eae6c
Signed-off-by: SangHyup Lee <sh53.lee@samsung.com>
SangHyup Lee [Thu, 28 Jan 2016 05:40:13 +0000 (14:40 +0900)]
fixup! [M48_2564] Chromium upversion to m48_2564 branch
RenderViewHost was separated from RenderWidgetHost after [1].
So we need to get RWHI from RVHI by |GetWidget()| to access
RWHI's member functions.
[1] https://codereview.chromium.org/
1413643002
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15384
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15385
Reviewed by: a1.gomes, sns.park
Change-Id: I51b483144524172ffb428db180d94ffc162d5eec
Signed-off-by: SangHyup Lee <sh53.lee@samsung.com>
Grzegorz Ludwikowski [Wed, 27 Jan 2016 10:16:22 +0000 (11:16 +0100)]
Remove warning from |NavigationPolicyHandlerEfl::DownloadNavigation|
|NavigationPolicyHandlerEfl::DownloadNavigation| contains '#pragma
message', which results in a warning during compilation. The message
warns about code commented out because of change in s-chromium around
M37.
The code was commented out so long ago, that 'git blame' wouldn't point
to actual change. Despite this, no bug was filed for broken
functionallity. This suggests that the code wasn't used anyway.
The code is called through ewk_policy_decision_download, when used on
Ewk_Policy_Decision object of policy type POLICY_NAVIGATION (recevied
from "policy,navigation,decide" callback). Assuming that we call
ewk_policy_decision_download only for Ewk_Policy_Decision objects which
return EWK_POLICY_DECISION_DOWNLOAD decision type from
ewk_policy_decision_type_get, the code isn't used, because objects with
this decision type don't appear in the callback.
Both browser and crosswalk do not use ewk_policy_decision_download API
at the moment.
This change removes the compile time warning, and adds
'NOTIMPLEMENTED()' macro instead, so we will see at runtime that not
implemented code path is used.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15638
Reviewed by: sns.park
Change-Id: Ie07171f41c8c45ba70699ed42b8956b5de0deece
Signed-off-by: Grzegorz Ludwikowski <g.ludwikowsk@samsung.com>
Antonio Gomes [Wed, 20 Jan 2016 23:13:59 +0000 (19:13 -0400)]
Unify ContentMainDelegateEfl and WebProcessContentMainDelegateEfl
Both classes are EWK delegates to the main process kick off routines.
It turns out though that chromium-efl is the only content-based
app that has two different delegates for browser and renderer processes.
Chrome (ChromeMainDelegate) and ContentShell (ShellMainDelegate) for
instance have this class unified.
That said, patch merges WebProcessContentMainDelegateEfl into
ContentMainDelegateEfl, simplifying our logic.
WebProcessContentMainDelegateEfl's main method PreSandboxStartup
was merged into ContentMainDelegateEfl counterpart.
Original beta/m47 patch:
- http://165.213.202.130/gerrit/#/c/105265/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14350
Reviewed by: djmix.kim, sns.park
Change-Id: Idff02ac417b8247d0fd498db7d99cf3a60b60fb7
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Antonio Gomes [Wed, 13 Jan 2016 20:51:26 +0000 (16:51 -0400)]
RenderProcessHostImpl dtor is not executed.
Since forcibly shutdown our UI message loop, it is possible
that important pending Tasks are simply not executed.
Among these, the destruction of RenderProcessHostImpl class
(scheduled with DeleteSoon) performs various process um-plumbing
stuff.
Particularly, RenderProcessHostImpl holds a strong references of lots
of other objects (e.g. URLRequestContextGetterEfl, RenderWidgetHelper,
ChildProcessLauncher, etc) and not performing the proper destruction
of them might result in crashes at shutdown or other undefined problems.
Patch gives the UI message loop a extra run ('till it gets idle)
to perform such pending tasks.
In order to implement this, it was required to unstub the
implementation of MessagePumpForUIEfl::Run, based on the logic
described in base/message_loop/message_pump.h, as well as in
message_pump_{default,glib}.cc
Callstack looks like:
- EwkGlobalData::~EwkGlobalData
- MessageLoop::RunUntilIdle
- RunLoop::RunUntilIdle
- RunLoop::Run
- MessageLoop::RunHandler
- MessagePumpForUIEfl::Run # idle state will break the loop
Original beta/m47 patches:
- http://165.213.202.130/gerrit/#/c/103446/
- http://165.213.202.130/gerrit/#/c/104287/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15557
Reviewed by: djmix.kim, sns.park
Change-Id: I2957a0a7fb942daf9bcbf74431d91acb8cff7902
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>
Leszek Syroka [Thu, 21 Jan 2016 15:00:01 +0000 (16:00 +0100)]
Remove warnings from MemoryPurger
Refactoring of MemoryPurger class. Warnings, unused includes and dead
code have been removed.
Erased code contained helper class calling non-existent memory purging
methods of ProxyService. MemomoryPurger was marked in previous commits
as obsolete https://code.google.com/p/chromium/issues/detail?id=350455.
MemoryPurger has not been removed from code base due to existing references
to its interface. Currently only EWebContext::NotifyLowMemory invokes
MemoryPurger methods.
Original beta/m47 patch: http://165.213.202.130/gerrit/#/c/104842/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=4822
Reviewed by: a1.gomes
Change-Id: Ib77e5d2983c38fa1281f37902370d2d1ad387748
Signed-off-by: Leszek Syroka <l.syroka@partner.samsung.com>
(cherry picked from commit
f776a5291d3af1bea61497155cb3724cf0ef3aa9)
Youngsoo Choi [Mon, 25 Jan 2016 12:42:24 +0000 (21:42 +0900)]
[tizen v3.0] Remove local build.conf files for better mainternance
Disabling rpmlint is for improving build time around 15 minutes.
The previous way to achieve the goal was to keep a custom build.conf inside
source tree, which was made by cloning & modifiying specific platform version's.
Refer to: http://165.213.202.130/gerrit/#/c/75855/
But problem in the approach is that
content of build.conf can be suddenly changed from gbs repo, such as
build macros, options, and package list, which can cause build break.
Whenever build.conf is changed from gbs repo, custom build.conf has to be also
fixed in source tree and uploaded to git repo.
To prevent the burden, build.conf needs to be automatically sync'ed with
the latest one from gbs repo.
So, this CL downloads build.conf from remote repo as part of build procedure,
and optionally modifies the content of the file if user does not wants to
run rpmlint (which is default behaviour).
- Disable rpmlint : build_{target}.sh
- Enable rpmlint : build_{target}.sh --rpmlint
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15671
Reviewed by: sns.park
Change-Id: I6ae5a468748d476db62d877d29080ef4239a7906
Signed-off-by: Youngsoo Choi <kenshin.choi@samsung.com>
SeungSeop Park [Mon, 25 Jan 2016 08:33:20 +0000 (17:33 +0900)]
Enforce "treat warning as error" policy for ubrowser
SSIA.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15612
Reviewed by: sns.park
Change-Id: I4623cd66a8d9213fa3b70c594ee7f5a0eec422b9
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
Leszek Syroka [Tue, 26 Jan 2016 11:40:14 +0000 (12:40 +0100)]
Fix compiler warnings for uBrowser
Warnings caused by calling deprecated API have been erased. Switching
tab to private mode in uBrowser have been removed because it is deprecated
and will not be supported in further versions of Chromium.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15664
Reviewed by: sns.park
Change-Id: Ideb1cb9700e951d2e759494eed99ecdde6c32bd6
Signed-off-by: Leszek Syroka <l.syroka@partner.samsung.com>
SeungSeop Park [Mon, 25 Jan 2016 08:26:11 +0000 (17:26 +0900)]
Enforce "treat warning as error" policy for efl_webview_app and mini_browser
SSIA.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15612
Reviewed by: sns.park
Change-Id: Ic1276552cdae91ce1054b9cb8a03dbc0cfc0a472
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
Youngcheol Kang [Wed, 30 Dec 2015 11:28:40 +0000 (20:28 +0900)]
fixup! Replacing deprecated 'efl-assist' with 'efl-extension'
This patch changes the codes in order to use the 'efl-extension'
in both Mobile and TV profile.
Reviewed by: g.czajkowski, sns.park
Change-Id: I4db84b345ebb7360459695944342412fe2145c1f
Signed-off-by: Youngcheol Kang <ychul.kang@samsung.com>
ws29.jung [Tue, 19 Jan 2016 10:28:50 +0000 (19:28 +0900)]
fixup! [MM] Introduce a switch for zero-copy support from multimedia
Previously, build break happens from missing libyuv headers
when |TIZEN_MULTIMEDIA_ZEROCOPY_SUPPORT| is turned off.
This fix includes changes:
- added missing libyuv headers.
- replaced "base::Process::Current().Handle()" with
"base::GetCurrentProcessHandle()"
Reviewed by: sns.park
Change-Id: I1b8f54297ac52366d8274ccf61fcbf9d1979ff94
Signed-off-by: ws29.jung <ws29.jung@samsung.com>
y0.kim [Thu, 21 Jan 2016 11:45:00 +0000 (20:45 +0900)]
Fix compiler warnings in ewk/efl_webview_app/app.c
Fixes following warnings in app.c:
> 'prompt_entry' defined but not used [-Wunused-variable]
> '__scroll_down_cb' defined but not used [-Wunused-function]
> '__scroll_up_cb' defined but not used [-Wunused-function]
> '__scroll_left_cb' defined but not used [-Wunused-function]
> '__scroll_right_cb' defined but not used [-Wunused-function]
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15559
Reviewed by: a1.gomes, sns.park
Change-Id: Ic9c7ae0109e3b2cb75ecdab0b05f644590ce28d3
Signed-off-by: y0.kim <y0.kim@samsung.com>
y0.kim [Thu, 21 Jan 2016 09:37:45 +0000 (18:37 +0900)]
Fix warnings in ewk/efl_webview_app/mini_browser.c
This patch fix 9 warnings in mini_browser.c
> passing argument 1 of 'ewk_context_vibration_client_callbacks_set'
from incompatible pointer type
> passing argument 3 of 'evas_object_smart_callback_add' from
incompatible pointer type
> initialization from incompatible pointer type
> (near initialization for 'values[0].strp')
> assignment from incompatible pointer typen
> unused variable 'permissionRequest' [-Wunused-variable]
> implicit declaration of function 'ewk_start_tracing'
[-Wimplicit-function-declaration]
> implicit declaration of function 'ewk_stop_tracing'
[-Wimplicit-function-declaration]
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15558
Reviewed by: sns.park
Change-Id: Iac5fe427fbde3b2093cce95c4651ac6c8261ee31
Signed-off-by: y0.kim <y0.kim@samsung.com>
SeungSeop Park [Fri, 22 Jan 2016 01:56:40 +0000 (10:56 +0900)]
fixup! [M48_2564] Chromium upversion to m48_2564 branch
Fix warning(error):
> .../ewk/efl_integration/renderer/render_view_observer_efl.cc:357:24:\
> error: unused variable 'failed_request' [-Werror=unused-variable]
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15496
Reviewed by: msu.koo, sm.venugopal, sns.park
Change-Id: Ia94f8d5ca56b52ed08749de9808ef7159306d87f
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
Suneel Kota [Wed, 20 Jan 2016 13:49:29 +0000 (19:19 +0530)]
fixup! [wayland][tizen v3.0] Initial support for TV and mobile
This CL fixes following warning(error) in building for tizen 3.0 hawkp:
> error: unused variable 'DownloadPath' [-Werror=unused-variable]
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15496
Reviewed by: sns.park
Change-Id: Ib4e45b2c847c0062f55c4445ec4caf87c34a225b
Signed-off-by: Suneel Kota <suneel.kota@samsung.com>
samyukta.c [Tue, 12 Jan 2016 04:10:55 +0000 (09:40 +0530)]
[MM] Remove |format| as argument
Removed the unused argument |format| and changed the order of resolution
parameters |width|, |height|.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15536
Reviewed by: k2.nagaraju, msu.koo, sm.venugopal
Change-Id: Ic7a9723e39d724f6a78cd777565b9e851cb5a411
Signed-off-by: samyukta.c <samyukta.c@samsung.com>
y0.kim [Thu, 7 Jan 2016 07:13:37 +0000 (16:13 +0900)]
fixup! Merge efl-port POC code into tizen_src
Below log is emitted during the build, which clutters the build log:
> #pramga message "TODO: Add proper delegate class for ..."
Patch demotes it to plain code comment for clean build log.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15496
Reviewed by: g.czajkowski, sns.park
Change-Id: Icdf3a4ea1b148dfdc133a79310738898afce6449
Signed-off-by: y0.kim <y0.kim@samsung.com>
Chandan Padhi [Tue, 19 Jan 2016 13:28:12 +0000 (18:58 +0530)]
White screen at content shell launch.
White screen is caused due to failure of gl draw calls.
GPU was blacklisted for content_shell due to which context
provider creation failed. This resulted in creation of
software resources instead of GL resources, for which GL
draw calls stand invalid.
This commit ignores gpu blacklist for content_shell similar
to efl_webview_app and ubrowser on desktop.
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15626
Reviewed by: a1.gomes, g.czajkowski
Change-Id: I53444ce02e6005a7a5021dbd5e540dde9b0a382b
Signed-off-by: Chandan Padhi <c.padhi@samsung.com>
SeungSeop Park [Fri, 15 Jan 2016 00:25:46 +0000 (09:25 +0900)]
Enforce "treat warning as error" policy for chromium-ewk
1. Remove top-level "-Dwerror=" gyp define from gyp_chromiumefl.sh.
This will let chromium targets use their own policy regarding whether to treat
warnings as error or ignore them.
2. Add '-Werror' cflag to chromium-ewk target.
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15612
Reviewed by: a1.gomes, g.czajkowski
Change-Id: Id2634bcb1958c53ddc53a4259404d3cf54047a66
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
SeungSeop Park [Tue, 19 Jan 2016 02:47:04 +0000 (11:47 +0900)]
fixup! Implement device display info and get device scale factor.
Fix following warning in tv build:
> 'int gfx::{anonymous}::GetDensityRange(int)' defined but not used \
> [-Werror=unused-function]
Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=15496
Reviewed by: a1.gomes, msu.koo, sns.park
Change-Id: I4f01bd642ec2117015bc502b136df9bb979b63f9
Signed-off-by: SeungSeop Park <sns.park@samsung.com>
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>