App respawns if manually closed while it's launching

[Michael Terry]

On latest rc-proposed:

1) Launch an app
2) While it's launching, enter right-edge spread and swipe the spawning app up, causing it to close
3) Wait, and watch the app appear again

I've looked into qtmir slightly and I have a guess, but I'm not super familiar with the code and lifecycles. I'd *guess* that Application::onSessionStateChanged needs its final few lines to switch from:

        } else {
            setInternalState(InternalState::StoppedResumable);
        }

to

        } else if (m_state != InternalState::Stopped) {
            setInternalState(InternalState::StoppedResumable);
        }

But it needs more investigation.

Gerry said to pass on to Daniel.

current build number: 321
device name: krillin
channel: ubuntu-touch/rc-proposed/bq-aquaris.en
last update: 2016-04-27 02:59:26
version version: 321
version ubuntu: 20160427
version device: 20160329-a9bacdb
version custom: 20160324--36-54-vivid

[Daniel d'Andrada (dandrader)]

Can't reproduce on Nexus 7 (flo)

[Daniel d'Andrada (dandrader)]

Can't reproduce with krillin

[Michael Terry (mterry)]

I just now reproduced on krillin. :-/

[Daniel d'Andrada (dandrader)]

Providing unity8.log and recording a video would help.

[Michael Terry (mterry)]

Here's a log from a video that will also be uploaded shortly. Telegram is the program that I closed and then it came back. Note that this log also contains a (failed) reproduction for uNav before I did it successfully with Telegram.

[Michael Terry (mterry)]

And here's the video.

[Daniel d'Andrada (dandrader)]

Thanks. Will get back to it on Monday.

[Daniel d'Andrada (dandrader)]

Could you please try out my fix? It's pretty much what you proposed but I took the opportunity to refactor that code as well.

[Michael Terry (mterry)]

Tested the fix, works for me! Thanks.

[Launchpad Janitor (janitor)]

This bug was fixed in the package qtmir - 0.4.8+16.10.20160520.1-0ubuntu1

---------------
qtmir (0.4.8+16.10.20160520.1-0ubuntu1) yakkety; urgency=medium

  [ Alan Griffiths ]
  * Reduce coupling to MirServer - it has been misused as a context
    object.

  [ Daniel d'Andrada ]
  * Also interpret the cursor names used by Mir client API (LP:
    #1447839)
  * Application: Don't respawn if closed while still starting up (LP:
    #1575577)

  [ Daniel d'Andrada, Michał Sawicz ]
  * Centralize logging categories

  [ Gerry Boland ]
  * UAL can throw on creating Application if invalid appId, catch
    instead of aborting (LP: #1578258)

  [ Timo Jyrinki ]
  * Use FindQt5PlatformSupport to find platform support, other methods
    not available on Qt 5.6 anymore. (LP: #1554404)

 -- Michał Sawicz <email address hidden> Fri, 20 May 2016 08:41:51 +0000