433357 - [Yahoo Mail Notifier] Could not convert JavaScript argument arg 0 [nsISupports.QueryInterface]

Status: RESOLVED FIXED

(addons.mozilla.org Graveyard :: Administration, defect)

Description

[Carsten Book [:Tomcat]]

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9pre) Gecko/2008051218 Firefox/3.0pre

Steps to reproduce:
-> Install the Yahoo Mail Notifier https://addons.mozilla.org/en-US/firefox/addon/1264
-> Configure the Mail Notifier with your Yahoo Mail Account (make sure there are some mails in this account) and save the password
-> Restart Firefox 
-> JS Error:

* Call to xpconnect wrapped JSObject produced this error:  *
[Exception... "Could not convert JavaScript argument arg 0 [nsISupports.QueryInterface]"  nsresult: "0x80570009 (NS_ERROR_XPC_BAD_CONVERT_JS)"  location: "JS frame :: file:///Users/carstenbook/Library/Application%20Support/Firefox/Profiles/rn08739c.default/extensions/%7B89f8dde0-010a-11da-8cd6-0800200c9a66%7D/platform/Darwin/components/ymNotifierGrowlNotifications.js :: observer :: line 41"  data: no]

Comment 1

[Michael Kraft [:morac]]

I found the problem.  Yahoo Mail Notifier will use the Growl Notifications addon if it's available, but I never checked to see if it was available in my component before trying to use it.  

I never tested this myself since I don't have a Mac, but the person who tested it never had an issue (probably because he had the Growl Notification addon installed).

The error won't cause problems, but I'll fix it for my next release.  I'll have to make changes anyway because Growl support is integrated into Firefox 3, so the addon won't be required.

Thanks for the heads up.

Comment 2

[Michael Kraft [:morac]]

Attachment: Updated addon to get rid of error.

I just noticed that this was filed on the Mozilla bug site.  I'm not sure that's the correct place to file a bug report for an addons.

In any case since it's here and I don't have a Mac, can you please test this updated extension to see if it solves the problem.

Comment 3

[Carsten Book [:Tomcat]]

Yes, this new Version fixes the problem for me, i don't see the error message anymore - thanks Michael

Comment 4

[Michael Kraft [:morac]]

Okay good.

I don't have the ability to edit this bug (or take it).  Can someone with permission be close this out?

Comment 5

[Carsten Book [:Tomcat]]

(In reply to comment #4)
> Okay good.
> 
> I don't have the ability to edit this bug (or take it).  Can someone with
> permission be close this out?
> 

Closing as fixed :)