PointerConfinement.test_we_update_our_confined_region_on_a_resize
Bug #1606418 reported by
Brandon Schaefer
This bug affects 3 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mir |
Fix Released
|
High
|
Brandon Schaefer | ||
mir (Ubuntu) |
Fix Released
|
High
|
Brandon Schaefer |
Bug Description
Randomly fails 1/100 times.
https:/
Looking into it, most likely a race with surface resize...
Related branches
lp:~brandontschaefer/mir/update-pointer-confinement-v2
- Mir CI Bot: Approve (continuous-integration)
- Alan Griffiths: Approve
- Andreas Pokorny (community): Approve
-
Diff: 1486 lines (+641/-86)43 files modifieddoc/component_reports.md (+1/-0)
include/server/mir/input/seat_report.h (+55/-0)
include/server/mir/scene/null_surface_observer.h (+0/-1)
include/server/mir/scene/surface_observer.h (+0/-1)
include/server/mir/server.h (+7/-0)
include/server/mir/shell/abstract_shell.h (+3/-1)
include/test/mir/test/doubles/mock_seat_report.h (+52/-0)
include/test/mir_test_framework/placement_applying_shell.h (+3/-0)
src/include/platform/mir/options/configuration.h (+1/-0)
src/include/server/mir/default_server_configuration.h (+3/-0)
src/include/server/mir/scene/surface_observers.h (+0/-1)
src/platform/options/default_configuration.cpp (+3/-0)
src/platform/symbols.map (+1/-0)
src/server/input/basic_seat.cpp (+10/-2)
src/server/input/basic_seat.h (+3/-1)
src/server/input/default_configuration.cpp (+2/-1)
src/server/input/seat_input_device_tracker.cpp (+37/-11)
src/server/input/seat_input_device_tracker.h (+9/-2)
src/server/report/default_server_configuration.cpp (+9/-0)
src/server/report/logging/CMakeLists.txt (+1/-0)
src/server/report/logging/logging_report_factory.cpp (+6/-0)
src/server/report/logging/seat_report.cpp (+146/-0)
src/server/report/logging/seat_report.h (+67/-0)
src/server/report/logging_report_factory.h (+1/-0)
src/server/report/lttng/lttng_report_factory.cpp (+5/-0)
src/server/report/lttng_report_factory.h (+1/-0)
src/server/report/null/CMakeLists.txt (+1/-0)
src/server/report/null/null_report_factory.cpp (+11/-0)
src/server/report/null/seat_report.cpp (+58/-0)
src/server/report/null/seat_report.h (+50/-0)
src/server/report/null_report_factory.h (+2/-0)
src/server/report/report_factory.h (+2/-0)
src/server/scene/basic_surface.cpp (+0/-21)
src/server/scene/legacy_surface_change_notification.cpp (+0/-4)
src/server/scene/legacy_surface_change_notification.h (+0/-1)
src/server/scene/null_surface_observer.cpp (+0/-1)
src/server/server.cpp (+1/-0)
src/server/shell/abstract_shell.cpp (+34/-12)
src/server/symbols.map (+1/-1)
tests/acceptance-tests/test_confined_pointer.cpp (+47/-23)
tests/integration-tests/input/test_single_seat_setup.cpp (+3/-1)
tests/mir_test_framework/placement_applying_shell.cpp (+1/-0)
tests/unit-tests/input/test_seat_input_device_tracker.cpp (+4/-1)
lp:~brandontschaefer/mir/fix-confined-region-test
- Daniel van Vugt: Approve
- Mir CI Bot: Approve (continuous-integration)
-
Diff: 59 lines (+18/-7)2 files modifiedinclude/test/mir/test/event_matchers.h (+5/-0)
tests/acceptance-tests/test_confined_pointer.cpp (+13/-7)
Changed in mir: | |
status: | New → In Progress |
importance: | Undecided → High |
assignee: | nobody → Brandon Schaefer (brandontschaefer) |
importance: | High → Medium |
tags: | added: testsfail |
Changed in mir: | |
milestone: | none → 0.25.0 |
Changed in mir: | |
milestone: | none → 0.26.0 |
Changed in mir: | |
importance: | Medium → High |
Changed in mir (Ubuntu): | |
status: | In Progress → Triaged |
importance: | Medium → High |
Changed in mir: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Seen again here: https:/ /mir-jenkins. ubuntu. com/job/ build-2- binpkg- mir/arch= amd64,compiler= clang,platform= mesa,release= vivid+overlay/ 1651/consoleFul l