searching for reviewer(nika)
5038a85d05e3c6298d5b813735bf306c9601be11: Bug 1649684 - Use the current nsISerialEventTarget with the various process hosts. r=nika
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 02 Jul 2020 22:59:24 +0000 - rev 538538
Push 37564 by nbeleuzu@mozilla.com at Fri, 03 Jul 2020 03:56:55 +0000
Bug 1649684 - Use the current nsISerialEventTarget with the various process hosts. r=nika In all those cases, the current nsISerialEventTarget is either the main thread or the MessageChannel's nsISerialEventTarget (since bug 1634846) Differential Revision: https://phabricator.services.mozilla.com/D81966
795a9aed1ec7502711b513844dbf5236e4d5727d: Bug 1650179 - init WC_FILED 'AutoplayPermission' r=nika
alwu <alwu@mozilla.com> - Thu, 02 Jul 2020 21:46:58 +0000 - rev 538521
Push 37564 by nbeleuzu@mozilla.com at Fri, 03 Jul 2020 03:56:55 +0000
Bug 1650179 - init WC_FILED 'AutoplayPermission' r=nika We added that field in bug1597450, but we forgot to set its init value. Differential Revision: https://phabricator.services.mozilla.com/D82077
a32b5c88973552c5ebdd34fafc5f10bcfc3aabeb: Bug 1648679 - Allow setting UseErrorPages from the parent process. r=nika
Matt Woodrow <mwoodrow@mozilla.com> - Thu, 02 Jul 2020 00:53:18 +0000 - rev 538345
Push 37561 by ncsoregi@mozilla.com at Thu, 02 Jul 2020 09:46:06 +0000
Bug 1648679 - Allow setting UseErrorPages from the parent process. r=nika Differential Revision: https://phabricator.services.mozilla.com/D81484
132e64564bdd24a11a23d7da6dd6b3c3978a8716: Bug 1634846 - P3. Get around NS_INLINE_DECL_REFCOUNTING not working with TaskQueue. r=nika,froydnj
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 02 Jul 2020 00:26:43 +0000 - rev 538340
Push 37561 by ncsoregi@mozilla.com at Thu, 02 Jul 2020 09:46:06 +0000
Bug 1634846 - P3. Get around NS_INLINE_DECL_REFCOUNTING not working with TaskQueue. r=nika,froydnj NS_INLINE_DECL_REFCOUNTING macro doesn't properly work when the object is used on a thread that isn't backed by a single PRThread (such as TaskQueue). See bug 1648031. The resolution of this issue is rather complex, and outside the scope of this series of change. So for now, we create a new macro NS_INLINE_DECL_REFCOUNTING_ONEVENTTHREAD which will use a different mechanism to ensure the thread-safe usage of a class. Differential Revision: https://phabricator.services.mozilla.com/D81269
ffa41c2b08b3612f90cdf160f2fef0252db7dbd0: Bug 1634846 - P2. Make ipc's MessageChannel works with TaskQueue, r=nika
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 02 Jul 2020 00:26:41 +0000 - rev 538339
Push 37561 by ncsoregi@mozilla.com at Thu, 02 Jul 2020 09:46:06 +0000
Bug 1634846 - P2. Make ipc's MessageChannel works with TaskQueue, r=nika We no longer rely of having a message loop for the worker thread. Differential Revision: https://phabricator.services.mozilla.com/D80655
681e77b6fe1b8df5e54eeb175a48f0afaaae2c5f: Bug 1634846 - P1. Remove unused method. r=nika,jld
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 02 Jul 2020 00:25:35 +0000 - rev 538338
Push 37561 by ncsoregi@mozilla.com at Thu, 02 Jul 2020 09:46:06 +0000
Bug 1634846 - P1. Remove unused method. r=nika,jld And the less use of MessageLoop, the better. Differential Revision: https://phabricator.services.mozilla.com/D80654
9cf50d4ea19fb05de3b0822f7e3aab36c96c4e63: Bug 1644795 - Expose OS Pid on nsIDOMProcessParent. r=nika
Alexandre Poirot <poirot.alex@gmail.com> - Wed, 01 Jul 2020 09:09:52 +0000 - rev 538166
Push 37559 by rmaries@mozilla.com at Wed, 01 Jul 2020 21:32:12 +0000
Bug 1644795 - Expose OS Pid on nsIDOMProcessParent. r=nika Differential Revision: https://phabricator.services.mozilla.com/D81122
7cc7ff20205ef335a54cea79690cb76181102cea: Bug 1644795 - Expose all nsIDOMProcessParent via ChromeUtils.getAllDOMProcesses(). r=nika
Alexandre Poirot <poirot.alex@gmail.com> - Wed, 01 Jul 2020 09:09:44 +0000 - rev 538165
Push 37559 by rmaries@mozilla.com at Wed, 01 Jul 2020 21:32:12 +0000
Bug 1644795 - Expose all nsIDOMProcessParent via ChromeUtils.getAllDOMProcesses(). r=nika Differential Revision: https://phabricator.services.mozilla.com/D81121
6debf3390ba2274e7eddb5830b45fd31088de6b8: Bug 1646609: Don't assert when an inactive/OOP inner window tries to navigate an OOP BrowsingContext. r=nika
Kris Maglione <maglione.k@gmail.com> - Tue, 30 Jun 2020 22:19:56 +0000 - rev 538119
Push 37557 by abutkovits@mozilla.com at Wed, 01 Jul 2020 03:27:11 +0000
Bug 1646609: Don't assert when an inactive/OOP inner window tries to navigate an OOP BrowsingContext. r=nika Differential Revision: https://phabricator.services.mozilla.com/D81731
6d882a42cfbe0c49521ba2513f5d176466c3fa91: Bug 1646519: Fix GetInProcessTopInternal usage in GetZone. r=nika
Kris Maglione <maglione.k@gmail.com> - Thu, 25 Jun 2020 21:13:10 +0000 - rev 538088
Push 37557 by abutkovits@mozilla.com at Wed, 01 Jul 2020 03:27:11 +0000
Bug 1646519: Fix GetInProcessTopInternal usage in GetZone. r=nika Differential Revision: https://phabricator.services.mozilla.com/D80092
f17be2edac145591c3559810d5023f83ca0d28d0: Bug 1646513: Fix GetInProcessParentDocument usage in CheckForSubFrameDrop. r=nika
Kris Maglione <maglione.k@gmail.com> - Thu, 25 Jun 2020 21:01:51 +0000 - rev 538087
Push 37557 by abutkovits@mozilla.com at Wed, 01 Jul 2020 03:27:11 +0000
Bug 1646513: Fix GetInProcessParentDocument usage in CheckForSubFrameDrop. r=nika We only allow drops from descendant frames into ancestors if they're same origin anyway, which implies same-process, but outside of Fission, we allow cross-origin interstitial frames, whereas with Fission we currently don't due to our walking the ancestor tree using GetInProcessParentDocument. Differential Revision: https://phabricator.services.mozilla.com/D80081
c83ae75568b4d22a36b6260a891e00e69d52c418: Bug 1646482: Stop using GetInProcessParentDocument in DoSecureToInsecureSubmitCheck. r=nika
Kris Maglione <maglione.k@gmail.com> - Thu, 25 Jun 2020 20:58:46 +0000 - rev 538083
Push 37557 by abutkovits@mozilla.com at Wed, 01 Jul 2020 03:27:11 +0000
Bug 1646482: Stop using GetInProcessParentDocument in DoSecureToInsecureSubmitCheck. r=nika Differential Revision: https://phabricator.services.mozilla.com/D80068
9a07058c8c33bcf912fcc97e4a131abd255d6d5d: Bug 1647958 - P3. Have GetCurrentSerialEventTarget returns the currently running MessageLoop. r=nika
Jean-Yves Avenard <jyavenard@mozilla.com> - Tue, 30 Jun 2020 08:04:10 +0000 - rev 537971
Push 37555 by cbrindusan@mozilla.com at Tue, 30 Jun 2020 14:45:59 +0000
Bug 1647958 - P3. Have GetCurrentSerialEventTarget returns the currently running MessageLoop. r=nika We want it to returning the actual nsThread if that's where the MessageLoop would dispatch its tasks; otherwise return the MessageLoop's EventTarget Depends on D80357 Differential Revision: https://phabricator.services.mozilla.com/D80811
1080acf92cfc281e0085444dc1ebba4deffc2d95: Bug 1647958 - P1. Protects MessageLoop::EventTarget's mLoop member. r=nika
Jean-Yves Avenard <jyavenard@mozilla.com> - Tue, 30 Jun 2020 08:04:13 +0000 - rev 537970
Push 37555 by cbrindusan@mozilla.com at Tue, 30 Jun 2020 14:45:59 +0000
Bug 1647958 - P1. Protects MessageLoop::EventTarget's mLoop member. r=nika EventTarget::Dispatch can be called on any threads ; there's a potential of a race when accessing the mLoop member. Differential Revision: https://phabricator.services.mozilla.com/D80810
5863f9c5229ffd4fe3ba63573231e58b300ce6e1: Bug 1634846 - P3. Get around NS_INLINE_DECL_REFCOUNTING not working with TaskQueue. r=nika,froydnj
Jean-Yves Avenard <jyavenard@mozilla.com> - Tue, 30 Jun 2020 02:50:07 +0000 - rev 537899
Push 37555 by cbrindusan@mozilla.com at Tue, 30 Jun 2020 14:45:59 +0000
Bug 1634846 - P3. Get around NS_INLINE_DECL_REFCOUNTING not working with TaskQueue. r=nika,froydnj NS_INLINE_DECL_REFCOUNTING macro doesn't properly work when the object is used on a thread that isn't backed by a single PRThread (such as TaskQueue). See bug 1648031. The resolution of this issue is rather complex, and outside the scope of this series of change. So for now, we create a new macro NS_INLINE_DECL_REFCOUNTING_ONEVENTTHREAD which will use a different mechanism to ensure the thread-safe usage of a class. Differential Revision: https://phabricator.services.mozilla.com/D81269
bca79526745d1c283b4a57546ec38c0067a8f888: Bug 1634846 - P2. Make ipc's MessageChannel works with TaskQueue, r=nika
Jean-Yves Avenard <jyavenard@mozilla.com> - Fri, 26 Jun 2020 09:34:47 +0000 - rev 537898
Push 37555 by cbrindusan@mozilla.com at Tue, 30 Jun 2020 14:45:59 +0000
Bug 1634846 - P2. Make ipc's MessageChannel works with TaskQueue, r=nika We no longer rely of having a message loop for the worker thread. Differential Revision: https://phabricator.services.mozilla.com/D80655
d539408a0048cd8f6336c76627ad730829779212: Bug 1634846 - P1. Remove unused method. r=nika,jld
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 25 Jun 2020 18:28:04 +0000 - rev 537897
Push 37555 by cbrindusan@mozilla.com at Tue, 30 Jun 2020 14:45:59 +0000
Bug 1634846 - P1. Remove unused method. r=nika,jld And the less use of MessageLoop, the better. Differential Revision: https://phabricator.services.mozilla.com/D80654
33b415f44869de360738666d1eb4e857dbabef61: Bug 1647557 - Add preffed-off code for controlling document loads directly from CanonicalBrowsingContext. r=nika,jya,necko-reviewers,dragana
Matt Woodrow <mwoodrow@mozilla.com> - Tue, 30 Jun 2020 01:19:15 +0000 - rev 537894
Push 37555 by cbrindusan@mozilla.com at Tue, 30 Jun 2020 14:45:59 +0000
Bug 1647557 - Add preffed-off code for controlling document loads directly from CanonicalBrowsingContext. r=nika,jya,necko-reviewers,dragana Differential Revision: https://phabricator.services.mozilla.com/D70629
55ecb48a05043c97f77dcfcb1e3ed4c66671523f: Bug 1647958 - P3. Have GetCurrentSerialEventTarget returns the currently running MessageLoop. r=nika
Jean-Yves Avenard <jyavenard@mozilla.com> - Tue, 30 Jun 2020 02:49:05 +0000 - rev 537886
Push 37555 by cbrindusan@mozilla.com at Tue, 30 Jun 2020 14:45:59 +0000
Bug 1647958 - P3. Have GetCurrentSerialEventTarget returns the currently running MessageLoop. r=nika We want it to returning the actual nsThread if that's where the MessageLoop would dispatch its tasks; otherwise return the MessageLoop's EventTarget Depends on D80357 Differential Revision: https://phabricator.services.mozilla.com/D80811
af210e0df79fe695f3399b6e9992a4ebd426b6af: Bug 1647958 - P1. Protects MessageLoop::EventTarget's mLoop member. r=nika
Jean-Yves Avenard <jyavenard@mozilla.com> - Tue, 30 Jun 2020 02:49:06 +0000 - rev 537885
Push 37555 by cbrindusan@mozilla.com at Tue, 30 Jun 2020 14:45:59 +0000
Bug 1647958 - P1. Protects MessageLoop::EventTarget's mLoop member. r=nika EventTarget::Dispatch can be called on any threads ; there's a potential of a race when accessing the mLoop member. Differential Revision: https://phabricator.services.mozilla.com/D80810
e6c12eec1cef32987b7a0b79ccf4420ee097b868: Bug 1633935 - P4 Remove On[Start|Stop]Request/OnTransportAndData in PHttpChannel, r=mayhemer,nika,necko-reviewers
Junior Hsu <juhsu@mozilla.com> - Tue, 30 Jun 2020 00:33:01 +0000 - rev 537872
Push 37554 by nbeleuzu@mozilla.com at Tue, 30 Jun 2020 10:00:42 +0000
Bug 1633935 - P4 Remove On[Start|Stop]Request/OnTransportAndData in PHttpChannel, r=mayhemer,nika,necko-reviewers We move OnStartRequest from PHttpChannel to PHttpBackgroundChannel, thus adjusting message-metadata.ini Depends on D76970 Differential Revision: https://phabricator.services.mozilla.com/D76971
c7d327088f5200996f77880e09117aa558b5006f: Bug 1648787 - remove MOZ_GUARD_OBJECT bits from RecursiveMutex; r=dmajor,xpcom-reviewers,nika
Nathan Froyd <froydnj@mozilla.com> - Mon, 29 Jun 2020 15:37:21 +0000 - rev 537816
Push 37553 by ncsoregi@mozilla.com at Tue, 30 Jun 2020 02:09:30 +0000
Bug 1648787 - remove MOZ_GUARD_OBJECT bits from RecursiveMutex; r=dmajor,xpcom-reviewers,nika Apparently I added these in the initial commit for RecursiveMutex. I'm not quite sure what I was thinking, but we don't need them for the RecursiveMutex itself. (We have them on the corresponding `*Auto*Lock` classes, which are also `MOZ_RAII`.) Differential Revision: https://phabricator.services.mozilla.com/D81345
ed28dd59831c72ce8009538c448172ebffe1c319: Bug 1646892 - Disable DocumentChannel process switching into the parent process when the old-RDM is running, since we don't support switching out of a nested <iframe mozbrowser>. r=nika
Matt Woodrow <mwoodrow@mozilla.com> - Sat, 27 Jun 2020 04:10:48 +0000 - rev 537690
Push 37546 by nbeleuzu@mozilla.com at Sat, 27 Jun 2020 21:48:19 +0000
Bug 1646892 - Disable DocumentChannel process switching into the parent process when the old-RDM is running, since we don't support switching out of a nested <iframe mozbrowser>. r=nika Differential Revision: https://phabricator.services.mozilla.com/D81262
9404dd36d892589b8478352b988c5704c7c352e4: Bug 1646892 - Allow DocumentChannel process switches into the parent process. r=jya,nika
Matt Woodrow <mwoodrow@mozilla.com> - Sat, 27 Jun 2020 04:10:23 +0000 - rev 537688
Push 37546 by nbeleuzu@mozilla.com at Sat, 27 Jun 2020 21:48:19 +0000
Bug 1646892 - Allow DocumentChannel process switches into the parent process. r=jya,nika Differential Revision: https://phabricator.services.mozilla.com/D80327
c2649a68a3a75f092e133ea5fef66297ea7673f5: Bug 1646892 - Move ParentChannelWrapper into a separate file. r=jya,nika
Matt Woodrow <mwoodrow@mozilla.com> - Sat, 27 Jun 2020 04:09:35 +0000 - rev 537687
Push 37546 by nbeleuzu@mozilla.com at Sat, 27 Jun 2020 21:48:19 +0000
Bug 1646892 - Move ParentChannelWrapper into a separate file. r=jya,nika Differential Revision: https://phabricator.services.mozilla.com/D80326
035d17b431efa2c9df104474aaacb17cb935c5ab: Bug 1647470: Fix MinTabSelector to take into account empty but reusable content processes; r=nika
Aaron Klotz <aklotz@mozilla.com> - Sat, 27 Jun 2020 06:14:27 +0000 - rev 537684
Push 37546 by nbeleuzu@mozilla.com at Sat, 27 Jun 2020 21:48:19 +0000
Bug 1647470: Fix MinTabSelector to take into account empty but reusable content processes; r=nika When the current number of existing content processes is less than the maximum, `MinTabSelector` previously always created a new process. This is inefficient in the case where we have `dom.ipc.keepProcessesAlive.web` set and there are content processes that are idle with zero tabs -- we should allow those to be reused. Differential Revision: https://phabricator.services.mozilla.com/D80560
bce7350f629b73cb5917df230ec657cfca9c6c70: Bug 1648270: Get rid of unused ContentParent 'opener' field. r=nika
Kris Maglione <maglione.k@gmail.com> - Fri, 26 Jun 2020 16:26:50 +0000 - rev 537612
Push 37545 by nerli@mozilla.com at Sat, 27 Jun 2020 09:38:32 +0000
Bug 1648270: Get rid of unused ContentParent 'opener' field. r=nika Differential Revision: https://phabricator.services.mozilla.com/D80971
165af6024bdc9b25255c964d4516f64b7efe50f7: Bug 1628249 - Return null from nsFrameLoader::GetExtantBrowsingContext if we're in the middle of destroying the nsFrameLoader. r=nika
Peter Van der Beken <peterv@propagandism.org> - Fri, 26 Jun 2020 14:33:44 +0000 - rev 537596
Push 37545 by nerli@mozilla.com at Sat, 27 Jun 2020 09:38:32 +0000
Bug 1628249 - Return null from nsFrameLoader::GetExtantBrowsingContext if we're in the middle of destroying the nsFrameLoader. r=nika Differential Revision: https://phabricator.services.mozilla.com/D70165
30cde47f9364e5c7da78fd08fa8ab21737d22399: Bug 1626362 - Don't return a failure from nsDocumentOpenInfo::OnStartRequest if we decide to handle the response with the external helper app and then fail. r=nika
Matt Woodrow <mwoodrow@mozilla.com> - Thu, 25 Jun 2020 22:13:31 +0000 - rev 537513
Push 37543 by abutkovits@mozilla.com at Fri, 26 Jun 2020 03:24:35 +0000
Bug 1626362 - Don't return a failure from nsDocumentOpenInfo::OnStartRequest if we decide to handle the response with the external helper app and then fail. r=nika The WPT a-download-click-404.html requires this behaviour for links with the download attribute, and this is also the current behaviour for Content-Disposition: Attachment (see bug 1604308). This previously worked because the parent process version of nsDocumentOpenInfo failed (with NS_ERROR_FILE_NOT_FOUND), but the error code was discarded and we forwarded the channel to the content process. The content process version then would then return NS_ERROR_WONT_HANDLE_CONTENT since the load requires downloading, but we don't allow that in the content process. This new error code is one that doesn't have an associated error page (unlilke the original error), so was silently discarded. Differential Revision: https://phabricator.services.mozilla.com/D81014
b6efc270e9a0474f3dd41b05c8f264145af21cbe: Bug 1626362 - Don't forward loads to DocumentLoadListener if nsDocumentOpenInfo returned a failure, since doing so would lose the error code. r=nika
Matt Woodrow <mwoodrow@mozilla.com> - Thu, 25 Jun 2020 04:33:46 +0000 - rev 537512
Push 37543 by abutkovits@mozilla.com at Fri, 26 Jun 2020 03:24:35 +0000
Bug 1626362 - Don't forward loads to DocumentLoadListener if nsDocumentOpenInfo returned a failure, since doing so would lose the error code. r=nika Differential Revision: https://phabricator.services.mozilla.com/D79409
53cb6ca4a42f0d0deb7dc5f1eaecad7aebcf008f: Bug 1626362 - Let DocumentLoadListener determine if docshell will display content (from the channel or from loading an error page), and only allow switching to a new process if so. r=nika
Matt Woodrow <mwoodrow@mozilla.com> - Thu, 25 Jun 2020 04:33:27 +0000 - rev 537511
Push 37543 by abutkovits@mozilla.com at Fri, 26 Jun 2020 03:24:35 +0000
Bug 1626362 - Let DocumentLoadListener determine if docshell will display content (from the channel or from loading an error page), and only allow switching to a new process if so. r=nika Differential Revision: https://phabricator.services.mozilla.com/D80111
45ec43c2ad0f980a0b2cfc4ef975a07040611b8a: Bug 1626362 - Make mOriginalUriString available in DocumentLoadListener and docshell. r=nika,necko-reviewers,valentin
Matt Woodrow <mwoodrow@mozilla.com> - Thu, 25 Jun 2020 04:33:14 +0000 - rev 537510
Push 37543 by abutkovits@mozilla.com at Fri, 26 Jun 2020 03:24:35 +0000
Bug 1626362 - Make mOriginalUriString available in DocumentLoadListener and docshell. r=nika,necko-reviewers,valentin This also fixes a bug where we were setting mOriginalUriString in docshell before InternalLoad (which clears it), instead of after. Differential Revision: https://phabricator.services.mozilla.com/D80110
4e545131f22ad2021445ca24f2701f4be759cd77: Bug 1626362 - Move nsDocShell::mUserErrorPages to BrowsingContext. r=nika
Matt Woodrow <mwoodrow@mozilla.com> - Thu, 25 Jun 2020 04:32:51 +0000 - rev 537509
Push 37543 by abutkovits@mozilla.com at Fri, 26 Jun 2020 03:24:35 +0000
Bug 1626362 - Move nsDocShell::mUserErrorPages to BrowsingContext. r=nika This is only used by Thunderbird, and is always true for Firefox. I've made CanSet only allow the embedder process, which is the desired behaviour, and should work for non-e10s. Differential Revision: https://phabricator.services.mozilla.com/D80109
5c59b040c35a7225aa2c28afb2f1eaa4b849dd4f: Bug 1626362 - Move url classifier notifications to parent process. r=nika,necko-reviewers,JuniorHsu
Matt Woodrow <mwoodrow@mozilla.com> - Thu, 25 Jun 2020 04:32:28 +0000 - rev 537508
Push 37543 by abutkovits@mozilla.com at Fri, 26 Jun 2020 03:24:35 +0000
Bug 1626362 - Move url classifier notifications to parent process. r=nika,necko-reviewers,JuniorHsu We shouldn't need to handle this with an early return in docshell, since the classifier failure codes won't display an error page anyway. Differential Revision: https://phabricator.services.mozilla.com/D80108
741edfcbb68d57d6ed1204f77fc4eae603327d1b: Bug 1626362 - Combine two load error page functions into one. r=nika
Matt Woodrow <mwoodrow@mozilla.com> - Thu, 25 Jun 2020 04:32:06 +0000 - rev 537507
Push 37543 by abutkovits@mozilla.com at Fri, 26 Jun 2020 03:24:35 +0000
Bug 1626362 - Combine two load error page functions into one. r=nika I don't think we need these to be done at different times, since the set of checks that happen before uri fixup are error codes that won't be affected by fixup. Differential Revision: https://phabricator.services.mozilla.com/D80107
a9e250dce7ed9041e963de9ca4a7ea5e4c95abaa: Bug 1626362 - Move checks for deciding if we should load an error page into helpers. r=nika
Matt Woodrow <mwoodrow@mozilla.com> - Thu, 25 Jun 2020 04:32:04 +0000 - rev 537506
Push 37543 by abutkovits@mozilla.com at Fri, 26 Jun 2020 03:24:35 +0000
Bug 1626362 - Move checks for deciding if we should load an error page into helpers. r=nika Differential Revision: https://phabricator.services.mozilla.com/D80106
721b94e6e3ce201fe1999e89807cc36f2934ef4e: Bug 1626362 - Move docshell uri fixup code into a static helper. r=nika
Matt Woodrow <mwoodrow@mozilla.com> - Thu, 25 Jun 2020 04:32:01 +0000 - rev 537505
Push 37543 by abutkovits@mozilla.com at Fri, 26 Jun 2020 03:24:35 +0000
Bug 1626362 - Move docshell uri fixup code into a static helper. r=nika Differential Revision: https://phabricator.services.mozilla.com/D80104
840d3dc313f1d9fda0cf396e3c7e27634ead28a4: Bug 1648270: Get rid of unused ContentParent 'opener' field. r=nika
Kris Maglione <maglione.k@gmail.com> - Thu, 25 Jun 2020 20:30:33 +0000 - rev 537490
Push 37543 by abutkovits@mozilla.com at Fri, 26 Jun 2020 03:24:35 +0000
Bug 1648270: Get rid of unused ContentParent 'opener' field. r=nika Differential Revision: https://phabricator.services.mozilla.com/D80971
3468660179b1acde1b969d1af576e0b1ea647551: Bug 1648270: Get rid of unused ContentParent 'opener' field. r=nika
Kris Maglione <maglione.k@gmail.com> - Thu, 25 Jun 2020 17:53:39 +0000 - rev 537477
Push 37543 by abutkovits@mozilla.com at Fri, 26 Jun 2020 03:24:35 +0000
Bug 1648270: Get rid of unused ContentParent 'opener' field. r=nika Differential Revision: https://phabricator.services.mozilla.com/D80971
00d72d88ddce8b62e794f186eea452f27ae10b01: Bug 1619649 - Update a few wpt tests' expectations; r=nika,annevk
Tom Tung <shes050117@gmail.com> - Thu, 25 Jun 2020 16:48:38 +0000 - rev 537448
Push 37542 by abutkovits@mozilla.com at Thu, 25 Jun 2020 21:45:10 +0000
Bug 1619649 - Update a few wpt tests' expectations; r=nika,annevk Differential Revision: https://phabricator.services.mozilla.com/D80877
e82b2460bff013c639b610e8bd7eba3f240f3106: Bug 1619649 - Set the prefs for COOP header, COEP header, and postMessage SAB to true on all channels; r=nika
Tom Tung <shes050117@gmail.com> - Wed, 24 Jun 2020 18:25:06 +0000 - rev 537447
Push 37542 by abutkovits@mozilla.com at Thu, 25 Jun 2020 21:45:10 +0000
Bug 1619649 - Set the prefs for COOP header, COEP header, and postMessage SAB to true on all channels; r=nika Differential Revision: https://phabricator.services.mozilla.com/D80858
865f107670e4b81634d166a8a00b6359a7b8b9f5: Bug 1646211 - Fix some clang-11 warnings -Wvoid-pointer-to-int-cast in ContentParent.cpp r=nika
David Major <dmajor@mozilla.com> - Thu, 25 Jun 2020 03:18:14 +0000 - rev 537302
Push 37540 by cbrindusan@mozilla.com at Thu, 25 Jun 2020 09:44:52 +0000
Bug 1646211 - Fix some clang-11 warnings -Wvoid-pointer-to-int-cast in ContentParent.cpp r=nika clang-11 doesn't like the `(long)` casts: `error: cast to smaller integer type 'long' from 'void *' [-Werror,-Wvoid-pointer-to-int-cast]` Differential Revision: https://phabricator.services.mozilla.com/D79919
ec3fb1bc94eae81cbe7b40eaf23ed683ec32b5ec: Bug 1637035 - Do not inherit COEP from opener for non-http or non-initial-about:blank documents r=nika
Eden Chuang <echuang@mozilla.com> - Thu, 25 Jun 2020 02:14:29 +0000 - rev 537298
Push 37540 by cbrindusan@mozilla.com at Thu, 25 Jun 2020 09:44:52 +0000
Bug 1637035 - Do not inherit COEP from opener for non-http or non-initial-about:blank documents r=nika Differential Revision: https://phabricator.services.mozilla.com/D80414
d29ac651a4fd75e4eb32a8dbd26cb07ce0f44b23: Bug 1626362 - Don't forward loads to DocumentLoadListener if nsDocumentOpenInfo returned a failure, since doing so would lose the error code. r=nika
Matt Woodrow <mwoodrow@mozilla.com> - Wed, 24 Jun 2020 22:04:07 +0000 - rev 537269
Push 37540 by cbrindusan@mozilla.com at Thu, 25 Jun 2020 09:44:52 +0000
Bug 1626362 - Don't forward loads to DocumentLoadListener if nsDocumentOpenInfo returned a failure, since doing so would lose the error code. r=nika Differential Revision: https://phabricator.services.mozilla.com/D79409
5ba5cf98a95d1862f6d344df511a536db289f628: Bug 1626362 - Let DocumentLoadListener determine if docshell will display content (from the channel or from loading an error page), and only allow switching to a new process if so. r=nika
Matt Woodrow <mwoodrow@mozilla.com> - Wed, 24 Jun 2020 22:03:52 +0000 - rev 537268
Push 37540 by cbrindusan@mozilla.com at Thu, 25 Jun 2020 09:44:52 +0000
Bug 1626362 - Let DocumentLoadListener determine if docshell will display content (from the channel or from loading an error page), and only allow switching to a new process if so. r=nika Differential Revision: https://phabricator.services.mozilla.com/D80111
69b5b01da9c14f654a76c22a7447f86231c5f141: Bug 1626362 - Make mOriginalUriString available in DocumentLoadListener and docshell. r=nika,necko-reviewers,valentin
Matt Woodrow <mwoodrow@mozilla.com> - Wed, 24 Jun 2020 22:03:39 +0000 - rev 537267
Push 37540 by cbrindusan@mozilla.com at Thu, 25 Jun 2020 09:44:52 +0000
Bug 1626362 - Make mOriginalUriString available in DocumentLoadListener and docshell. r=nika,necko-reviewers,valentin This also fixes a bug where we were setting mOriginalUriString in docshell before InternalLoad (which clears it), instead of after. Differential Revision: https://phabricator.services.mozilla.com/D80110
1d31061f44217f36d01820cbf2bc8ddf3f0c033a: Bug 1626362 - Move nsDocShell::mUserErrorPages to BrowsingContext. r=nika
Matt Woodrow <mwoodrow@mozilla.com> - Wed, 24 Jun 2020 22:03:16 +0000 - rev 537266
Push 37540 by cbrindusan@mozilla.com at Thu, 25 Jun 2020 09:44:52 +0000
Bug 1626362 - Move nsDocShell::mUserErrorPages to BrowsingContext. r=nika This is only used by Thunderbird, and is always true for Firefox. I've made CanSet only allow the embedder process, which is the desired behaviour, and should work for non-e10s. Differential Revision: https://phabricator.services.mozilla.com/D80109
a627870e35fb885618e5bf205a2bc81551ae32dc: Bug 1626362 - Move url classifier notifications to parent process. r=nika,necko-reviewers,JuniorHsu
Matt Woodrow <mwoodrow@mozilla.com> - Wed, 24 Jun 2020 22:03:04 +0000 - rev 537265
Push 37540 by cbrindusan@mozilla.com at Thu, 25 Jun 2020 09:44:52 +0000
Bug 1626362 - Move url classifier notifications to parent process. r=nika,necko-reviewers,JuniorHsu We shouldn't need to handle this with an early return in docshell, since the classifier failure codes won't display an error page anyway. Differential Revision: https://phabricator.services.mozilla.com/D80108
5f3ee46032b6fe65e14d3ac06ab7331394d7e318: Bug 1626362 - Combine two load error page functions into one. r=nika
Matt Woodrow <mwoodrow@mozilla.com> - Wed, 24 Jun 2020 22:02:56 +0000 - rev 537264
Push 37540 by cbrindusan@mozilla.com at Thu, 25 Jun 2020 09:44:52 +0000
Bug 1626362 - Combine two load error page functions into one. r=nika I don't think we need these to be done at different times, since the set of checks that happen before uri fixup are error codes that won't be affected by fixup. Differential Revision: https://phabricator.services.mozilla.com/D80107
5b170666991e6d1c119edb134b78fe9916222cb4: Bug 1626362 - Move checks for deciding if we should load an error page into helpers. r=nika
Matt Woodrow <mwoodrow@mozilla.com> - Wed, 24 Jun 2020 22:01:55 +0000 - rev 537263
Push 37540 by cbrindusan@mozilla.com at Thu, 25 Jun 2020 09:44:52 +0000
Bug 1626362 - Move checks for deciding if we should load an error page into helpers. r=nika Differential Revision: https://phabricator.services.mozilla.com/D80106