Bug 1376655 - Stop using EnsureSameStructsCached in ViewportFrame::UpdateStyle. r=emilio
authorBobby Holley <bobbyholley@gmail.com>
Tue, 27 Jun 2017 18:46:28 -0700
changeset 366461 0eb7e3863c5f3750c05497fbff10fbdd7d5ebf97
parent 366460 378597d9a6c3703ea4eec7d3e594799bd3991d84
child 366462 f3e0f0a4ab7b48483dcb6c6283220282b7aec651
push id32104
push usercbook@mozilla.com
push dateThu, 29 Jun 2017 13:46:04 +0000
treeherdermozilla-central@d2aff6fc075d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersemilio
bugs1376655
milestone56.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1376655 - Stop using EnsureSameStructsCached in ViewportFrame::UpdateStyle. r=emilio
layout/generic/ViewportFrame.cpp
--- a/layout/generic/ViewportFrame.cpp
+++ b/layout/generic/ViewportFrame.cpp
@@ -422,17 +422,17 @@ ViewportFrame::UpdateStyle(ServoRestyleS
   nsIAtom* pseudo = oldContext->GetPseudo();
   RefPtr<ServoStyleContext> newContext =
     aRestyleState.StyleSet().ResolveInheritingAnonymousBoxStyle(pseudo, nullptr);
 
   // We're special because we have a null GetContent(), so don't call things
   // like UpdateStyleOfOwnedChildFrame that try to append changes for the
   // content to the change list.  Nor do we computed a changehint, since we have
   // no way to apply it anyway.
-  newContext->EnsureSameStructsCached(oldContext);
+  newContext->ResolveSameStructsAs(PresContext(), oldContext);
 
   MOZ_ASSERT(!GetNextContinuation(), "Viewport has continuations?");
   SetStyleContext(newContext);
 
   UpdateStyleOfOwnedAnonBoxes(aRestyleState);
 }
 
 void