Bug 1394935: Add a (commented out for now) assertion about clobbering dirty bits. r?bholley draft
authorEmilio Cobos Álvarez <emilio@crisal.io>
Thu, 31 Aug 2017 09:42:20 +0200
changeset 656814 3134045df751c2d6a251cff182131c8fc4df7ea1
parent 656813 7e15bea104df62ee4cef9dba0f8509853d20601e
child 656815 2253ef9871c6d1ea3c3de8e8d5f20a529e9083cf
push id77325
push userbmo:emilio@crisal.io
push dateThu, 31 Aug 2017 18:09:13 +0000
reviewersbholley
bugs1394935
milestone57.0a1
Bug 1394935: Add a (commented out for now) assertion about clobbering dirty bits. r?bholley Will get this fixed tomorrow, I have patches but forgot to push to try. MozReview-Commit-ID: 3Dcb1MTo8g5
dom/base/nsIDocumentInlines.h
--- a/dom/base/nsIDocumentInlines.h
+++ b/dom/base/nsIDocumentInlines.h
@@ -59,16 +59,20 @@ nsIDocument::FindDocStyleSheetInsertionP
 }
 
 inline void
 nsIDocument::SetServoRestyleRoot(nsINode* aRoot, uint32_t aDirtyBits)
 {
   MOZ_ASSERT(aRoot);
   MOZ_ASSERT(aDirtyBits);
   MOZ_ASSERT((aDirtyBits & ~Element::kAllServoDescendantBits) == 0);
+  // FIXME(emilio): This should hold, but we're clobbering it from
+  // nsCSSFrameConstructor::ConstructDocElementFrame, which is wrong!
+  //
+  // MOZ_ASSERT((aDirtyBits & mServoRestyleRootDirtyBits) == mServoRestyleRootDirtyBits);
 
   MOZ_ASSERT(!mServoRestyleRoot ||
              mServoRestyleRoot == aRoot ||
              nsContentUtils::ContentIsFlattenedTreeDescendantOfForStyle(mServoRestyleRoot, aRoot));
   MOZ_ASSERT(aRoot == aRoot->OwnerDocAsNode() ||
              (aRoot->IsElement() && aRoot->IsInComposedDoc()));
   mServoRestyleRoot = aRoot;
   mServoRestyleRootDirtyBits = aDirtyBits;