Ubuntu
firefox package

  1. Bug #239462
  2. Comment #26

Comment 26 for bug 239462

Revision history for this message
In , Enn (enndeakin) wrote :

Comment on attachment 345448
Proposed patch making timeout user configurable

>+ if (sTooltipAutoHideTimeListenerCount++ == 0) {

You don't need a separate 'count' variable here; just put the code in the same block as the other preference that is checked.

>+int
>+nsXULTooltipListener::TooltipAutoHideTimePrefChanged(const char *aPref,
>+ void *aClosure)
>+{
>+ sTooltipAutoHideTime = nsContentUtils::GetIntPref("browser.preferences.tooltipHideTime",
>+ sTooltipAutoHideTime);

Align the second line with the first, after the parenthesis.