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

Unified Diff: third_party/WebKit/Source/core/input/PointerEventManager.h

Issue 2350433002: Extract more of the mouse logic from EventHandler (Closed)
Patch Set: applying comments Created 4 years, 3 months 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/PointerEventManager.h
diff --git a/third_party/WebKit/Source/core/input/PointerEventManager.h b/third_party/WebKit/Source/core/input/PointerEventManager.h
index 0239332cd9ef5721240cc37b171e32a43c050875..357c2c4e6e3ae437762fc62b650a27e1af87e4e6 100644
--- a/third_party/WebKit/Source/core/input/PointerEventManager.h
+++ b/third_party/WebKit/Source/core/input/PointerEventManager.h
@@ -33,30 +33,20 @@ public:
// and sets the newNodeUnderMouse if the capturing is set
// in this function.
WebInputEventResult sendMousePointerEvent(
- Node* target, const AtomicString& type,
- int clickCount, const PlatformMouseEvent&,
- Node* lastNodeUnderMouse,
- Node** newNodeUnderMouse);
+ Node* target, const AtomicString& type, const PlatformMouseEvent&);
WebInputEventResult handleTouchEvents(
const PlatformTouchEvent&);
- // Sends boundary events mouseout/leave/over/enter to the
- // corresponding targets. This function sends pointerout/leave/over/enter
- // only when isFrameBoundaryTransition is true which indicates the
- // transition is over the document boundary and not only the elements border
- // inside the document. If isFrameBoundaryTransition is false,
- // then the event is a compatibility event like those created by touch
- // and in that case the corresponding pointer events will be handled by
- // sendTouchPointerEvent for example and there is no need to send pointer
- // boundary events. Note that normal mouse events (e.g. mousemove/down/up)
+ // Sends boundary events pointerout/leave/over/enter and
+ // mouseout/leave/over/enter to the corresponding targets.
+ // inside the document. This functions handles the cases that pointer is
+ // leaving a frame. Note that normal mouse events (e.g. mousemove/down/up)
// and their corresponding boundary events will be handled altogether by
// sendMousePointerEvent function.
- void sendMouseAndPossiblyPointerBoundaryEvents(
- Node* exitedNode,
+ void sendMouseAndPointerBoundaryEvents(
Node* enteredNode,
- const PlatformMouseEvent&,
- bool isFrameBoundaryTransition);
+ const PlatformMouseEvent&);
// Resets the internal state of this object.
void clear();
@@ -163,10 +153,8 @@ private:
EventTarget* processCaptureAndPositionOfPointerEvent(
PointerEvent*,
EventTarget* hitTestTarget,
- EventTarget* lastNodeUnderMouse = nullptr,
const PlatformMouseEvent& = PlatformMouseEvent(),
- bool sendMouseEvent = false,
- bool setPointerPosition = true);
+ bool sendMouseEvent = false);
void removeTargetFromPointerCapturingMapping(
PointerCapturingMap&, const EventTarget*);

Powered by Google App Engine
This is Rietveld 408576698