Closed Bug 770063 Opened 13 years ago Closed 13 years ago

Intermittent test_sandbox.xul | Check number of loaded content types: application/javascript,text/html - got 2 (or 3), expected 4; or Check * was not loaded - -1 should equal -1

Categories

(Core Graveyard :: Identity, defect)

Product:
Core Graveyard
Component:
Identity
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox15 unaffected, firefox16 affected, firefox17 affected)

Status:
RESOLVED FIXED
Milestone:
mozilla16
Tracking Flags:
Tracking Status
firefox15 --- unaffected
firefox16 --- affected
firefox17 --- affected

People

(Reporter: philor, Assigned: MattN)

References

Details

(Keywords: intermittent-failure)

Attachments

(2 files)

Disable most of check_loaded_content on 10.5
1.17 KB, patch
Dolske
: review+
Details | Diff | Splinter Review
disable intermittently passing todo tests
1.38 KB, patch
MattN
: review+
Details | Diff | Splinter Review
https://tbpl.mozilla.org/php/getParsedLog.php?id=13149210&tree=Firefox Rev3 MacOSX Leopard 10.5.8 mozilla-central debug test mochitest-other on 2012-07-01 09:06:26 PDT for push 3be950fe9e1e slave: talos-r3-leopard-009 27544 INFO TEST-PASS | chrome://mochitests/content/chrome/toolkit/identity/tests/chrome/test_sandbox.xul | Check successful response - 200 should equal 200 27545 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/toolkit/identity/tests/chrome/test_sandbox.xul | Check number of loaded content types: application/javascript,text/html - got 2, expected 4 27546 ERROR TEST-UNEXPECTED-PASS | chrome://mochitests/content/chrome/toolkit/identity/tests/chrome/test_sandbox.xul | Check number of loaded content types: application/javascript,text/html - 2 should equal 2 27547 INFO TEST-PASS | chrome://mochitests/content/chrome/toolkit/identity/tests/chrome/test_sandbox.xul | Check JS was loaded - 0 should not equal -1 27548 INFO TEST-PASS | chrome://mochitests/content/chrome/toolkit/identity/tests/chrome/test_sandbox.xul | Check iframe was loaded - 1 should not equal -1 27549 ERROR TEST-UNEXPECTED-PASS | chrome://mochitests/content/chrome/toolkit/identity/tests/chrome/test_sandbox.xul | Check webm was not loaded - -1 should equal -1 27550 ERROR TEST-UNEXPECTED-PASS | chrome://mochitests/content/chrome/toolkit/identity/tests/chrome/test_sandbox.xul | Check ogg was not loaded - -1 should equal -1 Sandbox: free: 1657 https://tbpl.mozilla.org/php/getParsedLog.php?id=13150845&tree=Mozilla-Inbound Rev3 MacOSX Leopard 10.5.8 mozilla-inbound debug test mochitest-other on 2012-07-01 12:47:02 PDT for push d444d01ef71c slave: talos-r3-leopard-040 27599 INFO TEST-PASS | chrome://mochitests/content/chrome/toolkit/identity/tests/chrome/test_sandbox.xul | Check successful response - 200 should equal 200 27600 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/toolkit/identity/tests/chrome/test_sandbox.xul | Check number of loaded content types: application/javascript,text/html - got 2, expected 4 27601 ERROR TEST-UNEXPECTED-PASS | chrome://mochitests/content/chrome/toolkit/identity/tests/chrome/test_sandbox.xul | Check number of loaded content types: application/javascript,text/html - 2 should equal 2 27602 INFO TEST-PASS | chrome://mochitests/content/chrome/toolkit/identity/tests/chrome/test_sandbox.xul | Check JS was loaded - 0 should not equal -1 27603 INFO TEST-PASS | chrome://mochitests/content/chrome/toolkit/identity/tests/chrome/test_sandbox.xul | Check iframe was loaded - 1 should not equal -1 27604 ERROR TEST-UNEXPECTED-PASS | chrome://mochitests/content/chrome/toolkit/identity/tests/chrome/test_sandbox.xul | Check webm was not loaded - -1 should equal -1 27605 ERROR TEST-UNEXPECTED-PASS | chrome://mochitests/content/chrome/toolkit/identity/tests/chrome/test_sandbox.xul | Check ogg was not loaded - -1 should equal -1 27606 INFO TEST-INFO | chrome://mochitests/content/chrome/toolkit/identity/tests/chrome/test_sandbox.xul | reload the sandbox content Sandbox: reload: 1679 : http://mochi.test:8888/chrome/toolkit/identity/tests/chrome/sandbox_content_redirect.html ++DOMWINDOW == 156 (0x3b25d85c) [serial = 1684] [outer = 0x33e90d20] Sandbox: _makeSandboxContentLoaded : 1679 : http://mochi.test:8888/chrome/toolkit/identity/tests/chrome/sandbox_content.html Sandbox: reloaded sandbox id: : 1679
Assignee: nobody → mnoorenberghe+bmo
Status: NEW → ASSIGNED
I increased the timeout interval for now to reduce this: https://hg.mozilla.org/integration/mozilla-inbound/rev/1dca83015433
Target Milestone: --- → mozilla16
MattN, increasing the timeout limit doesn't seem to have helped. Any ideas?
(In reply to Ed Morley [:edmorley] from comment #90) > MattN, increasing the timeout limit doesn't seem to have helped. Any ideas? I think it did help since it reduced the failures but not as much as I thought. I made a patch to disable this portion of the test for 10.5 since that's where about 90% of the failures are and I don't have time to figure out the real cause at the moment. Currently going through a try run: https://hg.mozilla.org/try/rev/67f31f74c489
Not sure why it's been silent here the last 13 hours but we should probably disable this on 10.5 anyways.
Attachment #642178 - Flags: review?(dolske)
Comment on attachment 642178 [details] [diff] [review] Disable most of check_loaded_content on 10.5 Thankfully there's no way this can break on 10.6 or 10.7... ahem. But good enough for now!
Attachment #642178 - Flags: review?(dolske) → review+
> Comment on attachment 642178 [details] [diff] [review] > Disable most of check_loaded_content on 10.5 https://hg.mozilla.org/integration/mozilla-inbound/rev/7551b8fb7564
https://hg.mozilla.org/mozilla-central/rev/7551b8fb7564
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Summary: Intermittent test_sandbox.xul | Check number of loaded content types: application/javascript,text/html - got 2, expected 4 and three more failures → Intermittent test_sandbox.xul | Check number of loaded content types: application/javascript,text/html - got 2 (or 3), expected 4; and more failures
MattN, do you have any ideas for this one? (It is appearing on the orangefactor top oranges list)
Summary: Intermittent test_sandbox.xul | Check number of loaded content types: application/javascript,text/html - got 2 (or 3), expected 4; and more failures → Intermittent test_sandbox.xul | Check number of loaded content types: application/javascript,text/html - got 2 (or 3), expected 4; or Check * was not loaded - -1 should equal -1
MattN, ping for comment 196.
(In reply to Ed Morley [:edmorley] from comment #196) > MattN, do you have any ideas for this one? > (It is appearing on the orangefactor top oranges list) Some options in order of decreasing preference: 1) Fix bug 759964 then change these to expected passes 2) Change check_loaded_content to poll a few times until and wait longer for the webm/ogg to load 3) Increase the setTimeout intervals in check_disabled_content. They are already at 5s which seems like ample time.
(In reply to Matthew N. [:MattN] from comment #259) > Some options in order of decreasing preference: > 1) Fix bug 759964 then change these to expected passes So these are just intermittently failing todos? Why not just remove these checks until we fix bug 759964?
It's not clear to me from this bug why we thought it was a 10.5-specific issue, or why this is intermittently passing. But since it's just a todo, there's no use having this create a bunch of oranges.
Attachment #664166 - Flags: review?(mnoorenberghe+bmo)
Comment on attachment 664166 [details] [diff] [review] disable intermittently passing todo tests Review of attachment 664166 [details] [diff] [review]: ----------------------------------------------------------------- You beat me to it.
Attachment #664166 - Flags: review?(mnoorenberghe+bmo) → review+
https://hg.mozilla.org/mozilla-central/rev/e7e7255b9b1a
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Whiteboard: [orange] → [orange][parts of test disabled]
We don't need to track anything here, AFAIK. The disabled parts will be re-enabled once bug 759964 is fixed.
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
Whiteboard: [orange][parts of test disabled] → [orange]
Whiteboard: [orange]
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: