Revert "Delay Session::exec() for 250 ms to allow the StackView transition to complete" due to regressing spinner performance on Windows and Linux

This reverts commit b8ffb6e53d.
This commit is contained in:
Cameron Gutman 2018-09-13 14:38:56 -07:00
parent b8ffb6e53d
commit 54885c3e5f

View file

@ -57,26 +57,6 @@ Item {
toast.visible = true
}
// This timer delay allows the transition
// to this StreamSegue to complete before doing
// long operations in Session::exec(). It would
// be nice if we could find a way to avoid having
// to do this though.
Timer {
id: delayedStartTimer
interval: 250
onTriggered: {
// Run the streaming session to completion
session.exec(Screen.virtualX, Screen.virtualY)
// Show the Qt window again after streaming
window.visible = true
// Exit this view
stackView.pop()
}
}
onVisibleChanged: {
if (visible) {
// Hide the toolbar before we start loading
@ -89,8 +69,14 @@ Item {
session.displayLaunchError.connect(displayLaunchError)
session.displayLaunchWarning.connect(displayLaunchWarning)
// Kick off the delayed start
delayedStartTimer.running = true
// Run the streaming session to completion
session.exec(Screen.virtualX, Screen.virtualY)
// Show the Qt window again after streaming
window.visible = true
// Exit this view
stackView.pop()
}
else {
// Show the toolbar again when we become hidden