Trusted Prompt Sessions should be able to be on a Trusted Prompt Provider
Bug #1358388 reported by
Ted Gould
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| qtmir (Ubuntu) |
Fix Released
|
Critical
|
Nick Dedekind | ||
| unity8 (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Bug Description
For payments we create a trusted prompt session and then put the Payments UI as a prompt provider in that session. In most cases today this is the payments UI being overlayed on top of the dash. In the case that the user doesn't have a registered Ubuntu One account on the device they'll need to create/register one. In this situation we use Online Accounts to setup the account. Online Accounts then creates a trusted prompt session on top of Pay UI with the Ubuntu One account UI as the prompt provider. After the user creates/registers the account then we need to move back down the stack.
Related branches
lp:~nick-dedekind/qtmir/prompt-in-prompt
- Michał Sawicz: Approve (silo)
- PS Jenkins bot (community): Approve (continuous-integration)
- Daniel d'Andrada (community): Abstain
- Gerry Boland (community): Approve
-
Diff: 4382 lines (+2108/-1028)32 files modifieddebian/changelog (+9/-0)
src/modules/Unity/Application/Application.pro (+7/-0)
src/modules/Unity/Application/application.cpp (+60/-294)
src/modules/Unity/Application/application.h (+15/-56)
src/modules/Unity/Application/application_manager.cpp (+35/-79)
src/modules/Unity/Application/application_manager.h (+8/-8)
src/modules/Unity/Application/applicationscreenshotprovider.cpp (+4/-2)
src/modules/Unity/Application/dbuswindowstack.cpp (+1/-1)
src/modules/Unity/Application/mirsurfaceitem.cpp (+48/-128)
src/modules/Unity/Application/mirsurfaceitem.h (+11/-28)
src/modules/Unity/Application/mirsurfaceitemmodel.h (+32/-0)
src/modules/Unity/Application/mirsurfacemanager.cpp (+15/-193)
src/modules/Unity/Application/mirsurfacemanager.h (+14/-37)
src/modules/Unity/Application/objectlistmodel.h (+110/-0)
src/modules/Unity/Application/plugin.cpp (+16/-1)
src/modules/Unity/Application/session.cpp (+363/-0)
src/modules/Unity/Application/session.h (+136/-0)
src/modules/Unity/Application/sessionmanager.cpp (+208/-0)
src/modules/Unity/Application/sessionmanager.h (+90/-0)
src/modules/Unity/Application/sessionmodel.h (+32/-0)
src/platforms/mirserver/logging.h (+1/-0)
tests/modules/ApplicationManager/application_manager_test.cpp (+77/-199)
tests/modules/General/General.pro (+15/-0)
tests/modules/General/objectlistmodel_test.cpp (+132/-0)
tests/modules/SessionManager/SessionManager.pro (+17/-0)
tests/modules/SessionManager/session_manager_test.cpp (+124/-0)
tests/modules/SessionManager/session_test.cpp (+194/-0)
tests/modules/common/common.pri (+4/-1)
tests/modules/common/qtmir_test.h (+163/-0)
tests/modules/common/stub_input_channel.h (+59/-0)
tests/modules/common/stub_scene_surface.h (+107/-0)
tests/modules/modules.pro (+1/-1)
lp:~nick-dedekind/unity8/prompts-in-prompts
- Daniel d'Andrada (community): Approve
- PS Jenkins bot (community): Needs Fixing (continuous-integration)
- Michał Sawicz: Approve (silo)
- kevin gunn (community): Abstain
-
Diff: 3899 lines (+1838/-792)46 files modifieddebian/control (+1/-1)
plugins/AccountsService/AccountsService.qmltypes (+1/-0)
plugins/LightDM/LightDM.qmltypes (+2/-0)
plugins/Ubuntu/SystemImage/SystemImage.qmltypes (+2/-0)
plugins/Unity/Indicators/Indicators.qmltypes (+6/-4)
qml/Shell.qml (+9/-0)
qml/Stages/Animations/BaseSessionAnimation.qml (+25/-22)
qml/Stages/Animations/DarkenAndFadeInAnimation.qml (+0/-48)
qml/Stages/Animations/SwipeFromBottomAnimation.qml (+24/-19)
qml/Stages/Animations/SwipeUpAnimation.qml (+0/-49)
qml/Stages/ApplicationWindow.qml (+52/-72)
qml/Stages/SessionContainer.qml (+143/-0)
qml/Stages/SurfaceContainer.qml (+27/-72)
tests/mocks/AccountsService/AccountsService.qmltypes (+1/-0)
tests/mocks/LightDM/LightDM.qmltypes (+2/-0)
tests/mocks/Ubuntu/SystemImage/SystemImage.qmltypes (+3/-1)
tests/mocks/Unity/Application/Application.qmltypes (+122/-77)
tests/mocks/Unity/Application/ApplicationInfo.cpp (+42/-91)
tests/mocks/Unity/Application/ApplicationInfo.h (+11/-23)
tests/mocks/Unity/Application/ApplicationManager.cpp (+9/-16)
tests/mocks/Unity/Application/ApplicationManager.h (+1/-2)
tests/mocks/Unity/Application/ApplicationTestInterface.cpp (+81/-45)
tests/mocks/Unity/Application/ApplicationTestInterface.h (+13/-11)
tests/mocks/Unity/Application/CMakeLists.txt (+6/-1)
tests/mocks/Unity/Application/MirSurfaceItem.cpp (+29/-89)
tests/mocks/Unity/Application/MirSurfaceItem.h (+24/-31)
tests/mocks/Unity/Application/MirSurfaceItemModel.h (+35/-0)
tests/mocks/Unity/Application/ObjectListModel.h (+106/-0)
tests/mocks/Unity/Application/Session.cpp (+174/-0)
tests/mocks/Unity/Application/Session.h (+90/-0)
tests/mocks/Unity/Application/SessionManager.cpp (+55/-0)
tests/mocks/Unity/Application/SessionManager.h (+44/-0)
tests/mocks/Unity/Application/SessionModel.h (+35/-0)
tests/mocks/Unity/Application/SurfaceManager.cpp (+20/-7)
tests/mocks/Unity/Application/SurfaceManager.h (+6/-2)
tests/mocks/Unity/Application/plugin.cpp (+38/-5)
tests/mocks/Unity/Indicators/Indicators.qmltypes (+20/-0)
tests/mocks/Unity/Unity.qmltypes (+30/-2)
tests/qmltests/CMakeLists.txt (+1/-0)
tests/qmltests/Panel/tst_ActiveCallHint.qml (+1/-1)
tests/qmltests/Stages/RecursingChildSessionControl.qml (+142/-0)
tests/qmltests/Stages/tst_ApplicationWindow.qml (+100/-48)
tests/qmltests/Stages/tst_PhoneStage.qml (+1/-1)
tests/qmltests/Stages/tst_SessionContainer.qml (+267/-0)
tests/qmltests/Stages/tst_SurfaceContainer.qml (+28/-43)
tests/qmltests/tst_Shell.qml (+9/-9)
Revision history for this message
| Ted Gould (ted) wrote | #1 |
| Changed in qtmir: | |
| importance: | Undecided → Critical |
| tags: | added: rtm14 |
| Changed in qtmir: | |
| assignee: | nobody → Nick Dedekind (nick-dedekind) |
| Changed in qtmir: | |
| status: | New → In Progress |
| tags: | added: touch-2014-09-11 |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #2 |
| Changed in unity8 (Ubuntu): | |
| status: | New → Fix Released |
| Changed in qtmir: | |
| status: | In Progress → Fix Released |
| affects: | qtmir → qtmir (Ubuntu) |
To post a comment you must log in.
Marking as Critical since as once the Online Accounts trusted prompt session support migrates to the archive payments will be broken on the phone.