Signal the main window about vnc disconnection
Needs ReviewPublic

Authored by yuyichao on Wed, May 20, 5:04 AM.

Details

Summary

When the client is quitted and have stopped trying to reconnect.

Previously, a failed connection will leave the main window in a "connected" state
and require manually pressing the "Disconnect" button before a new connection can be made.

Use of the disconnected signal copied from the RDP backend, which handles this correctly.

Test Plan

Tested behavior on 20.04 branch. Tested compilation also on master.

Diff Detail

Repository
R436 KRDC
Branch
vnc-quit (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 27038
Build 27056: arc lint + arc unit
yuyichao requested review of this revision.Wed, May 20, 5:04 AM
yuyichao created this revision.
yuyichao updated this revision to Diff 83081.Wed, May 20, 5:38 AM

Include more disconnection cases than the ones from the vnc library.

yuyichao updated this revision to Diff 83082.Wed, May 20, 5:41 AM

Add comment about preserving error messages for the user.

uwolfer added inline comments.Mon, May 25, 6:09 PM
vnc/vncview.cpp
208

this signal was never emitted before in this class, right? are you sure you need to emit this manually?

222

is this required? m_quitFlag is initialized with false at the top (m_quitFlag(false),).

yuyichao marked 2 inline comments as done.Mon, May 25, 8:12 PM
yuyichao added inline comments.
vnc/vncview.cpp
208

Yes, this must be emitted manually and all other backends (rdp and the unfunctional nx) does this. This never done in this class is exactly the cause for the bug I mentioned in the description.

222

See the comment I added for this. I can also make this an assert.