Bug 1092363 - Disable bug 931668 optimizations for the time being. r=dbaron a=abillings
authorCameron McCormack <cam@mcc.id.au>
Fri, 23 Jan 2015 10:42:20 +1100
changeset 243008 126d92ac00e9
parent 243007 b3792d13df24
child 243011 afab84ec4e34
push id4361
push usercmccormack@mozilla.com
push date2015-01-22 23:43 +0000
treeherdermozilla-beta@126d92ac00e9 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdbaron, abillings
bugs1092363, 931668
milestone36.0
Bug 1092363 - Disable bug 931668 optimizations for the time being. r=dbaron a=abillings
layout/base/RestyleManager.cpp
--- a/layout/base/RestyleManager.cpp
+++ b/layout/base/RestyleManager.cpp
@@ -2925,17 +2925,17 @@ ElementRestyler::RestyleSelf(nsIFrame* a
 
   LOG_RESTYLE("RestyleSelf %s, aRestyleHint = %s",
               FrameTagToString(aSelf).get(),
               RestyleManager::RestyleHintToString(aRestyleHint).get());
   LOG_RESTYLE_INDENT();
 
   RestyleResult result;
 
-  if (aRestyleHint) {
+  if (aRestyleHint || true /* XXX bug 1092363 */) {
     result = eRestyleResult_Continue;
   } else {
     result = ComputeRestyleResultFromFrame(aSelf);
   }
 
   nsChangeHint assumeDifferenceHint = NS_STYLE_HINT_NONE;
   nsRefPtr<nsStyleContext> oldContext = aSelf->StyleContext();
   nsStyleSet* styleSet = mPresContext->StyleSet();
@@ -3223,17 +3223,19 @@ ElementRestyler::RestyleSelf(nsIFrame* a
       // which is important to maintain various invariants about
       // frame types matching their style contexts.
       // Note that this check even makes sense if we didn't call
       // CaptureChange because of copyFromContinuation being true,
       // since we'll have copied the existing context from the
       // previous continuation, so newContext == oldContext.
 
       if (result != eRestyleResult_Stop) {
-        if (copyFromContinuation) {
+        if (true) {
+          // XXX bug 1092363
+        } else if (copyFromContinuation) {
           LOG_RESTYLE("not swapping style structs, since we copied from a "
                       "continuation");
         } else if (oldContext->IsShared() && newContext->IsShared()) {
           LOG_RESTYLE("not swapping style structs, since both old and contexts "
                       "are shared");
         } else if (oldContext->IsShared()) {
           LOG_RESTYLE("not swapping style structs, since the old context is "
                       "shared");