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

Bug 1757575 - Fix non-unified-build bustage in dom/html/HTMLFormElement.cpp; r=smaug

Browse Source 
Differential Revision: https://phabricator.services.mozilla.com/D139947
This commit is contained in:
Jan Varga
2022-03-01 17:39:18 +00:00
parent 4e1a9ecb09
commit b93e399d5c
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
dom/html/HTMLFormElement.cpp
View File

@@ -1132,10 +1132,9 @@ int32_t HTMLFormElement::CompareFormControlPosition(Element* aElement1,
/* static */
void HTMLFormElement::AssertDocumentOrder(
const nsTArray<nsGenericHTMLFormElement*>& aControls, nsIContent* aForm) {
// TODO: remove the return statement with bug 598468.
// TODO: remove the if directive with bug 598468.
// This is done to prevent asserts in some edge cases.
return;
# if 0
// Only iterate if aControls is not empty, since otherwise
// |aControls.Length() - 1| will be a very large unsigned number... not what
// we want here.
@@ -1146,6 +1145,7 @@ void HTMLFormElement::AssertDocumentOrder(
"Form controls not ordered correctly");
}
}
# endif
}
/**
@@ -1158,10 +1158,9 @@ void HTMLFormElement::AssertDocumentOrder(
void HTMLFormElement::AssertDocumentOrder(
const nsTArray<RefPtr<nsGenericHTMLFormElement>>& aControls,
nsIContent* aForm) {
// TODO: remove the return statement with bug 598468.
// TODO: remove the if directive with bug 598468.
// This is done to prevent asserts in some edge cases.
return;
# if 0
// Only iterate if aControls is not empty, since otherwise
// |aControls.Length() - 1| will be a very large unsigned number... not what
// we want here.
@@ -1172,6 +1171,7 @@ void HTMLFormElement::AssertDocumentOrder(
"Form controls not ordered correctly");
}
}
# endif
}
#endif