[InputEvent] Support |deleteByDrag|, |insertFromDrop| and fire in sequential order

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());
+}
+
+DocumentFragment* documentFragmentFromDragData(DragData* dragData, LocalFrame* frame, Range* context, bool allowPlainText, bool& chosePlainText)
+{
+    DCHECK(dragData);
+    chosePlainText = false;
+
+    Document& document = context->ownerDocument();
+    if (dragData->containsCompatibleContent()) {
+        if (DocumentFragment* fragment = dragData->asFragment(frame))
+            return fragment;
+
+        if (dragData->containsURL(DragData::DoNotConvertFilenames)) {
+            String title;
+            String url = dragData->asURL(DragData::DoNotConvertFilenames, &title);
+            if (!url.isEmpty()) {
+                HTMLAnchorElement* anchor = HTMLAnchorElement::create(document);
+                anchor->setHref(AtomicString(url));
+                if (title.isEmpty()) {
+                    // Try the plain text first because the url might be normalized or escaped.
+                    if (dragData->containsPlainText())
+                        title = dragData->asPlainText();
+                    if (title.isEmpty())
+                        title = url;
+                }
+                Node* anchorText = document.createTextNode(title);
+                anchor->appendChild(anchorText);
+                DocumentFragment* fragment = document.createDocumentFragment();
+                fragment->appendChild(anchor);
+                return fragment;
+            }
+        }
+    }
+    if (allowPlainText && dragData->containsPlainText()) {
+        chosePlainText = true;
+        return createFragmentFromText(EphemeralRange(context), dragData->asPlainText());
+    }
+
+    return nullptr;
+}
+
+bool doDeleteByDragWithEvents(Element* dragTarget, bool smartDelete, const Position& referenceMovePosition)

[yosin_UTC9, 2016/09/28 04:19:03]

Please add comment about |bool| return value.

[chongz, 2016/09/29 02:36:22]

Done.

+{

[yosin_UTC9, 2016/09/28 04:19:03]

This function should be in Editor class and avoid to use |bool| parameter.

[chongz, 2016/09/29 02:36:22]

Done.

+    if (!dragTarget || !dragTarget->isConnected())
+        return true;
+
+    LocalFrame* dragFrame = dragTarget->document().frame();
+
+    // Dispatch 'beforeinput'.
+    const bool shouldDelete = dispatchBeforeInputEditorCommand(dragTarget, InputEvent::InputType::DeleteByDrag, nullptr) == DispatchEventResult::NotCanceled;

[yosin_UTC9, 2016/09/28 04:19:03]

I think we should dispatch event to drag source rather than drag target since
deletion is occurred at drag source rather than drag target.

[chongz, 2016/09/29 02:36:22]

Sorry for the confusion, I've renamed it to |dragSource|.

The naming story is I have 2 event targets |dragTarget| and |dropTarget|, where
|dragTarget| is essentially the event target of the "Drag" operation (i.g.
Source of Drag&Drop).

+
+    // 'beforeinput' event handler may destroy frame, return false to cancel remaining actions;
+    if (dragFrame != dragTarget->document().frame())
+        return false;
+
+    if (shouldDelete && dragTarget->isConnected())
+        dragFrame->editor().deleteSelectionWithSmartDelete(smartDelete, InputEvent::InputType::DeleteByDrag, referenceMovePosition);
+
+    return true;
+}
+
+bool doInsertFromDropWithEvents(Element* dropTarget, DragData* dragData, DocumentFragment* fragment, Range* dropCaretRange, bool smartInsert, bool chosePlainText)

[yosin_UTC9, 2016/09/28 04:19:03]

Please add comment about |bool| return value.

[chongz, 2016/09/29 02:36:22]

Done.

+{

[yosin_UTC9, 2016/09/28 04:19:03]

This function should be in Editor class and avoid to use |bool| parameter.

[chongz, 2016/09/29 02:36:22]

Done.

+    if (!dropTarget || !dropTarget->isConnected())
+        return true;
+
+    LocalFrame* dropFrame = dropTarget->document().frame();
+
+    // Dispatch 'beforeinput'.
+    DataTransfer* dataTransfer = createDraggingDataTransfer(DataTransferReadable, dragData);
+    dataTransfer->setSourceOperation(dragData->draggingSourceOperationMask());
+    const bool shouldInsert = dispatchBeforeInputDataTransfer(dropTarget, InputEvent::InputType::InsertFromDrop, dataTransfer, nullptr) == DispatchEventResult::NotCanceled;
+
+    // 'beforeinput' event handler may destroy frame, return false to cancel remaining actions;
+    if (dropFrame != dropTarget->document().frame())
+        return false;
+
+    if (shouldInsert && dropTarget->isConnected())
+        dropFrame->editor().replaceSelectionAfterDragging(fragment, smartInsert, chosePlainText);
+
+    return true;
+}
+
+} // 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)
 {
 }
 
 DragController* DragController::create(Page* page)
 {
     return new DragController(page);
 }
 
