reflow comments in core/input

third_party/WebKit/Source/core/input/GestureManager.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/input/GestureManager.h"
 
 #include "core/dom/Document.h"
 #include "core/editing/SelectionController.h"
 #include "core/events/GestureEvent.h"
 #include "core/frame/FrameHost.h"
@@ skipping 35 matching lines @@
 }
 
 HitTestRequest::HitTestRequestType GestureManager::getHitTypeForGestureType(
     PlatformEvent::EventType type) {
   HitTestRequest::HitTestRequestType hitType = HitTestRequest::TouchEvent;
   switch (type) {
     case PlatformEvent::GestureShowPress:
     case PlatformEvent::GestureTapUnconfirmed:
       return hitType | HitTestRequest::Active;
     case PlatformEvent::GestureTapDownCancel:
-      // A TapDownCancel received when no element is active shouldn't really be changing hover state.
+      // A TapDownCancel received when no element is active shouldn't really be
+      // changing hover state.
       if (!m_frame->document()->activeHoverElement())
         hitType |= HitTestRequest::ReadOnly;
       return hitType | HitTestRequest::Release;
     case PlatformEvent::GestureTap:
       return hitType | HitTestRequest::Release;
     case PlatformEvent::GestureTapDown:
     case PlatformEvent::GestureLongPress:
     case PlatformEvent::GestureLongTap:
     case PlatformEvent::GestureTwoFingerTap:
       // FIXME: Shouldn't LongTap and TwoFingerTap clear the Active state?
@@ skipping 68 matching lines @@
   const PlatformGestureEvent& gestureEvent = targetedEvent.event();
   HitTestRequest::HitTestRequestType hitType =
       getHitTypeForGestureType(gestureEvent.type());
   uint64_t preDispatchDomTreeVersion = m_frame->document()->domTreeVersion();
   uint64_t preDispatchStyleVersion = m_frame->document()->styleVersion();
 
   UserGestureIndicator gestureIndicator(DefinitelyProcessingUserGesture);
 
   HitTestResult currentHitTest = targetedEvent.hitTestResult();
 
-  // We use the adjusted position so the application isn't surprised to see a event with
-  // co-ordinates outside the target's bounds.
+  // We use the adjusted position so the application isn't surprised to see a
+  // event with co-ordinates outside the target's bounds.
   IntPoint adjustedPoint =
       frameView->rootFrameToContents(gestureEvent.position());
 
   const unsigned modifiers = gestureEvent.getModifiers();
 
   if (!m_suppressMouseEventsFromGestures) {
     PlatformMouseEvent fakeMouseMove(
         gestureEvent.position(), gestureEvent.globalPosition(),
         WebPointerProperties::Button::NoButton, PlatformEvent::MouseMoved,
         /* clickCount */ 0, static_cast<PlatformEvent::Modifiers>(modifiers),
         PlatformMouseEvent::FromTouch, gestureEvent.timestamp(),
         WebPointerProperties::PointerType::Mouse);
     m_mouseEventManager->setMousePositionAndDispatchMouseEvent(
         currentHitTest.innerNode(), EventTypeNames::mousemove, fakeMouseMove);
   }
 
   // Do a new hit-test in case the mousemove event changed the DOM.
-  // Note that if the original hit test wasn't over an element (eg. was over a scrollbar) we
-  // don't want to re-hit-test because it may be in the wrong frame (and there's no way the page
-  // could have seen the event anyway).
-  // Also note that the position of the frame may have changed, so we need to recompute the content
-  // co-ordinates (updating layout/style as hitTestResultAtPoint normally would).
-  // FIXME: Use a hit-test cache to avoid unnecessary hit tests. http://crbug.com/398920
+  // Note that if the original hit test wasn't over an element (eg. was over a
+  // scrollbar) we don't want to re-hit-test because it may be in the wrong
+  // frame (and there's no way the page could have seen the event anyway).  Also
+  // note that the position of the frame may have changed, so we need to
+  // recompute the content co-ordinates (updating layout/style as
+  // hitTestResultAtPoint normally would).
+  // FIXME: Use a hit-test cache to avoid unnecessary hit tests.
+  // http://crbug.com/398920
   if (currentHitTest.innerNode()) {
     LocalFrame* mainFrame = m_frame->localFrameRoot();
     if (mainFrame && mainFrame->view())
       mainFrame->view()->updateLifecycleToCompositingCleanPlusScrolling();
     adjustedPoint = frameView->rootFrameToContents(gestureEvent.position());
     currentHitTest = EventHandlingUtil::hitTestResultInFrame(
         m_frame, adjustedPoint, hitType);
   }
 
   // Capture data for showUnhandledTapUIIfNeeded.
@@ skipping 35 matching lines @@
           MouseEventWithHitTestResults(fakeMouseDown, currentHitTest));
   }
 
   if (currentHitTest.innerNode()) {
     DCHECK(gestureEvent.type() == PlatformEvent::GestureTap);
     HitTestResult result = currentHitTest;
     result.setToShadowHostIfInUserAgentShadowRoot();
     m_frame->chromeClient().onMouseDown(result.innerNode());
   }
 
