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.
Comment on attachment 345448
Proposed patch making timeout user configurable
>+ if (sTooltipAutoHi deTimeListenerC ount++ == 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 istener: :TooltipAutoHid eTimePrefChange d(const char *aPref, eTime = nsContentUtils: :GetIntPref( "browser. preferences. tooltipHideTime ", eTime);
>+nsXULTooltipL
>+ void *aClosure)
>+{
>+ sTooltipAutoHid
>+ sTooltipAutoHid
Align the second line with the first, after the parenthesis.