Closed Bug 1635728 Opened 5 years ago Closed 5 years ago

Move nsMenuFrame acceltext stuff to js

Categories

(Core :: XUL, task)

task

Tracking

()

RESOLVED FIXED
mozilla78
Tracking Status
firefox78 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

Details

Attachments

(1 file)

No description provided.

The place where it's done right now is just unsound. It modifies
attributes during frame construction and it causes assertions to fire
with lazy frame-construction enabled.

I removed support for the keytext attribute while at it because it was
unused, but lmk if you want to keep it.

Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4755418f8b4c Move nsMenuFrame::BuildAcceleratorText to JS. r=Gijs
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla78
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: