Bug 1535507 - Don't force display item rebuilding for ActiveLayerTracker expiry, since we should only need to update the Layer tree. r=miko
☠☠ backed out by 5fd21107e394 ☠ ☠
authorMatt Woodrow <mwoodrow@mozilla.com>
Mon, 18 Mar 2019 21:10:46 +0000
changeset 464872 f2061889974a1a7ce5410689e542a0c39a084540
parent 464871 e3745459533b0c3cde01d4506bff44423230f527
child 464873 a9c6e9e27b17013a4ff9793b052ec1383c7f9b7f
push id35727
push userdvarga@mozilla.com
push dateTue, 19 Mar 2019 09:48:59 +0000
treeherdermozilla-central@70baa37ae1eb [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmiko
bugs1535507
milestone68.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 1535507 - Don't force display item rebuilding for ActiveLayerTracker expiry, since we should only need to update the Layer tree. r=miko Differential Revision: https://phabricator.services.mozilla.com/D23608
layout/painting/ActiveLayerTracker.cpp
--- a/layout/painting/ActiveLayerTracker.cpp
+++ b/layout/painting/ActiveLayerTracker.cpp
@@ -187,17 +187,17 @@ void LayerActivityTracker::NotifyExpired
   MOZ_ASSERT((f == nullptr) != (c == nullptr),
              "A LayerActivity object should always have a reference to either "
              "its frame or its content");
 
   if (f) {
     // The pres context might have been detached during the delay -
     // that's fine, just skip the paint.
     if (f->PresContext()->GetContainerWeak()) {
-      f->SchedulePaint();
+      f->SchedulePaint(nsIFrame::PAINT_DEFAULT, false);
     }
     f->RemoveStateBits(NS_FRAME_HAS_LAYER_ACTIVITY_PROPERTY);
     f->DeleteProperty(LayerActivityProperty());
   } else {
     c->DeleteProperty(nsGkAtoms::LayerActivity);
   }
 }