556119 - ASSERTION: bad hint in chrome code: 'hint != XOW && hint != SOW && hint != COW' when calling nsIInstallLocation.getItemFile()

Status: RESOLVED WORKSFORME

(Core :: XPConnect, defect)

Description

[Jonathan Griffin (:jgriffin)]

When calling nsIInstallLocation.getItemFile() within an extension, the following assertion occurs:

###!!! ASSERTION: bad hint in chrome code: 'hint != XOW && hint != SOW && hint != COW', file c:/mozilla/mozilla-central/src/js/src/xpconnect/src/xpcwrappednativescope.cpp, line 1043

This happens on Windows XP and Windows 7, but not Linux (CentOS 5).

To reproduce:

1 - install the attached extension on a Windows machine
2 - from Firefox's Tools menu, select "Grafx Bot"
3 - press the Start button
>>>> you will see the assertion mentioned

In debugging this, it appears the assertion occurs when calling this line of code (in content/test-driver.js):

    var jsm = CC["@mozilla.org/extensions/manager;1"]
              .getService(CI.nsIExtensionManager)
              .getInstallLocation(id)
              .getItemFile(id, "modules/grafxbot.jsm");

where id = "[e-mail]".

The call completes successfully despite the assertion.

Comment 1

[Jonathan Griffin (:jgriffin)]

Attachment: test extension

Comment 2

[Johnny Stenback (:jst)]

We need to at least understand what this means before we ship.

Comment 3

[Blake Kaplan (:mrbkap) (inactive)]

jgriffin, I only have Linux and OSX here, so I can't test... does this still happen?

Comment 4

[Jonathan Griffin (:jgriffin)]

This problem no longer occurs with Grafx Bot, as I refactored some code for other reasons and the code that was leading to this assertion is no longer present.

However, I've also seen it in another context, since the new AddOn manager landed on trunk.  If you install or uninstall any addon, then click Restart, the same assertion occurs.  I'll check and see if this occurs on the Mac as well.

Comment 5

[Jonathan Griffin (:jgriffin)]

Yes, this assertion occurs on Mac as well, in the context described in comment #4.

Comment 6

[Mike Beltzner [:beltzner, not reading bugmail]]

Moving to betaN based on comment 2

Comment 7

[Damon Sicore (:damons)]

Who owns this?  Need an owner ASAP.

Comment 8

[Johnny Stenback (:jst)]

Blake should own this. Blake, please see comment 5.

Comment 9

[Blake Kaplan (:mrbkap) (inactive)]

The code that caused this is gone thanks to compartments.