Skip test_handlerApps.xhtml on OS X SeaMonkey, until bug 449763 is fixed

ASSIGNED
Assigned to

Status

()

defect
P3
major
ASSIGNED
7 years ago
10 months ago

People

(Reporter: sgautherie, Assigned: sgautherie)

Tracking

(Depends on 1 bug, Blocks 1 bug)

unspecified
All
macOS
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox12 wontfix, firefox13 affected, firefox14 affected)

Details

(Whiteboard: [fixed in moz15: Bv1] [perma-orange], )

Attachments

(2 attachments)

(In reply to Serge Gautherie (:sgautherie) from bug 556687 comment #17)
> (In reply to comment #14)
> > Looks like [something] turned this into a hang of the test.
> 
> At least, this test (now) triggers a "specific" warning:


(In reply to Serge Gautherie (:sgautherie) from bug 556687 comment #23)
> http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey-Aurora/1335224588.
> 1335225531.3665.gz
> OS X 10.5 comm-aurora debug test mochitests-2/5 on 2012/04/23 16:43:08
> s: cb-sea-miniosx02
> +
> http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey-Beta/1334930379.
> 1334931388.10687.gz
> OS X 10.5 comm-beta debug test mochitests-2/5 on 2012/04/20 06:59:39
> s: cb-sea-miniosx01
> {
> TEST-UNEXPECTED-FAIL | /tests/docshell/test/test_bug511449.html |
> application timed out after 330 seconds with no output
> 
> SCREENSHOT: data:image/png;base64,[...]
> }
> 
> Screenshot shows:
> *Top menu is 'SeaMonkeyDebug'.
> *Open window is 'iCal'.
> *Open dialog is 'Subscribe to calendar' for
> "webcal://127.0.0.1/rheeeeet.html" url.
> 
> ***
> 
> That url is in
> http://mxr.mozilla.org/mozilla-central/source/uriloader/exthandler/tests/
> mochitest/handlerApps.js
> which is part of
> http://mxr.mozilla.org/mozilla-central/source/uriloader/exthandler/tests/
> mochitest/test_handlerApps.xhtml
> 
> That test (would) run after the one which the test suite times out in.
> My guess would be that test_handlerApps.xhtml gets run once after reboot
> (whenever that might happen), then iCal stays there and breaks all following
> runs (because SeaMonkey can't get focus anymore?)...
> 
> This looks similar to Core bug 449763.
> 
> NB: I'm not sure why Firefox is not affected: are FF boxes rebooting after
> each test suite run?
Depends on: 394483
Whiteboard: [perma-orange] → [perma-orange] [autoland:619273:-u mochitest-5 -t none]
Try run for 741a5f568f44 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=741a5f568f44
Results (out of 31 total builds):
    success: 29
    warnings: 2
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/sgautherie.bz@free.fr-741a5f568f44
(In reply to Mozilla RelEng Bot from comment #3)

>     https://tbpl.mozilla.org/?tree=Try&rev=741a5f568f44

Succeeded.

*****

Ftr,


https://tbpl.mozilla.org/php/getParsedLog.php?id=11353963&tree=Try&full=1
Rev3 Fedora 12x64 try opt test mochitests-5/5 on 2012-05-01 03:12:07 PDT for push 741a5f568f44
{
6874 INFO TEST-START | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml
NPP_Destroy
6875 INFO TEST-PASS | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | webHandler launchWithURI (new window/tab) test started
6876 INFO TEST-PASS | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | webHandler launchWithURI (existing window/tab) test started
webcal://127.0.0.1/rheeeeet.html
6877 INFO TEST-PASS | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | localHandler launchWithURI test succeeded
Closing web-handler app which has no window.opener
6878 INFO TEST-INFO | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | Received unload event 1/2
6879 INFO TEST-PASS | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | uri passed to web-handler app - ?uri=webcal%3A%2F%2F127.0.0.1%2Frheeeeet.html should equal ?uri=webcal%3A%2F%2F127.0.0.1%2Frheeeeet.html
Closing web-handler app which has a window.opener
6880 INFO TEST-INFO | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | Received unload event 2/2
6881 INFO TEST-END | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | finished in 510ms
}
NB: 'echo' prints the url :-)


https://tbpl.mozilla.org/php/getParsedLog.php?id=11353806&tree=Try&full=1
Rev4 MacOSX Snow Leopard 10.6 try opt test mochitests-5/5 on 2012-05-01 02:55:12 PDT for push 741a5f568f44
{
7576 INFO TEST-START | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml
NPP_Destroy
7577 INFO TEST-PASS | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | webHandler launchWithURI (new window/tab) test started
7578 INFO TEST-PASS | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | webHandler launchWithURI (existing window/tab) test started
7579 INFO TEST-PASS | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | localHandler launchWithURI test succeeded
Closing web-handler app which has no window.opener
7580 INFO TEST-INFO | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | Received unload event 1/2
7581 INFO TEST-PASS | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | uri passed to web-handler app - ?uri=webcal%3A%2F%2F127.0.0.1%2Frheeeeet.html should equal ?uri=webcal%3A%2F%2F127.0.0.1%2Frheeeeet.html
Closing web-handler app which has a window.opener
7582 INFO TEST-INFO | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | Received unload event 2/2
7583 INFO TEST-END | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | finished in 500ms
}
NB: iCal doesn't print anything :-|


https://tbpl.mozilla.org/php/getParsedLog.php?id=11354249&tree=Try&full=1
Rev3 WINNT 6.1 try opt test mochitests-5/5 on 2012-05-01 03:29:15 PDT for push 741a5f568f44
{
8060 INFO TEST-START | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml
8061 INFO TEST-PASS | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | webHandler launchWithURI (new window/tab) test started
8062 INFO TEST-PASS | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | webHandler launchWithURI (existing window/tab) test started
8063 INFO TEST-PASS | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | localHandler launchWithURI test succeeded
Closing web-handler app which has no window.opener
sethostname: Use the Network Control Panel Applet to set hostname.
hostname -s is not supported.
8064 INFO TEST-INFO | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | Received unload event 1/2
8065 INFO TEST-PASS | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | uri passed to web-handler app - ?uri=webcal%3A%2F%2F127.0.0.1%2Frheeeeet.html should equal ?uri=webcal%3A%2F%2F127.0.0.1%2Frheeeeet.html
Closing web-handler app which has a window.opener
8066 INFO TEST-INFO | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | Received unload event 2/2
8067 INFO TEST-END | /tests/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml | finished in 840ms
}
NB: 'hostname' (at least on Windows 2000) doesn't support any parameter (= i.e. the url) :-/
Whiteboard: [perma-orange] [autoland:619273:-u mochitest-5 -t none] → [perma-orange]
Comment on attachment 619273 [details] [diff] [review]
(Av1) test_handlerApps.xhtml: Reorder tests, Add a dump() call, Remove extra spaces

Ping for review.
Attachment #619273 - Flags: review?(jonas)
Attachment #619275 - Flags: review?(jonas)
(In reply to Serge Gautherie (:sgautherie) from comment #0)
> > NB: I'm not sure why Firefox is not affected: are FF boxes rebooting after
> > each test suite run?

SM bug 755626 will do that too.
Depends on: 755626
I won't have time to review this anytime soon, so feel free to find another reviewer.
Attachment #619273 - Flags: review?(jst)
Attachment #619273 - Flags: review?(cbiesinger)
Attachment #619275 - Flags: review?(jst)
Attachment #619275 - Flags: review?(cbiesinger)
Attachment #619275 - Flags: review?(dmose) → review+
Comment on attachment 619275 [details] [diff] [review]
(Bv1) test_handlerApps.xhtml: OS X SeaMonkey workaround wrt iCal
[Checked in: Comments 8 and 10]

https://hg.mozilla.org/mozilla-central/rev/10f674b12089


[Approval Request Comment]
Work around bug 449763.
No risk, SeaMonkey test run only. But it needs to be deployed on all branches to be fully effective.
Attachment #619275 - Attachment description: (Bv1) test_handlerApps.xhtml: OS X SeaMonkey workaround wrt iCal → (Bv1) test_handlerApps.xhtml: OS X SeaMonkey workaround wrt iCal [Checked in: Comment 8]
Attachment #619275 - Flags: review?(jst)
Attachment #619275 - Flags: review?(jonas)
Attachment #619275 - Flags: review?(cbiesinger)
Attachment #619275 - Flags: approval-mozilla-beta?
Attachment #619275 - Flags: approval-mozilla-aurora?
Flags: in-testsuite+
Whiteboard: [perma-orange] → [fixed in moz15: Bv1] [perma-orange]
Comment on attachment 619275 [details] [diff] [review]
(Bv1) test_handlerApps.xhtml: OS X SeaMonkey workaround wrt iCal
[Checked in: Comments 8 and 10]

we've already built our last beta for this cycle, so not approving for beta but please go ahead and land to aurora before Monday June 4th when we merge.
Attachment #619275 - Flags: approval-mozilla-beta?
Attachment #619275 - Flags: approval-mozilla-beta-
Attachment #619275 - Flags: approval-mozilla-aurora?
Attachment #619275 - Flags: approval-mozilla-aurora+
Keywords: checkin-needed
Whiteboard: [fixed in moz15: Bv1] [perma-orange] → [c-n: 10f674b12089 to m-a] [fixed in moz15: Bv1] [perma-orange]
Comment on attachment 619275 [details] [diff] [review]
(Bv1) test_handlerApps.xhtml: OS X SeaMonkey workaround wrt iCal
[Checked in: Comments 8 and 10]

http://hg.mozilla.org/releases/mozilla-aurora/rev/d92042ceb4cb
Attachment #619275 - Attachment description: (Bv1) test_handlerApps.xhtml: OS X SeaMonkey workaround wrt iCal [Checked in: Comment 8] → (Bv1) test_handlerApps.xhtml: OS X SeaMonkey workaround wrt iCal [Checked in: Comments 8 and 10]
Keywords: checkin-needed
Whiteboard: [c-n: 10f674b12089 to m-a] [fixed in moz15: Bv1] [perma-orange] → [fixed in moz15: Bv1] [perma-orange]
Comment on attachment 619273 [details] [diff] [review]
(Av1) test_handlerApps.xhtml: Reorder tests, Add a dump() call, Remove extra spaces

please find someone else to review this
Attachment #619273 - Flags: review?(cbiesinger)
Comment on attachment 619273 [details] [diff] [review]
(Av1) test_handlerApps.xhtml: Reorder tests, Add a dump() call, Remove extra spaces

Review of attachment 619273 [details] [diff] [review]:
-----------------------------------------------------------------

I'm not the right person to review this
Attachment #619273 - Flags: review?(jonas)
Comment on attachment 619273 [details] [diff] [review]
(Av1) test_handlerApps.xhtml: Reorder tests, Add a dump() call, Remove extra spaces

Apologies; I'm not going to have time to review this any time soon.
Attachment #619273 - Flags: review?(dmose)
Attachment #619273 - Flags: review?(jst) → review+
Product: Core → Firefox
Target Milestone: mozilla15 → ---
Version: Trunk → unspecified
Moving to p3 because no activity for at least 1 year(s).
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information
Priority: P2 → P3
Moving to p3 because no activity for at least 1 year(s).
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information
You need to log in before you can comment on or make changes to this bug.