Closed Bug 1048628 Opened 11 years ago Closed 9 years ago

dom/media/test/crashtests/0-timescale.html is perma-fail on Android

Categories

(Core :: Audio/Video: Playback, defect)

Product:
Core
Component:
Audio/Video: Playback
Platform:
ARM
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: RyanVM, Assigned: esawin)

References

Details

Attachments

(2 files, 1 obsolete file)

0002-Bug-1048628-2.1-Enabled-autoplay-in-0-timescale-test.patch
677 bytes, patch
snorp
: review+
Details | Diff | Splinter Review
0001-Bug-1048628-1.2-Enabled-0-timescale-test-on-Android-.patch
578 bytes, patch
esawin
: review+
Details | Diff | Splinter Review
In bug 1048624, I'm trying to re-enable the content/media crashtests on Android (which are currently all being skipped). In the course of that testing, I found that 0-timescale.html currently perma-fails per the log below. I plan to skip the test for now while the cause is investigated and a fix landed. https://tbpl.mozilla.org/php/getParsedLog.php?id=45177140&tree=Try Android 4.0 Panda try opt test crashtest on 2014-08-04 06:54:26 PDT for push 669303742d9e slave: panda-0742 07:06:42 INFO - REFTEST TEST-START | http://10.26.135.20:30742/tests/content/media/test/crashtests/0-timescale.html 07:06:42 INFO - REFTEST TEST-LOAD | http://10.26.135.20:30742/tests/content/media/test/crashtests/0-timescale.html | 231 / 2704 (8%) 07:11:43 INFO - REFTEST TEST-UNEXPECTED-FAIL | http://10.26.135.20:30742/tests/content/media/test/crashtests/0-timescale.html | load failed: timed out waiting for reftest-wait to be removed 07:11:43 INFO - REFTEST INFO | Saved log: START http://10.26.135.20:30742/tests/content/media/test/crashtests/0-timescale.html 07:11:43 INFO - REFTEST INFO | Saved log: [CONTENT] OnDocumentLoad triggering WaitForTestEnd 07:11:43 INFO - REFTEST INFO | Saved log: [CONTENT] WaitForTestEnd: Adding listeners 07:11:43 INFO - REFTEST INFO | Saved log: Initializing canvas snapshot 07:11:43 INFO - REFTEST INFO | Saved log: [CONTENT] MakeProgress: STATE_WAITING_TO_FIRE_INVALIDATE_EVENT 07:11:43 INFO - REFTEST INFO | Saved log: [CONTENT] MakeProgress: dispatching MozReftestInvalidate 07:11:43 INFO - REFTEST INFO | Saved log: [CONTENT] MakeProgress: STATE_WAITING_FOR_REFTEST_WAIT_REMOVAL 07:11:43 INFO - REFTEST INFO | Saved log: [CONTENT] MakeProgress: waiting for reftest-wait to be removed 07:11:43 INFO - REFTEST INFO | Saved log: [CONTENT] AfterPaintListener in http://10.26.135.20:30742/tests/content/media/test/crashtests/0-timescale.html 07:11:43 INFO - REFTEST INFO | Saved log: [CONTENT] SendUpdateCanvasForEvent with 1 rects 07:11:43 INFO - REFTEST INFO | Saved log: [CONTENT] Rect: 0 0 800 672 07:11:43 INFO - REFTEST INFO | Saved log: Updating canvas for invalidation 07:11:43 INFO - REFTEST INFO | Saved log: [CONTENT] MakeProgress: STATE_WAITING_FOR_REFTEST_WAIT_REMOVAL 07:11:43 INFO - REFTEST INFO | Saved log: [CONTENT] MakeProgress: waiting for reftest-wait to be removed 07:11:43 INFO - REFTEST INFO | Loading a blank page 07:11:43 INFO - REFTEST TEST-END | http://10.26.135.20:30742/tests/content/media/test/crashtests/0-timescale.html
It's worth noting that this is already being skipped on B2G as well.
This test should be disabled for Android because the root cause is the libstagefright library provided by the platform.
Test disabled on Android. https://hg.mozilla.org/integration/mozilla-inbound/rev/446650efe0b6
Assignee: nobody → ryanvm
Whiteboard: [test disabled on Android][leave open]
Assignee: ryanvm → nobody
Still permafail. Though based on comment 2, I suppose this should just be wontfix?
Summary: content/media/test/crashtests/0-timescale.html is perma-fail on Android → dom/media/test/crashtests/0-timescale.html is perma-fail on Android/B2G
Whiteboard: [test disabled on Android][leave open] → [test disabled on Android/B2G][leave open]
It should be fine on Android 4.1 and later. I'm guessing our infrastructure needs updating.
It still fails on the Android 4.3 emulator runs. Any suggestions for what needs updating so it can be enabled?
James - this test should be testing the same path on 4.3 Android as it is on desktop. Want to take a peek?
Flags: needinfo?(snorp)
Component: Audio/Video → Audio/Video: Playback
Assignee: nobody → esawin
Flags: needinfo?(snorp)
On Fennec, we don't try to parse metadata on page load, we only do so with autoplay enabled or on explicit request.
Attachment #8692937 - Flags: review?(snorp)
Comment on attachment 8692936 [details] [diff] [review] 0001-Bug-1048628-1.1-Enabled-0-timescale-test-on-Android-.patch Review of attachment 8692936 [details] [diff] [review]: ----------------------------------------------------------------- ::: dom/media/test/crashtests/crashtests.list @@ +1,1 @@ > +load 0-timescale.html # bug 1048628 Please remove the comment pointing to this bug as well.
I would propose we only enable it on Android for now and investigate the B2G crash in a separate bug.
Comment on attachment 8692936 [details] [diff] [review] 0001-Bug-1048628-1.1-Enabled-0-timescale-test-on-Android-.patch Review of attachment 8692936 [details] [diff] [review]: ----------------------------------------------------------------- Yeah, what RyanVM said.
Attachment #8692936 - Flags: review?(snorp) → review+
Attachment #8692937 - Flags: review?(snorp) → review+
Filed bug to fix this test on B2G: bug 1229166.
Summary: dom/media/test/crashtests/0-timescale.html is perma-fail on Android/B2G → dom/media/test/crashtests/0-timescale.html is perma-fail on Android
Enable test on Android, leave disabled on B2G. Carrying over r+.
Attachment #8692936 - Attachment is obsolete: true
Attachment #8693794 - Flags: review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: [test disabled on Android/B2G][leave open]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: