assert when tooltip placed on html document

RESOLVED FIXED in M16

Status

()

Core
XUL
P2
major
RESOLVED FIXED
18 years ago
18 years ago

People

(Reporter: Ben Goodger (use ben at mozilla dot org for email), Assigned: Mike Pinkerton (not reading bugmail))

Tracking

Trunk
x86
Windows NT
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

I've checked in some code to enable tooltips in navigator. Unfortunately there's 
a really, /really/ annoying assertion that just repeats over and over and 
over... (see summary). You don't get this assert if the first thing you do when 
navigator comes up is to mouse over something with a tooltip (e.g. reload 
button). 

the tooltip is attached to the <window> tag, with tooltip="aTooltip". Applying 
the tooltip popup to other elements like toolbox or box did not help.
(Assignee)

Comment 1

18 years ago
if it's really annoying, why did you check it in!?!?!? grrrr. now i'm going to 
get 500 bugs on it. *grumble*

i'll play around with it, i have yet to see this assert, nor has kathy. maybe 
it's a win32 problem for some reason.
Status: NEW → ASSIGNED
Target Milestone: --- → M17
(Assignee)

Comment 2

18 years ago
*grumble* *grumble*
Priority: P3 → P1
Target Milestone: M17 → M16
(Assignee)

Comment 3

18 years ago
ok, this is happening because the tooltip is looking for a xul document, but the 
mouse is over html content and finding an html document. i'll look into this. the 
temp fix is to not put a tooltip for an html document.

cc'ing hyatt
Priority: P1 → P2
(Assignee)

Comment 4

18 years ago
adjusting summary.
Summary: 'tooltip listener attached to an element that is not in XUL!' assertion on navigator window → assert when tooltip placed on html document
(Assignee)

Updated

18 years ago
Keywords: nsbeta2
(Assignee)

Comment 5

18 years ago
fixed. we look for the document using the node on which the popup listener is 
registered, not the event target. that way we always get a xul document.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.