<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>200223</bug_id>
          
          <creation_ts>2019-07-29 06:04:56 -0700</creation_ts>
          <short_desc>[GTK] Compilation errors when GL is disabled</short_desc>
          <delta_ts>2019-09-18 19:34:57 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKitGTK</component>
          <version>WebKit Local Build</version>
          <rep_platform>All</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Gtk</keywords>
          <priority>P2</priority>
          <bug_severity>Major</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Kam Amini">k.amini</reporter>
          <assigned_to name="Michael Catanzaro">mcatanzaro</assigned_to>
          <cc>bugs-noreply</cc>
    
    <cc>commit-queue</cc>
    
    <cc>mcatanzaro</cc>
    
    <cc>pnormand</cc>
    
    <cc>yurys</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1556547</commentid>
    <comment_count>0</comment_count>
    <who name="Kam Amini">k.amini</who>
    <bug_when>2019-07-29 06:04:56 -0700</bug_when>
    <thetext>Hello,

I receive the following compilation error when the GL support is off. I&apos;m compiling 2.24.3 on a CentOS machine.

&gt; In file included from /usr/include/glib-2.0/gobject/gobject.h:28:0,
&gt;                  from /usr/include/glib-2.0/gobject/gbinding.h:29,
&gt;                  from /usr/include/glib-2.0/glib-object.h:23,
&gt;                  from /opt/activevideo/webkit-dependencies/include/gstreamer-1.0/gst/gstenumtypes.h:7,
&gt;                  from /opt/activevideo/webkit-dependencies/include/gstreamer-1.0/gst/gst.h:31,
&gt;                  from ../Source/WebCore/platform/graphics/gstreamer/GRefPtrGStreamer.h:24,
&gt;                  from ../Source/WebCore/platform/graphics/gstreamer/GStreamerCommon.h:24,
&gt;                  from ../Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.h:29,
&gt;                  from ../Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:27:
&gt; ../Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: In lambda function:
&gt; ../Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:2530:17: error: ‘class WebCore::MediaPlayerPrivateGStreamer’ has no member named ‘updateTextureMapperFlags’
&gt;          player-&gt;updateTextureMapperFlags();
&gt;                  ^
&gt;  [12/1006] Building CXX object Source/WebCore/CMakeFiles/WebCore.dir/platform/graphics/gstreamer/mse/MediaPlayerPrivateGStreamerMSE.cpp.o

Root Cause:
The issue is apparent in the following sources. The `updateTextureMapperFlags` is defined only when `TEXTURE_MAPPER_GL` is used.

Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.h:247
246 #if USE(TEXTURE_MAPPER_GL)
247     void updateTextureMapperFlags();
248     TextureMapperGL::Flags m_textureMapperFlags;
249 #endif

The following is the definition of the function:

Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:964
963 #if USE(TEXTURE_MAPPER_GL)
964 void MediaPlayerPrivateGStreamerBase::updateTextureMapperFlags()
965 {
966     switch (m_videoSourceOrientation) {
967     case DefaultImageOrientation:
968         m_textureMapperFlags = 0;

And the following is the usage of the function which does not check the `TEXTURE_MAPPER_GL` macro to be set.

Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:2491
2489             player-&gt;m_videoDecoderPlatform = WebKitGstVideoDecoderPlatform::ImxVPU;
2490
2491         player-&gt;updateTextureMapperFlags();
2492     }), this);

Thanks in advance.

Best regards,
Kam</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1556566</commentid>
    <comment_count>1</comment_count>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2019-07-29 08:29:10 -0700</bug_when>
    <thetext>Hi Kam,

Can you clarify the full list of CMake arguments you use please?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1556567</commentid>
    <comment_count>2</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2019-07-29 08:29:37 -0700</bug_when>
    <thetext>Can you confirm that adding the #if USE(TEXTURE_MAPPER_GL) guard around the call to player-&gt;updateTextureMapperFlags() is sufficient to fix the build, or is there a subsequent issue?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1556918</commentid>
    <comment_count>3</comment_count>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2019-07-30 02:19:12 -0700</bug_when>
    <thetext>Reproducer:

