Ubuntu
firefox package

  1. Bug #239462
  2. Comment #64

Comment 64 for bug 239462

Revision history for this message
In , Neil-httl (neil-httl) wrote :

Comment on attachment 360518
Working patch addressing previous patch bug

>+ // flag specifying if the tooltip has already been displayed
>+ PRBool mTooltipShown;
It took me some time to work out what this really means... what you're trying to say here is that you only ever show the tooltip once for any given element while the mouse is hovering over it. You could do with a better comment here, and maybe rename the flag to mRecentTooltip or something slightly more relevant.