repeating phantom key when bt disconnects
Bug #1550050 reported by
kevin gunn
This bug affects 3 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Canonical Pocket Desktop |
Fix Committed
|
Critical
|
kevin gunn | ||
| Canonical System Image |
Fix Released
|
Critical
|
kevin gunn | ||
| Mir |
Fix Released
|
Critical
|
Andreas Pokorny | ||
| 0.20 |
Fix Released
|
Critical
|
Andreas Pokorny | ||
| bluez (Ubuntu) |
Invalid
|
Undecided
|
Simon Fels | ||
| mir (Ubuntu) |
Fix Released
|
Critical
|
Andreas Pokorny | ||
Bug Description
Every now and then when having bt mouse/keyboard connected - you will see when selecting a text box there is a repeating character (there is no specific character, but always repeats)
It seems to be related somehow to bt devices disconnecting
the only cure is a reboot
This was reproduced by turning off the keyboard while holding down a key. We suspect the key up event is never delivered. This mimics the case where the BT connection drops off momentarily or perhaps an event is missed over the BT connection.
Related branches
lp://qastaging/~andreas-pokorny/mir/0.20-fix-1550050
- Mir CI Bot: Needs Fixing (continuous-integration)
- Kevin DuBois (community): Approve
-
Diff: 423 lines (+219/-53)6 files modifiedinclude/test/mir/test/doubles/mock_input_device_hub.h (+43/-0)
src/server/input/default_configuration.cpp (+37/-24)
src/server/input/default_input_device_hub.cpp (+14/-8)
src/server/input/key_repeat_dispatcher.cpp (+41/-0)
src/server/input/key_repeat_dispatcher.h (+7/-4)
tests/unit-tests/input/test_key_repeat_dispatcher.cpp (+77/-17)
lp://qastaging/~andreas-pokorny/mir/rebase-fix-1550050
- Alan Griffiths: Approve
- Mir CI Bot: Needs Fixing (continuous-integration)
- Andreas Pokorny (community): Needs Information
- Daniel van Vugt: Abstain
- Cemil Azizoglu (community): Approve
-
Diff: 343 lines (+153/-31)5 files modifiedsrc/server/input/default_configuration.cpp (+14/-2)
src/server/input/default_input_device_hub.cpp (+14/-8)
src/server/input/key_repeat_dispatcher.cpp (+41/-0)
src/server/input/key_repeat_dispatcher.h (+7/-4)
tests/unit-tests/input/test_key_repeat_dispatcher.cpp (+77/-17)
| Changed in canonical-devices-system-image: | |
| assignee: | nobody → John McAleely (john.mcaleely) |
| description: | updated |
| description: | updated |
| Changed in canonical-devices-system-image: | |
| status: | New → Confirmed |
| importance: | Undecided → Critical |
| milestone: | none → ww08-2016 |
| Changed in bluez (Ubuntu): | |
| status: | New → Confirmed |
| assignee: | nobody → Simon Fels (morphis) |
Revision history for this message
| Konrad Zapałowicz (kzapalowicz) wrote | #1 |
| tags: | added: ubuntu-classic |
| tags: | added: ubuntu-touch |
Revision history for this message
| Simon Fels (morphis) wrote | #2 |
Revision history for this message
| Simon Fels (morphis) wrote | #3 |
Revision history for this message
| Andreas Pokorny (andreas-pokorny) wrote | #5 |
| Changed in mir (Ubuntu): | |
| status: | New → Confirmed |
| assignee: | nobody → Andreas Pokorny (andreas-pokorny) |
| status: | Confirmed → In Progress |
| Changed in mir: | |
| status: | New → In Progress |
| assignee: | nobody → Andreas Pokorny (andreas-pokorny) |
| Changed in canonical-pocket-desktop: | |
| importance: | Undecided → Critical |
| assignee: | nobody → kevin gunn (kgunn72) |
| Changed in mir: | |
| importance: | Undecided → Critical |
| Changed in mir (Ubuntu): | |
| importance: | Undecided → Critical |
| Changed in mir: | |
| milestone: | none → 0.21.0 |
Revision history for this message
| Andreas Pokorny (andreas-pokorny) wrote | #6 |
| Changed in mir: | |
| status: | In Progress → Triaged |
Revision history for this message
| Simon Fels (morphis) wrote | #7 |
| Changed in canonical-devices-system-image: | |
| status: | Confirmed → In Progress |
| assignee: | John McAleely (john.mcaleely) → kevin gunn (kgunn72) |
| Changed in mir: | |
| status: | Triaged → In Progress |
Revision history for this message
| Simon Fels (morphis) wrote | #8 |
| Changed in bluez (Ubuntu): | |
| status: | Confirmed → Invalid |
| Changed in mir (Ubuntu): | |
| status: | In Progress → Fix Released |
| Changed in canonical-devices-system-image: | |
| status: | In Progress → Fix Committed |
| Changed in canonical-pocket-desktop: | |
| status: | New → Fix Committed |
Revision history for this message
| Daniel van Vugt (vanvugt) wrote | #9 |
| Changed in mir: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| KasperLau (liuxuan7000) wrote | #10 |
| Changed in mir: | |
| status: | Fix Committed → Fix Released |
| Changed in canonical-devices-system-image: | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| Venkat Satish BV (venkatsatish061) wrote | #11 |
To post a comment you must log in.
From a quick assessment I agree that the root cause might be a lack of "key up" event as this would explain the repetitions.