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

Bug 455314. RemoveIDTargetObserver doesn't need to do the full GetElementByIdInternal because we don't need or want to set up an mIdentifierMap entry if no entry is found for the ID. r+sr=jst

Browse Source
This commit is contained in:
Robert O'Callahan 2008-12-12 21:20:30 +13:00
parent e0206a44c5
commit 9106f372f7
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
content/base/src/nsDocument.cpp
View File

@ -3855,9 +3855,13 @@ nsDocument::RemoveIDTargetObserver(nsIAtom* aID,
if (!CheckGetElementByIdArg(aID))
return;
nsIdentifierMapEntry *entry = GetElementByIdInternal(aID);
if (!entry)
nsIdentifierMapEntry *entry = mIdentifierMap.GetEntry(aID);
if (!entry) {
// We don't need to do the stuff that GetElementByIdInternal does;
// if there's no entry already in mIdentifierMap, then there's no
// callback to remove.
return;
}
entry->RemoveContentChangeCallback(aObserver, aData);
}