Bug 1390409 part 1 - Remove unused ServoTraversalFlags param from ServoRestyleManager::ProcessPostTraversal. r?emilio
This was added in
bug 1384769 for passing into Servo_TakeChangeHint,
but
bug 1388031 changed Servo_TakeChangeHint to no longer take it, and
there is nothing else in the function which uses the flags.
MozReview-Commit-ID: LvZkJZHENUB
--- a/layout/base/ServoRestyleManager.cpp
+++ b/layout/base/ServoRestyleManager.cpp
@@ -645,17 +645,16 @@ UpdateFramePseudoElementStyles(nsIFrame*
aFrame, aRestyleState.StyleSet(), aRestyleState.ChangeList());
}
bool
ServoRestyleManager::ProcessPostTraversal(
Element* aElement,
ServoStyleContext* aParentContext,
ServoRestyleState& aRestyleState,
- ServoTraversalFlags aFlags,
bool aParentWasRestyled)
{
nsIFrame* styleFrame = nsLayoutUtils::GetStyleFrame(aElement);
nsIFrame* primaryFrame = aElement->GetPrimaryFrame();
// NOTE(emilio): This is needed because for table frames the bit is set on the
// table wrapper (which is the primary frame), not on the table itself.
const bool isOutOfFlow =
@@ -824,17 +823,16 @@ ServoRestyleManager::ProcessPostTraversa
upToDateContext,
displayContentsStyle && wasRestyled,
childrenRestyleState);
for (nsIContent* n = it.GetNextChild(); n; n = it.GetNextChild()) {
if (traverseElementChildren && n->IsElement()) {
recreatedAnyContext |= ProcessPostTraversal(n->AsElement(),
upToDateContext,
childrenRestyleState,
- aFlags,
wasRestyled);
} else if (traverseTextChildren && n->IsNodeOfType(nsINode::eTEXT)) {
recreatedAnyContext |= ProcessPostTraversalForText(n, textState,
childrenRestyleState,
wasRestyled);
}
}
}
@@ -1022,17 +1020,17 @@ ServoRestyleManager::DoProcessPendingRes
// lazy frame construction).
{
AutoRestyleTimelineMarker marker(mPresContext->GetDocShell(), false);
DocumentStyleRootIterator iter(doc->GetServoRestyleRoot());
while (Element* root = iter.GetNextStyleRoot()) {
nsTArray<nsIFrame*> wrappersToRestyle;
ServoRestyleState state(*styleSet, currentChanges, wrappersToRestyle);
anyStyleChanged |=
- ProcessPostTraversal(root, nullptr, state, aFlags,
+ ProcessPostTraversal(root, nullptr, state,
/* aParentWasRestyled = */ false);
}
}
doc->ClearServoRestyleRoot();
// Process the change hints.
//
--- a/layout/base/ServoRestyleManager.h
+++ b/layout/base/ServoRestyleManager.h
@@ -256,17 +256,16 @@ private:
* Returns whether any style did actually change. There may be cases where we
* didn't need to change any style after all, for example, when a content
* attribute changes that happens not to have any effect on the style of that
* element or any descendant or sibling.
*/
bool ProcessPostTraversal(Element* aElement,
ServoStyleContext* aParentContext,
ServoRestyleState& aRestyleState,
- ServoTraversalFlags aFlags,
bool aParentWasRestyled);
struct TextPostTraversalState;
bool ProcessPostTraversalForText(nsIContent* aTextNode,
TextPostTraversalState& aState,
ServoRestyleState& aRestyleState,
bool aParentWasRestyled);