806139 - Add "avc1.64001F" and "mp4v.20.3" to the list of supported h264 codecs, used by youtube

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[Jan Beich]

+++ This bug was initially created as a clone of Bug #761063 +++

YouTube seems to want "avc1.42001F" (h264 high 3.1) for 1080p/720p and "mp4v.20.3" (3gpp) for 240p videos. Here's an example, https://www.youtube.com/watch?v=hMDBwa4huUY . With media.webm.enabled -> false it lists for me "avc1.64001F, mp4a.40.2" (720p), "avc1.42001E, mp4a.40.2" (360p), "mp4v.20.3, mp4a.40.2" (240p) as available resolution options.

Comment 1

[Jan Beich]

Attachment: patch

Comment 2

[Jan Beich]

(In reply to Jan Beich from comment #0)
> "avc1.42001F"

Obvious typo. Also, if bug 800600 lands first this bug can be marked as DUPLICATE.

Comment 3

[cajbir (:cajbir)]

Comment on attachment 675900 [details] [diff] [review]
patch

Review of attachment 675900 [details] [diff] [review]:
-----------------------------------------------------------------

::: content/html/content/public/nsHTMLMediaElement.h
@@ +310,5 @@
>  #ifdef MOZ_GSTREAMER
>    static bool IsH264Enabled();
>    static bool IsH264Type(const nsACString& aType);
>    static const char gH264Types[3][16];
> +  static char const *const gH264Codecs[9];

You'll need to change this in the MOZ_WIDGET_GONK block below too. Or split this out into a block both defines share. r+ with that change.

Comment 4

[Jan Beich]

Attachment: patch, v2

Comment 5

[Jan Beich]

v2 stands for trivial changes (implicitly r+):

  #if defined(MOZ_GSTREAMER) || defined(MOZ_WIDGET_GONK)
    static char const *const gH264Codecs[9];
  #endif

making nsHTMLMediaElement.h and nsHTMLMediaElement.cpp both use same ifdef

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/ed7fb37591cf

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/ed7fb37591cf