Comment out DCHECK for now

R=chrishtr@chromium.org

Bug: 1166296
Change-Id: I26d9e83ce62d9b28a610cf7e747cc4403af3c2a0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2633027
Auto-Submit: Aaron Leventhal <aleventhal@chromium.org>
Commit-Queue: Chris Harrelson <chrishtr@chromium.org>
Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844278}

third_party/blink/renderer/core/editing/element_inner_text.cc

@@ -469,17 +469,15 @@ String Element::innerText() {
 
 // Used for callers that must ensure no document lifecycle rewind.
 String Element::GetInnerTextWithoutUpdate() {
-  // TODO(https:://crbug.com/1165850) Layout should always be clean here, but
-  // the lifecycle does not report the correctly updated value unless servicing
-  // animations. Fix the UpdateStyleAndLayout() to correctly advance the
-  // lifecycle, and then update the following DCHECK to always require clean
-  // layout in active documents.
-  DCHECK(!GetDocument().IsActive() || !GetDocument().GetPage() ||
-         !GetDocument().GetPage()->Animator().IsServicingAnimations() ||
-         GetDocument().Lifecycle().GetState() >=
-             DocumentLifecycle::kLayoutClean)
-      << "If GetInnerTextWithoutUpdate() is called while servicing animations, "
-         "then layout must be clean.";
+  // TODO(https:://crbug.com/1165850 https:://crbug.com/1166296) Layout should
+  // always be clean here, but the lifecycle does not report the correctly
+  // updated value unless servicing animations. Fix the UpdateStyleAndLayout()
+  // to correctly advance the lifecycle, and then update the following DCHECK to
+  // always require clean layout in active documents.
+  // DCHECK(!GetDocument().IsActive() || !GetDocument().GetPage() ||
+  //        GetDocument().Lifecycle().GetState() >=
+  //            DocumentLifecycle::kLayoutClean)
+  //     << "Layout must be clean when GetInnerTextWithoutUpdate() is called.";
   return ElementInnerTextCollector().RunOn(*this);
 }