-  // FIXME: Use a hit-test cache to avoid unnecessary hit tests. http://crbug.com/398920
+  // FIXME: Use a hit-test cache to avoid unnecessary hit tests.
+  // http://crbug.com/398920
   if (currentHitTest.innerNode()) {
     LocalFrame* mainFrame = m_frame->localFrameRoot();
     if (mainFrame && mainFrame->view())
       mainFrame->view()->updateAllLifecyclePhases();
     adjustedPoint = frameView->rootFrameToContents(gestureEvent.position());
     currentHitTest = EventHandlingUtil::hitTestResultInFrame(
         m_frame, adjustedPoint, hitType);
   }
 
   PlatformMouseEvent fakeMouseUp(
       gestureEvent.position(), gestureEvent.globalPosition(),
       WebPointerProperties::Button::Left, PlatformEvent::MouseReleased,
       gestureEvent.tapCount(), static_cast<PlatformEvent::Modifiers>(modifiers),
       PlatformMouseEvent::FromTouch, gestureEvent.timestamp(),
       WebPointerProperties::PointerType::Mouse);
   WebInputEventResult mouseUpEventResult =
       m_suppressMouseEventsFromGestures
           ? WebInputEventResult::HandledSuppressed
           : m_mouseEventManager->setMousePositionAndDispatchMouseEvent(
                 currentHitTest.innerNode(), EventTypeNames::mouseup,
                 fakeMouseUp);
 
   WebInputEventResult clickEventResult = WebInputEventResult::NotHandled;
   if (tappedNonTextNode) {
     if (currentHitTest.innerNode()) {
-      // Updates distribution because a mouseup (or mousedown) event listener can make the
-      // tree dirty at dispatchMouseEvent() invocation above.
-      // Unless distribution is updated, commonAncestor would hit DCHECK.
-      // Both tappedNonTextNode and currentHitTest.innerNode()) don't need to be updated
-      // because commonAncestor() will exit early if their documents are different.
+      // Updates distribution because a mouseup (or mousedown) event listener
+      // can make the tree dirty at dispatchMouseEvent() invocation above.
+      // Unless distribution is updated, commonAncestor would hit DCHECK.  Both
+      // tappedNonTextNode and currentHitTest.innerNode()) don't need to be
+      // updated because commonAncestor() will exit early if their documents are
+      // different.
       tappedNonTextNode->updateDistribution();
       Node* clickTargetNode = currentHitTest.innerNode()->commonAncestor(
           *tappedNonTextNode, EventHandlingUtil::parentForClickEvent);
       clickEventResult =
           m_mouseEventManager->setMousePositionAndDispatchMouseEvent(
               clickTargetNode, EventTypeNames::click, fakeMouseUp);
     }
     m_mouseEventManager->setClickNode(nullptr);
   }
 
@@ skipping 18 matching lines @@
     m_frame->chromeClient().showUnhandledTapUIIfNeeded(
         tappedPositionInViewport, tappedNode, domTreeChanged || styleChanged);
   }
   return eventResult;
 }
 
 WebInputEventResult GestureManager::handleGestureLongPress(
     const GestureEventWithHitTestResults& targetedEvent) {
   const PlatformGestureEvent& gestureEvent = targetedEvent.event();
 
-  // FIXME: Ideally we should try to remove the extra mouse-specific hit-tests here (re-using the
-  // supplied HitTestResult), but that will require some overhaul of the touch drag-and-drop code
-  // and LongPress is such a special scenario that it's unlikely to matter much in practice.
+  // FIXME: Ideally we should try to remove the extra mouse-specific hit-tests
+  // here (re-using the supplied HitTestResult), but that will require some
+  // overhaul of the touch drag-and-drop code and LongPress is such a special
+  // scenario that it's unlikely to matter much in practice.
 
   IntPoint hitTestPoint =
       m_frame->view()->rootFrameToContents(gestureEvent.position());
   HitTestResult hitTestResult =
       m_frame->eventHandler().hitTestResultAtPoint(hitTestPoint);
 
   m_longTapShouldInvokeContextMenu = false;
   bool hitTestContainsLinks = hitTestResult.URLElement() ||
                               !hitTestResult.absoluteImageURL().isNull() ||
                               !hitTestResult.absoluteMediaURL().isNull();
@@ skipping 105 matching lines @@
     return nullptr;
 
   return &m_frame->page()->frameHost();
 }
 
 double GestureManager::getLastShowPressTimestamp() const {
   return m_lastShowPressTimestamp;
 }
 
 }  // namespace blink