Closed Bug 933303 Opened 11 years ago Closed 3 years ago

handlerApps.js fails to kill iCal on Mac OS X (cannot create mochitest-plain tests in widget/tests)

Categories

(Firefox :: File Handling, defect)

x86_64
macOS
defect

Tracking

()

RESOLVED DUPLICATE of bug 786938

People

(Reporter: masayuki, Unassigned)

Details

Attachments

(1 file)

Attached image Screenshot (deleted) —
I'm creating first mochitest-plain test in widget/tests in bug 930374. But I meet this bug which causes orange. https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=9bc46d1a711d However, preceding test, uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml launches iCal on Mac and it fails to kill iCal process at the end of the test. The evidence is the attached screenshot. At least, our window should be activated even if we fail to kill iCal. Otherwise, I think that we should disable this test on Mac. http://mxr.mozilla.org/mozilla-central/source/uriloader/exthandler/tests/mochitest/test_handlerApps.xhtml http://mxr.mozilla.org/mozilla-central/source/uriloader/exthandler/tests/mochitest/handlerApps.js
I thought we had an existing bug on this iCal bustage....
How do we open iCal and why is that window different than all other windows?
Hmm, looking at the code, this is what we do: <http://mxr.mozilla.org/mozilla-central/source/uriloader/exthandler/nsLocalHandlerApp.cpp#123> which results in a call to posix_spawnp <http://mxr.mozilla.org/mozilla-central/source/xpcom/threads/nsProcessCommon.cpp#492> which seems quite normal. Can we register webcal: with a custom protocol handler or something like that?
The intent of that code is specifically to spawn OS-native applications. I suspect the right fix is to stop using iCal in the tests and instead create a "hello world" native Mac app that shuts down appropriately. I could also imagine having the test shell out and send a "kill -9" signal to iCal or something...
(In reply to comment #4) > The intent of that code is specifically to spawn OS-native applications. I > suspect the right fix is to stop using iCal in the tests and instead create a > "hello world" native Mac app that shuts down appropriately. I could also > imagine having the test shell out and send a "kill -9" signal to iCal or > something... kill -9 sounds like a good option to me!
Product: Core → Firefox
Version: Trunk → unspecified
No longer blocks: 394483, 930374
Severity: blocker → --
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: