Bug 26237

Summary: Improve the device event controller code.
Product: at-spi2 Reporter: Mark Doffman <mark.doffman>
Component: coreAssignee: Mark Doffman <mark.doffman>
Status: NEW --- QA Contact:
Severity: enhancement    
Priority: low    
Version: unspecified   
Hardware: Other   
OS: All   
i915 platform: i915 features:

Description Mark Doffman 2010-01-26 03:37:06 UTC
The DeviceEventController code of the registryd is in severe need of a Re-factoring.

The device event controller has the following functionality.

1. Converting and forwarding X events on to AT's
2. Registering Keystroke listeners and delivering keystrokes from applications to AT's
   - This requires taking KeyGrabs using X, and keeping track of the KeyGrabs.
3. Polling for mouse movement events and forwarding them on to AT's
4. Providing a D-Bus interface to clients for all its functionality.

All of these complicated features are implemented inside one 3000 line file, deviceventcontroller.c. It is almost impossible to see what is going on, and in the future, when we may wish to make changes to the keystroke delivery, we will run in-to great difficulty.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.