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

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

Issue 1686483002: Oilpan: Remove most WillBe types from the code base (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 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/EventHandler.cpp
diff --git a/third_party/WebKit/Source/core/input/EventHandler.cpp b/third_party/WebKit/Source/core/input/EventHandler.cpp
index e14f7c15c02dce70cc6892170708d3f88ae452ce..6bf7001cd5c8636668abd873781146d1ad90ecc2 100644
--- a/third_party/WebKit/Source/core/input/EventHandler.cpp
+++ b/third_party/WebKit/Source/core/input/EventHandler.cpp
@@ -125,7 +125,7 @@ WebInputEventResult mergeEventResult(WebInputEventResult responseA, WebInputEven
return static_cast<WebInputEventResult>(max(static_cast<int>(responseA), static_cast<int>(responseB)));
}
-WebInputEventResult eventToEventResult(PassRefPtrWillBeRawPtr<Event> event, bool res)
+WebInputEventResult eventToEventResult(RawPtr<Event> event, bool res)
{
if (event->defaultPrevented())
return WebInputEventResult::HandledApplication;
@@ -989,7 +989,7 @@ WebInputEventResult EventHandler::handleMousePressEvent(const PlatformMouseEvent
{
TRACE_EVENT0("blink", "EventHandler::handleMousePressEvent");
- RefPtrWillBeRawPtr<FrameView> protector(m_frame->view());
+ RawPtr<FrameView> protector(m_frame->view());
UserGestureIndicator gestureIndicator(DefinitelyProcessingUserGesture);
m_frame->localFrameRoot()->eventHandler().m_lastMouseDownUserGestureToken = gestureIndicator.currentToken();
@@ -1024,7 +1024,7 @@ WebInputEventResult EventHandler::handleMousePressEvent(const PlatformMouseEvent
m_mousePressNode = mev.innerNode();
- RefPtrWillBeRawPtr<LocalFrame> subframe = subframeForHitTestResult(mev);
+ RawPtr<LocalFrame> subframe = subframeForHitTestResult(mev);
if (subframe) {
WebInputEventResult result = passMousePressEventToSubframe(mev, subframe.get());
// Start capturing future events for this frame. We only do this if we didn't clear
@@ -1152,7 +1152,7 @@ WebInputEventResult EventHandler::handleMouseMoveEvent(const PlatformMouseEvent&
conditionallyEnableMouseEventForPointerTypeMouse(event);
- RefPtrWillBeRawPtr<FrameView> protector(m_frame->view());
+ RawPtr<FrameView> protector(m_frame->view());
HitTestResult hoveredNode = HitTestResult();
WebInputEventResult result = handleMouseMoveOrLeaveEvent(event, &hoveredNode);
@@ -1181,7 +1181,7 @@ void EventHandler::handleMouseLeaveEvent(const PlatformMouseEvent& event)
conditionallyEnableMouseEventForPointerTypeMouse(event);
- RefPtrWillBeRawPtr<FrameView> protector(m_frame->view());
+ RawPtr<FrameView> protector(m_frame->view());
handleMouseMoveOrLeaveEvent(event, 0, false, true);
}
@@ -1257,7 +1257,7 @@ WebInputEventResult EventHandler::handleMouseMoveOrLeaveEvent(const PlatformMous
}
WebInputEventResult eventResult = WebInputEventResult::NotHandled;
- RefPtrWillBeRawPtr<LocalFrame> newSubframe = m_capturingMouseEventsNode.get() ? subframeForTargetNode(m_capturingMouseEventsNode.get()) : subframeForHitTestResult(mev);
+ RawPtr<LocalFrame> newSubframe = m_capturingMouseEventsNode.get() ? subframeForTargetNode(m_capturingMouseEventsNode.get()) : subframeForHitTestResult(mev);
// We want mouseouts to happen first, from the inside out. First send a move event to the last subframe so that it will fire mouseouts.
if (m_lastMouseMoveEventSubframe && m_lastMouseMoveEventSubframe->tree().isDescendantOf(m_frame) && m_lastMouseMoveEventSubframe != newSubframe)
@@ -1314,7 +1314,7 @@ WebInputEventResult EventHandler::handleMouseReleaseEvent(const PlatformMouseEve
{
TRACE_EVENT0("blink", "EventHandler::handleMouseReleaseEvent");
- RefPtrWillBeRawPtr<FrameView> protector(m_frame->view());
+ RawPtr<FrameView> protector(m_frame->view());
m_frame->selection().setCaretBlinkingSuspended(false);
@@ -1415,7 +1415,7 @@ WebInputEventResult EventHandler::dispatchDragEvent(const AtomicString& eventTyp
if (!view)
return WebInputEventResult::NotHandled;
- RefPtrWillBeRawPtr<DragEvent> me = DragEvent::create(eventType,
+ RawPtr<DragEvent> me = DragEvent::create(eventType,
true, true, m_frame->document()->domWindow(),
0, event.globalPosition().x(), event.globalPosition().y(), event.position().x(), event.position().y(),
event.movementDelta().x(), event.movementDelta().y(),
@@ -1489,7 +1489,7 @@ WebInputEventResult EventHandler::updateDragAndDrop(const PlatformMouseEvent& ev
MouseEventWithHitTestResults mev = prepareMouseEvent(request, event);
// Drag events should never go to text nodes (following IE, and proper mouseover/out dispatch)
- RefPtrWillBeRawPtr<Node> newTarget = mev.innerNode();
+ RawPtr<Node> newTarget = mev.innerNode();
if (newTarget && newTarget->isTextNode())
newTarget = FlatTreeTraversal::parent(*newTarget);
@@ -1587,7 +1587,7 @@ void EventHandler::clearDragState()
m_shouldOnlyFireDragOverEvent = false;
}
-void EventHandler::setCapturingMouseEventsNode(PassRefPtrWillBeRawPtr<Node> n)
+void EventHandler::setCapturingMouseEventsNode(RawPtr<Node> n)
{
m_capturingMouseEventsNode = n;
m_eventHandlerWillResetCapturingMouseEventsNode = false;
@@ -1613,7 +1613,7 @@ void EventHandler::updateMouseEventTargetNode(Node* targetNode, const PlatformMo
if (result && result->isTextNode())
result = FlatTreeTraversal::parent(*result);
}
- RefPtrWillBeMember<Node> lastNodeUnderMouse = m_nodeUnderMouse;
+ Member<Node> lastNodeUnderMouse = m_nodeUnderMouse;
m_nodeUnderMouse = result;
PaintLayer* layerForLastNode = layerForNode(lastNodeUnderMouse.get());
@@ -1653,7 +1653,7 @@ void EventHandler::updateMouseEventTargetNode(Node* targetNode, const PlatformMo
sendNodeTransitionEvents(lastNodeUnderMouse.get(), m_nodeUnderMouse.get(), mouseEvent);
}
-WebInputEventResult EventHandler::dispatchPointerEvent(EventTarget* target, PassRefPtrWillBeRawPtr<PointerEvent> pointerEvent)
+WebInputEventResult EventHandler::dispatchPointerEvent(EventTarget* target, RawPtr<PointerEvent> pointerEvent)
{
if (!RuntimeEnabledFeatures::pointerEventEnabled())
return WebInputEventResult::NotHandled;
@@ -1687,8 +1687,8 @@ void EventHandler::sendNodeTransitionEvents(Node* exitedNode, Node* enteredNode,
// TODO(mustaq): Confirm spec conformance, double-check with other browsers.
// Create lists of all exited/entered ancestors, locate the common ancestor & capturing listeners.
- WillBeHeapVector<RefPtrWillBeMember<Node>, 32> exitedAncestors;
- WillBeHeapVector<RefPtrWillBeMember<Node>, 32> enteredAncestors;
+ HeapVector<Member<Node>, 32> exitedAncestors;
+ HeapVector<Member<Node>, 32> enteredAncestors;
if (isNodeInDocument(exitedNode)) {
exitedNode->updateDistribution();
for (Node* node = exitedNode; node; node = FlatTreeTraversal::parent(*node)) {
@@ -1759,13 +1759,13 @@ WebInputEventResult EventHandler::dispatchMouseEvent(const AtomicString& eventTy
if (!m_nodeUnderMouse)
return WebInputEventResult::NotHandled;
- RefPtrWillBeRawPtr<MouseEvent> event = MouseEvent::create(eventType, m_nodeUnderMouse->document().domWindow(), mouseEvent, clickCount, nullptr);
+ RawPtr<MouseEvent> event = MouseEvent::create(eventType, m_nodeUnderMouse->document().domWindow(), mouseEvent, clickCount, nullptr);
bool dispatchResult = m_nodeUnderMouse->dispatchEvent(event);
return eventToEventResult(event, dispatchResult);
}
EventTarget* EventHandler::getEffectiveTargetForPointerEvent(
- EventTarget* target, PassRefPtrWillBeRawPtr<PointerEvent> pointerEvent)
+ EventTarget* target, RawPtr<PointerEvent> pointerEvent)
{
EventTarget* capturingNode = m_pointerEventManager.getCapturingNode(pointerEvent.get());
if (capturingNode)
@@ -1782,7 +1782,7 @@ void EventHandler::sendPointerAndMouseTransitionEvents(Node* target, const Atomi
|| mouseEventType == EventTypeNames::mouseout);
AtomicString pointerEventType = pointerEventNameForMouseEventName(mouseEventType);
- RefPtrWillBeRawPtr<PointerEvent> pointerEvent = m_pointerEventManager.create(pointerEventType,
+ RawPtr<PointerEvent> pointerEvent = m_pointerEventManager.create(pointerEventType,
mouseEvent, relatedTarget, m_frame->document()->domWindow());
// Suppress these events if the target is not the capturing element
@@ -1815,7 +1815,7 @@ WebInputEventResult EventHandler::updatePointerTargetAndDispatchEvents(const Ato
|| (pointerEventType == EventTypeNames::pointerup && pointerButtonsPressed != 0))
pointerEventType = EventTypeNames::pointermove;
- RefPtrWillBeRawPtr<PointerEvent> pointerEvent = m_pointerEventManager.create(pointerEventType,
+ RawPtr<PointerEvent> pointerEvent = m_pointerEventManager.create(pointerEventType,
mouseEvent, nullptr, m_frame->document()->domWindow());
EventTarget* target = getEffectiveTargetForPointerEvent(m_nodeUnderMouse.get(), pointerEvent);
@@ -1825,7 +1825,7 @@ WebInputEventResult EventHandler::updatePointerTargetAndDispatchEvents(const Ato
m_preventMouseEventForPointerTypeMouse = true;
if (!m_preventMouseEventForPointerTypeMouse) {
- RefPtrWillBeRawPtr<MouseEvent> event = MouseEvent::create(mouseEventType, m_nodeUnderMouse->document().domWindow(), mouseEvent, clickCount, nullptr);
+ RawPtr<MouseEvent> event = MouseEvent::create(mouseEventType, m_nodeUnderMouse->document().domWindow(), mouseEvent, clickCount, nullptr);
bool dispatchResult = target->dispatchEvent(event);
result = mergeEventResult(result, eventToEventResult(event, dispatchResult));
}
@@ -1961,7 +1961,7 @@ WebInputEventResult EventHandler::handleWheelEvent(const PlatformWheelEvent& eve
if (!doc->layoutView())
return WebInputEventResult::NotHandled;
- RefPtrWillBeRawPtr<FrameView> protector(m_frame->view());
+ RawPtr<FrameView> protector(m_frame->view());
FrameView* view = m_frame->view();
if (!view)
@@ -1997,7 +1997,7 @@ WebInputEventResult EventHandler::handleWheelEvent(const PlatformWheelEvent& eve
}
}
- RefPtrWillBeRawPtr<Event> domEvent = WheelEvent::create(event, node->document().domWindow());
+ RawPtr<Event> domEvent = WheelEvent::create(event, node->document().domWindow());
if (!node->dispatchEvent(domEvent)) {
setFrameWasScrolledByUser();
return eventToEventResult(domEvent, false);
@@ -2117,8 +2117,8 @@ WebInputEventResult EventHandler::handleGestureEventInFrame(const GestureEventWi
{
ASSERT(!targetedEvent.event().isScrollEvent());
- RefPtrWillBeRawPtr<Node> eventTarget = targetedEvent.hitTestResult().innerNode();
- RefPtrWillBeRawPtr<Scrollbar> scrollbar = targetedEvent.hitTestResult().scrollbar();
+ RawPtr<Node> eventTarget = targetedEvent.hitTestResult().innerNode();
+ RawPtr<Scrollbar> scrollbar = targetedEvent.hitTestResult().scrollbar();
const PlatformGestureEvent& gestureEvent = targetedEvent.event();
if (scrollbar) {
@@ -2130,7 +2130,7 @@ WebInputEventResult EventHandler::handleGestureEventInFrame(const GestureEventWi
}
if (eventTarget) {
- RefPtrWillBeRawPtr<GestureEvent> gestureDomEvent = GestureEvent::create(eventTarget->document().domWindow(), gestureEvent);
+ RawPtr<GestureEvent> gestureDomEvent = GestureEvent::create(eventTarget->document().domWindow(), gestureEvent);
// TODO(dtapuska): dispatchEvent is inverted for Gesture Events
// crbug.com/560357
if (gestureDomEvent.get() && eventTarget->dispatchEvent(gestureDomEvent))
@@ -2166,8 +2166,8 @@ WebInputEventResult EventHandler::handleGestureScrollEvent(const PlatformGesture
{
TRACE_EVENT0("input", "EventHandler::handleGestureScrollEvent");
- RefPtrWillBeRawPtr<Node> eventTarget = nullptr;
- RefPtrWillBeRawPtr<Scrollbar> scrollbar = nullptr;
+ RawPtr<Node> eventTarget = nullptr;
+ RawPtr<Scrollbar> scrollbar = nullptr;
if (gestureEvent.type() != PlatformEvent::GestureScrollBegin) {
scrollbar = m_scrollbarHandlingScrollGesture.get();
eventTarget = m_scrollGestureHandlingNode.get();
@@ -2209,7 +2209,7 @@ WebInputEventResult EventHandler::handleGestureScrollEvent(const PlatformGesture
if (handleScrollGestureOnResizer(eventTarget.get(), gestureEvent))
return WebInputEventResult::HandledSuppressed;
- RefPtrWillBeRawPtr<GestureEvent> gestureDomEvent = GestureEvent::create(eventTarget->document().domWindow(), gestureEvent);
+ RawPtr<GestureEvent> gestureDomEvent = GestureEvent::create(eventTarget->document().domWindow(), gestureEvent);
// TODO(dtapuska): dispatchEvent is inverted for Gesture Events
// crbug.com/560357
if (gestureDomEvent.get() && eventTarget->dispatchEvent(gestureDomEvent))
@@ -2236,7 +2236,7 @@ WebInputEventResult EventHandler::handleGestureScrollEvent(const PlatformGesture
WebInputEventResult EventHandler::handleGestureTap(const GestureEventWithHitTestResults& targetedEvent)
{
- RefPtrWillBeRawPtr<FrameView> frameView(m_frame->view());
+ RawPtr<FrameView> frameView(m_frame->view());
const PlatformGestureEvent& gestureEvent = targetedEvent.event();
HitTestRequest::HitTestRequestType hitType = getHitTypeForGestureType(gestureEvent.type());
uint64_t preDispatchDomTreeVersion = m_frame->document()->domTreeVersion();
@@ -2274,7 +2274,7 @@ WebInputEventResult EventHandler::handleGestureTap(const GestureEventWithHitTest
m_clickNode = currentHitTest.innerNode();
// Capture data for showUnhandledTapUIIfNeeded.
- RefPtrWillBeRawPtr<Node> tappedNode = m_clickNode;
+ RawPtr<Node> tappedNode = m_clickNode;
IntPoint tappedPosition = gestureEvent.position();
if (m_clickNode && m_clickNode->isTextNode())
@@ -2367,7 +2367,7 @@ WebInputEventResult EventHandler::handleGestureLongPress(const GestureEventWithH
m_mouseDownMayStartDrag = true;
dragState().m_dragSrc = nullptr;
m_mouseDownPos = m_frame->view()->rootFrameToContents(mouseDragEvent.position());
- RefPtrWillBeRawPtr<FrameView> protector(m_frame->view());
+ RawPtr<FrameView> protector(m_frame->view());
if (handleDrag(mev, DragInitiator::Touch)) {
m_longTapShouldInvokeContextMenu = true;
return WebInputEventResult::HandledSystem;
@@ -2376,7 +2376,7 @@ WebInputEventResult EventHandler::handleGestureLongPress(const GestureEventWithH
IntPoint hitTestPoint = m_frame->view()->rootFrameToContents(gestureEvent.position());
HitTestResult result = hitTestResultAtPoint(hitTestPoint);
- RefPtrWillBeRawPtr<FrameView> protector(m_frame->view());
+ RawPtr<FrameView> protector(m_frame->view());
if (selectionController().handleGestureLongPress(gestureEvent, result)) {
focusDocumentView();
return WebInputEventResult::HandledSystem;
@@ -2440,12 +2440,12 @@ WebInputEventResult EventHandler::passScrollGestureEventToWidget(const PlatformG
WebInputEventResult EventHandler::handleGestureScrollEnd(const PlatformGestureEvent& gestureEvent)
{
- RefPtrWillBeRawPtr<Node> node = m_scrollGestureHandlingNode;
+ RawPtr<Node> node = m_scrollGestureHandlingNode;
if (node) {
passScrollGestureEventToWidget(gestureEvent, node->layoutObject());
if (RuntimeEnabledFeatures::scrollCustomizationEnabled()) {
- RefPtrWillBeRawPtr<ScrollState> scrollState = ScrollState::create(
+ RawPtr<ScrollState> scrollState = ScrollState::create(
0, 0, 0, 0, 0, gestureEvent.inertial(), /* isBeginning */
false, /* isEnding */ true, /* fromUserInput */ true);
customizedScroll(*node.get(), *scrollState);
@@ -2482,7 +2482,7 @@ WebInputEventResult EventHandler::handleGestureScrollBegin(const PlatformGesture
passScrollGestureEventToWidget(gestureEvent, m_scrollGestureHandlingNode->layoutObject());
if (RuntimeEnabledFeatures::scrollCustomizationEnabled()) {
m_currentScrollChain.clear();
- RefPtrWillBeRawPtr<ScrollState> scrollState = ScrollState::create(
+ RawPtr<ScrollState> scrollState = ScrollState::create(
0, 0, 0, 0, 0, /* inInertialPhase */ false, /* isBeginning */
true, /* isEnding */ false, /* fromUserInput */ true);
customizedScroll(*m_scrollGestureHandlingNode.get(), *scrollState);
@@ -2540,7 +2540,7 @@ WebInputEventResult EventHandler::handleGestureScrollUpdate(const PlatformGestur
if (!layoutObject)
return WebInputEventResult::NotHandled;
- RefPtrWillBeRawPtr<FrameView> protector(m_frame->view());
+ RawPtr<FrameView> protector(m_frame->view());
Node* stopNode = nullptr;
@@ -2561,7 +2561,7 @@ WebInputEventResult EventHandler::handleGestureScrollUpdate(const PlatformGestur
bool scrolled = false;
if (handleScrollCustomization) {
- RefPtrWillBeRawPtr<ScrollState> scrollState = ScrollState::create(
+ RawPtr<ScrollState> scrollState = ScrollState::create(
gestureEvent.deltaX(), gestureEvent.deltaY(),
0, gestureEvent.velocityX(), gestureEvent.velocityY(),
gestureEvent.inertial(), /* isBeginning */
@@ -2659,11 +2659,11 @@ bool EventHandler::bestClickableNodeForHitTestResult(const HitTestResult& result
IntPoint touchCenter = m_frame->view()->contentsToRootFrame(result.roundedPointInMainFrame());
IntRect touchRect = m_frame->view()->contentsToRootFrame(result.hitTestLocation().boundingBox());
- WillBeHeapVector<RefPtrWillBeMember<Node>, 11> nodes;
+ HeapVector<Member<Node>, 11> nodes;
copyToVector(result.listBasedTestResult(), nodes);
// FIXME: the explicit Vector conversion copies into a temporary and is wasteful.
- return findBestClickableCandidate(targetNode, targetPoint, touchCenter, touchRect, WillBeHeapVector<RefPtrWillBeMember<Node>> (nodes));
+ return findBestClickableCandidate(targetNode, targetPoint, touchCenter, touchRect, HeapVector<Member<Node>> (nodes));
}
bool EventHandler::bestContextMenuNodeForHitTestResult(const HitTestResult& result, IntPoint& targetPoint, Node*& targetNode)
@@ -2671,11 +2671,11 @@ bool EventHandler::bestContextMenuNodeForHitTestResult(const HitTestResult& resu
ASSERT(result.isRectBasedTest());
IntPoint touchCenter = m_frame->view()->contentsToRootFrame(result.roundedPointInMainFrame());
IntRect touchRect = m_frame->view()->contentsToRootFrame(result.hitTestLocation().boundingBox());
- WillBeHeapVector<RefPtrWillBeMember<Node>, 11> nodes;
+ HeapVector<Member<Node>, 11> nodes;
copyToVector(result.listBasedTestResult(), nodes);
// FIXME: the explicit Vector conversion copies into a temporary and is wasteful.
- return findBestContextMenuCandidate(targetNode, targetPoint, touchCenter, touchRect, WillBeHeapVector<RefPtrWillBeMember<Node>>(nodes));
+ return findBestContextMenuCandidate(targetNode, targetPoint, touchCenter, touchRect, HeapVector<Member<Node>>(nodes));
}
bool EventHandler::bestZoomableAreaForTouchPoint(const IntPoint& touchCenter, const IntSize& touchRadius, IntRect& targetArea, Node*& targetNode)
@@ -2689,11 +2689,11 @@ bool EventHandler::bestZoomableAreaForTouchPoint(const IntPoint& touchCenter, co
HitTestResult result = hitTestResultAtPoint(hitTestPoint, hitType, LayoutSize(touchRadius));
IntRect touchRect(touchCenter - touchRadius, touchRadius + touchRadius);
- WillBeHeapVector<RefPtrWillBeMember<Node>, 11> nodes;
+ HeapVector<Member<Node>, 11> nodes;
copyToVector(result.listBasedTestResult(), nodes);
// FIXME: the explicit Vector conversion copies into a temporary and is wasteful.
- return findBestZoomableArea(targetNode, targetArea, touchCenter, touchRect, WillBeHeapVector<RefPtrWillBeMember<Node>>(nodes));
+ return findBestZoomableArea(targetNode, targetArea, touchCenter, touchRect, HeapVector<Member<Node>>(nodes));
}
// Update the hover and active state across all frames for this gesture.
@@ -2703,7 +2703,7 @@ void EventHandler::updateGestureHoverActiveState(const HitTestRequest& request,
{
ASSERT(m_frame == m_frame->localFrameRoot());
- WillBeHeapVector<RawPtrWillBeMember<LocalFrame>> newHoverFrameChain;
+ HeapVector<Member<LocalFrame>> newHoverFrameChain;
LocalFrame* newHoverFrameInDocument = innerElement ? innerElement->document().frame() : nullptr;
// Insert the ancestors of the frame having the new hovered node to the frame chain
// The frame chain doesn't include the main frame to avoid the redundant work that cleans the hover state.
@@ -2714,8 +2714,8 @@ void EventHandler::updateGestureHoverActiveState(const HitTestRequest& request,
newHoverFrameInDocument = parentFrame && parentFrame->isLocalFrame() ? toLocalFrame(parentFrame) : nullptr;
}
- RefPtrWillBeRawPtr<Node> oldHoverNodeInCurDoc = m_frame->document()->hoverNode();
- RefPtrWillBeRawPtr<Node> newInnermostHoverNode = innerElement;
+ RawPtr<Node> oldHoverNodeInCurDoc = m_frame->document()->hoverNode();
+ RawPtr<Node> newInnermostHoverNode = innerElement;
if (newInnermostHoverNode != oldHoverNodeInCurDoc) {
size_t indexFrameChain = newHoverFrameChain.size();
@@ -2763,7 +2763,7 @@ void EventHandler::updateGestureTargetNodeForMouseEvent(const GestureEventWithHi
// - Dispatch mouseover/mouseenter events of the entered frames into the inside.
// Insert the ancestors of the frame having the new target node to the entered frame chain
- WillBeHeapVector<RawPtrWillBeMember<LocalFrame>> enteredFrameChain;
+ HeapVector<Member<LocalFrame>> enteredFrameChain;
LocalFrame* enteredFrameInDocument = targetedEvent.hitTestResult().innerNodeFrame();
while (enteredFrameInDocument) {
enteredFrameChain.append(enteredFrameInDocument);
@@ -2773,7 +2773,7 @@ void EventHandler::updateGestureTargetNodeForMouseEvent(const GestureEventWithHi
size_t indexEnteredFrameChain = enteredFrameChain.size();
LocalFrame* exitedFrameInDocument = m_frame;
- WillBeHeapVector<RawPtrWillBeMember<LocalFrame>> exitedFrameChain;
+ HeapVector<Member<LocalFrame>> exitedFrameChain;
// Insert the frame from the disagreement between last frames and entered frames
while (exitedFrameInDocument) {
Node* lastNodeUnderTap = exitedFrameInDocument->eventHandler().m_nodeUnderMouse.get();
@@ -3235,7 +3235,7 @@ bool EventHandler::handleAccessKey(const PlatformKeyboardEvent& evt)
WebInputEventResult EventHandler::keyEvent(const PlatformKeyboardEvent& initialKeyEvent)
{
- RefPtrWillBeRawPtr<FrameView> protector(m_frame->view());
+ RawPtr<FrameView> protector(m_frame->view());
m_frame->chromeClient().clearToolTip();
if (initialKeyEvent.windowsVirtualKeyCode() == VK_CAPITAL)
@@ -3254,7 +3254,7 @@ WebInputEventResult EventHandler::keyEvent(const PlatformKeyboardEvent& initialK
// Check for cases where we are too early for events -- possible unmatched key up
// from pressing return in the location bar.
- RefPtrWillBeRawPtr<Node> node = eventTargetNodeForDocument(m_frame->document());
+ RawPtr<Node> node = eventTargetNodeForDocument(m_frame->document());
if (!node)
return WebInputEventResult::NotHandled;
@@ -3271,7 +3271,7 @@ WebInputEventResult EventHandler::keyEvent(const PlatformKeyboardEvent& initialK
// FIXME: it would be fair to let an input method handle KeyUp events before DOM dispatch.
if (initialKeyEvent.type() == PlatformEvent::KeyUp || initialKeyEvent.type() == PlatformEvent::Char) {
- RefPtrWillBeRawPtr<KeyboardEvent> domEvent = KeyboardEvent::create(initialKeyEvent, m_frame->document()->domWindow());
+ RawPtr<KeyboardEvent> domEvent = KeyboardEvent::create(initialKeyEvent, m_frame->document()->domWindow());
bool dispatchResult = node->dispatchEvent(domEvent);
return eventToEventResult(domEvent, dispatchResult);
@@ -3280,7 +3280,7 @@ WebInputEventResult EventHandler::keyEvent(const PlatformKeyboardEvent& initialK
PlatformKeyboardEvent keyDownEvent = initialKeyEvent;
if (keyDownEvent.type() != PlatformEvent::RawKeyDown)
keyDownEvent.disambiguateKeyDownEvent(PlatformEvent::RawKeyDown);
- RefPtrWillBeRawPtr<KeyboardEvent> keydown = KeyboardEvent::create(keyDownEvent, m_frame->document()->domWindow());
+ RawPtr<KeyboardEvent> keydown = KeyboardEvent::create(keyDownEvent, m_frame->document()->domWindow());
if (matchedAnAccessKey)
keydown->setDefaultPrevented(true);
keydown->setTarget(node);
@@ -3312,7 +3312,7 @@ WebInputEventResult EventHandler::keyEvent(const PlatformKeyboardEvent& initialK
keyPressEvent.disambiguateKeyDownEvent(PlatformEvent::Char);
if (keyPressEvent.text().isEmpty())
return WebInputEventResult::NotHandled;
- RefPtrWillBeRawPtr<KeyboardEvent> keypress = KeyboardEvent::create(keyPressEvent, m_frame->document()->domWindow());
+ RawPtr<KeyboardEvent> keypress = KeyboardEvent::create(keyPressEvent, m_frame->document()->domWindow());
keypress->setTarget(node);
bool dispatchResult = node->dispatchEvent(keypress);
return eventToEventResult(keypress, dispatchResult);
@@ -3557,7 +3557,7 @@ bool EventHandler::handleTextInputEvent(const String& text, Event* underlyingEve
if (!target)
return false;
- RefPtrWillBeRawPtr<TextEvent> event = TextEvent::create(m_frame->domWindow(), text, inputType);
+ RawPtr<TextEvent> event = TextEvent::create(m_frame->domWindow(), text, inputType);
event->setUnderlyingEvent(underlyingEvent);
target->dispatchEvent(event);
@@ -3734,7 +3734,7 @@ HitTestResult EventHandler::hitTestResultInFrame(LocalFrame* frame, const Layout
}
void EventHandler::dispatchPointerEvents(const PlatformTouchEvent& event,
- WillBeHeapVector<TouchInfo>& touchInfos)
+ HeapVector<TouchInfo>& touchInfos)
{
if (!RuntimeEnabledFeatures::pointerEventEnabled())
return;
@@ -3752,7 +3752,7 @@ void EventHandler::dispatchPointerEvents(const PlatformTouchEvent& event,
bool pointerReleasedOrCancelled = pointState == PlatformTouchPoint::TouchReleased
|| pointState == PlatformTouchPoint::TouchCancelled;
- RefPtrWillBeRawPtr<PointerEvent> pointerEvent = m_pointerEventManager.create(
+ RawPtr<PointerEvent> pointerEvent = m_pointerEventManager.create(
pointerEventNameForTouchPointState(pointState),
touchPoint, event.modifiers(),
touchInfo.adjustedRadius.width(), touchInfo.adjustedRadius.height(),
@@ -3769,7 +3769,7 @@ void EventHandler::dispatchPointerEvents(const PlatformTouchEvent& event,
}
}
-void EventHandler::sendPointerCancels(WillBeHeapVector<TouchInfo>& touchInfos)
+void EventHandler::sendPointerCancels(HeapVector<TouchInfo>& touchInfos)
{
if (!RuntimeEnabledFeatures::pointerEventEnabled())
return;
@@ -3783,7 +3783,7 @@ void EventHandler::sendPointerCancels(WillBeHeapVector<TouchInfo>& touchInfos)
|| pointState == PlatformTouchPoint::TouchCancelled)
continue;
- RefPtrWillBeRawPtr<PointerEvent> pointerEvent = m_pointerEventManager.createPointerCancel(point);
+ RawPtr<PointerEvent> pointerEvent = m_pointerEventManager.createPointerCancel(point);
// TODO(nzolghadr): crbug.com/579553 dealing with implicit touch capturing vs pointer event capturing
touchInfo.touchTarget->dispatchEvent(pointerEvent.get());
@@ -3806,9 +3806,9 @@ class ChangedTouches final {
public:
// The touches corresponding to the particular change state this struct
// instance represents.
- RefPtrWillBeMember<TouchList> m_touches;
+ Member<TouchList> m_touches;
- using EventTargetSet = WillBeHeapHashSet<RefPtrWillBeMember<EventTarget>>;
+ using EventTargetSet = HeapHashSet<Member<EventTarget>>;
// Set of targets involved in m_touches.
EventTargetSet m_targets;
};
@@ -3816,7 +3816,7 @@ public:
} // namespace
WebInputEventResult EventHandler::dispatchTouchEvents(const PlatformTouchEvent& event,
- WillBeHeapVector<TouchInfo>& touchInfos, bool freshTouchEvents, bool allTouchReleased)
+ HeapVector<TouchInfo>& touchInfos, bool freshTouchEvents, bool allTouchReleased)
{
// Build up the lists to use for the 'touches', 'targetTouches' and
// 'changedTouches' attributes in the JS event. See
@@ -3824,11 +3824,11 @@ WebInputEventResult EventHandler::dispatchTouchEvents(const PlatformTouchEvent&
// lists fit together.
// Holds the complete set of touches on the screen.
- RefPtrWillBeRawPtr<TouchList> touches = TouchList::create();
+ RawPtr<TouchList> touches = TouchList::create();
// A different view on the 'touches' list above, filtered and grouped by
// event target. Used for the 'targetTouches' list in the JS event.
- using TargetTouchesHeapMap = WillBeHeapHashMap<EventTarget*, RefPtrWillBeMember<TouchList>>;
+ using TargetTouchesHeapMap = HeapHashMap<EventTarget*, Member<TouchList>>;
TargetTouchesHeapMap touchesByTarget;
// Array of touches per state, used to assemble the 'changedTouches' list.
@@ -3842,7 +3842,7 @@ WebInputEventResult EventHandler::dispatchTouchEvents(const PlatformTouchEvent&
if (touchInfo.consumed)
continue;
- RefPtrWillBeRawPtr<Touch> touch = Touch::create(
+ RawPtr<Touch> touch = Touch::create(
touchInfo.targetFrame.get(),
touchInfo.touchTarget.get(),
point.id(),
@@ -3898,7 +3898,7 @@ WebInputEventResult EventHandler::dispatchTouchEvents(const PlatformTouchEvent&
const AtomicString& eventName(touchEventNameForTouchPointState(static_cast<PlatformTouchPoint::State>(state)));
for (const auto& eventTarget : changedTouches[state].m_targets) {
EventTarget* touchEventTarget = eventTarget.get();
- RefPtrWillBeRawPtr<TouchEvent> touchEvent = TouchEvent::create(
+ RawPtr<TouchEvent> touchEvent = TouchEvent::create(
touches.get(), touchesByTarget.get(touchEventTarget), changedTouches[state].m_touches.get(),
eventName, touchEventTarget->toNode()->document().domWindow(),
event.modifiers(), event.cancelable(), event.causesScrollingIfUncanceled(), event.timestamp());
@@ -4017,12 +4017,12 @@ WebInputEventResult EventHandler::handleTouchEvent(const PlatformTouchEvent& eve
}
// Compute and store the common info used by both PointerEvent and TouchEvent.
- WillBeHeapVector<TouchInfo> touchInfos(points.size());
+ HeapVector<TouchInfo> touchInfos(points.size());
for (unsigned i = 0; i < points.size(); ++i) {
const PlatformTouchPoint& point = points[i];
PlatformTouchPoint::State pointState = point.state();
- RefPtrWillBeRawPtr<EventTarget> touchTarget = nullptr;
+ RawPtr<EventTarget> touchTarget = nullptr;
if (pointState == PlatformTouchPoint::TouchReleased || pointState == PlatformTouchPoint::TouchCancelled) {
// The target should be the original target for this touch, so get

Powered by Google App Engine
This is Rietveld 408576698