cmake ~/WebKit -DPORT=GTK -DENABLE_OPENGL=OFF
ninja</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1556921</commentid>
    <comment_count>4</comment_count>
    <who name="Kam Amini">k.amini</who>
    <bug_when>2019-07-30 02:32:07 -0700</bug_when>
    <thetext>Thanks @Philip for the reproduction commands. I&apos;ll place the macro around that line and let you know @Michael.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1556922</commentid>
    <comment_count>5</comment_count>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2019-07-30 02:34:45 -0700</bug_when>
    <thetext>This is beyond GStreamer:

In file included from /usr/include/X11/Xlib.h:44,
                 from /usr/include/gtk-3.0/gdk/gdkx.h:30,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/PlatformDisplay.cpp:54,
                 from DerivedSources/WebCore/unified-sources/UnifiedSource-3c72abbe-24.cpp:2:
DerivedSources/ForwardingHeaders/wtf/Threading.h:75:5: error: expected identifier before numeric constant
     None = 0,
     ^~~~
DerivedSources/ForwardingHeaders/wtf/Threading.h:75:5: error: expected ‘}’ before numeric constant
In file included from DerivedSources/ForwardingHeaders/wtf/WeakPtr.h:33,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/FontCascade.h:34,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/GraphicsContext.h:31,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/cairo/CairoOperations.h:38,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/cairo/CairoOperations.cpp:34,
                 from DerivedSources/WebCore/unified-sources/UnifiedSource-3c72abbe-24.cpp:4:
