75499 - Tooltips disappear immediately

Status: VERIFIED FIXED

(Core :: XUL, defect)

Description

[Henri Sivonen (:hsivonen)]

Steps to reproduce:
1) Hover the mouse cursor over an item that has a tooltip

Actual results:
A tooltip appears and disappears right away before a human can read it.

Expected results:
Expected the tooltip to stay visible until the mouse is moved again.

Comment 1

[Henri Sivonen (:hsivonen)]

And the build was: 2001-03-26 FizzillaCFM

Comment 2

[Peter Trudelle]

->pink

Comment 3

[Mike Pinkerton (not reading bugmail)]

moving my osx bugs to 0.9.2

Comment 4

[Adam]

Tooltips appear to work (sort of) in the 5/09 binary under Mac OS X, 
v10.0.3. They don't always appear the first time you mouse over 
something, but they usually do pop up and stay on the screen when they 
do. Only problem is they're not the right size and never in the right place 
(apparently, a lot of them appear off-screen). They do look nice with the 
drop shadow, though.

Comment 5

[Mike Pinkerton (not reading bugmail)]

that's probably because of the other popup/context menu issues we're having right 
now. once those are fixed, i expect it to revert to "normal" behavior.

Comment 6

[Mike Pinkerton (not reading bugmail)]

Attachment: [patch] use TickCount instead of event time.

Comment 7

[Mike Pinkerton (not reading bugmail)]

patch attached. it seems |event.when| is not what we think it should be on osx.
switching to using ::TickCount() seems to fix it and not cause any adverse
effects on os9. I tested

- timers in mail
- click hold context menus
- animated gifs
- tooltips timing out
- urlbar autocomplete widget

anything else i should test? 

Comment 8

[Mike Pinkerton (not reading bugmail)]

i put in some printf's on macos9 and we seem to always be within 5 ticks (5/60
sec) from event.when. Is that good enough?

Comment 9

[Simon Fraser [no longer active]]

Sounds good to me. We should probably try to use higher-resolution time values 
for timers anyway. Maybe pav's impl will fix that.

Comment 10

[Mike Pinkerton (not reading bugmail)]

r=pchen, can i get sr?

Comment 11

[Simon Fraser [no longer active]]

sr=sfraser

Comment 12

[Asa Dotzler [:asa]]

a= [e-mail] for checkin to the trunk.
(on behalf of drivers)

Comment 13

[Mike Pinkerton (not reading bugmail)]

fixed on trunk.

Comment 14

[Chris Petersen]

Marking verified in the June 5th Fizzilla build.