Shutdown dialog appears on resume

[Michael Terry]

This is very similar to old friends bug 1383277 and bug 1410830.

But we never fully vanquished the beast. The dialog still appears on resume in some situations where the system is very loaded.
(also when not at all loaded)

I suspect what is happening is that we process the power-button-release event so much later than the -press event because of the load. And think that the user has been holding it down this whole time.

But instead, we should compare event timestamps to determine that. (Which I'm assuming we aren't doing because this bug exists.)

[Michael Terry (mterry)]

Specifically, the load seems to only matter for dbus (which makes sense, it would delay IPC). So this mostly comes up when we see bug 1480877.

The following line reproduces this bug fairly easily if you press the power button a few times after running it:
sudo cpulimit -l 1 -c 1 -p `pidof dbus-daemon`

I have a proposed fix, but want broader comment on whether it's crack or not.

[Michael Terry (mterry)]

Uh... Use the following command instead of pidof to get the pid:
ps ax | grep dbus-daemon | head -n 1 | awk '{print $1;}'

I forgot there are two dbus-daemons.

[Pat McGowan (pat-mcgowan)]

FWIW I reproduced getting the dialog on resume quite easily , I simply pressed the power button for more than 1 sec and less than 5, released it just before the dialog appeared. It was there when I resumed.
arale and krillin

[Jean-Baptiste Lallement (jibel)]

Fixed in unity8 (8.11+15.04.20151104-0ubuntu1)

[Launchpad Janitor (janitor)]

This bug was fixed in the package unity8 - 8.11+16.04.20151104-0ubuntu1

---------------
unity8 (8.11+16.04.20151104-0ubuntu1) xenial; urgency=medium

  [ Albert Astals Cid ]
  * Add an Item that proxies for old and new audio roles (LP: #1493851)
  * CroppedImageMinimumSourceSize: Fix 'Binding loop detected for
    property "imageAspectRatio"'
  * Make the scope settings list scroll to text visible area (LP:
    #1499084)
  * Move to importing QtQuick 2.4 & friends
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)

  [ Andrea Cimitan ]
  * Add sharing widget to zoomable image and video playback
  * Make zoomable and video playback widgets edge to edge
  * Updated all 1.3 UbuntuShape to use new APIs where possible
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)

  [ CI Train Bot ]
  * Resync trunk.

  [ Christopher Lee ]
  * Quick spelling fix in process control output.

  [ Daniel d'Andrada ]
  * Enable support for overriding application orientations based on
    device type (LP: #1478637)
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)

  [ Josh Arenson ]
  * Enable support for overriding application orientations based on
    device type (LP: #1478637)

  [ Michael Terry ]
  * Avoid showing the shutdown dialog when turning on the screen if your
    device is under heavy load. (LP: #1508563)
  * Handle lifecycle policy exceptions ourselves, instead of letting
    qtmir do it for us and allow non-Touch apps to opt-out of the Touch
    lifecycle.

  [ Michael Zanetti ]
  * Fixes for the panel buttons (LP: #1510360, #1504269, #1431566,
    #1443319)
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)

  [ Michał Sawicz ]
  * Add missing copyright to Cursor.qml
  * Enable support for overriding application orientations based on
    device type (LP: #1478637)
  * Support server->client visibility change to stop rendering
    (lp:#1475678) (LP: #1475678)

  [ Nick Dedekind ]
  * Support server->client visibility change to stop rendering
    (lp:#1475678) (LP: #1475678)

 -- Michał Sawicz <email address hidden> Wed, 04 Nov 2015 14:58:41 +0000