DerivedSources/ForwardingHeaders/wtf/Threading.h:74:35: note: to match this ‘{’
 enum class GCThreadType : uint8_t {
                                   ^
In file included from /usr/include/X11/Xlib.h:44,
                 from /usr/include/gtk-3.0/gdk/gdkx.h:30,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/PlatformDisplay.cpp:54,
                 from DerivedSources/WebCore/unified-sources/UnifiedSource-3c72abbe-24.cpp:2:
DerivedSources/ForwardingHeaders/wtf/Threading.h:75:5: error: expected unqualified-id before numeric constant
     None = 0,
     ^~~~
In file included from DerivedSources/ForwardingHeaders/wtf/WeakPtr.h:33,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/FontCascade.h:34,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/GraphicsContext.h:31,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/cairo/CairoOperations.h:38,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/cairo/CairoOperations.cpp:34,
                 from DerivedSources/WebCore/unified-sources/UnifiedSource-3c72abbe-24.cpp:4:
DerivedSources/ForwardingHeaders/wtf/Threading.h:147:53: error: ‘GCThreadType’ has not been declared
     WTF_EXPORT_PRIVATE static void registerGCThread(GCThreadType);
                                                     ^~~~~~~~~~~~
DerivedSources/ForwardingHeaders/wtf/Threading.h:203:5: error: ‘GCThreadType’ does not name a type; did you mean ‘GThreadPool’?
     GCThreadType gcThreadType() const { return static_cast&lt;GCThreadType&gt;(m_gcThreadType); }
     ^~~~~~~~~~~~
     GThreadPool
DerivedSources/ForwardingHeaders/wtf/Threading.h:216:42: error: ‘PlatformThreadHandle’ has not been declared
     void establishPlatformSpecificHandle(PlatformThreadHandle);
                                          ^~~~~~~~~~~~~~~~~~~~
In file included from DerivedSources/ForwardingHeaders/wtf/WeakPtr.h:33,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/FontCascade.h:34,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/GraphicsContext.h:31,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/cairo/CairoOperations.h:38,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/cairo/CairoOperations.cpp:34,
                 from DerivedSources/WebCore/unified-sources/UnifiedSource-3c72abbe-24.cpp:4:
DerivedSources/ForwardingHeaders/wtf/Threading.h:248:5: error: ‘ThreadGroupAddResult’ does not name a type; did you mean ‘ThreadGroup’?
     ThreadGroupAddResult addToThreadGroup(const AbstractLocker&amp; threadGroupLocker, ThreadGroup&amp;);
     ^~~~~~~~~~~~~~~~~~~~
     ThreadGroup
DerivedSources/ForwardingHeaders/wtf/Threading.h:249:73: error: ‘ThreadGroup’ has not been declared
     void removeFromThreadGroup(const AbstractLocker&amp; threadGroupLocker, ThreadGroup&amp;);
                                                                         ^~~~~~~~~~~
DerivedSources/ForwardingHeaders/wtf/Threading.h:257:31: error: ‘ThreadSpecificKey’ does not name a type; did you mean ‘ThreadSafeRefCounted’?
     static WTF_EXPORT_PRIVATE ThreadSpecificKey s_key;
                               ^~~~~~~~~~~~~~~~~
                               ThreadSafeRefCounted
DerivedSources/ForwardingHeaders/wtf/Threading.h:293:26: error: ‘ThreadGroup’ was not declared in this scope
     Vector&lt;std::weak_ptr&lt;ThreadGroup&gt;&gt; m_threadGroups;
                          ^~~~~~~~~~~
DerivedSources/ForwardingHeaders/wtf/Threading.h:293:26: note: suggested alternative:
In file included from DerivedSources/ForwardingHeaders/wtf/WeakPtr.h:33,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/FontCascade.h:34,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/GraphicsContext.h:31,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/cairo/CairoOperations.h:38,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/cairo/CairoOperations.cpp:34,
                 from DerivedSources/WebCore/unified-sources/UnifiedSource-3c72abbe-24.cpp:4:
DerivedSources/ForwardingHeaders/wtf/Threading.h:61:7: note:   ‘WTF::ThreadGroup’
 class ThreadGroup;
       ^~~~~~~~~~~
In file included from DerivedSources/ForwardingHeaders/wtf/WeakPtr.h:33,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/FontCascade.h:34,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/GraphicsContext.h:31,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/cairo/CairoOperations.h:38,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/cairo/CairoOperations.cpp:34,
                 from DerivedSources/WebCore/unified-sources/UnifiedSource-3c72abbe-24.cpp:4:
DerivedSources/ForwardingHeaders/wtf/Threading.h:293:26: error: template argument 1 is invalid
     Vector&lt;std::weak_ptr&lt;ThreadGroup&gt;&gt; m_threadGroups;
                          ^~~~~~~~~~~
DerivedSources/ForwardingHeaders/wtf/Threading.h:293:37: error: template argument 1 is invalid
     Vector&lt;std::weak_ptr&lt;ThreadGroup&gt;&gt; m_threadGroups;
                                     ^~
DerivedSources/ForwardingHeaders/wtf/Threading.h:294:5: error: ‘PlatformThreadHandle’ does not name a type
     PlatformThreadHandle m_handle;
     ^~~~~~~~~~~~~~~~~~~~
DerivedSources/ForwardingHeaders/wtf/Threading.h: In constructor ‘Thread::Thread()’:
DerivedSources/ForwardingHeaders/wtf/Threading.h:321:44: error: ‘GCThreadType’ has not been declared
     , m_gcThreadType(static_cast&lt;unsigned&gt;(GCThreadType::None))
                                            ^~~~~~~~~~~~
In file included from /usr/include/X11/Xlib.h:44,
                 from /usr/include/gtk-3.0/gdk/gdkx.h:30,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/PlatformDisplay.cpp:54,
                 from DerivedSources/WebCore/unified-sources/UnifiedSource-3c72abbe-24.cpp:2:
DerivedSources/ForwardingHeaders/wtf/Threading.h:321:58: error: expected unqualified-id before numeric constant
     , m_gcThreadType(static_cast&lt;unsigned&gt;(GCThreadType::None))
                                                          ^~~~
In file included from DerivedSources/ForwardingHeaders/wtf/WeakPtr.h:33,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/FontCascade.h:34,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/GraphicsContext.h:31,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/cairo/CairoOperations.h:38,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/cairo/CairoOperations.cpp:34,
                 from DerivedSources/WebCore/unified-sources/UnifiedSource-3c72abbe-24.cpp:4:
DerivedSources/ForwardingHeaders/wtf/Threading.h:321:58: error: expected ‘)’ before numeric constant
     , m_gcThreadType(static_cast&lt;unsigned&gt;(GCThreadType::None))
                                                          ^
                                                          )
