Title: [242055] trunk
Revision
242055
Author
[email protected]
Date
2019-02-25 13:48:24 -0800 (Mon, 25 Feb 2019)

Log Message

[WPE] Bump WPEBackend-fdo requirement to API version 1.0
https://bugs.webkit.org/show_bug.cgi?id=195001

Reviewed by Carlos Garcia Campos.

.:

* Source/cmake/FindWPEBackend-fdo.cmake: Use WPEBackend-fdo-1.0.
* Source/cmake/OptionsWPE.cmake: Ditto.

Source/WebKit:

API version 1.0 always includes the functionality previously guarded with
WPE_BACKEND_CHECK_VERSION(): remove the guards and always use the new functions
unconditionally.

* UIProcess/API/wpe/WPEView.cpp:
(WKWPE::View::View): Remove usage of WPE_BACKEND_CHECK_VERSION().
(WKWPE::m_backend): Ditto.
* UIProcess/API/wpe/qt/WPEQtViewBackend.cpp:
(WPEQtViewBackend::WPEQtViewBackend): Use libWPEBackend-fdo-1.0 as
library name, remove call to wpe_fdo_initialize_for_egl_display().
(WPEQtViewBackend::create): Call wpe_fdo_initialize_for_egl_display()
here, to bail out early in case initialization fails.
* UIProcess/glib/WebProcessPoolGLib.cpp:
(WebKit::WebProcessPool::platformInitializeWebProcess): Remove usage of
WPE_BACKEND_CHECK_VERSION().

Tools:

API version 1.0 always includes the functionality previously guarded with
WPE_BACKEND_CHECK_VERSION(): remove the guards and always use the new functions
unconditionally.

* TestWebKitAPI/glib/WebKitGLib/TestMain.h:
(Test::createWebViewBackend): Remove usage of WPE_BACKEND_CHECK_VERSION().
* TestWebKitAPI/glib/WebKitGLib/wpe/WebViewTestWPE.cpp:
(WebViewTest::showInWindow): Ditto.
(WebViewTest::hideView): Ditto.
* wpe/backends/HeadlessViewBackend.cpp:
(WPEToolingBackends::HeadlessViewBackend::HeadlessViewBackend): Ditto.
* wpe/backends/ViewBackend.cpp:
(WPEToolingBackends::ViewBackend::ViewBackend): Use libWPEBackend-fdo-1.0 as library name.
* wpe/backends/WindowViewBackend.cpp:
(WPEToolingBackends::WindowViewBackend::WindowViewBackend): Remove usage of
WPE_BACKEND_CHECK_VERSION().
* wpe/jhbuild.modules: Build a version WPEBackend-fdo with the updated API version.

Modified Paths

Diff

Modified: trunk/ChangeLog (242054 => 242055)


--- trunk/ChangeLog	2019-02-25 20:15:53 UTC (rev 242054)
+++ trunk/ChangeLog	2019-02-25 21:48:24 UTC (rev 242055)
@@ -1,3 +1,13 @@
+2019-02-25  Adrian Perez de Castro  <[email protected]>
+
+        [WPE] Bump WPEBackend-fdo requirement to API version 1.0
+        https://bugs.webkit.org/show_bug.cgi?id=195001
+
+        Reviewed by Carlos Garcia Campos.
+
+        * Source/cmake/FindWPEBackend-fdo.cmake: Use WPEBackend-fdo-1.0.
+        * Source/cmake/OptionsWPE.cmake: Ditto.
+
 2019-02-25  Miguel Gomez  <[email protected]>
 
         [WPE] Add support for holepunch using an external video player

Modified: trunk/Source/WebKit/ChangeLog (242054 => 242055)


