Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-10-10 03:45:46 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1596768 - Remove GetBindingParentUsage in xpath. r=smaug

Browse Source 
Wants to check that the change happens in the same anon tree, use the right
function for that.

Depends on D53199

Differential Revision: https://phabricator.services.mozilla.com/D53200

--HG--
extra : moz-landing-system : lando
This commit is contained in:
Emilio Cobos Álvarez 2019-11-15 17:09:15 +00:00
parent bbbd04084e
commit 1881ee2c15
1 changed files with 5 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
dom/xslt/xpath/XPathResult.cpp
View File

@ -205,22 +205,11 @@ nsresult XPathResult::SetExprResult(txAExprResult* aExprResult,
void XPathResult::Invalidate(const nsIContent* aChangeRoot) {
nsCOMPtr<nsINode> contextNode = do_QueryReferent(mContextNode);
if (contextNode && aChangeRoot && aChangeRoot->GetBindingParent()) {
// If context node is in anonymous content, changes to
// non-anonymous content need to invalidate the XPathResult. If
// the changes are happening in a different anonymous trees, no
// invalidation should happen.
nsIContent* ctxBindingParent = nullptr;
if (contextNode->IsContent()) {
ctxBindingParent = contextNode->AsContent()->GetBindingParent();
} else if (auto* attr = Attr::FromNode(contextNode)) {
if (Element* parent = attr->GetElement()) {
ctxBindingParent = parent->GetBindingParent();
}
}
if (ctxBindingParent != aChangeRoot->GetBindingParent()) {
return;
}
// If the changes are happening in a different anonymous trees, no
// invalidation should happen.
if (contextNode && aChangeRoot &&
!nsContentUtils::IsInSameAnonymousTree(contextNode, aChangeRoot)) {
return;
}
mInvalidIteratorState = true;