Bug 184574 - [GTK] Webkit 2.20.x crashes frequently on Youtube.com when changing resolution during video playback, and sometimes while clicking the link to another video.
Summary: [GTK] Webkit 2.20.x crashes frequently on Youtube.com when changing resolutio...
Status: RESOLVED MOVED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: Other
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
: 185642 (view as bug list)
Depends on:
Blocks:
 
Reported: 2018-04-12 17:36 PDT by Ryan Farmer
Modified: 2018-05-15 09:19 PDT (History)
6 users (show)

See Also:


Attachments
Stack Trace (21.46 KB, text/plain)
2018-04-27 18:35 PDT, Ryan Farmer
no flags Details
GDB Stack Trace (143.79 KB, text/plain)
2018-04-27 19:22 PDT, Ryan Farmer
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ryan Farmer 2018-04-12 17:36:48 PDT
While I was testing to see if the distorted video rendering related to bug 173796 was still an issue, I noticed that while Youtube videos now play fine, at least initially, the Webkit Web Process crashes frequently if you change the resolution during playback and (occasionally) if you click on another video while one is playing.

Tested on Fedora 28 with the Gstreamer VA-API plugin and the Intel VA driver installed on Iris 540 graphics on a Skylake Core i7-6560U.
Comment 1 Michael Catanzaro 2018-04-13 08:54:24 PDT
Please post a backtrace
Comment 2 Hyunjun Ko 2018-04-15 23:45:54 PDT
(In reply to Ryan Farmer from comment #0)
> While I was testing to see if the distorted video rendering related to bug
> 173796 was still an issue, I noticed that while Youtube videos now play
> fine, at least initially, the Webkit Web Process crashes frequently if you
> change the resolution during playback and (occasionally) if you click on
> another video while one is playing.
> 
> Tested on Fedora 28 with the Gstreamer VA-API plugin and the Intel VA driver
> installed on Iris 540 graphics on a Skylake Core i7-6560U.

Could you let us know which version of gstreamer you're using?
Comment 3 Hyunjun Ko 2018-04-17 02:16:53 PDT
I found an issue in gst-vaapi, i'm still not sure if that is same as this issue.
If you're using gst-vaapi 1.14, it's likely to be same.
Comment 4 Ryan Farmer 2018-04-18 19:08:08 PDT
Yes, it's Gstreamer 1.14. 

Sorry for the delay. The bug mail was getting sent into my spam folder.
Comment 5 Michael Catanzaro 2018-04-18 21:33:04 PDT
Backtrace still needed :)
Comment 6 Ryan Farmer 2018-04-27 18:35:27 PDT
Created attachment 339047 [details]
Stack Trace
Comment 7 Ryan Farmer 2018-04-27 18:38:31 PDT
I can't get ABRT to come up anymore when this happens, but this is what it leaves in my system log. 

https://www.youtube.com/watch?v=07G4xhSefuI

There's a video that does it every time. Just click the gear icon and change the resolution to 720p.
Comment 8 Michael Catanzaro 2018-04-27 18:43:40 PDT
Hi, please see https://wiki.gnome.org/Community/GettingInTouch/Bugzilla/GettingTraces for instructions on getting a higher-quality backtrace. Be sure to install debuginfo for all the libraries that appear in the backtrace, and take the backtrace in gdb using 'thread apply all bt full'.
Comment 9 Michael Catanzaro 2018-04-27 18:50:23 PDT
(In reply to Ryan Farmer from comment #7)
> I can't get ABRT to come up anymore when this happens, but this is what it
> leaves in my system log. 

Ah, sorry. Fedora makes this easy: just run 'coredumpctl gdb'. Then quit gdb and run the dnf debuginfo-install command it gives you. Then 'coredumpctl gdb' again,  'thread apply all bt full'.
Comment 10 Ryan Farmer 2018-04-27 19:22:40 PDT
Created attachment 339049 [details]
GDB Stack Trace
Comment 11 Ryan Farmer 2018-04-27 19:24:23 PDT
I hope that's better. I had to install a _lot_ of debuginfo packages to get it. 

It still complained that a few were needed, but when I went to install the last batch, dnf says I already have them. :/
Comment 12 Michael Catanzaro 2018-04-28 14:57:51 PDT
(In reply to Ryan Farmer from comment #11)
> I hope that's better. I had to install a _lot_ of debuginfo packages to get
> it. 
> 
> It still complained that a few were needed, but when I went to install the
> last batch, dnf says I already have them. :/

Don't worry, what you posted there is great, thanks. That should be enough for Hyunjun to figure out if he's seeing the same bug as you.

I'll also CC Victor since this appears to be a bug in gst-vaapi.
Comment 13 Hyunjun Ko 2018-04-30 03:17:21 PDT
(In reply to Ryan Farmer from comment #10)
> Created attachment 339049 [details]
> GDB Stack Trace

This is same as mine :)

I'm working on this, refer to the issue https://bugzilla.gnome.org/show_bug.cgi?id=795391.
Comment 14 Michael Catanzaro 2018-04-30 08:11:17 PDT
I guess it's sufficient to track this on GNOME Bugzilla, right? Closing this, then. Thanks Hyunjun!
Comment 15 Hyunjun Ko 2018-05-01 18:20:21 PDT
(In reply to Michael Catanzaro from comment #14)
> I guess it's sufficient to track this on GNOME Bugzilla, right? Closing
> this, then. Thanks Hyunjun!

That's fine. 

@Ryan, for more information, you can test with patches on https://bugzilla.gnome.org/show_bug.cgi?id=793643. (on the master now)
Comment 16 Víctor M. Jáquez L. 2018-05-02 02:29:10 PDT
(In reply to Hyunjun Ko from comment #15)
> (In reply to Michael Catanzaro from comment #14)
> > I guess it's sufficient to track this on GNOME Bugzilla, right? Closing
> > this, then. Thanks Hyunjun!
> 
> That's fine. 
> 
> @Ryan, for more information, you can test with patches on
> https://bugzilla.gnome.org/show_bug.cgi?id=793643. (on the master now)

Also they are merged in the branch 1.14
Comment 17 Ryan Farmer 2018-05-11 10:00:40 PDT
Interestingly, I can't make this crash happen no matter what I do. 

Something in WebkitGTK 2.20.2 appears to have fixed it?
Comment 18 Michael Catanzaro 2018-05-15 09:19:14 PDT
*** Bug 185642 has been marked as a duplicate of this bug. ***