792899 - docshell/test/chrome/test_bug789773.xul introduced in Bug 789773 uses about:home which SeaMonkey doesn't have.

Status: RESOLVED FIXED

(Core :: DOM: Navigation, defect)

Description

[Philip Chee]

From Bug 789773 Comment 96:

> (In reply to Philip Chee from comment #95)
>> about:home doesn't exist in SeaMonkey. Can you use some other about: page
>> like about:neterror ?

> Fine by me assuming the test continues to pass. File a separate bug?

I looked around and I think about:rights and about:support seem to be available to Firefox, SeaMonkey, and Mobile (xul, android). Will have to test of course.

Comment 1

[Philip Chee]

Neil sez:
> These are all certain to exist:
> about:
> about:about
> about:addons (unsuitable, runs lots of script)
> about:buildconfig
> about:compartments (unsuitable, runs lots of script)
> about:config (unsuitable, runs lots of script)
> about:credits
> about:license
> about:logo
> about:memory (unsuitable, runs lots of script)
> about:mozilla
> about:neterror (unsuitable)
> about:newaddon (probably unsuitable)
> about:plugins (probably unsuitable)
> about:support (probably unsuitable)

Comment 2

[Frank Wein [:mcsmurf]]

Attachment: Patch

Fixes the about:home issue only (tested it on try for Firefox and locally for SeaMonkey), the first test still fails:
if (/browser.xul/.test(req.name)) {

It's looking for browser.xul in the load request for the new window (SeaMonkey uses navigator.xul instead). Maybe another way/test can be uses here to check this:
    1.36       ok(calledListenerForBrowserXUL, "Should have called the progress listener for browser.xul");

Comment 3

[Frank Wein [:mcsmurf]]

Comment on attachment 684992 [details] [diff] [review]
Patch

Although this fixes only one issue, it's better than nothing for now :) that way the test run does not have to wait for the timeout.

Comment 4

[Bobby Holley (:bholley)]

Comment on attachment 684992 [details] [diff] [review]
Patch

I _think_ this is ok, but the different characteristics of all these builtin pages are all pretty tricky. So please grab a build right before bug 789773, and verify that the test still fails with this patch. If so, r=bholley, and thanks for fixing it. :-)

Comment 5

[Frank Wein [:mcsmurf]]

I created a try build without the patch from Bug 792899 and with the test fix from this bug here applied.
The test failed as expected, as a consequence(?) other tests failed, too (though some of these seem to be known test failures; sample output). The browser failed to shutdown:
3687 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/docshell/test/chrome/test_bug789773.xul | Test timed out.
 8545 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/dom/tests/mochitest/chrome/test_focus.xul | Test timed out.
 8625 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/dom/tests/mochitest/chrome/test_nodesFromRect.html | Test timed out.
 8628 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/dom/tests/mochitest/chrome/test_popup_blocker_chrome.xul | Test timed out.
 8629 ERROR TEST-UNEXPECTED-FAIL | (SimpleTest/TestRunner.js) | 4 test timeouts, giving up.
8630 ERROR TEST-UNEXPECTED-FAIL | (SimpleTest/TestRunner.js) | Skipping 479 remaining tests.
8631 ERROR TEST-UNEXPECTED-FAIL | (SimpleTest/TestRunner.js) | chrome://mochitests/content/chrome/dom/tests/mochitest/chrome/test_popup_blocker_chrome.xul finished in a non-clean fashion, probably because it didn't call SimpleTest.finish()
TEST-UNEXPECTED-FAIL | Shutdown | application timed out after 330 seconds with no output

Pushed: https://hg.mozilla.org/integration/mozilla-inbound/rev/c6a9dce2248a

Comment 6

[Bobby Holley (:bholley)]

(In reply to Frank Wein [:mcsmurf] from comment #5)
> I created a try build without the patch from Bug 792899 and with the test
> fix from this bug here applied.
> The test failed as expected, as a consequence(?) other tests failed, too
> (though some of these seem to be known test failures; sample output)

Yep, that looks fine. Thanks for doing that!

Comment 7

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/c6a9dce2248a