Commit 90ae5f9e authored by Olli Pettay's avatar Olli Pettay Committed by Pier Angelo Vendrame
Browse files

Bug 1880429, clear source browsing context early, if possible, r=nika 

Differential Revision: https://phabricator.services.mozilla.com/D242075
parent 909a2d69

dom/base/PostMessageEvent.cpp

+8 −0
Original line number Diff line number Diff line
@@ -14,6 +14,7 @@ 
#include "mozilla/dom/MessageEventBinding.h"

#include "mozilla/dom/MessagePort.h"

#include "mozilla/dom/RootedDictionary.h"

#include "mozilla/dom/WindowContext.h"

#include "mozilla/BasePrincipal.h"

#include "mozilla/EventDispatcher.h"

#include "mozilla/StaticPrefs_dom.h"
@@ -52,6 +53,13 @@ PostMessageEvent::~PostMessageEvent() = default; 


// TODO: Convert this to MOZ_CAN_RUN_SCRIPT (bug 1415230, bug 1535398)

MOZ_CAN_RUN_SCRIPT_BOUNDARY NS_IMETHODIMP PostMessageEvent::Run() {

  if (mCallerWindowID) {

    RefPtr<WindowContext> wc = WindowContext::GetById(mCallerWindowID);

    if (!wc || !wc->IsCurrent()) {

      mSource = nullptr;

    }

  }



  // Note: We don't init this AutoJSAPI with targetWindow, because we do not

  // want exceptions during message deserialization to trigger error events on

  // targetWindow.