Bug 201434

Summary: [GTK] Build failure in Debian Stable and Ubuntu LTS bots after r249427
Product: WebKit Reporter: Pablo Saavedra <psaavedra>
Component: WebCore Misc.Assignee: Pablo Saavedra <psaavedra>
Status: RESOLVED FIXED    
Severity: Normal CC: bugs-noreply, calvaris, commit-queue, pnormand, sabouhallawa, simon.fraser, webkit-bug-importer, zan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
patch
none
patch none

Description Pablo Saavedra 2019-09-03 13:49:49 PDT
```
../../Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:179:9: error: ‘GstGLSyncMeta’ was not declared in this scope
         GstGLSyncMeta* meta = gst_buffer_get_gl_sync_meta(m_buffer);
         ^~~~~~~~~~~~~
..```
Comment 1 Pablo Saavedra 2019-09-03 13:51:56 PDT
Created attachment 377918 [details]
patch
Comment 2 Pablo Saavedra 2019-09-03 13:57:36 PDT
The current patch fix the build but maybe the right patch for this case also it should restore the logic replaced by r249427 in L719:

```
719            GLuint textureID = frameHolder->textureID();
720	 	            std::unique_ptr<TextureMapperPlatformLayerBuffer> layerBuffer;
721	 	            if (textureID) {
722	 	                layerBuffer = makeUnique<TextureMapperPlatformLayerBuffer>(textureID, frameHolder->size(), frameHolder->flags(), GraphicsContext3D::RGBA);
```

https://trac.webkit.org/browser/webkit/trunk/Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp?rev=249321#L719
Comment 3 Pablo Saavedra 2019-09-03 14:18:57 PDT
(In reply to Pablo Saavedra from comment #2)
> The current patch fix the build but maybe the right patch for this case also
> it should restore the logic replaced by r249427 in L719:
> 
> ```
> 719            GLuint textureID = frameHolder->textureID();
> 720	 	            std::unique_ptr<TextureMapperPlatformLayerBuffer>
> layerBuffer;
> 721	 	            if (textureID) {
> 722	 	                layerBuffer =
> makeUnique<TextureMapperPlatformLayerBuffer>(textureID, frameHolder->size(),
> frameHolder->flags(), GraphicsContext3D::RGBA);
> ```
> 
> https://trac.webkit.org/browser/webkit/trunk/Source/WebCore/platform/
> graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp?rev=249321#L719

Nah,  only is invoked in sections of the code already safeguarded by `#if USE(GSTREAMER_GL)`
Comment 4 Pablo Saavedra 2019-09-03 23:07:57 PDT
Created attachment 377957 [details]
patch
Comment 5 WebKit Commit Bot 2019-09-04 01:29:20 PDT
Comment on attachment 377957 [details]
patch

Clearing flags on attachment: 377957

Committed r249474: <https://trac.webkit.org/changeset/249474>
Comment 6 WebKit Commit Bot 2019-09-04 01:29:22 PDT
All reviewed patches have been landed.  Closing bug.
Comment 7 Radar WebKit Bug Importer 2019-09-04 01:30:19 PDT
<rdar://problem/55010966>