Bug 174222 – [GStreamer] Live twitch.tv videos do not play

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Charlie Turner]

Not investigated yet, but apparently we're missing a plugin to play whatever twitch.tv is serving in its live streams.

[Charlie Turner]

This was an upstream in bug in typefind, it was incorrectly deducing the type of Twitch HLS streams as text/uri-list. It's fixed by

e68c071bf01c1b4b796c48a87b511005e8c0c9f6 typefind: Switch to normal mode before have-type

I recommend we update our GStreamer dependency. Conservatively can go to 1.10.5, since there's nothing (apart from the fix above) adventurous,

$ git log --pretty=oneline 1.10.4...1.10.5
2bec169793cddeaad35a1e50a0e7d5f1d7946a58 Release 1.10.5
4eb8908fa9589d7ff555c44798f19fa65711acc2 Update .po files
37a9b02391e468226498f26ae8922cea68bb6e22 message: Don't pass a NULL debug string to g_utf8_validate()
e68c071bf01c1b4b796c48a87b511005e8c0c9f6 typefind: Switch to normal mode before have-type
f727d67d5300b85fae405d371dd171b6528eb9ed meson: update version
9c80d89d74244658675ef281b3ceceb32ae1fd66 meson: Fix build with latest upstream git

[Michael Catanzaro]

(In reply to Charlie Turner from comment #1)
> This was an upstream in bug in typefind, it was incorrectly deducing the
> type of Twitch HLS streams as text/uri-list. It's fixed by
> 
> e68c071bf01c1b4b796c48a87b511005e8c0c9f6 typefind: Switch to normal mode
> before have-type
> 
> I recommend we update our GStreamer dependency. Conservatively can go to
> 1.10.5, since there's nothing (apart from the fix above) adventurous,

OK. Do you want to prepare a patch for the WPE and GTK JHBuild modulesets?

[Charlie Turner]

Yep, I am working on that. I will do another test run once I'm finished to make sure there's no regressions.

[Charlie Turner]

This is looking alright to be, current patch review (no changes needed)

<patch file="gstreamer-0001-protection-added-function-to-filter-system-ids.patch" strip="1"/>
bda8440 in version 1.12.0 - still needed

<patch file="gst-plugins-good-0003-rtpbin-receive-bundle-support.patch" strip="1"/>
dcd3ce9 in version 1.11.0 - still needed

<patch file="gst-plugins-good-0004-qtdemux-add-context-for-a-preferred-protection.patch" strip="1"/>
not committed upstream - still needed.

There's some noise on stderr related to things like OpenWebRTC and MediaStream, neither of which has complete implementation, so I don't know whether to make this block the otherwise good upgrade.

[Charlie Turner]

WPE has for some reason lagged behind the previous GStreamer version bumps. I'm not setup yet to adequately test upgrades on WPE, but when I am I will look at upgrading it also.

[Charlie Turner]

Created attachment 314982 [details]
Patch

[Michael Catanzaro]

Comment on attachment 314982 [details]
Patch

Of course, it'd be even better to upgrade to 1.12, so we can test against the software our users are actually running. But that would probably require rebaselining more layout tests.

[WebKit Commit Bot]

Comment on attachment 314982 [details]
Patch

Clearing flags on attachment: 314982

Committed r219292: <http://trac.webkit.org/changeset/219292>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Michael Catanzaro]

Oops, for future reference, we should have done the JHBuild moduleset update in a separate bug report.