Invalidate caret during paint invalidation

third_party/WebKit/Source/core/editing/DragCaret.cpp

 /*
  * Copyright (C) 2004, 2008, 2009, 2010 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
  *
  * THIS SOFTWARE IS PROVIDED BY APPLE COMPUTER, INC. ``AS IS'' AND ANY
  * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
  * PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL APPLE COMPUTER, INC. OR
 * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
  * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
  * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
  * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
  * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "core/editing/DragCaret.h"
 
+#include "core/editing/CaretDisplayItemClient.h"
 #include "core/editing/EditingUtilities.h"
 #include "core/frame/Settings.h"
 #include "core/layout/api/LayoutViewItem.h"
 #include "core/paint/PaintLayer.h"
 
 namespace blink {
 
 DragCaret::DragCaret() : m_caretBase(new CaretDisplayItemClient()) {}
 
 DragCaret::~DragCaret() = default;
 
 DragCaret* DragCaret::create() {
   return new DragCaret;
 }
 
-bool DragCaret::hasCaretIn(const LayoutBlock& layoutBlock) const {
-  Node* node = m_position.anchorNode();
-  if (!node)
-    return false;
-  if (layoutBlock != CaretDisplayItemClient::caretLayoutObject(node))
-    return false;
-  return rootEditableElementOf(m_position.position());
+void DragCaret::clearPreviousVisualRect(const LayoutBlock& block) {
+  m_caretBase->clearPreviousVisualRect(block);
+}
+
+void DragCaret::layoutBlockWillBeDestroyed(const LayoutBlock& block) {
+  m_caretBase->layoutBlockWillBeDestroyed(block);
+}
+
+void DragCaret::updateStyleAndLayoutIfNeeded() {
+  m_caretBase->updateStyleAndLayoutIfNeeded(
+      rootEditableElementOf(m_position.position()) ? m_position
+                                                   : PositionWithAffinity());
+}
+
+void DragCaret::invalidatePaintIfNeeded(const LayoutBlock& block,
+                                        const PaintInvalidatorContext& context,
+                                        PaintInvalidationReason reason) {
+  m_caretBase->invalidatePaintIfNeeded(block, context, reason);
 }
 
 bool DragCaret::isContentRichlyEditable() const {
   return isRichlyEditablePosition(m_position.position());
 }
 
-void DragCaret::invalidateCaretRect(Node* node,
-                                    const LayoutRect& caretLocalRect) {
-  // TODO(editing-dev): The use of updateStyleAndLayout
-  // needs to be audited.  See http://crbug.com/590369 for more details.
-  // In the long term we should use idle time spell checker to prevent
-  // synchronous layout caused by spell checking (see crbug.com/517298).
-  node->document().updateStyleAndLayoutTree();
-  if (!hasEditableStyle(*node))
-    return;
-  m_caretBase->invalidateLocalCaretRect(node, caretLocalRect);
-}
-
 void DragCaret::setCaretPosition(const PositionWithAffinity& position) {
-  // for querying Layer::compositingState()
-  // This code is probably correct, since it doesn't occur in a stack that
-  // involves updating compositing state.
-  DisableCompositingQueryAsserts disabler;
-
-  if (Node* node = m_position.anchorNode())
-    invalidateCaretRect(node, m_caretLocalRect);
   m_position = createVisiblePosition(position).toPositionWithAffinity();
   Document* document = nullptr;
   if (Node* node = m_position.anchorNode()) {
-    invalidateCaretRect(node, m_caretLocalRect);
     document = &node->document();
     setContext(document);
   }
-  if (m_position.isNull()) {
-    m_caretLocalRect = LayoutRect();
-  } else {
+  if (!m_position.isNull()) {
     DCHECK(!m_position.isOrphan());
     document->updateStyleAndLayoutTree();
-    m_caretLocalRect = CaretDisplayItemClient::computeCaretRect(m_position);
   }
 }
 
 void DragCaret::nodeChildrenWillBeRemoved(ContainerNode& container) {
   if (!hasCaret() || !container.inActiveDocument())
     return;
   Node* const anchorNode = m_position.position().anchorNode();
   if (!anchorNode || anchorNode == container)
     return;
   if (!container.isShadowIncludingInclusiveAncestorOf(anchorNode))
@@ skipping 12 matching lines @@
     return;
   m_position.document()->layoutViewItem().clearSelection();
   clear();
 }
 
 DEFINE_TRACE(DragCaret) {
   visitor->trace(m_position);
   SynchronousMutationObserver::trace(visitor);
 }
 
-void DragCaret::paintDragCaret(LocalFrame* frame,
+bool DragCaret::shouldPaintCaret(const LayoutBlock& block) const {
+  return m_caretBase->shouldPaintCaret(block);
+}
+
+void DragCaret::paintDragCaret(const LocalFrame* frame,
                                GraphicsContext& context,
                                const LayoutPoint& paintOffset) const {
-  if (m_position.anchorNode()->document().frame() == frame) {
-    m_caretBase->paintCaret(m_position.anchorNode(), context, m_caretLocalRect,
-                            paintOffset, DisplayItem::kDragCaret);
-  }
+  if (m_position.anchorNode()->document().frame() == frame)
+    m_caretBase->paintCaret(context, paintOffset, DisplayItem::kDragCaret);
 }
 
 }  // namespace blink