author Kyle Machulis <>
Thu, 18 Apr 2019 17:31:41 -0700
changeset 473699 8ff2ff524489f0ae67d90ed4ce2c6d2fe4986d5d
parent 473698 27492a30286c2cc22509e7ccdd84ff22f1425841
child 473704 f7e477858ab7bc94193910e0d7fc5a11920729a7
permissions -rw-r--r--
Bug 1540839 - Don't reuse BCs on top level windows being made local; r=nika Don't preserve contexts when we have a chrome window that's being made local. Differential Revision:

/* -*- Mode: C++; tab-width: 8; indent-tabs-mode: nil; c-basic-offset: 2 -*- */
/* vim: set ts=8 sts=2 et sw=2 tw=80: */
/* This Source Code Form is subject to the terms of the Mozilla Public
 * License, v. 2.0. If a copy of the MPL was not distributed with this
 * file, You can obtain one at */

#include "nsFrameLoaderOwner.h"
#include "nsFrameLoader.h"
#include "nsFocusManager.h"
#include "nsSubDocumentFrame.h"
#include "nsQueryObject.h"
#include "mozilla/AsyncEventDispatcher.h"
#include "mozilla/dom/BrowsingContext.h"
#include "mozilla/dom/FrameLoaderBinding.h"
#include "mozilla/dom/MozFrameLoaderOwnerBinding.h"

already_AddRefed<nsFrameLoader> nsFrameLoaderOwner::GetFrameLoader() {
  return do_AddRef(mFrameLoader);

void nsFrameLoaderOwner::SetFrameLoader(nsFrameLoader* aNewFrameLoader) {
  mFrameLoader = aNewFrameLoader;

nsFrameLoaderOwner::GetBrowsingContext() {
  if (mFrameLoader) {
    return mFrameLoader->GetBrowsingContext();
  return nullptr;

void nsFrameLoaderOwner::ChangeRemoteness(
    const mozilla::dom::RemotenessOptions& aOptions, mozilla::ErrorResult& rv) {
  RefPtr<BrowsingContext> bc;
  // If we already have a Frameloader, destroy it, possibly preserving its
  // browsing context.
  if (mFrameLoader) {
    // Don't preserve contexts if this is a chrome (parent process) window that
    // is changing from remote to local.
    bool isChromeRemoteToLocal =
        XRE_IsParentProcess() && (!aOptions.mRemoteType.WasPassed() ||

    // If this is a process switch due to a difference in Cross Origin Opener
    // Policy, do not preserve the browsing context. Otherwise, save off the
    // browsing context and use it when creating our new FrameLoader.
    if (!aOptions.mReplaceBrowsingContext &&
        !isChromeRemoteToLocal) {
      bc = mFrameLoader->GetBrowsingContext();

    mFrameLoader = nullptr;

  // In this case, we're not reparenting a frameloader, we're just destroying
  // our current one and creating a new one, so we can use ourselves as the
  // owner.
  RefPtr<Element> owner = do_QueryObject(this);
  mFrameLoader = nsFrameLoader::Create(owner, bc, aOptions);

  if (NS_WARN_IF(!mFrameLoader)) {

  if (aOptions.mPendingSwitchID.WasPassed()) {
  } else {

  // Now that we've got a new FrameLoader, we need to reset our
  // nsSubDocumentFrame to use the new FrameLoader.
  if (nsSubDocumentFrame* ourFrame = do_QueryFrame(owner->GetPrimaryFrame())) {

  if (nsFocusManager* fm = nsFocusManager::GetFocusManager()) {
    if (fm->GetFocusedElement() == owner) {

  // Assuming this element is a XULFrameElement, once we've reset our
  // FrameLoader, fire an event to act like we've recreated ourselves, similar
  // to what XULFrameElement does after rebinding to the tree.
  // ChromeOnlyDispatch is turns on to make sure this isn't fired into content.
  (new mozilla::AsyncEventDispatcher(
       owner, NS_LITERAL_STRING("XULFrameLoaderCreated"),
       mozilla::CanBubble::eYes, mozilla::ChromeOnlyDispatch::eYes))