Use #ifdef in a file to declare shortcut keys in dom/xbl/builtin instead of using ShortcutKeyDefinitions.cpp per platform

ASSIGNED
Assigned to

Status

()

Product:
Core
Component:
User events and focus handling
Type:
enhancement
Importance:
P3
normal
Status:
ASSIGNED
Reported:
6 months ago
Modified:
2 months ago

People

(Reporter: masayuki, Assigned: masayuki)

Tracking

Version:
Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details
Assignee

Description

•
6 months ago 
Because of the fix of bug 1419091, some shortcut keys (mainly for editor) are declared with array of C++ instead of XUL files. However, for making the review easier in the bug, platform specific shortcut keys are declared in separated ShortcutKeyDefinitions.cpp for each platform. Therefore, when I investigate shortcut keys, it's not unclear which shortcut key won't work in which platforms with synthesized key events, etc.

So, I'd like to combine them into a C++ file and using #ifdefs to enable/disable some of them only on specific platforms.
Priority: -- → P3
Component: Event Handling → User events and focus handling
Product: Core → Core
You need to log in before you can comment on or make changes to this bug.