Closed Bug 623063 Opened 9 years ago Closed 9 years ago

Remove useless <tooltip> from browser.xul

Categories

(Firefox for Android Graveyard :: General, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: fabrice, Assigned: fabrice)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch fix (obsolete) — Splinter Review
It's not used anywhere in m-b, and hidden by a css rule.
Attachment #501179 - Flags: review?(mark.finkle)
This was added by bug 528472 for desktop showing some tooltips. I'd be happy to remove it if we can find a different way to suppress tooltips. Maybe we can using a pref?

"browser.chrome.toolbar_tips" = false might do it:

http://mxr.mozilla.org/mozilla-central/source/layout/xul/base/src/nsXULTooltipListener.cpp#192
Attached patch fix v2Splinter Review
Setting browser.chrome.toolbar_tips to false indeed prevents the platform code from looking for a place to display tooltips.
Assignee: nobody → fabrice
Attachment #501179 - Attachment is obsolete: true
Attachment #501446 - Flags: review?(mark.finkle)
Attachment #501179 - Flags: review?(mark.finkle)
Attachment #501446 - Flags: review?(mark.finkle) → review+
pushed:
http://hg.mozilla.org/mobile-browser/rev/e32b5f783463
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Is there a way for us to verify this?
You need to log in before you can comment on or make changes to this bug.