primary provisioning uncaught exception: target window is same as present window




4 years ago
2 years ago


(Reporter: ozten, Unassigned)


Firefox Tracking Flags

(Not tracked)



(1 attachment)



4 years ago
Based on patch from Bug#929243 and a Fx with dom.identity.enabled enabled.

Steps to reproduce:

1) Go to Persona test page like
2) Click Sign in
3) Enter

Identity IDP: _provisionIdentity: identity: url:
Identity IDP: _createProvisioningSandbox:
Identity sandbox: Creating sandbox for:
Identity sandbox: _makeSandboxContentLoaded: 48
Identity IDP: _provisionIdentity: Created sandbox and provisioning flow with id: 48
nsDOMIdentity (undefined): init was called from
JavaScript error: , line 0: uncaught exception: target window is same as present window -- not allowed
JavaScript error:, line 16: is not a function

A Sandboxed iframe loads and use is provisioned (since always says "yes, provision this user").


4 years ago
Blocks: 882884

Comment 1

4 years ago
This error message comes from jschannel, a Persona server side script.
We should not be using the SHIM here...

It appears that should be defined by the native dom, but isn't.

Comment 2

4 years ago
The issue is that in identity.manifest ( we've renamed the DOM API to mozId.

Comment 3

4 years ago
Created attachment 832394 [details] [diff] [review]
Fix	sandbox	API and	Desktop	Persona DOM API

Server side support landed.

This patch gets tip Firefox working again, with navigator.mozId instead of

It also provides "rpInfo" inputs into the /auth authentication flow of the fallback IdP.
Attachment #832394 - Flags: review?(MattN+bmo)
Attachment #832394 - Flags: feedback?(jparsons)
Comment on attachment 832394 [details] [diff] [review]
Fix	sandbox	API and	Desktop	Persona DOM API

Review of attachment 832394 [details] [diff] [review]:

(In reply to Austin King [:ozten] from comment #4)
> Try server:

In the future, please include mochitest and xpcshell test suites.

(In reply to Austin King [:ozten] from comment #3)
> Server side support landed.

Could you point me to the server-side change? I'd like to understand what was wrong and why all of these changes are necessary. It seems like this may be fixing many things in one patch which makes it hard to review.

::: toolkit/identity/IdentityProvider.jsm
@@ +336,3 @@
>      // extract authentication URL from idpParams
> +    let authPath = aIDPParams.idpParams.authentication +
> +      '?' + rpLoginOptions.join('&');

We only want to do this for a fallback provider, right?

::: toolkit/identity/Sandbox.jsm
@@ +102,3 @@
>      docShell.allowWindowControl = false;
> +    // TODO: disable media (bug 759964)
> +    // docShell.allowMedia = false;

I think you want to revert this change
Attachment #832394 - Flags: review?(MattN+bmo) → feedback+
Comment on attachment 832394 [details] [diff] [review]
Fix	sandbox	API and	Desktop	Persona DOM API

Thank you for doing this, Austin.  I give feedback+, and hope that Persona will someday again have support to move forward.
Attachment #832394 - Flags: feedback?(jed+bmo) → feedback+
I'm WONTFIXing persona-related bugs now that we've committed to decommissioning in the persona.or service.
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.