-static DocumentFragment* documentFragmentFromDragData(DragData* dragData, LocalFrame* frame, Range* context, bool allowPlainText, bool& chosePlainText)
-{
-    ASSERT(dragData);
-    chosePlainText = false;
-
-    Document& document = context->ownerDocument();
-    if (dragData->containsCompatibleContent()) {
-        if (DocumentFragment* fragment = dragData->asFragment(frame))
-            return fragment;
-
-        if (dragData->containsURL(DragData::DoNotConvertFilenames)) {
-            String title;
-            String url = dragData->asURL(DragData::DoNotConvertFilenames, &title);
-            if (!url.isEmpty()) {
-                HTMLAnchorElement* anchor = HTMLAnchorElement::create(document);
-                anchor->setHref(AtomicString(url));
-                if (title.isEmpty()) {
-                    // Try the plain text first because the url might be normalized or escaped.
-                    if (dragData->containsPlainText())
-                        title = dragData->asPlainText();
-                    if (title.isEmpty())
-                        title = url;
-                }
-                Node* anchorText = document.createTextNode(title);
-                anchor->appendChild(anchorText);
-                DocumentFragment* fragment = document.createDocumentFragment();
-                fragment->appendChild(anchor);
-                return fragment;
-            }
-        }
-    }
-    if (allowPlainText && dragData->containsPlainText()) {
-        chosePlainText = true;
-        return createFragmentFromText(EphemeralRange(context), dragData->asPlainText());
-    }
-
-    return nullptr;
-}
-
 bool DragController::dragIsMove(FrameSelection& selection, DragData* dragData)
 {
     return m_documentUnderMouse == m_dragInitiator && selection.isContentEditable() && selection.isRange() && !isCopyKeyDown(dragData);
 }
 
 // FIXME: This method is poorly named.  We're just clearing the selection from the document this drag is exiting.
 void DragController::cancelDrag()
 {
     m_page->dragCaretController().clear();
 }
@@ skipping 309 matching lines @@
     ResourceCacheValidationSuppressor validationSuppressor(fetcher);
     if (dragIsMove(innerFrame->selection(), dragData) || dragCaret.isContentRichlyEditable()) {
         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);
+            const bool smartDelete = innerFrame->editor().smartInsertDeleteEnabled();
+            const bool smartInsert = smartDelete && innerFrame->selection().granularity() == WordGranularity && dragData->canSmartReplace();
+
+            if (!doDeleteByDragWithEvents(innerFrame->editor().findEventTargetFromSelection(), smartDelete, dragCaret.base()))
+                return false;
+
+            innerFrame->selection().setSelection(VisibleSelection(range->startPosition(), range->endPosition()));
+            if (innerFrame->selection().isAvailable()) {
+                DCHECK(m_documentUnderMouse);
+                if (!doInsertFromDropWithEvents(element, dragData, fragment, range, smartInsert, chosePlainText))
+                    return false;
+            }
         } else {
             if (setSelectionToDragCaret(innerFrame, dragCaret, range, point)) {
-                ASSERT(m_documentUnderMouse);
-                m_documentUnderMouse->frame()->editor().replaceSelectionAfterDragging(fragment, dragData->canSmartReplace(), chosePlainText);
+                DCHECK(m_documentUnderMouse);
+                if (!doInsertFromDropWithEvents(element, dragData, fragment, range, dragData->canSmartReplace(), chosePlainText))
+                    return false;
             }
         }
     } 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);
+            DCHECK(m_documentUnderMouse);
+            if (!doInsertFromDropWithEvents(element, dragData, createFragmentFromText(EphemeralRange(range), text), range, canSmartReplace, chosePlainText))
+                return false;
         }
     }
 
     if (rootEditableElement) {
         if (LocalFrame* frame = rootEditableElement->document().frame())
             frame->eventHandler().updateDragStateAfterEditDragIfNeeded(rootEditableElement);
     }
 
     return true;
 }
@@ skipping 452 matching lines @@
 
 DEFINE_TRACE(DragController)
 {
     visitor->trace(m_page);
     visitor->trace(m_documentUnderMouse);
     visitor->trace(m_dragInitiator);
     visitor->trace(m_fileInputElementUnderMouse);
 }
 
 } // namespace blink