Closed Bug 567954 Opened 14 years ago Closed 3 years ago

Intermittent failure in test_mixed_principals.html | Failed to throw exception in toDataURL for v1

Categories

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

Product:
Core
Component:
Audio/Video: Playback
Type:
defect

Tracking

()

Status:
RESOLVED INCOMPLETE
Milestone:
mozilla52
Tracking Flags:
Tracking Status
blocking2.0 --- -
firefox50 --- disabled
firefox51 --- disabled
firefox52 --- fixed

People

(Reporter: philor, Unassigned)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file, 3 obsolete files)

Bug 567954 - Use onloadeddata to contorl the suspend and resume of MediaResource. .
(deleted), text/x-review-board-request
jwwang
: review+
Details
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1274758494.1274759712.1228.gz Linux mozilla-central debug test mochitests-1/5 on 2010/05/24 20:34:54 s: moz2-linux-slave28 45051 INFO Running /tests/content/media/test/test_mixed_principals.html... WARNING: NS_ENSURE_TRUE(mSuspendCount > 0) failed: file /builds/slave/mozilla-central-linux-debug/build/netwerk/protocol/http/src/nsHttpChannel.cpp, line 4219 WARNING: NS_ENSURE_TRUE(mSuspendCount > 0) failed: file /builds/slave/mozilla-central-linux-debug/build/netwerk/protocol/http/src/nsHttpChannel.cpp, line 4219 WARNING: NS_ENSURE_TRUE(mSuspendCount > 0) failed: file /builds/slave/mozilla-central-linux-debug/build/netwerk/protocol/http/src/nsHttpChannel.cpp, line 4219 WARNING: NS_ENSURE_TRUE(mSuspendCount > 0) failed: file /builds/slave/mozilla-central-linux-debug/build/netwerk/protocol/http/src/nsHttpChannel.cpp, line 4219 WARNING: NS_ENSURE_TRUE(mSuspendCount > 0) failed: file /builds/slave/mozilla-central-linux-debug/build/netwerk/protocol/http/src/nsHttpChannel.cpp, line 4219 WARNING: NS_ENSURE_TRUE(mSuspendCount > 0) failed: file /builds/slave/mozilla-central-linux-debug/build/netwerk/protocol/http/src/nsHttpChannel.cpp, line 4219 WARNING: NS_ENSURE_TRUE(mSuspendCount > 0) failed: file /builds/slave/mozilla-central-linux-debug/build/netwerk/protocol/http/src/nsHttpChannel.cpp, line 4219 WARNING: NS_ENSURE_TRUE(mSuspendCount > 0) failed: file /builds/slave/mozilla-central-linux-debug/build/netwerk/protocol/http/src/nsHttpChannel.cpp, line 4219 ++DOMWINDOW == 16 (0xcd09434) [serial = 1409] [outer = 0xaa7caf8] file=[xpconnect wrapped nsILocalFile @ 0xd54c598 (native @ 0xd54c518)] file=[xpconnect wrapped nsILocalFile @ 0xbf3c3a0 (native @ 0xbf3c320)] --DOMWINDOW == 15 (0xd87c64c) [serial = 1408] [outer = 0xaa7caf8] [url = http://mochi.test:8888/tests/content/media/test/test_media_selection.html] --DOMWINDOW == 14 (0xce73ba4) [serial = 1407] [outer = 0xaa7caf8] [url = http://mochi.test:8888/tests/content/media/test/test_load_candidates.html] --DOMWINDOW == 13 (0xe62ebdc) [serial = 1406] [outer = 0xaa7caf8] [url = http://mochi.test:8888/tests/content/media/test/test_load.html] 45052 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_mixed_principals.html | Failed to throw exception in toDataURL for v1
blocking2.0: --- → ?
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox-Release/1281104322.1281104900.2233.gz Rev3 Fedora 12x64 release-mozilla-central opt test mochitests-1/5 on 2010/08/06 07:18:42 s: talos-r3-fed64-049 49015 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_mixed_principals.html | Failed to throw exception in toDataURL for v1
We have hit this on beta 3 build 3.
philringnalda%gmail.com http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1281214486.1281216115.1649.gz Rev3 Fedora 12x64 mozilla-central opt test mochitests-1/5 on 2010/08/07 13:54:46 s: talos-r3-fed64-024 49042 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_mixed_principals.html | Failed to throw exception in toDataURL for v1
philringnalda%gmail.com http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1281284399.1281285085.27934.gz Rev3 Fedora 12x64 mozilla-central opt test mochitests-1/5 on 2010/08/08 09:19:59 s: talos-r3-fed64-024 49058 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_mixed_principals.html | Failed to throw exception in toDataURL for v1
philringnalda%gmail.com http://tinderbox.mozilla.org/showlog.cgi?log=TraceMonkey/1281249037.1281249629.11819.gz Rev3 Fedora 12 tracemonkey opt test mochitests-1/5 on 2010/08/07 23:30:37 s: talos-r3-fed-035 49048 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_mixed_principals.html | Failed to throw exception in toDataURL for v1
philringnalda%gmail.com http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1281295768.1281296458.6365.gz Rev3 Fedora 12 mozilla-central opt test mochitests-1/5 on 2010/08/08 12:29:28 s: talos-r3-fed-050 49043 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_mixed_principals.html | Failed to throw exception in toDataURL for v1
philringnalda%gmail.com http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1281297137.1281297923.12618.gz Rev3 Fedora 12x64 mozilla-central opt test mochitests-1/5 on 2010/08/08 12:52:17 s: talos-r3-fed64-038 49028 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_mixed_principals.html | Failed to throw exception in toDataURL for v1
philringnalda%gmail.com http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1281521356.1281521960.24889.gz Rev3 Fedora 12 mozilla-central opt test mochitests-1/5 on 2010/08/11 03:09:16 s: talos-r3-fed-025 49033 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_mixed_principals.html | Failed to throw exception in toDataURL for v1
philringnalda%gmail.com http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1281520557.1281521231.20955.gz Rev3 Fedora 12x64 mozilla-central opt test mochitests-1/5 on 2010/08/11 02:55:57 s: talos-r3-fed64-045 49026 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_mixed_principals.html | Failed to throw exception in toDataURL for v1
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1281557325.1281558322.6158.gz Rev3 WINNT 6.1 mozilla-central opt test mochitests-1/5 49779 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_mixed_principals.html | Failed to throw exception in toDataURL for v1
philringnalda%gmail.com http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1281561143.1281561696.22748.gz Rev3 Fedora 12x64 mozilla-central opt test mochitests-1/5 on 2010/08/11 14:12:23 s: talos-r3-fed64-003 49031 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_mixed_principals.html | Failed to throw exception in toDataURL for v1
OS: Linux → All
Hardware: x86 → All
Does anyone who cares about this test want to rewrite it so it skips this part of the test, or should I just disable the entire test once the tree reopens?
philringnalda%gmail.com http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1281936449.1281937002.1352.gz Rev3 Fedora 12x64 mozilla-central opt test mochitests-1/5 on 2010/08/15 22:27:29 s: talos-r3-fed64-019 49061 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_mixed_principals.html | Failed to throw exception in toDataURL for v1
philringnalda%gmail.com http://tinderbox.mozilla.org/showlog.cgi?log=TraceMonkey/1282267707.1282268326.4959.gz Rev3 Fedora 12 tracemonkey opt test mochitests-1/5 on 2010/08/19 18:28:27 s: talos-r3-fed-025 49076 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_mixed_principals.html | Failed to throw exception in toDataURL for v1
philringnalda%gmail.com http://tinderbox.mozilla.org/showlog.cgi?log=TraceMonkey/1282267430.1282267967.3589.gz Rev3 Fedora 12x64 tracemonkey opt test mochitests-1/5 on 2010/08/19 18:23:50 s: talos-r3-fed64-003 49086 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_mixed_principals.html | Failed to throw exception in toDataURL for v1
philringnalda%gmail.com http://tinderbox.mozilla.org/showlog.cgi?log=MozillaTry/1282266355.1282267036.31617.gz Rev3 Fedora 12x64 tryserver opt test mochitests-1/5 on 2010/08/19 18:05:55 s: talos-r3-fed64-012 51610 ERROR TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_mixed_principals.html | Failed to throw exception in toDataURL for v1
Summary: Intermittent failure in test_mixed_principals.html | Failed to throw exception in toDataURL for v1 → [test disabled on mozilla-central] Intermittent failure in test_mixed_principals.html | Failed to throw exception in toDataURL for v1
Depends on: 590028
Whiteboard: [orange] → [orange][test disabled]
Whiteboard: [orange][test disabled] → [test disabled]
We can still hit this failure on Android debug: https://tbpl.mozilla.org/php/getParsedLog.php?id=41248365&tree=Try Also, this intermittently leaks on Linux debug: https://tbpl.mozilla.org/php/getParsedLog.php?id=41255678&full=1&branch=try#error0 https://tbpl.mozilla.org/php/getParsedLog.php?id=41254000&tree=Try&full=1#error0
Summary: [test disabled on mozilla-central] Intermittent failure in test_mixed_principals.html | Failed to throw exception in toDataURL for v1 → Intermittent failure in test_mixed_principals.html | Failed to throw exception in toDataURL for v1
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #329) > Also, this intermittently leaks on Linux debug: > https://tbpl.mozilla.org/php/getParsedLog. > php?id=41255678&full=1&branch=try#error0 > https://tbpl.mozilla.org/php/getParsedLog. > php?id=41254000&tree=Try&full=1#error0 18:39:05 INFO - Leaked URLs: 18:39:05 INFO - http://example.org/tests/content/media/test/dynamic_redirect.sjs?key=v2_51515434&res=seek.ogv 18:39:05 INFO - http://mochi.test:8888/tests/content/media/test/test_mixed_principals.html 18:39:05 INFO - http://mochi.test:8888/tests/content/media/test/seek.ogv 18:39:05 INFO - http://example.org/tests/content/media/test/dynamic_redirect.sjs?key=v2_51515434&res=seek.ogv
(In reply to TBPL Robot from comment #331) > 06-07 17:34:41.508 766 766 I GeckoDump: 1406 INFO TEST-UNEXPECTED-FAIL | > /tests/content/media/test/test_mixed_principals.html | Failed to throw > exception in toDataURL for v1 Doh. Will disable on B2G as well.
Turns out that given enough opportunities to do so, it'll eventually leak on Windows too. I give up. Re-disabled across the board. https://hg.mozilla.org/integration/mozilla-inbound/rev/55f5561a23ec
Assignee: nobody → ryanvm
Go home mcMerge, you're drunk.
Assignee: ryanvm → nobody
Component: Audio/Video → Audio/Video: Playback
Priority: -- → P5
Comment on attachment 8786583 [details] Bug 567954 - Change the media.cache.size to force the test case trigger second HttpRequest. . https://reviewboard.mozilla.org/r/75484/#review73470 ::: dom/media/test/test_mixed_principals.html:23 (Diff revision 1) > <video id="v2" autoplay onended="loaded('v2')"></video> > > <pre id="test"> > <script type="text/javascript"> > SimpleTest.waitForExplicitFinish(); > +// This test rely on an assumption that when Ogg do a seek to end and then go The test case is fragile because: 1. it relys on the particular implementation of ogg decoder. 2. getPlayableVideo() doesn't necessarily return an ogg file. 3. we have changed the implementation of ogg decoder (bug 1168674). I think we should re-write the whole test case to be more robust and cover more test files other than ogg.
Attachment #8786583 - Flags: review?(jwwang) → review-
Comment on attachment 8786582 [details] Bug 567954 - Correct coding style. . https://reviewboard.mozilla.org/r/75482/#review73472
Attachment #8786582 - Flags: review?(jwwang) → review+
Attachment #8786582 - Attachment is obsolete: true
Attachment #8786583 - Attachment is obsolete: true
Attachment #8786584 - Attachment is obsolete: true
Attachment #8786584 - Flags: review?(jwwang)
Comment on attachment 8800129 [details] Bug 567954 - Use onloadeddata to contorl the suspend and resume of MediaResource. . https://reviewboard.mozilla.org/r/85144/#review83708 ::: dom/media/test/test_mixed_principals.html:72 (Diff revision 1) > - // In v1, try loading from same-origin first and then getting redirected to > + // In v1, try loading from same-origin first and then getting redirected to > - // another origin. > + // another origin. > - v1.src = "http://mochi.test:8888/tests/dom/media/test/dynamic_redirect.sjs?key=v1_" + key + "&res=" + resource.name; > + v1.src = > + "http://mochi.test:8888/tests/dom/media/test/dynamic_redirect.sjs?key=v1_" + > + key + "&res=" + resource.name; > - v1.load(); > + v1.load(); Don't need to call load() because setting 'src' will run the load algorithm. ::: dom/media/test/test_mixed_principals.html:81 (Diff revision 1) > + } > > - // In v2, try loading cross-origin first and then getting redirected to > + // In v2, try loading cross-origin first and then getting redirected to > - // our origin. > + // our origin. > - v2.src = "http://example.org/tests/dom/media/test/dynamic_redirect.sjs?key=v2_" + key + "&res=" + resource.name; > + v2.src = "http://example.org/tests/dom/media/test/dynamic_redirect.sjs?key=v2_" + key + "&res=" + resource.name; > - v2.load(); > + v2.load(); Ditto. ::: dom/media/test/test_mixed_principals.html:89 (Diff revision 1) > - todo(false, "No types supported"); > + v2.play(); > + } > + }); > } > + > +testMixedPrincipals(getPlayableVideo(gPrincipalTest)) Just specify "pixel_aspect_ratio.mp4" in this test case. Because it is the exact and only one file we want to test.
Attachment #8800129 - Flags: review?(jwwang) → review+
Pushed by cbook@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/fe84473a739c Use onloadeddata to contorl the suspend and resume of MediaResource. r=jwwang.
Keywords: checkin-needed
sorry had to back this out for test failures like https://treeherder.mozilla.org/logviewer.html#?job_id=4993702&repo=autoland
Flags: needinfo?(ctai)
Backout by cbook@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/84286e454b4e Backed out changeset fe84473a739c for failing mda tests due to own test
Looks like the time out can easily reproduce in XP. I will take a look.
Flags: needinfo?(ctai)
Assignee: nobody → ctai
Disable the test case in XP and file another bug(bug 1311231) for XP case. (In reply to Chia-hung Tai [:ctai :ctai_mozilla :cht] from comment #348) > Comment on attachment 8800129 [details] > Bug 567954 - Use onloadeddata to contorl the suspend and resume of > MediaResource. . > > Review request updated; see interdiff: > https://reviewboard.mozilla.org/r/85144/diff/3-4/
Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/autoland/rev/626eb0f05882 Use onloadeddata to contorl the suspend and resume of MediaResource. r=jwwang.
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/626eb0f05882
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox52: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
status-firefox50: --- → disabled
status-firefox51: --- → disabled
Whiteboard: [test disabled]
Assignee: ctai → nobody

This test was rewritten in https://hg.mozilla.org/mozilla-central/rev/482dbcc619ce83241380b6e9376b9f9d1014b433 but bug 1736934 might plausible be due to a similar flaw.

Status: REOPENED → RESOLVED
Closed: 8 years ago → 3 years ago
Resolution: --- → INCOMPLETE
See Also: → https://bugzilla.mozilla.org/show_bug.cgi?id=1736934
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: