Bug 182968

Summary: [GStreamer] Create a Wayland GL display instead of EGL
Product: WebKit Reporter: Philippe Normand <pnormand>
Component: PlatformAssignee: Philippe Normand <pnormand>
Severity: Normal CC: calvaris, cgarcia, magomez, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 185207    
Bug Blocks: 182996    
Description Flags
Patch calvaris: review+

Description Philippe Normand 2018-02-20 06:17:22 PST
This looks like an old copy/paste error.
Comment 1 Philippe Normand 2018-02-20 06:20:21 PST
Created attachment 334264 [details]
Comment 2 Xabier Rodríguez Calvar 2018-02-21 00:44:52 PST
Comment on attachment 334264 [details]

Wait, I think we have a leak here. All these gst_gl_display_*_new_with_egl_display are [transfer full] and we are not adopting them.
Comment 3 Xabier Rodríguez Calvar 2018-02-21 01:27:41 PST
Comment on attachment 334264 [details]

Before the patch landed on https://bugzilla.gnome.org/show_bug.cgi?id=743062 these references were still floating, thus we can't adopt or we would get crashes in debug mode. After that patch, we need to adopt or we would be leaking. I think we can land as it is but we need a new bug to fix this.
Comment 4 Philippe Normand 2018-02-21 01:35:00 PST
Committed r228866: <https://trac.webkit.org/changeset/228866>
Comment 5 Radar WebKit Bug Importer 2018-02-21 01:35:24 PST
Comment 6 Carlos Garcia Campos 2018-02-26 06:21:55 PST
This is wrong, the display we want to use for rendering is the EGL display created and initialized by the PlatformDisplay. This patch causes runtime errors like:

0:00:46.130136714 13362 0x5609ff9524a0 ERROR      webkitmediaplayer MediaPlayerPrivateGStreamer.cpp:956:handleMessage: Error 3: Failed to create a OpenGL context: EGL_BAD_CONTEXT (url=webkit+https://video.twimg.com/ext_tw_video/968118107192602624/pu/vid/640x360/dFLZDeagge7DOJd6.mp4)

And then the web process hangs.
Comment 7 Carlos Garcia Campos 2018-02-26 06:23:51 PST
Reopening, not rolling out yet, just in case you have a quick fix. Otherwise, please roll it out.
Comment 8 Philippe Normand 2018-02-26 06:35:13 PST
Rolled out in r229022