corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1827516 - Cleanup some event target parent code. r=smaug

Browse Source 
No behavior change here, but I was looking and this code can be simpler.

Differential Revision: https://phabricator.services.mozilla.com/D175714
This commit is contained in:
Emilio Cobos Álvarez
2023-04-18 09:23:06 +00:00
parent 64ee6d1aa2
commit d2e44bf44a
1 changed files with 6 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
dom/base/FragmentOrElement.cpp
View File

@@ -731,23 +731,15 @@ static nsINode* FindChromeAccessOnlySubtreeOwner(nsINode* aNode) {
if (!aNode->ChromeOnlyAccess()) {
return aNode;
}
while (aNode && !aNode->IsRootOfChromeAccessOnlySubtree()) {
aNode = aNode->GetParentNode();
}
return aNode ? aNode->GetParentOrShadowHostNode() : nullptr;
return const_cast<nsIContent*>(aNode->GetChromeOnlyAccessSubtreeRootParent());
}
already_AddRefed<nsINode> FindChromeAccessOnlySubtreeOwner(
EventTarget* aTarget) {
nsCOMPtr<nsINode> node = nsINode::FromEventTargetOrNull(aTarget);
if (!node || !node->ChromeOnlyAccess()) {
return node.forget();
nsINode* FindChromeAccessOnlySubtreeOwner(EventTarget* aTarget) {
nsINode* node = nsINode::FromEventTargetOrNull(aTarget);
if (!node) {
return nullptr;
}
node = FindChromeAccessOnlySubtreeOwner(node);
return node.forget();
return FindChromeAccessOnlySubtreeOwner(node);
}
void nsIContent::GetEventTargetParent(EventChainPreVisitor& aVisitor) {