600785 - TM: intermittent failure in browser_bug471404.js | Test timed out after JavaScript Error: "dirSvc.unregisterProvider is not a function"

Reporter

Description

•

15 years ago

Not immediately obvious to me whether this is the test doing something it shouldn't, or jseng doing something it shouldn't, but sometime today browser_bug471404.js started intermittently failing like http://tinderbox.mozilla.org/showlog.cgi?log=TraceMonkey/1285806951.1285808484.5252.gz Rev3 WINNT 6.1 tracemonkey opt test mochitest-other on 2010/09/29 17:35:51 s: talos-r3-w7-033 TEST-INFO | chrome://mochikit/content/browser/toolkit/crashreporter/test/browser/browser_bug471404.js | Console message: [JavaScript Error: "dirSvc.unregisterProvider is not a function" {file: "chrome://mochikit/content/browser/toolkit/crashreporter/test/browser//aboutcrashes_utils.js" line: 60}] TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/toolkit/crashreporter/test/browser/browser_bug471404.js | Test timed out where http://mxr.mozilla.org/mozilla-central/source/toolkit/crashreporter/test/browser/aboutcrashes_utils.js#58 is just let dirSvc = Cc["@mozilla.org/file/directory_service;1"] .getService(Ci.nsIProperties); dirSvc.unregisterProvider(_provider);

Phil Ringnalda (:philor)

Comment 1

•

15 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=TraceMonkey/1285818235.1285819726.23826.gz Rev3 WINNT 6.1 tracemonkey opt test mochitest-other on 2010/09/29 20:43:55 s: talos-r3-w7-036 http://tinderbox.mozilla.org/showlog.cgi?log=TraceMonkey/1285823075.1285823932.10560.gz Rev3 MacOSX Leopard 10.5.8 tracemonkey opt test mochitest-other on 2010/09/29 22:04:35 s: talos-r3-leopard-044

blocking2.0: --- → ?

Phil Ringnalda (:philor)

Comment 2

•

15 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=TraceMonkey/1285804558.1285806517.28133.gz Rev3 WINNT 6.1 tracemonkey opt test mochitest-other on 2010/09/29 16:55:58 s: talos-r3-w7-045

Comment hidden (Legacy TBPL/Treeherder Robot)

Phil Ringnalda (:philor)

Comment 4

•

15 years ago

http://tinderbox.mozilla.org/showlog.cgi?log=TraceMonkey/1285806335.1285808342.4422.gz Rev3 MacOSX Snow Leopard 10.6.2 tracemonkey debug test mochitest-other on 2010/09/29 17:25:35 s: talos-r3-snow-012

Comment hidden (Legacy TBPL/Treeherder Robot)

Phil's Orange Reporter

Reporter

Updated

•

15 years ago

Blocks: 600883

Phil Ringnalda (:philor)

Updated

•

15 years ago

No longer blocks: 600883

Comment hidden (Legacy TBPL/Treeherder Robot)

philringnalda%gmail.com http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1285981493.1285984522.5579.gz WINNT 5.2 mozilla-central debug test mochitest-other on 2010/10/01 18:04:53 s: win32-slave38 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/sessionstore/test/browser/browser_586068-cascaded_restore.js | Test timed out TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | should have 2 tabs before hiding - Got 5, expected 2 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | just checking that browser state is correct - Got 6, expected 3 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | second is hidden TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | only restored 1 visible tab - Got 4, expected 1 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | second tab is still hidden TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | Found an unexpected tab at the end of test run: about:blank TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | Found an unexpected tab at the end of test run: about:blank TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | Found an unexpected tab at the end of test run: http://example.org/#2 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | Found an unexpected browser window at the end of test run TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | Found an unexpected browser window at the end of test run TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/crashreporter/test/browser/browser_bug471404.js | Test timed out TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/crashreporter/test/browser/browser_bug471404.js | Found a tab after previous test timed out: about:crashes TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 1685387 bytes during test execution TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 1607 instances of AtomImpl with size 20 bytes each (32140 bytes total) TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 1 instance of BackstagePass with size 24 bytes TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 5 instances of BodyRule with size 16 bytes each (80 bytes total) TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 6 instances of CSSImportRuleImpl with size 44 bytes each (264 bytes total) TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 111 instances of CSSImportantRule with size 16 bytes each (1776 bytes total)

