Comment 7 for bug 430203

As best I can read the code, this rate limiting in metacity affects audible and visual bells identically. Thus, this change should allow metacity to flash a visible bell at 10 Hz. (I can't figure out how to enable the visual bell in 9.10, so I can't actually test this.) This seems like a bad idea, especially where epileptics are concerned.

In my uniformed opinion, the responsibility for making system bell sounds should be moved out of metacity. Then the rate limiting can be left at 1Hz for the visual bell, while the audible bell can repeat as fast as possible. A patch I just submitted to bug #486154 removes metacity's ability to capture audible bell events, leaving the PC speaker or pulse audio's module-x11-bell with the responsibility for handling audible bell events. That could be a solution to this bug as well.