640700 - Set gAppData in content process

Status: RESOLVED WONTFIX

(Core :: XPCOM, defect)

Description

[Josh Matthews [:jdm]]

>  var appinfo = Components.classes["@mozilla.org/xre/app-info;1"]
>                          .getService(Components.interfaces.nsIXULAppInfo);
>  if (appinfo.ID == "{a23983c0-fd0e-11dc-95ff-0800200c9a66}") {

This currently doesn't work in a content process (think Mochitest here).  Benjamin, is this something we want to work?

Comment 1

[Benjamin Smedberg]

I don't think so, why does the content process need to know this info?

Comment 2

[Martijn Wargers (dead)]

Attachment: testcase

Comment 3

[(not currently active) Ted Mielczarek]

Just FYI, we're going to kill enablePrivilege in the near future anyway. Anything that you want to do in Mochitests that requires privileges should go through the SpecialPowers framework.

Comment 4

[Josh Matthews [:jdm]]

WONTFIX it is.

Comment 5

[Martijn Wargers (dead)]

(In reply to comment #3)
> Just FYI, we're going to kill enablePrivilege in the near future anyway.
> Anything that you want to do in Mochitests that requires privileges should go
> through the SpecialPowers framework.

How does that work then?
I tried adding this:
In SpecialPowersObserver.js:
      case "SPappinfo":
        return "string";
        break;

In specialpowers.js:
  getAppID: function() {
    var  msg = {'op':'get', 'prefName': 'browser.tabs.remote','prefType': 'BOOL'};
    return sendSyncMessage('SPappinfo', msg);
  },

I keep getting nothing returned.

Comment 6

[Josh Matthews [:jdm]]

You need to return sendSyncMessage('SPappinfo', {})[0];  Also the msg var is not used.

Comment 7

[Martijn Wargers (dead)]

I know that the msg var is not used. I'm just trying to get something that is working.
Anyway, I tried your "sendSyncMessage('SPappinfo', {})[0];" suggestion. It doesn't work. When I call "alert(SpecialPowers.getAppID());", I get 'undefined' returned.

Comment 8

[Martijn Wargers (dead)]

It turned out, I had to add "      messageManager.addMessageListener("SPappinfo", this);" in SpecialPowersObserver.js observe function.