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

Unified Diff: third_party/WebKit/Source/core/page/ChromeClient.cpp

Issue 2389073002: Fix link's hover state if the link under scrollbar (Closed)
Patch Set: add test Created 4 years, 1 month 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/page/ChromeClient.cpp
diff --git a/third_party/WebKit/Source/core/page/ChromeClient.cpp b/third_party/WebKit/Source/core/page/ChromeClient.cpp
index c048b73d418e85bef4138ffef2e6bb2874b45f22..5184b74ece32c83f25e0dc7f5a9687388b6680df 100644
--- a/third_party/WebKit/Source/core/page/ChromeClient.cpp
+++ b/third_party/WebKit/Source/core/page/ChromeClient.cpp
@@ -151,13 +151,16 @@ bool ChromeClient::openJavaScriptPrompt(LocalFrame* frame,
void ChromeClient::mouseDidMoveOverElement(LocalFrame& frame,
const HitTestResult& result) {
- if (result.innerNode() &&
+ if (!result.scrollbar() && result.innerNode() &&
result.innerNode()->document().isDNSPrefetchEnabled())
prefetchDNS(result.absoluteLinkURL().host());
showMouseOverURL(result);
- setToolTip(frame, result);
+ if (result.scrollbar())
+ clearToolTip(frame);
+ else
+ setToolTip(frame, result);
}
void ChromeClient::setToolTip(LocalFrame& frame, const HitTestResult& result) {

Powered by Google App Engine
This is Rietveld 408576698