--- a/dom/base/nsContentIterator.cpp
+++ b/dom/base/nsContentIterator.cpp
@@ -340,17 +340,17 @@ nsContentIterator::Init(nsINode* aStartC
return InitInternal(aStartContainer, aStartOffset,
aEndContainer, aEndOffset);
}
// XXX Argument names will be replaced in the following patch.
nsresult
nsContentIterator::InitInternal(nsINode* aStartContainer, uint32_t aStartOffset,
- nsINode* aEndContainer, uint32_t endIndx)
+ nsINode* aEndContainer, uint32_t aEndOffset)
{
// get common content parent
mCommonParent =
nsContentUtils::GetCommonAncestor(aStartContainer, aEndContainer);
if (NS_WARN_IF(!mCommonParent)) {
return NS_ERROR_FAILURE;
}
@@ -360,17 +360,17 @@ nsContentIterator::InitInternal(nsINode*
if (aStartContainer == aEndContainer) {
// Check to see if we have a collapsed range, if so, there is nothing to
// iterate over.
//
// XXX: CharacterDataNodes (text nodes) are currently an exception, since
// we always want to be able to iterate text nodes at the end points
// of a range.
- if (!startIsData && aStartOffset == endIndx) {
+ if (!startIsData && aStartOffset == aEndOffset) {
MakeEmpty();
return NS_OK;
}
if (startIsData) {
// It's a character data node.
mFirst = aStartContainer->AsContent();
mLast = mFirst;
@@ -416,17 +416,17 @@ nsContentIterator::InitInternal(nsINode*
mFirst = GetNextSibling(aStartContainer);
NS_WARNING_ASSERTION(mFirst, "GetNextSibling returned null");
// Does mFirst node really intersect the range? The range could be
// 'degenerate', i.e., not collapsed but still contain no content.
if (mFirst &&
NS_WARN_IF(!NodeIsInTraversalRange(mFirst, mPre,
aStartContainer, aStartOffset,
- aEndContainer, endIndx))) {
+ aEndContainer, aEndOffset))) {
mFirst = nullptr;
}
} else {
mFirst = aStartContainer->AsContent();
}
} else {
// post-order
if (NS_WARN_IF(!aStartContainer->IsContent())) {
@@ -444,50 +444,50 @@ nsContentIterator::InitInternal(nsINode*
mFirst = GetDeepFirstChild(cChild);
NS_WARNING_ASSERTION(mFirst, "GetDeepFirstChild returned null");
// Does mFirst node really intersect the range? The range could be
// 'degenerate', i.e., not collapsed but still contain no content.
if (mFirst &&
!NodeIsInTraversalRange(mFirst, mPre, aStartContainer, aStartOffset,
- aEndContainer, endIndx)) {
+ aEndContainer, aEndOffset)) {
mFirst = nullptr;
}
}
}
// Find last node in range.
bool endIsData = aEndContainer->IsNodeOfType(nsINode::eDATA_NODE);
- if (endIsData || !aEndContainer->HasChildren() || endIndx == 0) {
+ if (endIsData || !aEndContainer->HasChildren() || !aEndOffset) {
if (mPre) {
if (NS_WARN_IF(!aEndContainer->IsContent())) {
// Not much else to do here...
mLast = nullptr;
} else {
// If the end node is a non-container element and the end offset is 0,
// the last element should be the previous node (i.e., shouldn't
// include the end node in the range).
bool endIsContainer = true;
if (aEndContainer->IsHTMLElement()) {
if (nsIParserService* ps = nsContentUtils::GetParserService()) {
nsIAtom* name = aEndContainer->NodeInfo()->NameAtom();
ps->IsContainer(ps->HTMLAtomTagToId(name), endIsContainer);
}
}
- if (!endIsData && !endIsContainer && !endIndx) {
+ if (!endIsData && !endIsContainer && !aEndOffset) {
mLast = PrevNode(aEndContainer);
NS_WARNING_ASSERTION(mLast, "PrevNode returned null");
if (mLast && mLast != mFirst &&
NS_WARN_IF(!NodeIsInTraversalRange(mLast, mPre,
mFirst, 0,
- aEndContainer, endIndx))) {
+ aEndContainer, aEndOffset))) {
mLast = nullptr;
}
} else {
mLast = aEndContainer->AsContent();
}
}
} else {
// post-order
@@ -496,41 +496,41 @@ nsContentIterator::InitInternal(nsINode*
// cdata node, should we set mLast to the prev sibling?
if (!endIsData) {
mLast = GetPrevSibling(aEndContainer);
NS_WARNING_ASSERTION(mLast, "GetPrevSibling returned null");
if (!NodeIsInTraversalRange(mLast, mPre,
aStartContainer, aStartOffset,
- aEndContainer, endIndx)) {
+ aEndContainer, aEndOffset)) {
mLast = nullptr;
}
} else {
mLast = aEndContainer->AsContent();
}
}
} else {
- int32_t indx = endIndx;
+ int32_t indx = aEndOffset;
cChild = aEndContainer->GetChildAt(--indx);
if (NS_WARN_IF(!cChild)) {
// No child at offset!
NS_NOTREACHED("nsContentIterator::nsContentIterator");
return NS_ERROR_FAILURE;
}
if (mPre) {
mLast = GetDeepLastChild(cChild);
NS_WARNING_ASSERTION(mLast, "GetDeepLastChild returned null");
if (NS_WARN_IF(!NodeIsInTraversalRange(mLast, mPre,
aStartContainer, aStartOffset,
- aEndContainer, endIndx))) {
+ aEndContainer, aEndOffset))) {
mLast = nullptr;
}
} else {
// post-order
mLast = cChild;
}
}