Send an empty tooltip text when hovering over a different node

third_party/WebKit/Source/core/page/ChromeClient.cpp

 /*
  * Copyright (C) 2006, 2007, 2009, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2010 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) 2012, Samsung Electronics. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 20 matching lines @@
 #include "core/page/FrameTree.h"
 #include "core/page/ScopedPageSuspender.h"
 #include "core/page/WindowFeatures.h"
 #include "platform/geometry/IntRect.h"
 #include "platform/network/NetworkHints.h"
 #include "public/platform/WebScreenInfo.h"
 #include <algorithm>
 
 namespace blink {
 
+DEFINE_TRACE(ChromeClient) {
+  visitor->trace(m_lastMouseOverNode);
+  HostWindow::trace(visitor);
+}
+
 void ChromeClient::setWindowRectWithAdjustment(const IntRect& pendingRect,
                                                LocalFrame& frame) {
   IntRect screen = screenInfo().availableRect;
   IntRect window = pendingRect;
 
   IntSize minimumSize = minimumWindowSize();
   // Let size 0 pass through, since that indicates default size, not minimum
   // size.
   if (window.width())
     window.setWidth(std::min(std::max(minimumSize.width(), window.width()),
@@ skipping 132 matching lines @@
         // ChromeClient::setToolTip. We'll work on tooltip text
         // direction during bidi cleanup in form inputs.
         toolTipDirection = TextDirection::Ltr;
       }
     }
   }
 
   if (m_lastToolTipPoint == result.hitTestLocation().point() &&
       m_lastToolTipText == toolTip)
     return;
+
+  // If a tooltip was displayed earlier, and mouse cursor moves over
+  // a different node with the same tooltip text, make sure the previous
+  // tooltip is unset, so that it does not get stuck positioned relative
+  // to the previous node).
+  // The ::setToolTip overload, which is be called down the road,
+  // ensures a new tooltip to be displayed with the new context.
+  if (result.innerNodeOrImageMapImage() != m_lastMouseOverNode &&
+      !m_lastToolTipText.isEmpty() && toolTip == m_lastToolTipText)
+    clearToolTip(frame);
+
   m_lastToolTipPoint = result.hitTestLocation().point();
   m_lastToolTipText = toolTip;
+  m_lastMouseOverNode = result.innerNodeOrImageMapImage();
   setToolTip(frame, toolTip, toolTipDirection);
 }
 
 void ChromeClient::clearToolTip(LocalFrame& frame) {
   // Do not check m_lastToolTip* and do not update them intentionally.
   // We don't want to show tooltips with same content after clearToolTip().
   setToolTip(frame, String(), TextDirection::Ltr);
 }
 
 bool ChromeClient::print(LocalFrame* frame) {
   if (frame->document()->isSandboxed(SandboxModals)) {
     UseCounter::count(frame, UseCounter::DialogInSandboxedContext);
     frame->console().addMessage(ConsoleMessage::create(
         SecurityMessageSource, ErrorMessageLevel,
         "Ignored call to 'print()'. The document is sandboxed, and the "
         "'allow-modals' keyword is not set."));
     return false;
   }
 
   // Suspend pages in case the client method runs a new event loop that would
   // otherwise cause the load to continue while we're in the middle of
   // executing JavaScript.
   ScopedPageSuspender suspender;
 
   printDelegate(frame);
   return true;
 }
 
 }  // namespace blink