Merge lp:~agateau/unity-2d/wait-for-end-session into lp:unity-2d/3.0
Proposed by
Aurélien Gâteau
Status: | Merged |
---|---|
Approved by: | Aurélien Gâteau |
Approved revision: | 468 |
Merged at revision: | 474 |
Proposed branch: | lp:~agateau/unity-2d/wait-for-end-session |
Merge into: | lp:unity-2d/3.0 |
Diff against target: |
117 lines (+51/-8) 2 files modified
libunity-2d-private/src/gnomesessionclient.cpp (+50/-8) libunity-2d-private/src/gnomesessionclient.h (+1/-0) |
To merge this branch: | bzr merge lp:~agateau/unity-2d/wait-for-end-session |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Aurélien Gâteau (community) | Approve | ||
Bill Filler (community) | Needs Fixing | ||
Review via email: mp+54320@code.launchpad.net |
Commit message
[session] Avoid long delays on logout
Make sure we wait for gnome-session to call us back before we leave the event-loop.
Description of the change
Avoid long delays on logout
Make sure we wait for gnome-session to call us back before we leave the event-loop.
To post a comment you must log in.
tested and works great. I just made a couple of slight changes that you should consider:
See my branch here for diff
lp:~bfiller/unity-2d/wait-for-end-session
1) in queryEndSession() I set m_waitingForEnd Session = true as the first thing and only set it to false if an error occurs sending the response.
2) in endSession() don't set the m_waitingForEnd Session = false until after we send response
after merging these changes it's good to go unless you have comments