After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 734905 - Unpainted GtkClutterActor with the GDK backend
Unpainted GtkClutterActor with the GDK backend
Status: RESOLVED FIXED
Product: clutter-gtk
Classification: Platform
Component: GtkClutterActor
unspecified
Other Linux
: Normal normal
: ---
Assigned To: clutter-gtk maintainer(s)
clutter-gtk maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2014-08-16 14:01 UTC by Lionel Landwerlin
Modified: 2014-08-17 16:01 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
gtk-clutter-actor: fix inconsistent condition in actor update (1.55 KB, patch)
2014-08-16 14:02 UTC, Lionel Landwerlin 		committed Details | Review

Description Lionel Landwerlin 2014-08-16 14:01:48 UTC 
Using the gdk backend of Clutter, you could end up in situation where none of the embedded widgets were being displayed. That's because the conditions for  invalidating the canvas where inconsistent with the rest of the code.
Comment 1 Lionel Landwerlin 2014-08-16 14:02:57 UTC 
Created attachment 283593 [details] [review]
gtk-clutter-actor: fix inconsistent condition in actor update
Comment 2 Emmanuele Bassi (:ebassi) 2014-08-17 10:10:00 UTC 
Review of attachment 283593 [details] [review]:

looks good.

as a future improvement, we should:

 * drop ClutterX11TexturePixmap and use a CoglTexturePixmapX11 directly
 * write a WidgetContent that either uses a cairo image surface or a xlib surface bound to a Pixmap, depending on whether GDK is using the X11 backend
Comment 3 Lionel Landwerlin 2014-08-17 15:58:27 UTC 
Agreed, we could use the wayland surface too.