header's timestamp can take a minute to update after resume from suspend
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| indicator-datetime (Ubuntu) |
Fix Released
|
Critical
|
Charles Kerr | ||
Bug Description
Not the same as the bug #1328646, where the clock fell completely out-of-sync and never came back.
This bug has a milder but similar symptom, the clock is set to update once per minute, so after resume from suspend it will still show the wrong time until that periodic update timer triggers.
In addition to the periodic update, indicator-datetime should listen to powerd's Wakeup signal, or better yet, to com.canonical.
Reported by ogra and dednick in #ubuntu-touch.
*edit:* also discussed by kgunn72 and aacid @ https:/
Related branches
- PS Jenkins bot (community): Approve (continuous-integration)
- Ted Gould (community): Approve
-
Diff: 514 lines (+313/-73)7 files modifiedinclude/datetime/clock-mock.h (+10/-2)
include/datetime/clock.h (+3/-6)
include/datetime/dbus-shared.h (+5/-0)
src/clock.cpp (+185/-46)
src/snap.cpp (+10/-1)
tests/manual (+7/-0)
tests/test-clock.cpp (+93/-18)
| Changed in indicator-datetime (Ubuntu): | |
| assignee: | nobody → Charles Kerr (charlesk) |
| importance: | Undecided → High |
| status: | New → Triaged |
| description: | updated |
| summary: |
- header's timestamp can take a minute to update after screen turned on + header's timestamp can take a minute to update after resume from suspend |
| description: | updated |
| description: | updated |
| Michał Sawicz (saviq) wrote | #1 |
| Michał Sawicz (saviq) wrote | #2 |
| Changed in indicator-datetime (Ubuntu): | |
| status: | Triaged → In Progress |
| tags: | added: rtm14 touch-2014-09-25 |
| Nick Dedekind (nick-dedekind) wrote | #3 |
| Changed in indicator-datetime (Ubuntu): | |
| importance: | High → Critical |
| Launchpad Janitor (janitor) wrote | #4 |
| Changed in indicator-datetime (Ubuntu): | |
| status: | In Progress → Fix Released |
You could probably listen to the powerd dbus interface for power changes and update on that.