DerivedSources/ForwardingHeaders/wtf/Threading.h:321:63: error: expected ‘{’ before ‘)’ token
     , m_gcThreadType(static_cast&lt;unsigned&gt;(GCThreadType::None))
                                                               ^
DerivedSources/ForwardingHeaders/wtf/Threading.h: At global scope:
DerivedSources/ForwardingHeaders/wtf/Threading.h:321:63: error: expected unqualified-id before ‘)’ token
DerivedSources/ForwardingHeaders/wtf/Threading.h: In static member function ‘static Thread* Thread::currentMayBeNull()’:
DerivedSources/ForwardingHeaders/wtf/Threading.h:329:51: error: ‘s_key’ was not declared in this scope
     return static_cast&lt;Thread*&gt;(threadSpecificGet(s_key));
                                                   ^~~~~
DerivedSources/ForwardingHeaders/wtf/Threading.h:329:51: note: suggested alternative: ‘si_pkey’
     return static_cast&lt;Thread*&gt;(threadSpecificGet(s_key));
                                                   ^~~~~
                                                   si_pkey
DerivedSources/ForwardingHeaders/wtf/Threading.h:329:33: error: ‘threadSpecificGet’ was not declared in this scope
     return static_cast&lt;Thread*&gt;(threadSpecificGet(s_key));
                                 ^~~~~~~~~~~~~~~~~
DerivedSources/ForwardingHeaders/wtf/Threading.h:329:33: note: suggested alternative:
In file included from DerivedSources/ForwardingHeaders/wtf/MainThread.h:35,
                 from DerivedSources/ForwardingHeaders/wtf/ThreadSafeRefCounted.h:30,
                 from DerivedSources/ForwardingHeaders/wtf/FunctionDispatcher.h:29,
                 from DerivedSources/ForwardingHeaders/wtf/RunLoop.h:33,
                 from DerivedSources/ForwardingHeaders/pal/HysteresisActivity.h:28,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/cairo/BackingStoreBackendCairoImpl.h:23,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/cairo/BackingStoreBackendCairoImpl.cpp:21,
                 from DerivedSources/WebCore/unified-sources/UnifiedSource-3c72abbe-24.cpp:3:
DerivedSources/ForwardingHeaders/wtf/ThreadingPrimitives.h:143:14: note:   ‘WTF::threadSpecificGet’
 inline void* threadSpecificGet(ThreadSpecificKey key)
              ^~~~~~~~~~~~~~~~~
In file included from DerivedSources/ForwardingHeaders/wtf/Platform.h:31,
                 from /home/phil/WebKit/Source/WebCore/config.h:26,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/GraphicsContext3DPrivate.cpp:20,
                 from DerivedSources/WebCore/unified-sources/UnifiedSource-3c72abbe-24.cpp:1:
DerivedSources/ForwardingHeaders/wtf/Threading.h: In static member function ‘static Thread&amp; Thread::current()’:
DerivedSources/ForwardingHeaders/wtf/Threading.h:344:26: error: ‘s_key’ is not a member of ‘Thread’
     if (UNLIKELY(Thread::s_key == InvalidThreadSpecificKey))
                          ^~~~~
DerivedSources/ForwardingHeaders/wtf/Compiler.h:341:41: note: in definition of macro ‘UNLIKELY’
 #define UNLIKELY(x) __builtin_expect(!!(x), 0)
                                         ^
DerivedSources/ForwardingHeaders/wtf/Threading.h:344:35: error: ‘InvalidThreadSpecificKey’ was not declared in this scope
     if (UNLIKELY(Thread::s_key == InvalidThreadSpecificKey))
                                   ^~~~~~~~~~~~~~~~~~~~~~~~
DerivedSources/ForwardingHeaders/wtf/Compiler.h:341:41: note: in definition of macro ‘UNLIKELY’
 #define UNLIKELY(x) __builtin_expect(!!(x), 0)
                                         ^
DerivedSources/ForwardingHeaders/wtf/Threading.h:344:35: note: suggested alternative:
     if (UNLIKELY(Thread::s_key == InvalidThreadSpecificKey))
                                   ^~~~~~~~~~~~~~~~~~~~~~~~