--- trunk/Source/WebKit/ChangeLog	2019-02-25 20:15:53 UTC (rev 242054)
+++ trunk/Source/WebKit/ChangeLog	2019-02-25 21:48:24 UTC (rev 242055)
@@ -1,3 +1,26 @@
+2019-02-25  Adrian Perez de Castro  <[email protected]>
+
+        [WPE] Bump WPEBackend-fdo requirement to API version 1.0
+        https://bugs.webkit.org/show_bug.cgi?id=195001
+
+        Reviewed by Carlos Garcia Campos.
+
+        API version 1.0 always includes the functionality previously guarded with
+        WPE_BACKEND_CHECK_VERSION(): remove the guards and always use the new functions
+        unconditionally.
+
+        * UIProcess/API/wpe/WPEView.cpp:
+        (WKWPE::View::View): Remove usage of WPE_BACKEND_CHECK_VERSION().
+        (WKWPE::m_backend): Ditto.
+        * UIProcess/API/wpe/qt/WPEQtViewBackend.cpp:
+        (WPEQtViewBackend::WPEQtViewBackend): Use libWPEBackend-fdo-1.0 as
+        library name, remove call to wpe_fdo_initialize_for_egl_display().
+        (WPEQtViewBackend::create): Call wpe_fdo_initialize_for_egl_display()
+        here, to bail out early in case initialization fails.
+        * UIProcess/glib/WebProcessPoolGLib.cpp:
+        (WebKit::WebProcessPool::platformInitializeWebProcess): Remove usage of
+        WPE_BACKEND_CHECK_VERSION().
+
 2019-02-25  Per Arne Vollan  <[email protected]>
 
         A prewarmed Web process should have a process assertion.

Modified: trunk/Source/WebKit/UIProcess/API/wpe/WPEView.cpp (242054 => 242055)


--- trunk/Source/WebKit/UIProcess/API/wpe/WPEView.cpp	2019-02-25 20:15:53 UTC (rev 242054)
+++ trunk/Source/WebKit/UIProcess/API/wpe/WPEView.cpp	2019-02-25 21:48:24 UTC (rev 242055)
@@ -45,9 +45,7 @@
     : m_client(std::make_unique<API::ViewClient>())
     , m_pageClient(std::make_unique<PageClientImpl>(*this))
     , m_size { 800, 600 }
-#if !defined(WPE_BACKEND_CHECK_VERSION) || !WPE_BACKEND_CHECK_VERSION(1, 1, 0)
     , m_viewStateFlags { WebCore::ActivityState::WindowIsActive, WebCore::ActivityState::IsFocused, WebCore::ActivityState::IsVisible, WebCore::ActivityState::IsInWindow }
-#endif
     , m_backend(backend)
 {
     ASSERT(m_backend);
@@ -87,7 +85,6 @@
             auto& view = *reinterpret_cast<View*>(data);
             view.frameDisplayed();
         },
-#if defined(WPE_BACKEND_CHECK_VERSION) && WPE_BACKEND_CHECK_VERSION(1, 1, 0)
         // activity_state_changed
         [](void* data, uint32_t state)
         {
@@ -103,9 +100,6 @@
                 flags.add(WebCore::ActivityState::IsInWindow);
             view.setViewState(flags);
         },
-#else
-        nullptr,
-#endif
         // padding
         nullptr,
         nullptr,

Modified: trunk/Source/WebKit/UIProcess/API/wpe/qt/WPEQtViewBackend.cpp (242054 => 242055)


--- trunk/Source/WebKit/UIProcess/API/wpe/qt/WPEQtViewBackend.cpp	2019-02-25 20:15:53 UTC (rev 242054)
+++ trunk/Source/WebKit/UIProcess/API/wpe/qt/WPEQtViewBackend.cpp	2019-02-25 21:48:24 UTC (rev 242055)
@@ -37,7 +37,7 @@
 
     eglInitialize(eglDisplay, nullptr, nullptr);
 
