Changeset 217838 in webkit

Timestamp:

Jun 6, 2017 10:13:00 AM (7 years ago)

Author:

Adrian Perez de Castro

Message:

[GTK][Wayland] Do not pass value of $DISPLAY to wl_display_connect()
​https://bugs.webkit.org/show_bug.cgi?id=172993

Reviewed by Žan Doberšek.

No new tests needed.

• platform/graphics/wayland/PlatformDisplayWayland.cpp:

(WebCore::PlatformDisplayWayland::create): Instead of using the "DISPLAY" environment variable (meant for X11),
pass a null pointer to "wl_display_connect()". This function checks the value of the "WAYLAND_DISPLAY" environment
variable itself and therefore we do not need to pass it ourselves.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/wayland/PlatformDisplayWayland.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2017-06-06  Adrian Perez de Castro  <aperez@igalia.com>
+
+        [GTK][Wayland] Do not pass value of $DISPLAY to wl_display_connect()
+        https://bugs.webkit.org/show_bug.cgi?id=172993
+
+        Reviewed by Žan Doberšek.
+
+        No new tests needed.
+
+        * platform/graphics/wayland/PlatformDisplayWayland.cpp:
+        (WebCore::PlatformDisplayWayland::create): Instead of using the "DISPLAY" environment variable (meant for X11),
+        pass a null pointer to "wl_display_connect()". This function checks the value of the "WAYLAND_DISPLAY" environment
+        variable itself and therefore we do not need to pass it ourselves.
+
 2017-06-06  Antoine Quint  <graouts@apple.com>
 

trunk/Source/WebCore/platform/graphics/wayland/PlatformDisplayWayland.cpp

@@ -53 +53 @@
 std::unique_ptr<PlatformDisplay> PlatformDisplayWayland::create()
 {
-    struct wl_display* display = wl_display_connect(getenv("DISPLAY"));
+    struct wl_display* display = wl_display_connect(nullptr);
     if (!display)
         return nullptr;