[InputEvent] Move 'beforeinput' logic into |CompositeEditCommand::willApplyEditing()| (3/3)

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

 /*
  * Copyright (C) 2004, 2005, 2006, 2007 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 23 matching lines @@
 #include "core/dom/Range.h"
 #include "core/dom/Text.h"
 #include "core/dom/shadow/ShadowRoot.h"
 #include "core/editing/EditingStrategy.h"
 #include "core/editing/Editor.h"
 #include "core/editing/PlainTextRange.h"
 #include "core/editing/PositionIterator.h"
 #include "core/editing/VisiblePosition.h"
 #include "core/editing/VisibleSelection.h"
 #include "core/editing/VisibleUnits.h"
+#include "core/editing/commands/CompositeEditCommand.h"
+#include "core/editing/commands/TypingCommand.h"
 #include "core/editing/iterators/TextIterator.h"
 #include "core/editing/serializers/HTMLInterchange.h"
 #include "core/editing/state_machines/BackspaceStateMachine.h"
 #include "core/editing/state_machines/BackwardGraphemeBoundaryStateMachine.h"
 #include "core/editing/state_machines/ForwardGraphemeBoundaryStateMachine.h"
 #include "core/frame/FrameView.h"
 #include "core/frame/LocalFrame.h"
 #include "core/frame/UseCounter.h"
 #include "core/html/HTMLBRElement.h"
 #include "core/html/HTMLDivElement.h"
 #include "core/html/HTMLLIElement.h"
 #include "core/html/HTMLParagraphElement.h"
 #include "core/html/HTMLSpanElement.h"
 #include "core/html/HTMLTableCellElement.h"
 #include "core/html/HTMLUListElement.h"
 #include "core/layout/LayoutObject.h"
 #include "core/layout/LayoutTableCell.h"
-#include "platform/clipboard/ClipboardMimeTypes.h"
 #include "wtf/Assertions.h"
 #include "wtf/StdLibExtras.h"
 #include "wtf/text/StringBuilder.h"
 #include "wtf/text/Unicode.h"
 
 namespace blink {
 
 using namespace HTMLNames;
 
 namespace {
@@ skipping 1968 matching lines @@
   // otherwise, make sure to be at the start of the first selected node,
   // instead of possibly at the end of the last node before the selection
   return mostForwardCaretPosition(visiblePosition.deepEquivalent());
 }
 
 bool isTextSecurityNode(const Node* node) {
   return node && node->layoutObject() &&
          node->layoutObject()->style()->textSecurity() != TSNONE;
 }
 
-DispatchEventResult dispatchBeforeInputInsertText(EventTarget* target,

[chongz, 2016/12/20 23:27:51]

See header file.

-                                                  const String& data) {
-  if (!RuntimeEnabledFeatures::inputEventEnabled())
-    return DispatchEventResult::NotCanceled;
-  if (!target)
-    return DispatchEventResult::NotCanceled;
-  // TODO(chongz): Pass appropriate |ranges| after it's defined on spec.
-  // http://w3c.github.io/editing/input-events.html#dom-inputevent-inputtype
-  InputEvent* beforeInputEvent = InputEvent::createBeforeInput(
-      InputEvent::InputType::InsertText, data,
-      InputEvent::EventCancelable::IsCancelable,
-      InputEvent::EventIsComposing::NotComposing, nullptr);
-  return target->dispatchEvent(beforeInputEvent);
-}
-
-DispatchEventResult dispatchBeforeInputFromComposition(
-    EventTarget* target,
-    InputEvent::InputType inputType,
-    const String& data,
-    InputEvent::EventCancelable cancelable) {
-  if (!RuntimeEnabledFeatures::inputEventEnabled())
-    return DispatchEventResult::NotCanceled;
-  if (!target)
-    return DispatchEventResult::NotCanceled;
-  // TODO(chongz): Pass appropriate |ranges| after it's defined on spec.
-  // http://w3c.github.io/editing/input-events.html#dom-inputevent-inputtype
-  InputEvent* beforeInputEvent = InputEvent::createBeforeInput(
-      inputType, data, cancelable, InputEvent::EventIsComposing::IsComposing,
-      nullptr);
-  return target->dispatchEvent(beforeInputEvent);
-}
-
-DispatchEventResult dispatchBeforeInputEditorCommand(
-    EventTarget* target,
-    InputEvent::InputType inputType,
-    const RangeVector* ranges) {
-  if (!RuntimeEnabledFeatures::inputEventEnabled())
-    return DispatchEventResult::NotCanceled;
-  if (!target)
-    return DispatchEventResult::NotCanceled;
-  InputEvent* beforeInputEvent = InputEvent::createBeforeInput(
-      inputType, nullAtom, InputEvent::EventCancelable::IsCancelable,
-      InputEvent::EventIsComposing::NotComposing, ranges);
-  return target->dispatchEvent(beforeInputEvent);
-}
-
-DispatchEventResult dispatchBeforeInputDataTransfer(
-    EventTarget* target,
-    InputEvent::InputType inputType,
-    DataTransfer* dataTransfer,
-    const RangeVector* ranges) {
-  if (!RuntimeEnabledFeatures::inputEventEnabled())
-    return DispatchEventResult::NotCanceled;
-  if (!target)
-    return DispatchEventResult::NotCanceled;
-
-  DCHECK(inputType == InputEvent::InputType::InsertFromPaste ||
-         inputType == InputEvent::InputType::InsertReplacementText ||
-         inputType == InputEvent::InputType::InsertFromDrop ||
-         inputType == InputEvent::InputType::DeleteByCut)
-      << "Unsupported inputType: " << (int)inputType;
-
-  InputEvent* beforeInputEvent;
-
-  if (hasRichlyEditableStyle(*(target->toNode())) || !dataTransfer) {

[chongz, 2016/12/21 23:59:24]

This logic for data vs. dataTransfer are only used by paste, drop and
spellchecker, so it makes more sense to be moved to
|ReplaceSelectionCommand::textDataForInputEvent()| and
|ReplaceSelectionCommand::dataTransferForInputEvent()|.

-    beforeInputEvent = InputEvent::createBeforeInput(
-        inputType, dataTransfer, InputEvent::EventCancelable::IsCancelable,
-        InputEvent::EventIsComposing::NotComposing, ranges);
-  } else {
-    const String& data = dataTransfer->getData(mimeTypeTextPlain);
-    // TODO(chongz): Pass appropriate |ranges| after it's defined on spec.
-    // http://w3c.github.io/editing/input-events.html#dom-inputevent-inputtype
-    beforeInputEvent = InputEvent::createBeforeInput(
-        inputType, data, InputEvent::EventCancelable::IsCancelable,
-        InputEvent::EventIsComposing::NotComposing, nullptr);
-  }
-  return target->dispatchEvent(beforeInputEvent);
-}
-
 InputEvent::InputType deletionInputTypeFromTextGranularity(
     DeleteDirection direction,
     TextGranularity granularity) {
   using InputType = InputEvent::InputType;
   switch (direction) {
     case DeleteDirection::Forward:
       if (granularity == WordGranularity)
         return InputType::DeleteWordForward;
       if (granularity == LineBoundary)
         return InputType::DeleteLineForward;
       return InputType::DeleteContentForward;
     case DeleteDirection::Backward:
       if (granularity == WordGranularity)
         return InputType::DeleteWordBackward;
       if (granularity == LineBoundary)
         return InputType::DeleteLineBackward;
       return InputType::DeleteContentBackward;
     default:
       return InputType::None;
   }
 }
 
+InputEvent::EventIsComposing isComposingFromCommand(

[chongz, 2016/12/20 23:27:51]

See header file.

+    const CompositeEditCommand* command) {
+  if (command->isTypingCommand() &&
+      toTypingCommand(command)->compositionType() !=
+          TypingCommand::TextCompositionNone)
+    return InputEvent::IsComposing;
+  return InputEvent::NotComposing;
+}
+
+InputEvent::EventCancelable isCancelableFromCommand(
+    const CompositeEditCommand* command) {
+  if (command->isTypingCommand() &&
+      toTypingCommand(command)->compositionType() ==
+          TypingCommand::TextCompositionUpdate)
+    return InputEvent::NotCancelable;
+  return InputEvent::IsCancelable;
+}
+
+bool dispatchBeforeInputEvent(EditCommandSource source,
+                              LocalFrame* frame,
+                              Node* target,
+                              InputEvent::InputType inputType,
+                              const String& data,
+                              DataTransfer* dataTransfer,
+                              InputEvent::EventCancelable isCancelable,
+                              InputEvent::EventIsComposing isComposing,
+                              const RangeVector* ranges) {
+  // Don't fire 'beforeinput', return true to continue.
+  if (source != EditCommandSource::kMenuOrKeyBinding ||
+      inputType == InputEvent::InputType::None)
+    return true;
+  if (!RuntimeEnabledFeatures::inputEventEnabled())
+    return true;
+  if (!frame || !target || !target->isConnected())
+    return true;
+
+  if (!hasRichlyEditableStyle(*target)) {
+    // Plain-text only elements (<input>, <textarea>, etc.) don't support
+    // DataTransfer or Range.
+    dataTransfer = nullptr;
+    ranges = nullptr;
+  }
+
+  InputEvent* beforeInputEvent;
+
+  if (dataTransfer) {
+    beforeInputEvent = InputEvent::createBeforeInput(
+        inputType, dataTransfer, isCancelable, isComposing, ranges);
+  } else {
+    beforeInputEvent = InputEvent::createBeforeInput(
+        inputType, data, isCancelable, isComposing, ranges);
+  }
+
+  DispatchEventResult result = target->dispatchEvent(beforeInputEvent);
+  // 'beforeinput' event handler may destroy target frame.
+  if (frame->document()->frame() != frame)

[Xiaocheng, 2016/12/21 02:56:38]

Is |frame->document()| still non-null after the event?

Is it safer to compare |frame->document()| with the original document, like what
SpellChecker currently does?

[chongz, 2016/12/21 23:59:24]

Added check for |frame->document()|, but according to my tests it's always
non-null.
Also moved |isConnected()| check here.
Yeah yosin@ suggested this before, but according to my tests |frame->document()|
won't change, see discussion:
https://codereview.chromium.org/2258663003/diff/120001/third_party/WebKit/Sou...

Also see "fast/events/inputevents/beforeinput-remove-iframe-crash.html" for test
cases, where SpellChecker was not tested and I suspect it will fail.

[Xiaocheng, 2016/12/22 06:26:20]

Thanks for the clarification.

I thought |frame->document()->frame == frame| should be always true... I may
need to look into that later.

+    return false;
+  return result == DispatchEventResult::NotCanceled;
+}
+
 }  // namespace blink