Bug 1448413: Do a bit of misc cleanup. r=jwatt

Summary: Mostly removing unneeded AsServo() and such.

Reviewers: jwatt

Bug #: 1448413

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

MozReview-Commit-ID: 8muHF7uMA53
This commit is contained in:
Emilio Cobos Álvarez
2018-03-23 18:47:05 +01:00
parent 68804bb222
commit 0b3a18a950
7 changed files with 72 additions and 79 deletions

View File

@@ -8182,9 +8182,9 @@ nsTableFrame::UpdateStyleOfOwnedAnonBoxesForTableWrapper(
nsCSSAnonBoxes::tableWrapper,
"What happened to our parent?");
RefPtr<ComputedStyle> newContext =
RefPtr<ComputedStyle> newStyle =
aRestyleState.StyleSet().ResolveInheritingAnonymousBoxStyle(
nsCSSAnonBoxes::tableWrapper, aOwningFrame->Style()->AsServo());
nsCSSAnonBoxes::tableWrapper, aOwningFrame->Style());
// Figure out whether we have an actual change. It's important that we do
// this, even though all the wrapper's changes are due to properties it
@@ -8201,7 +8201,7 @@ nsTableFrame::UpdateStyleOfOwnedAnonBoxesForTableWrapper(
// compared to the owner frame.
uint32_t equalStructs, samePointerStructs; // Not used, actually.
nsChangeHint wrapperHint = aWrapperFrame->Style()->CalcStyleDifference(
newContext,
newStyle,
&equalStructs,
&samePointerStructs,
/* aIgnoreVariables = */ true);
@@ -8219,7 +8219,7 @@ nsTableFrame::UpdateStyleOfOwnedAnonBoxesForTableWrapper(
}
for (nsIFrame* cur = aWrapperFrame; cur; cur = cur->GetNextContinuation()) {
cur->SetComputedStyle(newContext);
cur->SetComputedStyle(newStyle);
}
MOZ_ASSERT(!(aWrapperFrame->GetStateBits() & NS_FRAME_OWNS_ANON_BOXES),