Comment hidden (Legacy TBPL/Treeherder Robot)

philringnalda%gmail.com http://tinderbox.mozilla.org/showlog.cgi?log=TraceMonkey/1285926839.1285929659.5720.gz WINNT 5.2 tracemonkey debug test mochitest-other on 2010/10/01 02:53:59 s: win32-slave34 620 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/content/xul/templates/tests/chrome/test_tmpl_simplesyntaxusingdontrecurse.xul | simple syntax using dont-recurse 621 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/content/xul/templates/tests/chrome/test_tmpl_simplesyntaxusingdontrecurse.xul | Error is: attribute container is '' instead of 'true' - got "<hbox id=\"http://www.some-fictitious-zoo.com/arachnids\"><label value=\"Arachnids\"/></hbox><hbox id=\"http://www.some-fictitious-zoo.com/birds\"><label value=\"Birds\"/></hbox>", expected "Same" TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/browser/components/sessionstore/test/browser/browser_586068-cascaded_restore.js | Test timed out TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | should have 2 tabs before hiding - Got 7, expected 2 TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | just checking that browser state is correct - Got 8, expected 3 TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | second is hidden TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | only restored 1 visible tab - Got 6, expected 1 TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | second tab is still hidden TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | Found an unexpected tab at the end of test run: about:blank TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | Found an unexpected tab at the end of test run: about:blank TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | Found an unexpected tab at the end of test run: about:blank TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | Found an unexpected tab at the end of test run: about:blank TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/browser/components/sessionstore/test/browser/browser_586147.js | Found an unexpected tab at the end of test run: about:blank TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/browser/components/test/browser/browser_bug538331.js | The url opened from the notification should be the url provided by the update - Got about:blank, expected http://example.com/ TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/toolkit/crashreporter/test/browser/browser_bug471404.js | Test timed out TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/toolkit/crashreporter/test/browser/browser_bug471404.js | Found a tab after previous test timed out: about:crashes TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 1680091 bytes during test execution TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 1607 instances of AtomImpl with size 20 bytes each (32140 bytes total) TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 1 instance of BackstagePass with size 24 bytes TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 5 instances of BodyRule with size 16 bytes each (80 bytes total) TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 6 instances of CSSImportRuleImpl with size 44 bytes each (264 bytes total) TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 109 instances of CSSImportantRule with size 16 bytes each (1744 bytes total)

Comment hidden (Legacy TBPL/Treeherder Robot)

David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)

Assignee

Updated

•

15 years ago

Assignee: general → dbaron

Component: JavaScript Engine → Breakpad Integration

Product: Core → Toolkit

QA Contact: general → breakpad.integration

David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)

Assignee

Comment 58

•

15 years ago

Attached patch patch — Details — Splinter Review

Pretty sure this is a bug in the test, which is racing with GC of the wrapper.

Attachment #481861 - Flags: review?(ted.mielczarek)

(not currently active) Ted Mielczarek

Comment 59

•

15 years ago

Comment on attachment 481861 [details] [diff] [review] patch Kind of awful that we have to think about wrappers like this. :-/ Is there no sane way to make sure that the wrapper hangs around? I can see us making this mistake fairly often, since it violates my expectations.

Attachment #481861 - Flags: review?(ted.mielczarek) → review+

David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)

Assignee

Comment 60

•

15 years ago

You can make the wrapper hang around by storing it in a global variable or property of something else; that has other dangers, though.

David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)

Assignee

Comment 61

•

15 years ago

Perhaps the better answer is that if the object has classinfo, you don't have to mess with QI at all.

Comment hidden (Legacy TBPL/Treeherder Robot)

David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)

Assignee

Comment 65

•

15 years ago

http://hg.mozilla.org/mozilla-central/rev/26291a8c4caf

Status: NEW → RESOLVED

Closed: 15 years ago

Resolution: --- → FIXED

Target Milestone: --- → mozilla2.0b8

David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)

Assignee

Updated

•

15 years ago

blocking2.0: ? → -

Serge Gautherie (:sgautherie)

Updated

•

15 years ago

Target Milestone: mozilla2.0b8 → mozilla2.0b7

Nobody; OK to take it and work on it

Updated

•

13 years ago

Keywords: intermittent-failure

Nobody; OK to take it and work on it

Updated

•

13 years ago

Whiteboard: [orange]