[InputEvent] Support |deleteByDrag| and |insertFromDrop|

third_party/WebKit/Source/core/page/DragController.cpp

 /*
  * Copyright (C) 2007, 2009, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Google Inc.
  *
  * 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
@@ skipping 74 matching lines @@
 #include "wtf/CurrentTime.h"
 #include "wtf/RefPtr.h"
 #include <memory>
 
 #if OS(WIN)
 #include <windows.h>
 #endif
 
 namespace blink {
 
+namespace {
+
+DataTransfer* createDraggingDataTransfer(DataTransferAccessPolicy policy, DragData* dragData)
+{
+    return DataTransfer::create(DataTransfer::DragAndDrop, policy, dragData->platformData());
+}
+
+enum class DragDropBeforeInputResult {
+    None,
+    DeleteOnly,
+    InsertOnly,
+    DeleteAndInsert,
+};
+
+DragDropBeforeInputResult dispatchDragDropBeforeInput(Element* dragTarget, Element* dropTarget, DragData* dragData, bool dragIsMove)
+{
+    LocalFrame* dragFrame = dragTarget->document().frame();
+    LocalFrame* dropFrame = dropTarget->document().frame();
+
+    // Handle DeleteByDrag.
+    bool shouldDelete = dragIsMove;
+    if (shouldDelete) {
+        DispatchEventResult dragBeforeInputResult = dispatchBeforeInputEditorCommand(dragTarget, InputEvent::InputType::DeleteByDrag, emptyString(), nullptr);
+        shouldDelete = (dragBeforeInputResult == DispatchEventResult::NotCanceled);

[yosin_UTC9, 2016/08/31 01:20:34]

nit: We don't need to have parenthesis.
Edit

[chongz, 2016/09/02 18:02:10]

Done.

+    }
+
+    // 'beforeinput' event handler may destroy frame.
+    if (dragFrame != dragTarget->document().frame() || dropFrame != dropTarget->document().frame())

[yosin_UTC9, 2016/08/31 01:20:34]

What is happened when |dropTarget| is removed?

[chongz, 2016/09/02 18:02:10]

Added check for removed element. So if |dropTarget| is removed:
  1. We still dispatch 'beforeinput' to |dragTarget| and |dropTarget|
  2. Only apply |DeleteSelectionCommand| to |dragTarget|
  3. Only fire 'input' for |dragTarget|

I think it's OK to dispatch 'beforeinput' to detached elements, but since
editing commands won't modify removed elements we shouldn't dispatch 'input' for
them.

Which means removing |dropTarget| behaves similar to cancelling
|insertFromDrop|, and this won't affect existing behavior for 'drop' and
'dragend' events.

+        return DragDropBeforeInputResult::None;
+
+    // Handle InsertFromDrop.
+    DataTransfer* dataTransfer = createDraggingDataTransfer(DataTransferReadable, dragData);
+    dataTransfer->setSourceOperation(dragData->draggingSourceOperationMask());
+    DispatchEventResult dropBeforeInputResult = dispatchBeforeInputDataTransfer(dropTarget, InputEvent::InputType::InsertFromDrop, dataTransfer, nullptr);
+    bool shouldInsert = (dropBeforeInputResult == DispatchEventResult::NotCanceled);

[yosin_UTC9, 2016/08/31 01:20:34]

nit: We don't need to have parenthesis.

[chongz, 2016/09/02 18:02:10]

Done.

+
+    // 'beforeinput' event handler may destroy frame.
+    if (dragFrame != dragTarget->document().frame() || dropFrame != dropTarget->document().frame())
+        return DragDropBeforeInputResult::None;
+
+    // Generate dispatch result.
+    if (shouldDelete && shouldInsert)
+        return DragDropBeforeInputResult::DeleteAndInsert;
+    if (shouldDelete)
+        return DragDropBeforeInputResult::DeleteOnly;
+    if (shouldInsert)
+        return DragDropBeforeInputResult::InsertOnly;
+    return DragDropBeforeInputResult::None;
+}
+
+} // anonymous namespace
+
 const int DragController::DragIconRightInset = 7;
 const int DragController::DragIconBottomInset = 3;
 
 static const int MaxOriginalImageArea = 1500 * 1500;
 static const int LinkDragBorderInset = 2;
 static const float DragImageAlpha = 0.75f;
 
 #if ENABLE(ASSERT)
 static bool dragTypeIsValid(DragSourceAction action)
 {
@@ skipping 12 matching lines @@
 #endif
 
 static PlatformMouseEvent createMouseEvent(DragData* dragData)
 {
     return PlatformMouseEvent(dragData->clientPosition(), dragData->globalPosition(),
         WebPointerProperties::Button::Left, PlatformEvent::MouseMoved, 0,
         static_cast<PlatformEvent::Modifiers>(dragData->modifiers()),
         PlatformMouseEvent::RealOrIndistinguishable, monotonicallyIncreasingTime());
 }
 
-static DataTransfer* createDraggingDataTransfer(DataTransferAccessPolicy policy, DragData* dragData)
-{
-    return DataTransfer::create(DataTransfer::DragAndDrop, policy, dragData->platformData());
-}
-
 DragController::DragController(Page* page)
     : m_page(page)
     , m_documentUnderMouse(nullptr)
     , m_dragInitiator(nullptr)
     , m_fileInputElementUnderMouse(nullptr)
     , m_documentIsHandlingDrag(false)
     , m_dragDestinationAction(DragDestinationActionNone)
     , m_didInitiateDrag(false)
 {
 }
@@ skipping 354 matching lines @@
         return false;
     }
     Range* range = createRange(dragCaret.toNormalizedEphemeralRange());
     Element* rootEditableElement = innerFrame->selection().rootEditableElement();
 
     // For range to be null a WebKit client must have done something bad while
     // manually controlling drag behaviour
     if (!range)
         return false;
     ResourceFetcher* fetcher = range->ownerDocument().fetcher();
-    ResourceCacheValidationSuppressor validationSuppressor(fetcher);
-    if (dragIsMove(innerFrame->selection(), dragData) || dragCaret.isContentRichlyEditable()) {
+    ResourceCacheValidationSuppressor validationSuppressorr(fetcher);
+
+    DragDropBeforeInputResult beforeInputResult = dispatchDragDropBeforeInput(innerFrame->editor().findEventTargetFromSelection(), element, dragData, dragIsMove(innerFrame->selection(), dragData));
+
+    switch (beforeInputResult) {
+    case DragDropBeforeInputResult::None:
+    default:
+        return true;
+
+    case DragDropBeforeInputResult::DeleteOnly:
+        innerFrame->editor().deleteSelectionWithSmartDelete(innerFrame->editor().smartInsertDeleteEnabled(), InputEvent::InputType::DeleteByDrag);
+        break;
+
+    case DragDropBeforeInputResult::InsertOnly:
+        if (dragCaret.isContentRichlyEditable()) {
+            bool chosePlainText = false;
+            DocumentFragment* fragment = documentFragmentFromDragData(dragData, innerFrame, range, true, chosePlainText);
+            if (!fragment)
+                return false;
+            if (setSelectionToDragCaret(innerFrame, dragCaret, range, point)) {
+                DCHECK(m_documentUnderMouse);
+                m_documentUnderMouse->frame()->editor().replaceSelectionAfterDragging(fragment, dragData->canSmartReplace(), chosePlainText);
+            }
+        } else {
+            String text = dragData->asPlainText();
+            if (text.isEmpty())
+                return false;
+
+            if (setSelectionToDragCaret(innerFrame, dragCaret, range, point)) {
+                const bool canSmartReplace = false;
+                const bool chosePlainText = true;
+                DCHECK(m_documentUnderMouse);
+                m_documentUnderMouse->frame()->editor().replaceSelectionAfterDragging(createFragmentFromText(EphemeralRange(range), text), canSmartReplace, chosePlainText);
+            }
+        }
+        break;
+
+    case DragDropBeforeInputResult::DeleteAndInsert:
         bool chosePlainText = false;
         DocumentFragment* fragment = documentFragmentFromDragData(dragData, innerFrame, range, true, chosePlainText);
         if (!fragment)
             return false;
-
-        if (dragIsMove(innerFrame->selection(), dragData)) {
-            // NSTextView behavior is to always smart delete on moving a selection,
-            // but only to smart insert if the selection granularity is word granularity.
-            bool smartDelete = innerFrame->editor().smartInsertDeleteEnabled();
-            bool smartInsert = smartDelete && innerFrame->selection().granularity() == WordGranularity && dragData->canSmartReplace();
-            innerFrame->editor().moveSelectionAfterDragging(fragment, dragCaret.base(), smartInsert, smartDelete);
-        } else {
-            if (setSelectionToDragCaret(innerFrame, dragCaret, range, point)) {
-                ASSERT(m_documentUnderMouse);
-                m_documentUnderMouse->frame()->editor().replaceSelectionAfterDragging(fragment, dragData->canSmartReplace(), chosePlainText);
-            }
-        }
-    } else {
-        String text = dragData->asPlainText();
-        if (text.isEmpty())
-            return false;
-
-        if (setSelectionToDragCaret(innerFrame, dragCaret, range, point)) {
-            const bool canSmartReplace = false;
-            const bool chosePlainText = true;
-            ASSERT(m_documentUnderMouse);
-            m_documentUnderMouse->frame()->editor().replaceSelectionAfterDragging(createFragmentFromText(EphemeralRange(range), text), canSmartReplace, chosePlainText);
-        }
+        // NSTextView behavior is to always smart delete on moving a selection,
+        // but only to smart insert if the selection granularity is word granularity.
+        bool smartDelete = innerFrame->editor().smartInsertDeleteEnabled();
+        bool smartInsert = smartDelete && innerFrame->selection().granularity() == WordGranularity && dragData->canSmartReplace();
+        innerFrame->editor().moveSelectionAfterDragging(fragment, dragCaret.base(), smartInsert, smartDelete);
+        break;
     }
 
     if (rootEditableElement) {
         if (LocalFrame* frame = rootEditableElement->document().frame())
             frame->eventHandler().updateDragStateAfterEditDragIfNeeded(rootEditableElement);
     }
 
     return true;
 }
 
@@ skipping 451 matching lines @@
 
 DEFINE_TRACE(DragController)
 {
     visitor->trace(m_page);
     visitor->trace(m_documentUnderMouse);
     visitor->trace(m_dragInitiator);
     visitor->trace(m_fileInputElementUnderMouse);
 }
 
 } // namespace blink