Bug 781320 - Part 3: Set <iframe mozbrowser>'s name for iframes created via window.open. r=smaug

dom/browser-element/BrowserElementParent.cpp

@@ -29,7 +29,7 @@ namespace {
  * aOpenerFrameElement.
  */
 already_AddRefed<nsHTMLIFrameElement>
-CreateIframe(Element* aOpenerFrameElement)
+CreateIframe(Element* aOpenerFrameElement, const nsAString& aName)
 {
   nsNodeInfoManager *nodeInfoManager =
     aOpenerFrameElement->OwnerDoc()->NodeInfoManager();
@@ -54,6 +54,10 @@ CreateIframe(Element* aOpenerFrameElement)
                                mozapp, /* aNotify = */ false);
   }
 
+  // Copy the window name onto the iframe.
+  popupFrameElement->SetAttr(kNameSpaceID_None, nsGkAtoms::name,
+                             aName, /* aNotify = */ false);
+
   return popupFrameElement.forget();
 }
 
@@ -131,7 +135,7 @@ BrowserElementParent::OpenWindowOOP(mozilla::dom::TabParent* aOpenerTabParent,
     do_QueryInterface(aOpenerTabParent->GetOwnerElement());
   NS_ENSURE_TRUE(openerFrameElement, false);
   nsRefPtr<nsHTMLIFrameElement> popupFrameElement =
-    CreateIframe(openerFrameElement);
+    CreateIframe(openerFrameElement, aName);
 
   // Normally an <iframe> element will try to create a frameLoader when the
   // page touches iframe.contentWindow or sets iframe.src.
@@ -189,7 +193,7 @@ BrowserElementParent::OpenWindowInProcess(nsIDOMWindow* aOpenerWindow,
     do_QueryInterface(openerFrameDOMElement);
 
   nsRefPtr<nsHTMLIFrameElement> popupFrameElement =
-    CreateIframe(openerFrameElement);
+    CreateIframe(openerFrameElement, aName);
   NS_ENSURE_TRUE(popupFrameElement, false);
 
   nsCAutoString spec;