Editing: Indent command: Do not insert BLOCKQUOTE under the root HTML element.

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

 /*
  * Copyright (C) 2005, 2006, 2007, 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 1934 matching lines @@
   for (CompositeEditCommand* command = this; command;
        command = command->parent()) {
     if (UndoStep* undoStep = command->undoStep()) {
       DCHECK(command->isTopLevelCommand());
       undoStep->setEndingSelection(selection);
     }
     command->m_endingSelection = selection;
   }
 }
 
+void CompositeEditCommand::shrinkEndingSelectionToFitBody() {

[yosin_UTC9, 2017/03/17 10:04:11]

I would like this function as local function and functional-style, like:

EphemeralRange adjustRangeNotCrossingBody(const EphemeralRange& range, Element*
body) {
  const Position&  bodyStart = Positin::firstPositionInNode(body);
  const Position&  bodyEnd = Positin::lastPositionInNode(body);
  const Position& start = range.startPosition() < bodyStart ? bodyStart :
range.startPosition();
  const Position& end = range.endPositon() > bodyEnd ? bodyEnd :
range.endPostion();
  return EphemeralRange(start, end);
}

ApplyBlockElementCommand::doApply() {
  ...
  {
    const Position& start = endingSelection().visibleStart().deepEquivalent();
    const Position& end = endingSelection().visibleEnd().deepEquivalent();
    if (start.isConnected() && end.isConnected()) {
      const EphemeralRange range(start, end);
      const EphemeralRange adjustedRange = adjustRangeNotCrossingBody(range,
document().body());
      if (newRange != range) {
         setEndingSelection(SelectionInDOMTree::Builder()
            .setBaseAndExtent(newRange)
            .setIsDirecitonal(endingSelection().isDirectional())
            .build());
      }
    }
  }
...

+  Position start = endingSelection().visibleStart().deepEquivalent();
+  Position end = endingSelection().visibleEnd().deepEquivalent();
+  if (start.isNull() || start.isOrphan() || end.isNull() || end.isOrphan())
+    return;
+  HTMLElement* body = start.document()->body();
+  if (!body)
+    return;
+  Position bodyStart(body, PositionAnchorType::BeforeChildren);
+  Position bodyEnd(body, PositionAnchorType::AfterChildren);
+  bool needsToShrink = false;
+
+  if (start < bodyStart) {
+    start = bodyStart;
+    needsToShrink = true;
+  }
+  if (end < bodyStart) {
+    end = bodyStart;
+    needsToShrink = true;
+  }
+  if (bodyEnd < start) {
+    start = bodyEnd;
+    needsToShrink = true;
+  }
+  if (bodyEnd < end) {
+    end = bodyEnd;
+    needsToShrink = true;
+  }
+  if (!needsToShrink)
+    return;
+  SelectionInDOMTree::Builder builder;
+  builder.collapse(start);
+  builder.extend(end);
+  builder.setIsDirectional(endingSelection().isDirectional());
+  setEndingSelection(builder.build());
+}
+
 void CompositeEditCommand::setParent(CompositeEditCommand* parent) {
   EditCommand::setParent(parent);
   if (!parent)
     return;
   m_startingSelection = parent->m_endingSelection;
   m_endingSelection = parent->m_endingSelection;
 }
 
 DEFINE_TRACE(CompositeEditCommand) {
   visitor->trace(m_commands);
   visitor->trace(m_startingSelection);
   visitor->trace(m_endingSelection);
   visitor->trace(m_undoStep);
   EditCommand::trace(visitor);
 }
 
 }  // namespace blink