Revert of Fix link's hover state if the link under scrollbar

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

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 edc9647675db56831317f6320b72ed099891a880..a40a5bc065dff8cd721b95a119fcf4bc36e0dad3 100644
--- a/third_party/WebKit/Source/core/input/EventHandler.cpp
+++ b/third_party/WebKit/Source/core/input/EventHandler.cpp
@@ -278,10 +278,8 @@
     return result;
 
   m_frame->contentLayoutItem().hitTest(result);
-  if (!request.readOnly()) {
-    m_frame->document()->updateHoverActiveState(request, result.innerElement(),
-                                                result.scrollbar());
-  }
+  if (!request.readOnly())
+    m_frame->document()->updateHoverActiveState(request, result.innerElement());
 
   return result;
 }
@@ -531,10 +529,6 @@
 OptionalCursor EventHandler::selectAutoCursor(const HitTestResult& result,
                                               Node* node,
                                               const Cursor& iBeam) {
-  if (result.scrollbar()) {
-    return pointerCursor();
-  }
-
   bool editable = (node && hasEditableStyle(*node));
 
   const bool isOverLink =
@@ -817,7 +811,7 @@
   // So we must force the hit-test to fail, while still clearing hover/active
   // state.
   if (forceLeave) {
-    m_frame->document()->updateHoverActiveState(request, nullptr, false);
+    m_frame->document()->updateHoverActiveState(request, 0);
   } else {
     mev = EventHandlingUtil::performMouseEventHitTest(m_frame, request,
                                                       mouseEvent);
@@ -1531,13 +1525,13 @@
       // If the old hovered frame is different from the new hovered frame.
       // we should clear the old hovered node from the old hovered frame.
       if (newHoverFrame != oldHoverFrame)
-        doc->updateHoverActiveState(request, nullptr, false);
+        doc->updateHoverActiveState(request, nullptr);
     }
   }
 
   // Recursively set the new active/hover states on every frame in the chain of
   // innerElement.
-  m_frame->document()->updateHoverActiveState(request, innerElement, false);
+  m_frame->document()->updateHoverActiveState(request, innerElement);
 }
 
 // Update the mouseover/mouseenter/mouseout/mouseleave events across all frames
@@ -1863,8 +1857,7 @@
   HitTestRequest request(HitTestRequest::Active);
   HitTestResult result(request, locationInRootFrame);
   result.setInnerNode(targetNode);
-  doc->updateHoverActiveState(request, result.innerElement(),
-                              result.scrollbar());
+  doc->updateHoverActiveState(request, result.innerElement());
 
   // The contextmenu event is a mouse event even when invoked using the
   // keyboard.  This is required for web compatibility.
@@ -1930,8 +1923,8 @@
                            view->rootFrameToContents(
                                m_mouseEventManager->lastKnownMousePosition()));
       layoutItem.hitTest(result);
-      m_frame->document()->updateHoverActiveState(
-          request, result.innerElement(), result.scrollbar());
+      m_frame->document()->updateHoverActiveState(request,
+                                                  result.innerElement());
     }
   }
 }
@@ -1945,8 +1938,8 @@
     // m_lastDeferredTapElement.get() == m_frame->document()->activeElement()
     HitTestRequest request(HitTestRequest::TouchEvent |
                            HitTestRequest::Release);
-    m_frame->document()->updateHoverActiveState(
-        request, m_lastDeferredTapElement.get(), false);
+    m_frame->document()->updateHoverActiveState(request,
+                                                m_lastDeferredTapElement.get());
   }
   m_lastDeferredTapElement = nullptr;
 }