1260766 - Moving tabs with non-default user context between windows needs rework

Status: RESOLVED FIXED

(Firefox :: Tabbed Browser, defect)

Description

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

As part of bug 1242644 (about to land), swapFrameLoaders will start to enforce that both frames being swapped have matching OriginAttributes before allow them to be swapped.

This breaks moving tabs between windows (which uses swapFrameLoaders) for tabs with non-default user context IDs.  (Regular tabs using the default context continue to move as expected.)

Most likely the issue is that the user context ID needs to be set earlier on the target browser we swap into when moving tabs.

Comment 2

[Andrea Marchesini [:baku]]

Attachment: fix.patch

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8745853 [details] [diff] [review]
fix.patch

Please add a comment that this expects that usercontextid attribute has been set on the xul:browser before swap. Don't know how to explain that clearly in a comment though.

+    int32_t namespaceID = mOwnerContent->GetNameSpaceID();
+    if ((namespaceID == kNameSpaceID_XUL) &&
I would use just mOwnerContent->IsXULElement()

Comment 4

[Andrea Marchesini [:baku]]

Attachment: fix.patch

Comment 5

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8745874 [details] [diff] [review]
fix.patch

You need to re-explain to me, why this works if "our" isn't the about:blank, but the actual page which we want to move to a new window.

Comment 6

[Andrea Marchesini [:baku]]

Attachment: fix.patch

Comment 7

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8745985 [details] [diff] [review]
fix.patch

You need to use the other frameloader for PopulateUserContextIdFromAttribute call when populating otherOriginAttributes. In both places.
With that r+

Comment 8

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e78a24e7938b

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/e78a24e7938b