DerivedSources/ForwardingHeaders/wtf/Compiler.h:341:41: note: in definition of macro ‘UNLIKELY’
 #define UNLIKELY(x) __builtin_expect(!!(x), 0)
                                         ^
In file included from DerivedSources/ForwardingHeaders/wtf/MainThread.h:35,
                 from DerivedSources/ForwardingHeaders/wtf/ThreadSafeRefCounted.h:30,
                 from DerivedSources/ForwardingHeaders/wtf/FunctionDispatcher.h:29,
                 from DerivedSources/ForwardingHeaders/wtf/RunLoop.h:33,
                 from DerivedSources/ForwardingHeaders/pal/HysteresisActivity.h:28,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/cairo/BackingStoreBackendCairoImpl.h:23,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/cairo/BackingStoreBackendCairoImpl.cpp:21,
                 from DerivedSources/WebCore/unified-sources/UnifiedSource-3c72abbe-24.cpp:3:
DerivedSources/ForwardingHeaders/wtf/ThreadingPrimitives.h:122:36: note:   ‘WTF::InvalidThreadSpecificKey’
 static constexpr ThreadSpecificKey InvalidThreadSpecificKey = PTHREAD_KEYS_MAX;
                                    ^~~~~~~~~~~~~~~~~~~~~~~~
In file included from DerivedSources/ForwardingHeaders/wtf/WeakPtr.h:33,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/FontCascade.h:34,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/GraphicsContext.h:31,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/cairo/CairoOperations.h:38,
                 from /home/phil/WebKit/Source/WebCore/platform/graphics/cairo/CairoOperations.cpp:34,
                 from DerivedSources/WebCore/unified-sources/UnifiedSource-3c72abbe-24.cpp:4:
