Closed
Bug 1383479
Opened 7 years ago
Closed 7 years ago
Can't play any video
Categories
(Firefox for Android Graveyard :: Audio/Video, defect, P1)
Tracking
(firefox54 unaffected, firefox55blocking verified, firefox56+ verified)
VERIFIED
FIXED
Firefox 56
Tracking | Status | |
---|---|---|
firefox54 | --- | unaffected |
firefox55 | blocking | verified |
firefox56 | + | verified |
People
(Reporter: sandken, Assigned: jhlin)
Details
(Keywords: regression, Whiteboard: [testcoverage])
Attachments
(3 files)
311.66 KB,
image/png
|
Details | |
59 bytes,
text/x-review-board-request
|
esawin
:
review+
jcristau
:
approval-mozilla-beta+
|
Details |
131.14 KB,
image/png
|
Details |
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:56.0) Gecko/20100101 Firefox/56.0 Build ID: 20170722120123 Steps to reproduce: Click on any Youtubd video' Actual results: "Buffering" icon shows, video doesn't play. See screenshot. Expected results: Video should have played. Videos play okay on stable 54 and beta 55. I don't know exactly when this happened, but 14 days ago, I could play videos when I encountered a different bug 1379389
Forgot to say I'm using a Verizon Ellipsis 10 tablet, Android 5.1, Verizon software update IR7_20C63.
Comment 2•7 years ago
|
||
Thanks for firing this bug. Can you still see this bug if you reboot your device?
status-firefox54:
--- → unaffected
status-firefox55:
--- → unaffected
status-firefox56:
--- → affected
Keywords: regression
Further investigation shows Firefox Nightly won't play any video.
Summary: Can't play any Youtube video → Can't play any video
(In reply to Blake Wu [:bwu][:blakewu] from comment #2) > Thanks for firing this bug. > Can you still see this bug if you reboot your device? Reboot doesn't fix; problem still exists. Also I've found this is not restricted to Youtube; can't play any video.
Comment 5•7 years ago
|
||
Thanks for this info. John, Could you check this?
Flags: needinfo?(jolin)
Priority: -- → P1
Reproducible on Huawei P10. Regression range https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=1b065ffd8a535a0ad4c39a912af18e948e6a42c1&tochange=0985725c848ec0cfc6f2f3c3a5aa3d71321e7620
Assignee | ||
Comment 7•7 years ago
|
||
(In reply to Anthony Jones (:kentuckyfriedtakahe, :k17e) from comment #6) > Reproducible on Huawei P10. Regression range > https://hg.mozilla.org/mozilla-central/ > pushloghtml?fromchange=1b065ffd8a535a0ad4c39a912af18e948e6a42c1&tochange=0985 > 725c848ec0cfc6f2f3c3a5aa3d71321e7620 Do you by any chance have the logcat dump to help investigation?
Flags: needinfo?(jolin)
Updated•7 years ago
|
Flags: needinfo?(ajones)
[Tracking Requested - why for this release]: This issue breaks all playback on some handsets including Huawei's flagship P10 phone.
What phone are you having this issue on?
Flags: needinfo?(sandken)
Updated•7 years ago
|
Whiteboard: [testcoverage]
Comment hidden (mozreview-request) |
Assignee | ||
Comment 12•7 years ago
|
||
Thanks to the excellent support from ssergiu@softvision, I've identify the root cause. This is a regression caused by the patch in bug 1365205, which rejects null output buffers. Unfortunately, some decoders return null output buffers when it's configured with output surface and all output buffers they produce were considered invalid [1]. [1] http://searchfox.org/mozilla-central/source/mobile/android/geckoview/src/main/java/org/mozilla/gecko/media/Codec.java#267
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → jolin
Comment 13•7 years ago
|
||
Sounds like this might be a blocker for 55 if it affects popular phones.
Reporter | ||
Comment 14•7 years ago
|
||
(In reply to Anthony Jones (:kentuckyfriedtakahe, :k17e) from comment #10) > What phone are you having this issue on? Verizon Ellipsis 10 tablet -desktop mode - Android 5.1 - Verizon software update level IR7_20C63.
Flags: needinfo?(sandken)
Comment 15•7 years ago
|
||
mozreview-review |
Comment on attachment 8890717 [details] Bug 1383479 - allow null output buffers when codec is configured with output surface. https://reviewboard.mozilla.org/r/161918/#review167770 I feel responsible for this one, good find!
Attachment #8890717 -
Flags: review?(esawin) → review+
Comment 16•7 years ago
|
||
Pushed by jolin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/59507e1a1b96 allow null output buffers when codec is configured with output surface. r=esawin
Comment 17•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/59507e1a1b96
Status: UNCONFIRMED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
Comment 18•7 years ago
|
||
Hi John, Could you please help request beta uplift approval right away today's morningin?
Flags: needinfo?(jolin)
Comment 19•7 years ago
|
||
(In reply to Hsin-Yi Tsai (55 Regression Engineering support) [:hsinyi] from comment #18) > Hi John, > Could you please help request beta uplift approval right away today's > morningin? typo: s/morningin/morning We will have to make it uplifted today's afternoon.
Assignee | ||
Comment 20•7 years ago
|
||
Comment on attachment 8890717 [details] Bug 1383479 - allow null output buffers when codec is configured with output surface. Approval Request Comment [Feature/Bug causing the regression]: 1365205 [User impact if declined]: cannot play video on certain devices [Is this code covered by automated tests?]:no [Has the fix been verified in Nightly?]:yes [Needs manual test from QE? If yes, steps to reproduce]: Yes. Play any video on Huawei P10 and see if works. [List of other uplifts needed for the feature/fix]:none [Is the change risky?]:low risk [Why is the change risky/not risky?]:for video decoding, only the buffer info is needed rather than its contents. [String changes made/needed]:none
Flags: needinfo?(jolin)
Attachment #8890717 -
Flags: approval-mozilla-beta?
Comment 21•7 years ago
|
||
Comment on attachment 8890717 [details] Bug 1383479 - allow null output buffers when codec is configured with output surface. video regression fix for fennec 55.0b14
Attachment #8890717 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 22•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/0f9963b51665
Comment 23•7 years ago
|
||
Devices: - Huawei P10 (Android 7.0); - Huawei MediaPad M2 (Adnroid 5.1.1). Builds: - Beta - 55.0b14; - Nightly - 56.0a1 (2017-07-31); Hello, Verified this issue and it is no longer reproducible in beta. Nightly still seems to be affected on the P10.
Assignee | ||
Comment 24•7 years ago
|
||
(In reply to Bogdan Surd, QA [:BogdanS] from comment #23) > Nightly still seems to be affected on the P10. Thanks a lot for the verification. Could you please type 'about:buildconfig' in the address bar of Nightly and upload the screenshot?
Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(bogdan.surd)
Assignee | ||
Comment 26•7 years ago
|
||
Bogdan, thanks a lot for the prompt support. It looks like the Nightly you tested was built from 2017-07-26 changeset which doesn't include the needed patch. Could you please download and install the APK from [1], and see if it is still affected? [1] https://archive.mozilla.org/pub/mobile/nightly/latest-mozilla-central-android-api-15/fennec-56.0a1.multi.android-arm.apk
Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(bogdan.surd)
Comment 27•7 years ago
|
||
Hello, Thanks John, with the apk provided video playback works without any issue for both mobile and desktop versions of the site. Marking as verified.
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•