Bug 1115812 patch 1 - Remove obsolete comment that was fixed by bug 1047928. r=heycam
authorL. David Baron <dbaron@dbaron.org>
Tue, 13 Jan 2015 21:03:11 -0800
changeset 223702 8b5e114c159c7d148603bd790883f3dce106e65c
parent 223701 7d3ef4f3fa8fc5509203c0290fa7941bdbaad07b
child 223703 8927072cd0fb0216ce2205c8652c217c0af91eef
push id10811
push usercbook@mozilla.com
push dateWed, 14 Jan 2015 13:52:17 +0000
treeherderfx-team@b8d38d19dbd9 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersheycam
bugs1115812, 1047928
milestone38.0a1
Bug 1115812 patch 1 - Remove obsolete comment that was fixed by bug 1047928. r=heycam
layout/base/RestyleManager.cpp
--- a/layout/base/RestyleManager.cpp
+++ b/layout/base/RestyleManager.cpp
@@ -1560,19 +1560,16 @@ RestyleManager::DoRebuildAllStyleData(Re
       aRestyleTracker.AddPendingRestyle(root, aRestyleHint, nsChangeHint(0));
     }
     aRestyleHint = nsRestyleHint(0);
   }
 
   // Recalculate all of the style contexts for the document
   // Note that we can ignore the return value of ComputeStyleChangeFor
   // because we never need to reframe the root frame
-  // XXX This could be made faster by not rerunning rule matching
-  // (but note that nsPresShell::SetPreferenceStyleRules currently depends
-  // on us re-running rule matching here
   // XXX Does it matter that we're passing aExtraHint to the real root
   // frame and not the root node's primary frame?  (We could do
   // roughly what we do for aRestyleHint above.)
   // Note: The restyle tracker we pass in here doesn't matter.
   ComputeAndProcessStyleChange(mPresContext->PresShell()->GetRootFrame(),
                                aExtraHint, aRestyleTracker, aRestyleHint);
   FlushOverflowChangedTracker();