DerivedSources/ForwardingHeaders/wtf/Threading.h: At global scope:
DerivedSources/ForwardingHeaders/wtf/Threading.h:356:1: error: expected declaration before ‘}’ token
 } // namespace WTF
 ^</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1556942</commentid>
    <comment_count>6</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2019-07-30 08:25:47 -0700</bug_when>
    <thetext>(In reply to Philippe Normand from comment #5)
&gt; This is beyond GStreamer:
&gt; 
&gt; In file included from /usr/include/X11/Xlib.h:44,
&gt;                  from /usr/include/gtk-3.0/gdk/gdkx.h:30,
&gt;                  from
&gt; /home/phil/WebKit/Source/WebCore/platform/graphics/PlatformDisplay.cpp:54,
&gt;                  from
&gt; DerivedSources/WebCore/unified-sources/UnifiedSource-3c72abbe-24.cpp:2:
&gt; DerivedSources/ForwardingHeaders/wtf/Threading.h:75:5: error: expected
&gt; identifier before numeric constant
&gt;      None = 0,
&gt;      ^~~~

Files including Xlib.h must be excluded from unified build. Solution is to add @no-unify to PlatformDisplay.cpp in Sources.txt.

If you&apos;re bored or have 25 hours today, you could git grep to try to find every .cpp that includes gdkx.h and Xlib.h and mark them all as @no-unify.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1556943</commentid>
    <comment_count>7</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2019-07-30 08:26:57 -0700</bug_when>
    <thetext>Actually I have time today to take this.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1556969</commentid>
    <comment_count>8</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2019-07-30 09:16:53 -0700</bug_when>
    <thetext>I tested the tarball, and these guards are all that&apos;s needed there. I&apos;ll check trunk next; I expect to see Phil&apos;s errors on trunk.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1556971</commentid>
    <comment_count>9</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2019-07-30 09:17:42 -0700</bug_when>
    <thetext>Committed r248009: &lt;https://trac.webkit.org/changeset/248009&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1556979</commentid>
    <comment_count>10</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2019-07-30 09:28:41 -0700</bug_when>
    <thetext>BTW the eventual 2.24.4 tarball will contain this fix, thanks for reporting.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1557001</commentid>
    <comment_count>11</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2019-07-30 10:29:02 -0700</bug_when>
    <thetext>Committed r248012: &lt;https://trac.webkit.org/changeset/248012&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1557003</commentid>
    <comment_count>12</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2019-07-30 10:31:06 -0700</bug_when>
    <thetext>Reopening since fixing trunk requires thinking, and I&apos;m not good at that. We have:

In file included from DerivedSources/WebKit/unified-sources/UnifiedSource-88d1702b-25.cpp:3:
/home/mcatanzaro/Projects/WebKit/Source/WebKit/UIProcess/glib/WebProcessPoolGLib.cpp: In member function ‘void WebKit::WebProcessPool::platformInitializeWebProcess(const WebKit::WebProcessProxy&amp;, WebKit::WebProcessCreationParameters&amp;)’:
/home/mcatanzaro/Projects/WebKit/Source/WebKit/UIProcess/glib/WebProcessPoolGLib.cpp:123:51: error: ‘WaylandCompositor’ has not been declared
  123 |         parameters.waylandCompositorDisplayName = WaylandCompositor::singleton().displayName();
      |                                                   ^~~~~~~~~~~~~~~~~

So WebProcessPoolGLib.cpp requires WaylandCompositor because PLATFORM(WAYLAND) is true but USE(WPE_RENDERER) is false. Problem is WaylandCompositor.h is guarded by #if PLATFORM(WAYLAND) &amp;&amp; USE(EGL) &amp;&amp; !USE(WPE_RENDERER). Carlos Garcia might know what to do.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1557224</commentid>
    <comment_count>13</comment_count>
    <who name="Kam Amini">k.amini</who>
    <bug_when>2019-07-31 01:34:14 -0700</bug_when>
    <thetext>(In reply to Michael Catanzaro from comment #10)
&gt; BTW the eventual 2.24.4 tarball will contain this fix, thanks for reporting.

Thank you for the quick fix. I&apos;ll wait until the bug is fully resolved.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1572044</commentid>
    <comment_count>14</comment_count>
      <attachid>379065</attachid>
    <who name="Yury Semikhatsky">yurys</who>
    <bug_when>2019-09-18 13:17:25 -0700</bug_when>
    <thetext>Created attachment 379065
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1572204</commentid>
    <comment_count>15</comment_count>
      <attachid>379065</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2019-09-18 19:34:56 -0700</bug_when>
    <thetext>Comment on attachment 379065
Patch

Clearing flags on attachment: 379065

Committed r250077: &lt;https://trac.webkit.org/changeset/250077&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1572205</commentid>
    <comment_count>16</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2019-09-18 19:34:57 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>379065</attachid>
            <date>2019-09-18 13:17:25 -0700</date>
            <delta_ts>2019-09-18 19:34:56 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-200223-20190918131724.patch</filename>
            <type>text/plain</type>
            <size>1968</size>
            <attacher name="Yury Semikhatsky">yurys</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjQ5OTU3CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCmluZGV4IGQ4YjQ3MjAyOGExZjVhMjdl
Y2E3M2FjNmFiZGJlZTY0Y2Q1M2EwZTYuLjhhMjgzM2ViZDgxMGNlZjUyNzJlZWYxZDA3NDU1Mjlk
MTdkOThlM2YgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCisrKyBiL1NvdXJj
ZS9XZWJLaXQvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTggQEAKKzIwMTktMDktMTggIFl1cnkgU2Vt
aWtoYXRza3kgIDx5dXJ5c0BjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgW0dUS10gQ29tcGlsYXRp
b24gZXJyb3JzIHdoZW4gR0wgaXMgZGlzYWJsZWQKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtp
dC5vcmcvc2hvd19idWcuY2dpP2lkPTIwMDIyMworCisgICAgICAgIEZpeCBmb2xsb3dpbmcgY29t
cGlsYXRpb24gZXJyb3Igd2hlbiBidWlsZGluZyB3aXRoIEVOQUJMRV9PUEVOR0w9T0ZGCisgICAg
ICAgIC4uLy4uL1NvdXJjZS9XZWJLaXQvVUlQcm9jZXNzL2dsaWIvV2ViUHJvY2Vzc1Bvb2xHTGli
LmNwcDoxMjM6NTE6IGVycm9yOiB1c2Ugb2YgdW5kZWNsYXJlZCBpZGVudGlmaWVyICdXYXlsYW5k
Q29tcG9zaXRvcicKKyAgICAgICAgcGFyYW1ldGVycy53YXlsYW5kQ29tcG9zaXRvckRpc3BsYXlO
YW1lID0gV2F5bGFuZENvbXBvc2l0b3I6OnNpbmdsZXRvbigpLmRpc3BsYXlOYW1lKCk7CisgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIF4KKyAgICAgICAg
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBVSVByb2Nlc3MvZ2xpYi9X
ZWJQcm9jZXNzUG9vbEdMaWIuY3BwOgorICAgICAgICAoV2ViS2l0OjpXZWJQcm9jZXNzUG9vbDo6
cGxhdGZvcm1Jbml0aWFsaXplV2ViUHJvY2Vzcyk6IG9ubHkgbWFrZSBhIGNhbGwgd2hlbiB1c2lu
ZyBFR0wsIHRoaXMgbWF0Y2hlcworICAgICAgICBndWFyZHMgaW4gV2F5bGFuZENvbXBvc2l0b3Iu
aC4gVGhlIGNvbmRpdGlvbiB3YXMgY2hhbmdlZCBpbiByMjQ1ODA3LgorCiAyMDE5LTA5LTE3ICBU
cnVpdHQgU2F2ZWxsICA8dHNhdmVsbEBhcHBsZS5jb20+CiAKICAgICAgICAgVW5yZXZpZXdlZCwg
cm9sbGluZyBvdXQgcjI0OTk1MC4KZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvVUlQcm9jZXNz
L2dsaWIvV2ViUHJvY2Vzc1Bvb2xHTGliLmNwcCBiL1NvdXJjZS9XZWJLaXQvVUlQcm9jZXNzL2ds
aWIvV2ViUHJvY2Vzc1Bvb2xHTGliLmNwcAppbmRleCAzN2VjZTJmNDU3NmRjZjg5NzA4NmI3ZTlk
YmRiNWU1ZGVlODNiZTgwLi5hMzgwYTI1ZmE0Yjk4YTNjNTQzNTJlZmY2ZDgwMWE3YzFjNDNjZWNh
IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0L1VJUHJvY2Vzcy9nbGliL1dlYlByb2Nlc3NQb29s
R0xpYi5jcHAKKysrIGIvU291cmNlL1dlYktpdC9VSVByb2Nlc3MvZ2xpYi9XZWJQcm9jZXNzUG9v
bEdMaWIuY3BwCkBAIC0xMTksNyArMTE5LDcgQEAgdm9pZCBXZWJQcm9jZXNzUG9vbDo6cGxhdGZv
cm1Jbml0aWFsaXplV2ViUHJvY2Vzcyhjb25zdCBXZWJQcm9jZXNzUHJveHkmIHByb2Nlc3MKICAg
ICAgICAgICAgIHBhcmFtZXRlcnMuaG9zdENsaWVudEZpbGVEZXNjcmlwdG9yID0gd3BlX3JlbmRl
cmVyX2hvc3RfY3JlYXRlX2NsaWVudCgpOwogICAgICAgICAgICAgcGFyYW1ldGVycy5pbXBsZW1l
bnRhdGlvbkxpYnJhcnlOYW1lID0gRmlsZVN5c3RlbTo6ZmlsZVN5c3RlbVJlcHJlc2VudGF0aW9u
KHdwZV9sb2FkZXJfZ2V0X2xvYWRlZF9pbXBsZW1lbnRhdGlvbl9saWJyYXJ5X25hbWUoKSk7CiAg
ICAgICAgIH0KLSNlbHNlCisjZWxpZiBVU0UoRUdMKQogICAgICAgICBwYXJhbWV0ZXJzLndheWxh
bmRDb21wb3NpdG9yRGlzcGxheU5hbWUgPSBXYXlsYW5kQ29tcG9zaXRvcjo6c2luZ2xldG9uKCku
ZGlzcGxheU5hbWUoKTsKICNlbmRpZgogICAgIH0K
</data>

          </attachment>
      

    </bug>

</bugzilla>