Bug 189224

Summary: REGRESSION(r235165): [GStreamer] Major video performance regression
Product: WebKit Reporter: Michael Catanzaro <mcatanzaro>
Component: WebKitGTKAssignee: Zan Dobersek <zan>
Status: RESOLVED FIXED    
Severity: Normal CC: bugs-noreply, calvaris, mcatanzaro, pgriffis, pnormand, zan
Priority: P2    
Version: Other   
Hardware: PC   
OS: Linux   
See Also: https://bugs.webkit.org/show_bug.cgi?id=188693
Attachments:
Description Flags
Patch none

Description Michael Catanzaro 2018-09-01 20:36:30 PDT 
r235165 "[CoordGraphics] Switch to Nicosia::CompositionLayer state tracking" has introduced a *very major* video performance regression that should be immediately noticeable e.g. when playing any video on YouTube. I can reproduce the same issue on http://download.blender.org/peach/bigbuckbunny_movies/big_buck_bunny_720p_stereo.ogg, which is a much simpler case than YouTube.

It's not straightforward to rollout due to many code changes in these files.

Fortunately, this commit is not on the 2.22 branch, so video playback in 2.21.92 is fine.
Comment 1 Zan Dobersek 2018-09-03 09:07:24 PDT 
Created attachment 348774 [details]
Patch
Comment 2 Philippe Normand 2018-09-03 09:16:14 PDT 
I tested this on desktop GTK and iMX6 WPE. Regression fixed. Thanks!
Comment 3 Xabier Rodríguez Calvar 2018-09-04 01:29:12 PDT 
I guess this didn't land yet, otherwise bug would be closed, right?
Comment 4 Zan Dobersek 2018-09-04 01:38:30 PDT 
Comment on attachment 348774 [details]
Patch

Clearing flags on attachment: 348774

Committed r235608: <https://trac.webkit.org/changeset/235608>
Comment 5 Zan Dobersek 2018-09-04 01:38:34 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 6 Michael Catanzaro 2018-09-04 18:17:48 PDT 
(In reply to Philippe Normand from comment #2)
> I tested this on desktop GTK and iMX6 WPE. Regression fixed. Thanks!

It's fixed for me too, yay!