394990 - Running xpcshell tests: Components.classes['@mozilla.org/xre/app-info;1'] has no properties

Status: RESOLVED FIXED

(Toolkit Graveyard :: XULRunner, defect)

Description

[Alex Vincent [:WeirdAl]]

Attachment: error log

Somehow, I'm getting a conflict between nsBrowserContentHandler.js and xpcshell tests for extension manager.  I'm also seeing our old friend:

###!!! ASSERTION: XPConnect is being called on a scope without a 'Components' property!

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a8pre) Gecko/2007090420 Minefield/3.0a8pre

Comment 1

[Dave Townsend [:mossop]]

What does this have to do with bug 378216?

Comment 2

[Alex Vincent [:WeirdAl]]

I don't recall off the top of my head.  I think it was just looking at cvs blame, where the code came from.

Comment 3

[Robert Sayre]

xpcshell bugs usually don't block

Comment 4

[Dave Townsend [:mossop]]

This isnt to do with any testcases. Simply running a debug build xpcshell on trunk currently produces the following:

************************************************************
* Call to xpconnect wrapped JSObject produced this error:  *
[Exception... "'[JavaScript Error: "Components.classes['@mozilla.org/xre/app-info;1'] has no properties" {file: "file:///Users/dave/mozilla/build/minefield-debug/dist/bin/components/nsBrowserContentHandler.js" line: 750}]' when calling method: [nsIModule::registerSelf]"  nsresult: "0x80570021 (NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS)"  location: "<unknown>"  data: yes]
************************************************************
js> quit()

WARNING: nsExceptionService ignoring thread destruction after shutdown: file /Users/dave/mozilla/source/HEAD/mozilla/xpcom/base/nsExceptionService.cpp, line 191
nsStringStats
 => mAllocCount:            936
 => mReallocCount:            0
 => mFreeCount:             936
 => mShareCount:             44
 => mAdoptCount:              7
 => mAdoptFreeCount:          7

Comment 5

[Dave Townsend [:mossop]]

Attachment: patch rev 1

This simply plows on in the event there is no app-info.

Comment 6

[Reed Loden [:reed]]

Checking in browser/components/nsBrowserContentHandler.js;
/cvsroot/mozilla/browser/components/nsBrowserContentHandler.js,v  <--  nsBrowserContentHandler.js
new revision: 1.46; previous revision: 1.45
done