956227 - Using chrome content in a panel does not work

Status: RESOLVED FIXED

(Add-on SDK Graveyard :: General, defect, P3)

Description

[Erik Vold [:erikvold] (please needinfo? me)]

This code:

    Panel({
      focus: false,
      contentURL: 'chrome://fantasticbar/content/fantasticBar.html',
      contentScriptWhen: 'ready',
      contentScriptFile: data.url('fantasticBar.js'),
      position: {
        top: 0,
        left: 0
      }
    });

Produced this error:

Error: System principal is not allowed in an expanded principal
Stack trace:
sandbox@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/loader/sandbox.js:26
WorkerSandbox@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/content/worker.js:144
Trait@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/deprecated/traits.js:114
Worker<._attach@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/content/worker.js:528
LegacyWorker<.attach@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/worker/utils.js:48
attach@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/worker/utils.js:95
@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://fantastic-bar-at-erikvold-dot-com/fantastic-bar/lib/panel.js:264
emit@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js:94
next@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/utils.js:36
filter/<@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/utils.js:51
transform/<@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/utils.js:42
emit@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js:94
next@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/utils.js:36
filter/<@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/utils.js:51
transform/<@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/utils.js:42
emit@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js:94
forward@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/panel/events.js:20
Observer<.observe@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/system/events.js:74
emit@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/system/events.js:50
onContentReady@resource://gre/modules/XPIProvider.jsm -> jar:file:///var/folders/v2/pq4lfnjx0nj43h97ppl6dtcm0000gn/T/tmp6bdiLN.mozrunner/extensions/[e-mail]!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/panel/utils.js:257

Comment 1

[Erik Vold [:erikvold] (please needinfo? me)]

I feel like this should work, but I have a feeling you will disagree Irakli.

Comment 2

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

(In reply to Erik Vold [:erikvold] [:ztatic] from comment #1)
> I feel like this should work, but I have a feeling you will disagree Irakli.

I think that expectation it to work is reasonable, although I do not think this bug is a high priority,
since given use case isn't idiomatic SDK way to make panels.

Comment 3

[Erik Vold [:erikvold] (please needinfo? me)]

This is working now, so we should just need a test.

Comment 4

[Erik Vold [:erikvold] (please needinfo? me)]

Attachment: Link to Github pull-request: https://github.com/mozilla/addon-sdk/pull/1478

Comment 5

[[github robot]]

Commits pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/b6c8647c568df583d2079d7bad5e1c769cc3e494
Bug 956227 - Adding test that using chrome content in a panel does work

https://github.com/mozilla/addon-sdk/commit/5ea96ee3aa847c469a04987ae31ea471bc1cfe1c
Merge pull request #1478 from erikvold/956227

Bug 956227 - Adding test that using chrome content in a panel does work r=@gozala