1013307 - Shutdown error: illegal operation

Status: RESOLVED FIXED

(Instantbird Graveyard :: Other, defect)

Description

[aleth [:aleth]]

* Call to xpconnect wrapped JSObject produced this error:  *
[Exception... "Component returned failure code: 0x8046001e [nsIWindowWatcher.openWindow]"  nsresult: "0x8046001e (<unknown>)"  location: "JS frame :: resource:///modules/ibCore.jsm :: Core.showWindow :: line 143"  data: no]

"NS_ERROR_ILLEGAL_DURING_SHUTDOWN (0x8046001E)
    Many operations cannot be performed once the application is being shutdown. This error will occur in this situation."

Comment 1

[Patrick Cloke [:clokep]]

Are there STR? Is this really a blocker?

Comment 2

[Florian Quèze [:florian]]

(In reply to Patrick Cloke [:clokep] from comment #1)
> Are there STR? Is this really a blocker?

I saw a similar error at more or less every shutdown of my debug build for months. I haven't checked recently if I still see it. I don't think it blocks.

Comment 3

[aleth [:aleth]]

Attachment: shutdownopenwindow.diff

Found this old fix sitting in the mq

Comment 4

[Florian Quèze [:florian]]

Could we instead make _showAccountManagerIfNeeded return early if imServices.core.initialized is false?

Comment 5

[aleth [:aleth]]

(In reply to Florian Quèze [:florian] [:flo] from comment #4)
> Could we instead make _showAccountManagerIfNeeded return early if
> imServices.core.initialized is false?

That'll only work if initialized is unset before all the accounts etc are uninited and not after (as is the case now). I don't know whether that would have side effects or not.

Comment 6

[Florian Quèze [:florian]]

Comment on attachment 8646575 [details] [diff] [review]
shutdownopenwindow.diff

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

::: im/modules/ibCore.jsm
@@ +292,5 @@
>        this._onQuitRequest(aSubject, aData);
>        return;
>      }
> +
> +    if (aTopic == "prpl-quit") {

Could we do this during "quit-application-granted" instead?

Comment 7

[aleth [:aleth]]

(In reply to Florian Quèze [:florian] [:flo] from comment #6)
> > +    if (aTopic == "prpl-quit") {
> 
> Could we do this during "quit-application-granted" instead?

I should think so, but why?
Nothing currently observes that apart from the listener that calls quit().

Comment 8

[Florian Quèze [:florian]]

At some point, the core service was designed/tested so that we could do .init() then .quit() and then .init() again. I think the goal of doing that was for test purposes, but it may also be helpful to install/uninstall prpl add-ons.

After quit-application-granted, we are sure the process will be terminated, so doing irreversible changes to functions doesn't matter.

Comment 9

[aleth [:aleth]]

Attachment: shutdownopenwindow.diff v2

I'm not sure I'd trust this feature right now, but let's not break it further ;)

Comment 10

[Florian Quèze [:florian]]

Comment on attachment 8646917 [details] [diff] [review]
shutdownopenwindow.diff v2

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

Thanks!

(In reply to aleth [:aleth] from comment #9)

> I'm not sure I'd trust this feature right now, but let's not break it
> further ;)

Yeah, I wouldn't bet either on it being in working conditions right now.

Comment 11

[aleth [:aleth]]

url:        https://hg.mozilla.org/comm-central/rev/639709ee79764cd725fb10ebecb41f1cd5547141
changeset:  639709ee79764cd725fb10ebecb41f1cd5547141
user:       aleth <[e-mail]>
date:       Wed Aug 12 19:31:52 2015 +0200
description:
Bug 1013307 - Don't try to open account manager at shutdown. r=florian