Use single AWTEventListener for key press/release and alt/shift/ctrl detection

[akks]

In #7991 and #3910 there was mentioned a problem: sometimes we need to detect repeated key press or release but do not want to receive events then key is hold. There are some workarounds using AWTEventListener that allow it but this should be synchronized and not copy-pasted to all mapmodes and plugins.

AWTEventListener was also widely used to detect changing of CTRL/ALT/SHIFT modifier keys.

The proposed change is to unify all this "advanced" key handling in separate class AdvancedKeyPressDetector and make it avalable as Main.map.keyDetector.

[akks]

In 7217/josm:

see #10104: refactor key press/release detection introducing Main.map.keyDetector

[akks]

It would be very good if someone can check that the behavior did not change on Linux and Mac .
(cursor changing on Alt-Shift-Ctrl could be affected for all modes and repeating&holding X and A)

Code improvements are also welcome.

[Don-vip]

The new file was forgotten in the commit :)

[akks]

Oops, sorry. 3 Files, actually :) 7219/josm

[Don-vip]

is there still something to do on this ticket?

[Don-vip]

In 7278/josm:

fix #10212, see #10104 - ImproveWayAccuracy: key listeners forgot to remove themselves

[akks]

In 7293/josm:

Fix resource leaks when closing map in Lasso mode, see #10104

[skyper]

Replying to akks:

In 7293/josm:

Fix resource leaks when closing map in Lasso mode, see #10104

Maybe this was the reason for #10233.