ASSERT -> DCHECK in core/editing.

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

 /*
  * Copyright (C) 2004, 2006, 2008 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
@@ skipping 82 matching lines @@
 
     // Returns |PositionIsAnchor| type |Position| which is compatible with
     // |RangeBoundaryPoint| as safe to pass |Range| constructor. Return value
     // of this function is different from |parentAnchoredEquivalent()| which
     // returns editing specific position.
     PositionTemplate<Strategy> toOffsetInAnchor() const;
 
     // Inline O(1) access for Positions which callers know to be parent-anchored
     int offsetInContainerNode() const
     {
-        ASSERT(isOffsetInAnchor());
+        DCHECK(isOffsetInAnchor());
         return m_offset;
     }
 
     // Returns an offset for editing based on anchor type for using with
     // |anchorNode()| function:
     //   - OffsetInAnchor  m_offset
     //   - BeforeChildren  0
     //   - BeforeAnchor    0
     //   - AfterChildren   last editing offset in anchor node
     //   - AfterAnchor     last editing offset in anchor node
@@ skipping 111 matching lines @@
     return !(a == b);
 }
 
 // We define position creation functions to make callsites more readable.
 // These are inline to prevent ref-churn when returning a Position object.
 // If we ever add a PassPosition we can make these non-inline.
 
 template <typename Strategy>
 PositionTemplate<Strategy> PositionTemplate<Strategy>::inParentBeforeNode(const Node& node)
 {
-    // FIXME: This should ASSERT(node.parentNode())
+    // FIXME: This should DCHECK(node.parentNode())
     // At least one caller currently hits this ASSERT though, which indicates
     // that the caller is trying to make a position relative to a disconnected node (which is likely an error)
-    // Specifically, editing/deleting/delete-ligature-001.html crashes with ASSERT(node->parentNode())
+    // Specifically, editing/deleting/delete-ligature-001.html crashes with DCHECK(node->parentNode())
     return PositionTemplate<Strategy>(Strategy::parent(node), Strategy::index(node));
 }
 
 inline Position positionInParentBeforeNode(const Node& node)
 {
     return Position::inParentBeforeNode(node);
 }
 
 template <typename Strategy>
 PositionTemplate<Strategy> PositionTemplate<Strategy>::inParentAfterNode(const Node& node)
 {
-    ASSERT(node.parentNode());
+    DCHECK(node.parentNode());

[yosin_UTC9, 2016/04/14 04:35:00]

How about adding |<< node|?

     return PositionTemplate<Strategy>(Strategy::parent(node), Strategy::index(node) + 1);
 }
 
 inline Position positionInParentAfterNode(const Node& node)
 {
     return Position::inParentAfterNode(node);
 }
 
 // positionBeforeNode and positionAfterNode return neighbor-anchored positions, construction is O(1)
 template <typename Strategy>
 PositionTemplate<Strategy> PositionTemplate<Strategy>::beforeNode(Node* anchorNode)
 {
-    ASSERT(anchorNode);
+    DCHECK(anchorNode);
     return PositionTemplate<Strategy>(anchorNode, PositionAnchorType::BeforeAnchor);
 }
 
 inline Position positionBeforeNode(Node* anchorNode)
 {
     return Position::beforeNode(anchorNode);
 }
 
 template <typename Strategy>
 PositionTemplate<Strategy> PositionTemplate<Strategy>::afterNode(Node* anchorNode)
 {
-    ASSERT(anchorNode);
+    DCHECK(anchorNode);
     return PositionTemplate<Strategy>(anchorNode, PositionAnchorType::AfterAnchor);
 }
 
 inline Position positionAfterNode(Node* anchorNode)
 {
     return Position::afterNode(anchorNode);
 }
 
 template <typename Strategy>
 int PositionTemplate<Strategy>::lastOffsetInNode(Node* node)
@@ skipping 110 matching lines @@
 
 } // namespace blink
 
 #ifndef NDEBUG
 // Outside the WebCore namespace for ease of invocation from gdb.
 void showTree(const blink::Position&);
 void showTree(const blink::Position*);
 #endif
 
 #endif // Position_h