Last Comment Bug 360589 - add conditional testing for accesskey conflicts in release binaries
: add conditional testing for accesskey conflicts in release binaries
Status: RESOLVED WONTFIX
:
Product: Core
Classification: Components
Component: Keyboard: Navigation (show other bugs)
: Trunk
: All All
: -- normal (vote)
: ---
Assigned To: Nobody; OK to take it and work on it
:
Mentors:
http://lxr.mozilla.org/seamonkey/sour...
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2006-11-13 10:57 PST by Axel Hecht [:Pike]
Modified: 2006-11-27 15:47 PST (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments

Description Axel Hecht [:Pike] 2006-11-13 10:57:54 PST
It would be good to have the testing code for accesskey conflicts in release code so that localization teams and tests could check for that in release software or tinderbox configs.

It should probably the same code that is current DEBUG_jag and protected by a pref.
Comment 1 Axel Hecht [:Pike] 2006-11-27 15:47:24 PST
Actually, the most important part would be accesskeys in menus, and that's a tad tougher, and doesn't go through nsEventStateManager.cpp, but through layout in 
http://lxr.mozilla.org/mozilla/source/layout/xul/base/src/nsMenuPopupFrame.cpp#1521.

The rest is not really worth it, so I'm WONTFIXing this one.

Note You need to log in before you can comment on or make changes to this bug.