Differential D9402

Fix error in WaylandSurface::testDisconnect
ClosedPublic
Actions

Authored by davidedmundson on Dec 18 2017, 10:38 PM.

Details

Reviewers
graesslin
Group Reviewers
Plasma
Commits
R127:edefb5629f6d: Fix error in WaylandSurface::testDisconnect
Summary

This test deletes the client wayland connection.

We then finish the test and call test cleanup which
releases m_idleInhititManager.
Calling release after the connection is destroyed is an error.

Hence the explicit destroy.

Test Plan

Built tests with ASAN
No more crash

Diff Detail

Repository
R127 KWayland
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Restricted Application added projects: Plasma on Wayland, Frameworks. · View Herald TranscriptDec 18 2017, 10:38 PM
Restricted Application added subscribers: Frameworks, plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.Dec 18 2017, 10:38 PM
graesslin accepted this revision.Dec 19 2017, 5:16 AM
This revision is now accepted and ready to land.Dec 19 2017, 5:16 AM
Restricted Application edited projects, added Plasma; removed Plasma on Wayland. · View Herald TranscriptDec 19 2017, 5:16 AM
Closed by commit R127:edefb5629f6d: Fix error in WaylandSurface::testDisconnect (authored by davidedmundson). · Explain WhyDec 19 2017, 7:45 AM
This revision was automatically updated to reflect the committed changes.
Restricted Application edited projects, added Plasma on Wayland; removed Plasma. · View Herald TranscriptDec 19 2017, 7:45 AM

Revision Contents
Changeset List

Diff 24088

View Options

autotests/client/test_wayland_surface.cpp

Loading...