-    if (!eglBindAPI(EGL_OPENGL_ES_API))
+    if (!eglBindAPI(EGL_OPENGL_ES_API) || !wpe_fdo_initialize_for_egl_display(eglDisplay))
         return nullptr;
 
     static const EGLint configAttributes[13] = {
@@ -74,12 +74,8 @@
     , m_view(view)
     , m_size(size)
 {
-#if defined(WPE_BACKEND_CHECK_VERSION) && WPE_BACKEND_CHECK_VERSION(0, 2, 0)
-    wpe_loader_init("libWPEBackend-fdo-0.1.so");
-#endif
+    wpe_loader_init("libWPEBackend-fdo-1.0.so");
 
-    wpe_fdo_initialize_for_egl_display(m_eglDisplay);
-
     imageTargetTexture2DOES = reinterpret_cast<PFNGLEGLIMAGETARGETTEXTURE2DOESPROC>(eglGetProcAddress("glEGLImageTargetTexture2DOES"));
 
     static const char* vertexShaderSource =
@@ -128,9 +124,7 @@
 
     m_exportable = wpe_view_backend_exportable_fdo_egl_create(&exportableClient, this, m_size.width(), m_size.height());
 
-#if defined(WPE_BACKEND_CHECK_VERSION) && WPE_BACKEND_CHECK_VERSION(1, 1, 0)
     wpe_view_backend_add_activity_state(backend(), wpe_view_activity_state_visible | wpe_view_activity_state_focused | wpe_view_activity_state_in_window);
-#endif
 
     m_surface.setFormat(context->format());
     m_surface.create();

Modified: trunk/Source/WebKit/UIProcess/glib/WebProcessPoolGLib.cpp (242054 => 242055)


--- trunk/Source/WebKit/UIProcess/glib/WebProcessPoolGLib.cpp	2019-02-25 20:15:53 UTC (rev 242054)
+++ trunk/Source/WebKit/UIProcess/glib/WebProcessPoolGLib.cpp	2019-02-25 21:48:24 UTC (rev 242055)
@@ -92,10 +92,8 @@
 {
 #if PLATFORM(WPE)
     parameters.hostClientFileDescriptor = wpe_renderer_host_create_client();
-#if defined(WPE_BACKEND_CHECK_VERSION) && WPE_BACKEND_CHECK_VERSION(0, 2, 0)
     parameters.implementationLibraryName = FileSystem::fileSystemRepresentation(wpe_loader_get_loaded_implementation_library_name());
 #endif
-#endif
 
     parameters.memoryCacheDisabled = m_memoryCacheDisabled || cacheModel() == CacheModel::DocumentViewer;
     parameters.proxySettings = m_networkProxySettings;

Modified: trunk/Source/cmake/FindWPEBackend-fdo.cmake (242054 => 242055)


--- trunk/Source/cmake/FindWPEBackend-fdo.cmake	2019-02-25 20:15:53 UTC (rev 242054)
+++ trunk/Source/cmake/FindWPEBackend-fdo.cmake	2019-02-25 21:48:24 UTC (rev 242055)
@@ -29,7 +29,7 @@
 # ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 find_package(PkgConfig)
-pkg_check_modules(PC_WPEBACKEND_FDO QUIET wpebackend-fdo-0.1)
+pkg_check_modules(PC_WPEBACKEND_FDO QUIET wpebackend-fdo-1.0)
 
 find_path(WPEBACKEND_FDO_INCLUDE_DIRS
     NAMES wpe/fdo.h
@@ -37,7 +37,7 @@
 )
 
 find_library(WPEBACKEND_FDO_LIBRARIES
-    NAMES WPEBackend-fdo-0.1
+    NAMES WPEBackend-fdo-1.0
     HINTS ${PC_WPEBACKEND_FDO_LIBDIR} ${PC_WPEBACKEND_FDO_LIBRARY_DIRS}
 )
 

Modified: trunk/Source/cmake/OptionsWPE.cmake (242054 => 242055)


--- trunk/Source/cmake/OptionsWPE.cmake	2019-02-25 20:15:53 UTC (rev 242054)
+++ trunk/Source/cmake/OptionsWPE.cmake	2019-02-25 21:48:24 UTC (rev 242055)
@@ -140,7 +140,7 @@
     find_package(Qt5 REQUIRED COMPONENTS Core Quick Gui)
     find_package(Qt5Test REQUIRED)
     find_package(PkgConfig)
-    pkg_check_modules(WPE_BACKEND_FDO REQUIRED wpebackend-fdo-0.1)
+    pkg_check_modules(WPE_BACKEND_FDO REQUIRED wpebackend-fdo-1.0)
 endif ()
 
 add_definitions(-DBUILDING_WPE__=1)

Modified: trunk/Tools/ChangeLog (242054 => 242055)


--- trunk/Tools/ChangeLog	2019-02-25 20:15:53 UTC (rev 242054)
+++ trunk/Tools/ChangeLog	2019-02-25 21:48:24 UTC (rev 242055)
@@ -1,3 +1,28 @@
+2019-02-25  Adrian Perez de Castro  <[email protected]>
+
+        [WPE] Bump WPEBackend-fdo requirement to API version 1.0
+        https://bugs.webkit.org/show_bug.cgi?id=195001
+
+        Reviewed by Carlos Garcia Campos.
+
+        API version 1.0 always includes the functionality previously guarded with
+        WPE_BACKEND_CHECK_VERSION(): remove the guards and always use the new functions
+        unconditionally.
+
+        * TestWebKitAPI/glib/WebKitGLib/TestMain.h:
+        (Test::createWebViewBackend): Remove usage of WPE_BACKEND_CHECK_VERSION().
+        * TestWebKitAPI/glib/WebKitGLib/wpe/WebViewTestWPE.cpp:
+        (WebViewTest::showInWindow): Ditto.
+        (WebViewTest::hideView): Ditto.
+        * wpe/backends/HeadlessViewBackend.cpp:
+        (WPEToolingBackends::HeadlessViewBackend::HeadlessViewBackend): Ditto.
+        * wpe/backends/ViewBackend.cpp:
+        (WPEToolingBackends::ViewBackend::ViewBackend): Use libWPEBackend-fdo-1.0 as library name.
+        * wpe/backends/WindowViewBackend.cpp:
+        (WPEToolingBackends::WindowViewBackend::WindowViewBackend): Remove usage of
+        WPE_BACKEND_CHECK_VERSION().
+        * wpe/jhbuild.modules: Build a version WPEBackend-fdo with the updated API version.
+
 2019-02-25  Thibault Saunier  <[email protected]>
 
         [Flatpak] Add support for flatpak > 1.1.2

Modified: trunk/Tools/TestWebKitAPI/glib/WebKitGLib/TestMain.h (242054 => 242055)


--- trunk/Tools/TestWebKitAPI/glib/WebKitGLib/TestMain.h	2019-02-25 20:15:53 UTC (rev 242054)
+++ trunk/Tools/TestWebKitAPI/glib/WebKitGLib/TestMain.h	2019-02-25 21:48:24 UTC (rev 242055)
@@ -143,10 +143,8 @@
     static WebKitWebViewBackend* createWebViewBackend()
     {
         auto* headlessBackend = new WPEToolingBackends::HeadlessViewBackend(800, 600);
-#if defined(WPE_BACKEND_CHECK_VERSION) && WPE_BACKEND_CHECK_VERSION(1, 1, 0)
         // Make the view initially hidden for consistency with GTK+ tests.
         wpe_view_backend_remove_activity_state(headlessBackend->backend(), wpe_view_activity_state_visible | wpe_view_activity_state_focused);
-#endif
         return webkit_web_view_backend_new(headlessBackend->backend(), [](gpointer userData) {
             delete static_cast<WPEToolingBackends::HeadlessViewBackend*>(userData);
         }, headlessBackend);

Modified: trunk/Tools/TestWebKitAPI/glib/WebKitGLib/wpe/WebViewTestWPE.cpp (242054 => 242055)


--- trunk/Tools/TestWebKitAPI/glib/WebKitGLib/wpe/WebViewTestWPE.cpp	2019-02-25 20:15:53 UTC (rev 242054)
+++ trunk/Tools/TestWebKitAPI/glib/WebKitGLib/wpe/WebViewTestWPE.cpp	2019-02-25 21:48:24 UTC (rev 242055)
@@ -43,18 +43,14 @@
 
 void WebViewTest::showInWindow()
 {
-#if defined(WPE_BACKEND_CHECK_VERSION) && WPE_BACKEND_CHECK_VERSION(1, 1, 0)
     auto* backend = webkit_web_view_backend_get_wpe_backend(webkit_web_view_get_backend(m_webView));
     wpe_view_backend_add_activity_state(backend, wpe_view_activity_state_visible | wpe_view_activity_state_in_window | wpe_view_activity_state_focused);
-#endif
 }
 
 void WebViewTest::hideView()
 {
-#if defined(WPE_BACKEND_CHECK_VERSION) && WPE_BACKEND_CHECK_VERSION(1, 1, 0)
     auto* backend = webkit_web_view_backend_get_wpe_backend(webkit_web_view_get_backend(m_webView));
     wpe_view_backend_remove_activity_state(backend, wpe_view_activity_state_visible | wpe_view_activity_state_focused);
-#endif
 }
 
 void WebViewTest::mouseMoveTo(int x, int y, unsigned mouseModifiers)

Modified: trunk/Tools/wpe/backends/HeadlessViewBackend.cpp (242054 => 242055)


--- trunk/Tools/wpe/backends/HeadlessViewBackend.cpp	2019-02-25 20:15:53 UTC (rev 242054)
+++ trunk/Tools/wpe/backends/HeadlessViewBackend.cpp	2019-02-25 21:48:24 UTC (rev 242055)
@@ -72,9 +72,7 @@
     if (!initialize())
         return;
 
-#if defined(WPE_BACKEND_CHECK_VERSION) && WPE_BACKEND_CHECK_VERSION(1, 1, 0)
     wpe_view_backend_add_activity_state(backend(), wpe_view_activity_state_visible | wpe_view_activity_state_focused | wpe_view_activity_state_in_window);
-#endif
 
     if (!eglMakeCurrent(m_eglDisplay, EGL_NO_SURFACE, EGL_NO_SURFACE, m_eglContext))
         return;

Modified: trunk/Tools/wpe/backends/ViewBackend.cpp (242054 => 242055)


--- trunk/Tools/wpe/backends/ViewBackend.cpp	2019-02-25 20:15:53 UTC (rev 242054)
+++ trunk/Tools/wpe/backends/ViewBackend.cpp	2019-02-25 21:48:24 UTC (rev 242055)
@@ -35,9 +35,7 @@
     : m_width(width)
     , m_height(height)
 {
-#if defined(WPE_BACKEND_CHECK_VERSION) && WPE_BACKEND_CHECK_VERSION(0, 2, 0)
-    wpe_loader_init("libWPEBackend-fdo-0.1.so");
-#endif
+    wpe_loader_init("libWPEBackend-fdo-1.0.so");
 }
 
 ViewBackend::~ViewBackend()

Modified: trunk/Tools/wpe/backends/WindowViewBackend.cpp (242054 => 242055)


--- trunk/Tools/wpe/backends/WindowViewBackend.cpp	2019-02-25 20:15:53 UTC (rev 242054)
+++ trunk/Tools/wpe/backends/WindowViewBackend.cpp	2019-02-25 21:48:24 UTC (rev 242055)
@@ -439,7 +439,6 @@
         auto& window = *static_cast<WindowViewBackend*>(data);
         wpe_view_backend_dispatch_set_size(window.backend(), width, height);
 
-#if defined(WPE_BACKEND_CHECK_VERSION) && WPE_BACKEND_CHECK_VERSION(1, 1, 0)
         bool isFocused = false;
         void* p;
         wl_array_for_each(p, states)
@@ -462,19 +461,12 @@
             wpe_view_backend_add_activity_state(window.backend(), wpe_view_activity_state_focused);
         else
             wpe_view_backend_remove_activity_state(window.backend(), wpe_view_activity_state_focused);
-#else
-        (void)states;
-#endif
     },
     // close
     [](void* data, struct zxdg_toplevel_v6*)
     {
-#if defined(WPE_BACKEND_CHECK_VERSION) && WPE_BACKEND_CHECK_VERSION(1, 1, 0)
         auto& window = *static_cast<WindowViewBackend*>(data);
         wpe_view_backend_remove_activity_state(window.backend(), wpe_view_activity_state_visible | wpe_view_activity_state_focused | wpe_view_activity_state_in_window);
-#else
-        (void)data;
-#endif
     },
 };
 
@@ -525,9 +517,7 @@
             zxdg_toplevel_v6_add_listener(m_xdgToplevel, &s_xdgToplevelListener, this);
             zxdg_toplevel_v6_set_title(m_xdgToplevel, "WPE");
             wl_surface_commit(m_surface);
-#if defined(WPE_BACKEND_CHECK_VERSION) && WPE_BACKEND_CHECK_VERSION(1, 1, 0)
             wpe_view_backend_add_activity_state(backend(), wpe_view_activity_state_visible | wpe_view_activity_state_in_window);
-#endif
         }
     }
 

Modified: trunk/Tools/wpe/jhbuild.modules (242054 => 242055)


--- trunk/Tools/wpe/jhbuild.modules	2019-02-25 20:15:53 UTC (rev 242054)
+++ trunk/Tools/wpe/jhbuild.modules	2019-02-25 21:48:24 UTC (rev 242055)
@@ -202,7 +202,7 @@
       <dep package="glib"/>
     </dependencies>
     <branch repo="github.com" module="Igalia/wpebackend-fdo.git" checkoutdir="wpebackend-fdo"
-            tag="d7eeab26375b1f08c6a35dbe2d34f02197708a03"/>
+            tag="fa227dac7c6a133d62b35d7a6af0332582b64d4d"/>
   </cmake>
 
   <autotools id="libgpg-error" autogen-sh="configure">
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to