FocusController::set_focus_to() no longer seems to raise a session to the top
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mir |
Fix Released
|
Medium
|
Alan Griffiths | ||
mir (Ubuntu) |
Fix Released
|
Medium
|
Unassigned |
Bug Description
I'm testing mir/devel with my split greeter branches which have multiple sessions. And I've noticed a regression this last week.
msh::FocusContr
On a related note, is there an easy way to raise a whole session without individually raising the surfaces (and without changing input focus like set_focus_to does)? Because I'm not sure when raising the surfaces inside a session if I've got the order right. (That is, I want to raise all of session B's surfaces above session A's, but keep their respective session-internal order.)
Related branches
- Daniel van Vugt: Approve
- PS Jenkins bot (community): Approve (continuous-integration)
- Alexandros Frantzis (community): Approve
- Chris Halse Rogers: Approve
-
Diff: 182 lines (+155/-1)3 files modifiedsrc/server/shell/default_focus_mechanism.cpp (+1/-1)
tests/integration-tests/CMakeLists.txt (+1/-0)
tests/integration-tests/session_management.cpp (+153/-0)
Changed in mir (Ubuntu): | |
assignee: | nobody → Alan Griffiths (alan-griffiths) |
status: | New → In Progress |
Changed in mir: | |
status: | New → In Progress |
assignee: | nobody → Alan Griffiths (alan-griffiths) |
Changed in mir (Ubuntu): | |
status: | In Progress → Triaged |
status: | Triaged → Confirmed |
assignee: | Alan Griffiths (alan-griffiths) → nobody |
Changed in mir: | |
milestone: | none → 0.1.9 |
summary: |
- Session raising problems + FocusController::set_focus_to() no longer seems to raise a session to + the top |
Changed in mir: | |
status: | Fix Committed → Fix Released |
Changed in mir: | |
importance: | Undecided → Medium |
Changed in mir (Ubuntu): | |
importance: | Undecided → Medium |
Changed in mir (Ubuntu): | |
status: | Confirmed → Triaged |
I'm not sure what's causing the observed symptoms - certainly calling set_focus_to() does seem to invoke SurfaceStack: :raise( ) as seen in the integration test added in the linked branch.