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

third_party/WebKit/Source/core/editing/commands/InsertLineBreakCommand.cpp

 /*
  * Copyright (C) 2005, 2006 Apple Computer, 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 53 matching lines @@
 
 void InsertLineBreakCommand::doApply(EditingState* editingState) {
   deleteSelection(editingState);
   if (editingState->isAborted())
     return;
   VisibleSelection selection = endingSelection();
   if (!selection.isNonOrphanedCaretOrRange())
     return;
 
   VisiblePosition caret(selection.visibleStartDeprecated());
-  // FIXME: If the node is hidden, we should still be able to insert text.
-  // For now, we return to avoid a crash.  https://bugs.webkit.org/show_bug.cgi?id=40342
+  // FIXME: If the node is hidden, we should still be able to insert text. For
+  // now, we return to avoid a crash.
+  // https://bugs.webkit.org/show_bug.cgi?id=40342
   if (caret.isNull())
     return;
 
   Position pos(caret.deepEquivalent());
 
   pos = positionAvoidingSpecialElementBoundary(pos, editingState);
   if (editingState->isAborted())
     return;
 
   pos = positionOutsideTabSpan(pos);
@@ skipping 46 matching lines @@
             VisiblePosition::beforeNode(nodeToInsert))) {
       insertNodeBefore(nodeToInsert->cloneNode(false), nodeToInsert,
                        editingState);
       if (editingState->isAborted())
         return;
     }
 
     setEndingSelection(createVisibleSelectionDeprecated(
         Position::inParentAfterNode(*nodeToInsert), TextAffinity::Downstream,
         endingSelection().isDirectional()));
-    // If we're inserting after all of the rendered text in a text node, or into a non-text node,
-    // a simple insertion is sufficient.
+    // If we're inserting after all of the rendered text in a text node, or into
+    // a non-text node, a simple insertion is sufficient.
   } else if (!pos.anchorNode()->isTextNode() ||
              pos.computeOffsetInContainerNode() >=
                  caretMaxOffset(pos.anchorNode())) {
     insertNodeAt(nodeToInsert, pos, editingState);
     if (editingState->isAborted())
       return;
     setEndingSelection(createVisibleSelectionDeprecated(
         Position::inParentAfterNode(*nodeToInsert), TextAffinity::Downstream,
         endingSelection().isDirectional()));
   } else if (pos.anchorNode()->isTextNode()) {
     // Split a text node
     Text* textNode = toText(pos.anchorNode());
     splitTextNode(textNode, pos.computeOffsetInContainerNode());
     insertNodeBefore(nodeToInsert, textNode, editingState);
     if (editingState->isAborted())
       return;
     Position endingPosition = Position::firstPositionInNode(textNode);
 
     // Handle whitespace that occurs after the split
     document().updateStyleAndLayoutIgnorePendingStylesheets();
     // TODO(yosin) |isRenderedCharacter()| should be removed, and we should
     // use |VisiblePosition::characterAfter()|.
     if (!isRenderedCharacter(endingPosition)) {
       Position positionBeforeTextNode(Position::inParentBeforeNode(*textNode));
       // Clear out all whitespace and insert one non-breaking space
       deleteInsignificantTextDownstream(endingPosition);
       DCHECK(!textNode->layoutObject() ||
              textNode->layoutObject()->style()->collapseWhiteSpace());
-      // Deleting insignificant whitespace will remove textNode if it contains nothing but insignificant whitespace.
+      // Deleting insignificant whitespace will remove textNode if it contains
+      // nothing but insignificant whitespace.
       if (textNode->isConnected()) {
         insertTextIntoNode(textNode, 0, nonBreakingSpaceString());
       } else {
         Text* nbspNode = document().createTextNode(nonBreakingSpaceString());
         insertNodeAt(nbspNode, positionBeforeTextNode, editingState);
         if (editingState->isAborted())
           return;
         endingPosition = Position::firstPositionInNode(nbspNode);
       }
     }
 
     setEndingSelection(createVisibleSelectionDeprecated(
         endingPosition, TextAffinity::Downstream,
         endingSelection().isDirectional()));
   }
 
   // Handle the case where there is a typing style.
 
   EditingStyle* typingStyle = document().frame()->selection().typingStyle();
 
   if (typingStyle && !typingStyle->isEmpty()) {
-    // Apply the typing style to the inserted line break, so that if the selection
-    // leaves and then comes back, new input will have the right style.
+    // Apply the typing style to the inserted line break, so that if the
+    // selection leaves and then comes back, new input will have the right
+    // style.
     // FIXME: We shouldn't always apply the typing style to the line break here,
     // see <rdar://problem/5794462>.
     applyStyle(typingStyle, firstPositionInOrBeforeNode(nodeToInsert),
                lastPositionInOrAfterNode(nodeToInsert), editingState);
     if (editingState->isAborted())
       return;
-    // Even though this applyStyle operates on a Range, it still sets an endingSelection().
-    // It tries to set a VisibleSelection around the content it operated on. So, that VisibleSelection
-    // will either (a) select the line break we inserted, or it will (b) be a caret just
-    // before the line break (if the line break is at the end of a block it isn't selectable).
-    // So, this next call sets the endingSelection() to a caret just after the line break
-    // that we inserted, or just before it if it's at the end of a block.
+    // Even though this applyStyle operates on a Range, it still sets an
+    // endingSelection(). It tries to set a VisibleSelection around the content
+    // it operated on. So, that VisibleSelection will either
+    //   (a) select the line break we inserted, or it will
+    //   (b) be a caret just before the line break (if the line break is at the
+    //       end of a block it isn't selectable).
+    // So, this next call sets the endingSelection() to a caret just after the
+    // line break that we inserted, or just before it if it's at the end of a
+    // block.
     setEndingSelection(endingSelection().visibleEndDeprecated());
   }
 
   rebalanceWhitespace();
 }
 
 }  // namespace blink