Closed Bug 625038 Opened 9 years ago Closed 9 years ago

[SeaMonkey] mochitest-plain-5: test_bug619644.html fails to load bug619644_inner.html, since it landed

Categories

(Toolkit :: General, defect, major)

defect
Not set
major

Tracking

()

VERIFIED FIXED
mozilla2.0b10
Tracking Status
blocking2.0 --- -

People

(Reporter: sgautherie, Assigned: khuey)

References

(Depends on 1 open bug, Blocks 1 open bug, )

Details

(Whiteboard: [test that aborts the suite] [perma-orange])

Attachments

(1 file)

http://tinderbox.mozilla.org/showbuilds.cgi?tree=SeaMonkey&maxdate=1293934820&hours=24&legend=0&norules=1
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1293861121.1293862312.6511.gz
Linux comm-central-trunk debug test mochitests-5/5 on 2010/12/31 21:52:01
{
...
Error loading URL http://mochi.test:8888/tests/toolkit/components/prompts/test/bug619644_inner.html : 804b0002 (NS_BINDING_ABORTED)
...
5741 ERROR TEST-UNEXPECTED-FAIL | /tests/toolkit/components/prompts/test/test_bug619644.html | Test timed out.
}

Then 1+ other test fails too then
"buildbot.slave.commands.TimeoutError: command timed out: 1200 seconds without output"
:-(
blocking2.0: --- → ?
Does Seamonkey use tab-modal prompts?  If not we should just skip the test.
> Does Seamonkey use tab-modal prompts?
No.
> If not we should just skip the test.
How about making the test conditional on whether the tab-modal pref is on or off?
(In reply to comment #2)
> > Does Seamonkey use tab-modal prompts?
> No.
> > If not we should just skip the test.
> How about making the test conditional on whether the tab-modal pref is on or
> off?

Right.
Assignee: nobody → khuey
Status: NEW → ASSIGNED
No longer blocks: 619644
Depends on: 625056, 619644
blocking2.0: ? → -
Yeah, the test should be a todo() when the pref is off.
Do Seamonkey, Thunderbird, etc have Services.jsm?  Or do I need to get this through xpcom proper?
(In reply to comment #6)
> Do Seamonkey, Thunderbird, etc have Services.jsm?  Or do I need to get this
> through xpcom proper?

SM and TB do, and all Toolkit apps should, iiuc:
http://mxr.mozilla.org/comm-central/source/mozilla/toolkit/content/Makefile.in#93
Cool.  I'll land this change tomorrow.
Comment on attachment 503296 [details] [diff] [review]
Do not run tab-modal specific tests if the app does not have tab-modal dialogs
[Checked in: See comment 11]

>+  return Services.prefs.getBoolPref("prompts.tab_modal.enabled");

This pref is undefined on other apps, so this code should be 

  var prefName = "prompts.tab_modal.enabled";
  return Services.prefs.getPrefType(prefName) == Services.prefs.PREF_BOOL &&
         Services.prefs.getBoolPref(prefName);

>+  todo(0, "No tab modal prompts");

Nit: I would prefer something like

  todo(false, "Test disabled as tab modal prompts are not enabled");
Attachment #503296 - Flags: review-
Bah, annoying that toolkit doesn't set a default value for this pref.
http://hg.mozilla.org/mozilla-central/rev/996e407e9735
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Attachment #503296 - Attachment description: Do not run tab-modal specific tests if the app does not have tab-modal dialogs. → Do not run tab-modal specific tests if the app does not have tab-modal dialogs [Checked in: See comment 11]
Blocks: 626294
V.Fixed, per bug 626294 comment 0.

*****

(In reply to comment #0)
> Then 1+ other test fails too

I filed bug 626294.
Status: RESOLVED → VERIFIED
Flags: in-testsuite+
Target Milestone: --- → mozilla2.0b10
(In reply to comment #11)
> http://hg.mozilla.org/mozilla-central/rev/996e407e9735

Nit: a 'var' is missing:
{
18   prefName = "prompts.tab_modal.enabled";
}
(In reply to comment #13)
> Nit: a 'var' is missing:

I fixed it in bug 626294.
You need to log in before you can comment on or make changes to this bug.