[Wayland-bugs] [Bug 762618] wayland: Switching between fullscreen and unfullscreen using F11 in google-chrome causes gnome-shell to hang

[mutter (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=762618

Ray Strode [halfline] <rstrode at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |rstrode at redhat.com

--- Comment #11 from Ray Strode [halfline] <rstrode at redhat.com> ---
some pared down discussion from irc:

<ofourdan> I meant Shell.util_get_content_for_window_actor() in
js/ui/windowManager.js
<rtcm_> that's to get a snapshot of the window for the (un)fullscreen animation
<ofourdan> yeap
<rtcm_> the animation is basically a fade out of the window as it was before
resize and a fade in of the current window's contents
<halfline> meta_shaped_texture_get_image seems a bit suspect
<halfline> it reads the texture into a cairo image surface and then does
compositing with cairo 
<halfline> it looks like it was originally added to implement screenshots
<halfline> using cairo for somethign that's going to get thrown in a png makes
sense
<rtcm_> I'm sure this isn't optimal. as halfline points out, the shell is
re-using code that was originally meant for screenshots
<ebassi> halfline: Looking at the code, I'm suspecting the reason why it goes
through cairo is because of the texture mask on top for shaped windows
<ebassi> halfline: We could create a new multi-texturing pipeline content,
though
<ebassi> It does; and when drawing it does multi-texturing to blend the texture
and mask together
<ebassi> So it should be possible to just extract the same thing into an object
and paint it again, instead of doing GPU → CPU → GPU
<halfline> though we'll want to keep the -> cairo for screenshots
<ebassi> halfline: Sure, that has to stay there 
<ebassi> halfline: I started looking at it, but I have been busy
<halfline> okay
<ebassi> halfline: The idea was to add a MetaWindowContent that would be used
by the window actor to paint itself
<ebassi> halfline: Then the code that currently returns a ClutterCanvas would
return that content object
<ebassi> halfline: Which can be used by the animation, and avoid the read-back

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/wayland-bugs/attachments/20160307/c7a87781/attachment.html>