From 6a5cd09de5458cd0f5bf443c5f6f29631e3646a8 Mon Sep 17 00:00:00 2001
From: "bzbarsky%mit.edu"
 <bzbarsky%mit.edu@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Sun, 18 Sep 2005 23:08:48 +0000
Subject: [PATCH] Correct tooltip coords in nsWebBrowser when frames are
 involved.  Bug 171229, r+sr=roc

git-svn-id: svn://10.0.0.236/trunk@180528 18797224-902f-48f8-a5cc-f745e15eee43
---
 .../embedding/browser/webBrowser/Makefile.in  |  1 +
 .../webBrowser/nsDocShellTreeOwner.cpp        | 49 +++++++++++++++++--
 .../browser/webBrowser/nsDocShellTreeOwner.h  |  3 +-
 3 files changed, 48 insertions(+), 5 deletions(-)

diff --git a/mozilla/embedding/browser/webBrowser/Makefile.in b/mozilla/embedding/browser/webBrowser/Makefile.in
index 85bc825e57a..e982241923e 100644
--- a/mozilla/embedding/browser/webBrowser/Makefile.in
+++ b/mozilla/embedding/browser/webBrowser/Makefile.in
@@ -68,6 +68,7 @@ REQUIRES	= xpcom \
 		  imglib2 \
 		  locale \
 		  embed_base \
+		  view \
 		  $(NULL)
 
 ifdef MOZ_PHOENIX
diff --git a/mozilla/embedding/browser/webBrowser/nsDocShellTreeOwner.cpp b/mozilla/embedding/browser/webBrowser/nsDocShellTreeOwner.cpp
index 926c209fc15..9af595de85a 100644
--- a/mozilla/embedding/browser/webBrowser/nsDocShellTreeOwner.cpp
+++ b/mozilla/embedding/browser/webBrowser/nsDocShellTreeOwner.cpp
@@ -91,6 +91,8 @@
 #include "imgIContainer.h"
 #include "nsContextMenuInfo.h"
 #include "nsPresContext.h"
+#include "nsIViewManager.h"
+#include "nsIView.h"
 
 //
 // GetEventReceiver
@@ -1306,6 +1308,8 @@ ChromeTooltipListener::MouseMove(nsIDOMEvent* aMouseEvent)
   if ( mMouseClientX == newMouseX && mMouseClientY == newMouseY )
     return NS_OK;
   mMouseClientX = newMouseX; mMouseClientY = newMouseY;
+  mouseEvent->GetScreenX(&mMouseScreenX);
+  mouseEvent->GetScreenY(&mMouseScreenY);
 
   // We want to close the tip if it is being displayed and the mouse moves. Recall 
   // that |mShowingTooltip| is set when the popup is showing. Furthermore, as the mouse
@@ -1415,8 +1419,40 @@ void
 ChromeTooltipListener::sTooltipCallback(nsITimer *aTimer,
                                         void *aChromeTooltipListener)
 {
-  ChromeTooltipListener* self = NS_STATIC_CAST(ChromeTooltipListener*, aChromeTooltipListener);
-  if ( self && self->mPossibleTooltipNode ) {
+  ChromeTooltipListener* self = NS_STATIC_CAST(ChromeTooltipListener*,
+                                               aChromeTooltipListener);
+  if ( self && self->mPossibleTooltipNode ){
+    // The actual coordinates we want to put the tooltip at are relative to the
+    // toplevel docshell of our mWebBrowser.  We know what the screen
+    // coordinates of the mouse event were, which means we just need the screen
+    // coordinates of the docshell.  Unfortunately, there is no good way to
+    // find those short of groveling for the presentation in that docshell and
+    // finding the screen coords of its toplevel widget...
+    nsCOMPtr<nsIDocShell> docShell =
+      do_GetInterface(NS_STATIC_CAST(nsIWebBrowser*, self->mWebBrowser));
+    nsCOMPtr<nsIPresShell> shell;
+    if (docShell) {
+      docShell->GetPresShell(getter_AddRefs(shell));
+    }
+
+    nsIWidget* widget = nsnull;
+    if (shell) {
+      nsIViewManager* vm = shell->GetViewManager();
+      if (vm) {
+        nsIView* view;
+        vm->GetRootView(view);
+        if (view) {
+          nsPoint offset;
+          widget = view->GetNearestWidget(&offset);
+        }
+      }
+    }
+
+    if (!widget) {
+      // release tooltip target if there is one, NO MATTER WHAT
+      self->mPossibleTooltipNode = nsnull;
+      return;
+    }
 
     // if there is text associated with the node, show the tip and fire
     // off a timer to auto-hide it.
@@ -1430,8 +1466,13 @@ ChromeTooltipListener::sTooltipCallback(nsITimer *aTimer,
       
       if (textFound) {
         nsString tipText(tooltipText);
-        self->CreateAutoHideTimer ( );  
-        self->ShowTooltip (self->mMouseClientX, self->mMouseClientY, tipText);
+        self->CreateAutoHideTimer();
+        nsRect widgetDot(0, 0, 1, 1);
+        nsRect screenDot;
+        widget->WidgetToScreen(widgetDot, screenDot);
+        self->ShowTooltip (self->mMouseScreenX - screenDot.x,
+                           self->mMouseScreenY - screenDot.y,
+                           tipText);
       }
     }
     
diff --git a/mozilla/embedding/browser/webBrowser/nsDocShellTreeOwner.h b/mozilla/embedding/browser/webBrowser/nsDocShellTreeOwner.h
index 908b9821610..f94e08647af 100644
--- a/mozilla/embedding/browser/webBrowser/nsDocShellTreeOwner.h
+++ b/mozilla/embedding/browser/webBrowser/nsDocShellTreeOwner.h
@@ -240,7 +240,8 @@ private:
 
   nsCOMPtr<nsITimer> mTooltipTimer;
   static void sTooltipCallback ( nsITimer* aTimer, void* aListener ) ;
-  PRInt32 mMouseClientX, mMouseClientY;       // mouse coordinates for tooltip event
+  PRInt32 mMouseClientX, mMouseClientY;       // mouse coordinates for last mousemove event we saw
+  PRInt32 mMouseScreenX, mMouseScreenY;       // mouse coordinates for tooltip event
   PRBool mShowingTooltip;
 
     // a timer for auto-hiding the tooltip after a certain delay