Closed
Bug 963621
Opened 10 years ago
Closed 10 years ago
Regression: No video playback (MP4); Samsung Galaxy S4 (Android 4.4.2) (I420ColorConverter)
Categories
(Core :: Audio/Video, defect)
Tracking
()
VERIFIED
FIXED
mozilla29
Tracking | Status | |
---|---|---|
firefox26 | --- | unaffected |
firefox27 | --- | unaffected |
firefox28 | + | verified |
firefox29 | + | fixed |
fennec | 29+ | --- |
People
(Reporter: aaronmt, Assigned: eflores)
References
()
Details
(Keywords: regression, reproducible, testcase)
Attachments
(4 files)
14.52 KB,
text/plain
|
Details | |
14.73 KB,
text/plain
|
Details | |
5.21 KB,
patch
|
cajbir
:
review+
lsblakk
:
approval-mozilla-release+
|
Details | Diff | Splinter Review |
32.05 KB,
text/plain
|
Details |
See attached log-cat for OMX related output. Of particular interest maybe? E/OMX-VDEC-1080P( 209): omx_vdec::component_init(): Start of New Playback : role = OMX.qcom.video.decoder.avc : DEVICE = /dev/msm_vidc_dec E/OMX-VDEC-1080P( 209): omx_vdec::component_init(): Open returned fd 39, errno 2 Playing the video http://images.apple.com/media/us/mac/30-years/2014/feature/mac-30/mac30-feature-us-20140124_640x360h.mp4 will yield no video playback, however audio will work. On Firefox 26/27/28 video playback works. This is reproducible on my Samsung Galaxy S4 (Android 4.4.2) with Nightly (01/24) Last good revision: 0d11fce4f845 (2013-12-16) First bad revision: b1e5ade62913 (2013-12-17) Pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=0d11fce4f845&tochange=b1e5ade62913
Flags: needinfo?(edwin)
Reporter | ||
Comment 1•10 years ago
|
||
Reporter | ||
Updated•10 years ago
|
Attachment #8365145 -
Attachment mime type: text/x-log → text/plain
Assignee | ||
Comment 3•10 years ago
|
||
We don't have an SGS4 here in the Auckland office. Could you try this build? http://people.mozilla.org/~eflores/apk/fennec-29.0a1.en-US.android-arm.apk
Flags: needinfo?(aaron.train)
Reporter | ||
Comment 4•10 years ago
|
||
With the build above, I continue to not see any video playback on my device with the video from comment #0
Flags: needinfo?(aaron.train)
Assignee | ||
Comment 5•10 years ago
|
||
The bug here is with Qualcomm's OMXCodec implementation that hadn't previously surfaced because the hardware decoder was returning a colour format we didn't support at the time. When we pass the kClientNeedsFramebuffer flag (needed by Samsung and others) to OMXCodec, internally it sets a flag called |mThumbnailMode|. Somebody at qcom (presumably) decided it would be efficient to handle this flag in their OMXCodec implementation by returning a single video frame and then return EOS on subsequent |read()| calls. This patch just makes sure we don't pass the flag into Qualcomm decoders.
Attachment #8367035 -
Flags: review?(chris.double)
Updated•10 years ago
|
Attachment #8367035 -
Flags: review?(chris.double) → review+
Updated•10 years ago
|
tracking-fennec: ? → 29+
Reporter | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 6•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/15a8086a2431
Keywords: checkin-needed
Comment 7•10 years ago
|
||
Backed out for Android bustage. https://hg.mozilla.org/integration/mozilla-inbound/rev/c0ee7e389205 https://tbpl.mozilla.org/php/getParsedLog.php?id=33822258&tree=Mozilla-Inbound
Assignee | ||
Comment 8•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/2a7a697f2d88
Comment 9•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/2a7a697f2d88
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Assignee | ||
Comment 10•9 years ago
|
||
Comment on attachment 8367035 [details] [diff] [review] 963621.patch [Approval Request Comment] Regression caused by (bug #): bug 866080 User impact if declined: H264 video doesn't work on "Galaxy S4, Galaxy Note 3 and LG G2" (https://bugzilla.mozilla.org/show_bug.cgi?id=984230#c23) Testing completed (on m-c, etc.): Has been in ff29 since 2014-01-31. Risk to taking this patch (and alternatives if risky): Video continuing to not work on some devices. String or IDL/UUID changes made by this patch:
Attachment #8367035 -
Flags: approval-mozilla-release?
Reporter | ||
Comment 11•9 years ago
|
||
Mind posting a build for me to test with the patch applied on 28?
Assignee | ||
Comment 12•9 years ago
|
||
(In reply to Aaron Train [:aaronmt] from comment #11) > Mind posting a build for me to test with the patch applied on 28? http://people.mozilla.org/~eflores/apk/fennec-28.0.en-US.android-arm.apk
Reporter | ||
Comment 13•9 years ago
|
||
(In reply to Edwin Flores [:eflores] [:edwin] from comment #12) > (In reply to Aaron Train [:aaronmt] from comment #11) > > Mind posting a build for me to test with the patch applied on 28? > > http://people.mozilla.org/~eflores/apk/fennec-28.0.en-US.android-arm.apk Confirmed working on my devices. I still have no idea why when I was testing Aurora 28.0a2 at the time and yesterday again it was not working, oh well. Lukas, I think we should get this out in a point release.
Flags: needinfo?(lsblakk)
Comment 14•9 years ago
|
||
Thanks for the detective work here, I'll get a proposed schedule out tomorrow morning to deal with this in a fennec only dot release.
Flags: needinfo?(lsblakk)
Comment 15•9 years ago
|
||
Comment on attachment 8367035 [details] [diff] [review] 963621.patch Let's get this landed in preparation
Attachment #8367035 -
Flags: approval-mozilla-release? → approval-mozilla-release+
Updated•9 years ago
|
tracking-firefox28:
--- → +
Comment 18•9 years ago
|
||
I'm still seeing this issue - no video playback when playing the video from comment#0 - on Firefox 28.0.1 and Nightly 31.0a1 (2014-04-24) on several devices: Samsung Galaxy Tab 2 (4.2.2) Samsung Galaxy Tab (4.0.4) Samsung Galaxy S3 (4.3) Samsung Galaxy Note (4.0.4) Motorola RAZR XT910 (4.0.4) ZTE x86 (4.0.4) I'll attach a log capture from Galaxy Tab 2 playing http://images.apple.com/media/us/mac/30-years/2014/feature/mac-30/mac30-feature-us-20140124_640x360h.mp4 Also I've tested the files from http://people.mozilla.org/~atrain/mobile/tests/media.html and all of them were played correctly on all devices mentioned above.
Comment 19•9 years ago
|
||
Reporter | ||
Comment 20•9 years ago
|
||
I believe that the aforementioned in comment #18 is a seperate issue and not nothing new in regard to video playback on that particular Apple video. In particular, this bug is focused on Qualcomm chipsets. As in this case, I do not see any frames at all. I see the scrubber and time marker work which wasn't in the case in this particular bug report. Can you please file a new bug? I am able to reproduce an issue with that particular video on my Samsung Galaxy Note II (Android 4.1.2), on all channels including Firefox 27. As for this bug in particular, I can verify the mozilla-release candidate is working.
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 21•9 years ago
|
||
I filed bug 987223.
You need to log in
before you can comment on or make changes to this bug.
Description
•