Fix link's hover state if the link under scrollbar

third_party/WebKit/Source/web/ChromeClientImpl.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies).
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
@@ skipping 609 matching lines @@
 
 void ChromeClientImpl::layoutUpdated(LocalFrame* frame) const {
   m_webView->layoutUpdated(WebLocalFrameImpl::fromFrame(frame));
 }
 
 void ChromeClientImpl::showMouseOverURL(const HitTestResult& result) {
   if (!m_webView->client())
     return;
 
   WebURL url;
-  // Find out if the mouse is over a link, and if so, let our UI know...
-  if (result.isLiveLink() && !result.absoluteLinkURL().getString().isEmpty()) {
-    url = result.absoluteLinkURL();
-  } else if (result.innerNode() && (isHTMLObjectElement(*result.innerNode()) ||
-                                    isHTMLEmbedElement(*result.innerNode()))) {
-    LayoutObject* object = result.innerNode()->layoutObject();
-    if (object && object->isLayoutPart()) {
-      Widget* widget = toLayoutPart(object)->widget();
-      if (widget && widget->isPluginContainer()) {
-        WebPluginContainerImpl* plugin = toWebPluginContainerImpl(widget);
-        url = plugin->plugin()->linkAtPosition(
-            result.roundedPointInInnerNodeFrame());
+
+  // Ignore URL if hitTest include scrollbar

[bokan, 2016/11/01 21:09:37]

I'add add to this comment " since we might have both a scrollbar and an element
in the case of overlay scrollbars."

+  if (!result.scrollbar()) {
+    // Find out if the mouse is over a link, and if so, let our UI know...
+    if (result.isLiveLink() &&
+        !result.absoluteLinkURL().getString().isEmpty()) {
+      url = result.absoluteLinkURL();
+    } else if (result.innerNode() &&
+               (isHTMLObjectElement(*result.innerNode()) ||
+                isHTMLEmbedElement(*result.innerNode()))) {
+      LayoutObject* object = result.innerNode()->layoutObject();
+      if (object && object->isLayoutPart()) {
+        Widget* widget = toLayoutPart(object)->widget();
+        if (widget && widget->isPluginContainer()) {
+          WebPluginContainerImpl* plugin = toWebPluginContainerImpl(widget);
+          url = plugin->plugin()->linkAtPosition(
+              result.roundedPointInInnerNodeFrame());
+        }
       }
     }
   }
 
   m_webView->client()->setMouseOverURL(url);
 }
 
 void ChromeClientImpl::setToolTip(LocalFrame& frame,
                                   const String& tooltipText,
                                   TextDirection dir) {
@@ skipping 503 matching lines @@
   if (RuntimeEnabledFeatures::presentationEnabled())
     PresentationController::provideTo(frame, client->presentationClient());
   if (RuntimeEnabledFeatures::audioOutputDevicesEnabled())
     provideAudioOutputDeviceClientTo(frame,
                                      AudioOutputDeviceClientImpl::create());
   if (RuntimeEnabledFeatures::installedAppEnabled())
     InstalledAppController::provideTo(frame, client->installedAppClient());
 }
 
 }  // namespace blink