Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1578)

Unified Diff: third_party/WebKit/Source/core/input/MouseEventManager.cpp

Issue 2536083002: SelectionAutoscroll should not happen in user-select:none. (Closed)
Patch Set: To pass the tests. Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/input/MouseEventManager.cpp
diff --git a/third_party/WebKit/Source/core/input/MouseEventManager.cpp b/third_party/WebKit/Source/core/input/MouseEventManager.cpp
index c38ab2fcb8db66b6adc03875a6e1446fdffb617a..4d9a743195e9de7df2541a700887aa4f02a771f7 100644
--- a/third_party/WebKit/Source/core/input/MouseEventManager.cpp
+++ b/third_party/WebKit/Source/core/input/MouseEventManager.cpp
@@ -732,8 +732,13 @@ WebInputEventResult MouseEventManager::handleMouseDraggedEvent(
m_mouseDownMayStartDrag = false;
+ m_frame->eventHandler().selectionController().handleMouseDraggedEvent(
+ event, m_mouseDownPos, m_dragStartPos, m_mousePressNode.get(),
+ m_lastKnownMousePosition);
+
if (m_mouseDownMayStartAutoscroll &&
- !m_scrollManager->middleClickAutoscrollInProgress()) {
+ !m_scrollManager->middleClickAutoscrollInProgress() &&
+ !m_frame->selection().selectedHTMLForClipboard().isEmpty()) {
if (AutoscrollController* controller =
m_scrollManager->autoscrollController()) {
controller->startAutoscrollForSelection(layoutObject);
@@ -741,9 +746,6 @@ WebInputEventResult MouseEventManager::handleMouseDraggedEvent(
}
}
- m_frame->eventHandler().selectionController().handleMouseDraggedEvent(
- event, m_mouseDownPos, m_dragStartPos, m_mousePressNode.get(),
- m_lastKnownMousePosition);
return WebInputEventResult::HandledSystem;
}

Powered by Google App Engine
This is Rietveld 408576698