For problems related to media (video/audio) -- especially when the reporter is not sure which area of the media stack the problem is in. This category will mostly be for untriaged video/audio issues. Bugs in this category will typically be moved to another media category during the triage process.

User Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:26.0) Gecko/20100101 Firefox/26.0 (Beta/Release)
Build ID: 20131205075310
Steps to reproduce:
Open
http://www.tagesschau.de/ausland/mexiko508.html
with my MotoG
Klick the Play Button
Actual results:
The video starts, but I can only hear the sound, the sreen is white.
This happens on all videos on tagesschau.de
Expected results:
The video should play with picture and sound
(And, if possible Firefox shouldn’t crash while doing :-) , as it does on my Asus Memopad HD 7 )

Comment on attachment 8356424[details][diff][review]
953394.patch
Review of attachment 8356424[details][diff][review]:
-----------------------------------------------------------------
::: content/media/plugins/MediaPluginHost.cpp
@@ +181,5 @@
> }
> + else if (version < 19) {
> + // Ice Cream Sandwich and Jellybean
> + return "libomxplugin.so";
> + }
The versions checks are currently in numerical order. Can you refactor so that the base case is libmoxplugin.so and the version check is for kitkat as the first check (since whatever version kitkat is is the highest).
::: media/omx-plugin/OmxPlugin.cpp
@@ +293,5 @@
> + * Look for a decoder that supports a colour format that we support.
> + */
> +static bool
> +FindPreferredDecoderAndColorFormat(const sp<IOMX>& aOmx,
> + char *decoderName,
I would prefer this take a 'length' parameter as well which is the size of the decoderName array. Then in the strncpy use that argument instead of MAX_DECODER_NAME_LEN. When calling this function pass in sizeof(decoderName) as that argument value.
@@ +318,5 @@
> + ColorFormatSupport supported = IsColorFormatSupported(color);
> +
> + if (supported) {
> + strncpy(decoderName, caps.mComponentName.string(), MAX_DECODER_NAME_LEN);
> + *colorFormat = (OMX_COLOR_FORMATTYPE)color;
Why is this cast needed?

Comment on attachment 8356424[details][diff][review]
953394.patch
[Approval Request Comment]
Bug caused by (feature/regressing bug #): Android
User impact if declined: No h264 playback on many high end KitKat devices.
Testing completed (on m-c, etc.): Has been on m-c for a week
Risk to taking this patch (and alternatives if risky): Small risk of breaking h.264 on other KitKat devices.
String or IDL/UUID changes made by this patch: None

(In reply to Edwin Flores [:eflores] [:edwin] from comment #13)
> Comment on attachment 8356424[details][diff][review]
> 953394.patch
>
> [Approval Request Comment]
> Bug caused by (feature/regressing bug #): Android
> User impact if declined: No h264 playback on many high end KitKat devices.
> Testing completed (on m-c, etc.): Has been on m-c for a week
> Risk to taking this patch (and alternatives if risky): Small risk of
> breaking h.264 on other KitKat devices.
> String or IDL/UUID changes made by this patch: None
Is this a new regression or did we have support of h264 in Firefox 26 and this is Fx27 regression ? Its too late in Fx27 cycle given we only have our final beta left, not convinced it is a good candidate since we usually only low risk very well tested patches on beta.
Approving for aurora though.

Comment on attachment 8366211[details][diff][review]
953394.patch rebased to Aurora
Not sure revision of Aurora what this was rebased on because I'm still hitting the same failed hunks as before.
patching file media/omx-plugin/OmxPlugin.cpp
Hunk #2 FAILED at 236
Hunk #3 FAILED at 734
2 out of 3 hunks FAILED -- saving rejects to file media/omx-plugin/OmxPlugin.cpp.rej

(In reply to Ryan VanderMeulen [:RyanVM UTC-5] from comment #19)
> Not sure revision of Aurora what this was rebased on because I'm still
> hitting the same failed hunks as before.
Seems to work for me. Is this on top of the patch from bug 866080, above?