Reflow comments in //third_party/WebKit/Source/core/editing

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

 /*
  * Copyright (C) 2004, 2005, 2006, 2009 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 157 matching lines @@
 }
 
 // Neighbor-anchored positions are invalid DOM positions, so they need to be
 // fixed up before handing them off to the Range object.
 template <typename Strategy>
 PositionTemplate<Strategy>
 PositionTemplate<Strategy>::parentAnchoredEquivalent() const {
   if (!m_anchorNode)
     return PositionTemplate<Strategy>();
 
-  // FIXME: This should only be necessary for legacy positions, but is also needed for positions before and after Tables
+  // FIXME: This should only be necessary for legacy positions, but is also
+  // needed for positions before and after Tables
   if (m_offset == 0 && !isAfterAnchorOrAfterChildren()) {
     if (Strategy::parent(*m_anchorNode) &&
         (Strategy::editingIgnoresContent(m_anchorNode.get()) ||
          isDisplayInsideTable(m_anchorNode.get())))
       return inParentBeforeNode(*m_anchorNode);
     return PositionTemplate<Strategy>(m_anchorNode.get(), 0);
   }
   if (!m_anchorNode->isCharacterDataNode() &&
       (isAfterAnchorOrAfterChildren() ||
        static_cast<unsigned>(m_offset) == m_anchorNode->countChildren()) &&
@@ skipping 152 matching lines @@
 template <typename Strategy>
 bool PositionTemplate<Strategy>::operator>=(
     const PositionTemplate<Strategy>& other) const {
   return comparePositions(*this, other) >= 0;
 }
 
 template <typename Strategy>
 bool PositionTemplate<Strategy>::atFirstEditingPositionForNode() const {
   if (isNull())
     return true;
-  // FIXME: Position before anchor shouldn't be considered as at the first editing position for node
-  // since that position resides outside of the node.
+  // FIXME: Position before anchor shouldn't be considered as at the first
+  // editing position for node since that position resides outside of the node.
   switch (m_anchorType) {
     case PositionAnchorType::OffsetInAnchor:
       return m_offset == 0;
     case PositionAnchorType::BeforeChildren:
     case PositionAnchorType::BeforeAnchor:
       return true;
     case PositionAnchorType::AfterChildren:
     case PositionAnchorType::AfterAnchor:
       // TODO(yosin) We should use |Strategy::lastOffsetForEditing()| instead
       // of DOM tree version.
@@ skipping 30 matching lines @@
   // TODO(yosin) We should use |Strategy::lastOffsetForEditing()| instead of
   // DOM tree version.
   return !Strategy::parent(*anchorNode()) &&
          m_offset >= EditingStrategy::lastOffsetForEditing(anchorNode());
 }
 
 // static
 template <typename Strategy>
 PositionTemplate<Strategy> PositionTemplate<Strategy>::inParentBeforeNode(
     const Node& node) {
-  // 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 DCHECK(node->parentNode())
+  // FIXME: This should DCHECK(node.parentNode()). At least one caller currently
+  // hits this DCHECK 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
+  // DCHECK(node->parentNode())
   return PositionTemplate<Strategy>(Strategy::parent(node),
                                     Strategy::index(node));
 }
 
 // static
 template <typename Strategy>
 PositionTemplate<Strategy> PositionTemplate<Strategy>::inParentAfterNode(
     const Node& node) {
   DCHECK(node.parentNode()) << node;
   return PositionTemplate<Strategy>(Strategy::parent(node),
@@ skipping 253 matching lines @@
 }
 
 void showTree(const blink::Position* pos) {
   if (pos)
     pos->showTreeForThis();
   else
     LOG(INFO) << "Cannot showTree for <null>";
 }
 
 #endif