author | Kris Maglione <maglione.k@gmail.com> |
Thu, 01 Aug 2019 19:46:32 -0700 | |
changeset 488063 | a50f085eb3e6c506fd8e17acb669ef91b6f31390 |
parent 488062 | a8ac7bcafc270b5be3eca84efebcde672fe8fba5 |
child 488064 | ba987e21b5329d07c3606a65ee0cb76e7bc99a74 |
push id | 36435 |
push user | cbrindusan@mozilla.com |
push date | Thu, 15 Aug 2019 09:46:49 +0000 |
treeherder | mozilla-central@0db07ff50ab5 [default view] [failures only] |
perfherder | [talos] [build metrics] [platform microbench] (compared to previous push) |
reviewers | nika |
bugs | 1562292 |
milestone | 70.0a1 |
first release with | nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
|
last release without | nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
|
--- a/dom/base/nsGlobalWindowOuter.cpp +++ b/dom/base/nsGlobalWindowOuter.cpp @@ -6327,16 +6327,18 @@ void nsGlobalWindowOuter::ForceClose() { FinalClose(); } void nsGlobalWindowOuter::FinalClose() { // Flag that we were closed. mIsClosed = true; + GetBrowsingContext()->SetClosed(true); + // If we get here from CloseOuter then it means that the parent process is // going to close our window for us. It's just important to set mIsClosed. if (XRE_GetProcessType() == GeckoProcessType_Content) { return; } // This stuff is non-sensical but incredibly fragile. The reasons for the // behavior here don't make sense today and may not have ever made sense,