Orientation sensor is kept on even when the display is off
Bug #1375297 reported by
Daniel d'Andrada
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Canonical System Image |
Fix Released
|
High
|
Canonical Devices Products | ||
| qtmir (Ubuntu) |
Fix Released
|
High
|
Josh Arenson | ||
| qtmir (Ubuntu RTM) |
Fix Released
|
High
|
Michał Sawicz | ||
Bug Description
The orientation sensor is kept on even when the display is off.
Steps to reproduce:
1 - make sure qtmir logging is all enabled (it currently is by default)
2 - turn off the display
3 - rotate the device
Expected outcome:
Nothing new gets printed in unity8.log
Actual outcome:
You get output like this:
"""
qtmir.sensor: Screen:
qtmir.sensor: Screen::customEvent - new orientation 8 handled
"""
This bug has a FIXME item in qtmir code, in src/platforms/
"""
// FIXME: nothing is using this method yet, but we should turn off sensors when display is off.
void Screen:
"""
Related branches
lp://staging/~josharenson/qtmir/disable_orientation_sensors_based_on_screen_state
- Gerry Boland (community): Approve
- Daniel d'Andrada (community): Approve
- PS Jenkins bot (community): Approve (continuous-integration)
-
Diff: 192 lines (+118/-3)5 files modifiedsrc/platforms/mirserver/screen.cpp (+28/-2)
src/platforms/mirserver/screen.h (+8/-0)
tests/mirserver/Screen/Screen.pro (+16/-0)
tests/mirserver/Screen/screen_test.cpp (+65/-0)
tests/mirserver/mirserver.pro (+1/-1)
lp://staging/~mir-team/qtmir/rtm-14.09-staging
- Michał Sawicz: Approve
-
Diff: 261 lines (+121/-6)10 files modifieddebian/changelog (+13/-0)
debian/control (+1/-1)
src/modules/Unity/Application/session.cpp (+14/-1)
src/platforms/mirserver/promptsessionlistener.cpp (+12/-0)
src/platforms/mirserver/promptsessionlistener.h (+4/-0)
src/platforms/mirserver/qteventfeeder.cpp (+7/-0)
tests/modules/SessionManager/session_test.cpp (+58/-0)
tests/modules/common/mock_mir_session.h (+3/-1)
tests/modules/common/mock_prompt_session.h (+5/-0)
tests/modules/common/mock_prompt_session_manager.h (+4/-3)
| tags: | added: rtm14 |
Revision history for this message
| Victor Tuson Palau (vtuson) wrote | #1 |
Revision history for this message
| Daniel d'Andrada (dandrader) wrote | #2 |
| Changed in qtmir: | |
| importance: | Undecided → High |
| tags: | added: touch-2014-11-06 |
| Changed in qtmir: | |
| assignee: | nobody → Josh Arenson (josharenson) |
| Changed in qtmir: | |
| status: | New → Confirmed |
| Changed in qtmir: | |
| status: | Confirmed → In Progress |
| tags: |
added: ota-1 removed: touch-2014-11-06 |
| Changed in qtmir (Ubuntu): | |
| assignee: | nobody → Josh Arenson (josharenson) |
| Changed in qtmir (Ubuntu RTM): | |
| assignee: | nobody → Josh Arenson (josharenson) |
| Changed in qtmir (Ubuntu): | |
| importance: | Undecided → High |
| Changed in qtmir (Ubuntu RTM): | |
| importance: | Undecided → High |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #3 |
| Changed in qtmir (Ubuntu): | |
| status: | New → Fix Released |
| Changed in qtmir (Ubuntu RTM): | |
| status: | New → In Progress |
| assignee: | Josh Arenson (josharenson) → Michał Sawicz (saviq) |
| Changed in canonical-devices-system-image: | |
| assignee: | nobody → Canonical Devices Products (canonical-devices-products-team) |
| importance: | Undecided → High |
| milestone: | none → r1 |
| status: | New → Confirmed |
| Changed in qtmir (Ubuntu RTM): | |
| status: | In Progress → Triaged |
| assignee: | Michał Sawicz (saviq) → nobody |
| milestone: | none → 14.09-ota-1 |
| Changed in qtmir: | |
| status: | In Progress → Fix Released |
| Changed in qtmir (Ubuntu RTM): | |
| status: | Triaged → In Progress |
| assignee: | nobody → Michał Sawicz (saviq) |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #4 |
| Changed in qtmir (Ubuntu RTM): | |
| status: | In Progress → Fix Released |
| Changed in canonical-devices-system-image: | |
| status: | Confirmed → Fix Released |
| no longer affects: | qtmir |
To post a comment you must log in.
kevin - I imagine this stop sensors from turning off and hence has an impact on battery life? If so I think this should be a high bug