Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-11-24 13:21:05 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1802346 - Use the owner principal in ParseFromSafeString, r=smaug

Browse Source 
This is done rather than using the system principal to avoid potential
cross-docgroup adoption issues.

Differential Revision: https://phabricator.services.mozilla.com/D163017
This commit is contained in:
Nika Layzell 2022-11-25 20:56:25 +00:00
parent a441ec4cdd
commit 102c1465fe
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
dom/base/DOMParser.cpp
View File

@ -103,12 +103,13 @@ already_AddRefed<Document> DOMParser::ParseFromString(const nsAString& aStr,
already_AddRefed<Document> DOMParser::ParseFromSafeString(const nsAString& aStr,
SupportedType aType,
ErrorResult& aRv) {
// Since we disable cross docGroup node adoption, it is safe to create
// new document with the system principal, then the new document will be
// placed in the same docGroup as the chrome document.
// Create the new document with the same principal as `mOwner`, even if it is
// the system principal. This will ensure that nodes from the returned
// document are in the same DocGroup as the owner global's document, allowing
// nodes to be adopted.
nsCOMPtr<nsIPrincipal> docPrincipal = mPrincipal;
if (!mPrincipal->IsSystemPrincipal()) {
mPrincipal = SystemPrincipal::Create();
if (mOwner && mOwner->PrincipalOrNull()) {
mPrincipal = mOwner->PrincipalOrNull();
}
RefPtr<Document> ret = ParseFromString(aStr, aType, aRv);