Invalidate the previous caret even if the parent text node is removed.

third_party/WebKit/Source/core/editing/FrameSelection.h

Index: third_party/WebKit/Source/core/editing/FrameSelection.h
diff --git a/third_party/WebKit/Source/core/editing/FrameSelection.h b/third_party/WebKit/Source/core/editing/FrameSelection.h
index 7ae465cda55a0e2d1333347929d9ce3b57d216f6..a6dcd683a9c31254696701ba01c539bedda23bf2 100644
--- a/third_party/WebKit/Source/core/editing/FrameSelection.h
+++ b/third_party/WebKit/Source/core/editing/FrameSelection.h
@@ -188,7 +188,6 @@ public:
     void scheduleVisualUpdate() const;
     void invalidateCaretRect();
     void paintCaret(GraphicsContext&, const LayoutPoint&);
-    bool ShouldPaintCaretForTesting() const { return m_shouldPaintCaret; }
 
     // Used to suspend caret blinking while the mouse is down.
     void setCaretBlinkingSuspended(bool suspended) { m_isCaretBlinkingSuspended = suspended; }
@@ -287,6 +286,10 @@ private:
 
     GranularityStrategy* granularityStrategy();
 
+    // For unittests
+    bool ShouldPaintCaretForTesting() const { return m_shouldPaintCaret; }

[jbroman, 2016/02/25 18:29:22]

nit: method names should begin with a lowercase letter in Blink.

[dshwang, 2016/02/25 20:02:49]

Done. I didn't touch existing method, but good idea to fix here.

+    bool isPreviousCaretDirtyForTesting() const { return m_previousCaretNode; }
+
     RawPtrWillBeMember<LocalFrame> m_frame;
     const OwnPtrWillBeMember<PendingSelection> m_pendingSelection;
     const OwnPtrWillBeMember<SelectionEditor> m_selectionEditor;