U1 login window opens behind pay-ui window

Bug #1586219 reported by Selene ToyKeeper
18
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
Critical
Michał Sawicz
qtmir (Ubuntu)
Invalid
Critical
Daniel d'Andrada
unity8 (Ubuntu)
Fix Released
Critical
Daniel d'Andrada

Bug Description

In today's phone images, a very basic test case fails: Purchasing an app on a freshly-flashed phone.

Specifically, after tapping the app's price button to initiate a purchase, the pay-ui's "Loading..." dialog appears and never disappears or times out. If I hit its "Cancel" button, I see the U1 login window. Completing the login then returns the view to the app's summary screen in the click store.

What's supposed to happen is:
- Tap the price button.
- Get a U1 login window.
- Complete the login.
- Get a payment confirmation window.

I'm not completely sure what causes this, but it looks like the U1 login window is simply appearing underneath the pay-ui dialog instead of above it. Dobey found this error in the logs:
2016-05-26 20:34:53,467 - WARNING - QWindow::fromWinId(): platform plugin does not support foreign windows.

This should probably be fixed before OTA12, since it's a step backward in the first-time app purchase flow.

Related branches

Changed in canonical-devices-system-image:
status: New → Confirmed
Revision history for this message
Alberto Mardegan (mardy) wrote :

I don't think that the log message is that relevant, since qtmir never supported foreign windows.
This looks like a mir/qtmir/unity8 issue in handling trusted sessions (unless something has changed in pay-service?).
Tentatively assigning to qtmir for investigation.

affects: ubuntu-system-settings-online-accounts (Ubuntu) → qtmir (Ubuntu)
Revision history for this message
Alberto Mardegan (mardy) wrote :

I also reproduced the issue. Indeed I suspect that Selene is right and that the U1 login window is opening, but behind the "Loading" prompt. Attaching the logs from online-accounts-service; despite the warnings, which are "normal" in this situation, the U1 login seems to be successfully loaded.

Revision history for this message
Alberto Mardegan (mardy) wrote :

I've been bisecting the rc-proposed images to find where the bug was introduced (arbitrarily assuming that this is a 100% reproducible issue):
Working images: 435,440,442; failing images: 445,443

So, it seems that the bug was introduced in image 443, which carries these changes:
http://people.canonical.com/~lzemczak/landing-team/ubuntu-touch/rc-proposed/ubuntu/mako/443.commitlog

The best candidates for investigation are qtmir and unity8:
======
  * Move prompt surfaces from MirSurface to Application and emit
    firstChanged signal
======

Adding unity8 too.

Changed in canonical-devices-system-image:
assignee: nobody → Michał Sawicz (saviq)
importance: Undecided → Critical
milestone: none → 12
Michał Sawicz (saviq)
Changed in canonical-devices-system-image:
status: Confirmed → Triaged
Changed in qtmir (Ubuntu):
importance: Undecided → Critical
Changed in unity8 (Ubuntu):
importance: Undecided → Critical
Changed in qtmir (Ubuntu):
status: New → Triaged
Changed in unity8 (Ubuntu):
status: New → Triaged
Changed in qtmir (Ubuntu):
assignee: nobody → Daniel d'Andrada (dandrader)
Changed in unity8 (Ubuntu):
assignee: nobody → Daniel d'Andrada (dandrader)
Changed in qtmir (Ubuntu):
status: Triaged → In Progress
Changed in unity8 (Ubuntu):
status: Triaged → In Progress
Changed in qtmir (Ubuntu):
status: In Progress → Invalid
Changed in canonical-devices-system-image:
status: Triaged → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 8.12+16.10.20160617-0ubuntu1

---------------
unity8 (8.12+16.10.20160617-0ubuntu1) yakkety; urgency=medium

  [ Michael Terry ]
  * Do not create Label/Icon if we don't need it
  * Fix Z-ordering of prompt surfaces (LP: #1586219)
  * Drop bottom edge tutorials completely and small fixes for other
    edges. (LP: #1565103)
  * Redo visuals for greeter prompt box. (LP: #1435923)
  * Make tutorial qmltests more robust. (LP: #1590810)
  * Stop showing an extra lockscreen in desktop unity8 sessions. (LP:
    #1582987)

  [ Albert Astals Cid ]
  * Improve dash plugin mock
  * Stabilize PreviewExpandableTest::test_all_widgets_height
  * Workaround QTBUG-53460
  * Split artShape code for cardTool card and regular card
  * Simplify the code since fixedArtSize code is mandatory for regular
    cards
  * Desktop stage: Support rotatesWindowContents (LP: #1547481)
  * Turn if if else into if elseif else
  * Fix leaks in QInputDeviceManagerPrivate::addDevice
  * Do not call |= on uninitialized sigterm.sa_flags
  * initialize udevMonitor
  * Make notifier and notifierFd function local variables
  * Fix crash if a component that is not an Item is given to
    sectionDelegate
  * Tests: Do not use dynamic_cast if we're not going to test it anyway
  * Minor coverity fixes
  * Fix warning about anchors.fill in ProportionalShape (LP: #1590963)
  * Make wait in cleanup() wait for snapTo animation to have finished

  [ Alexandros Frantzis ]
  * Improve the way notifications interact with power management (LP:
    #1570922)

  [ Andrea Cimitan ]
  * Correctly refresh cardcreator cache on artshapestyle changes (LP:
    #1588335)
  * Same tweaks we have for click scope now for libertine
  * add a timer to pull to refresh to work on small windows (LP:
    #1589060)

  [ Daniel d'Andrada ]
  * Clean up debug leftovers
  * Ensure mouse and window movement are pixel-aligned (LP: #1510382)
  * Fix SpreadDelegate rotation animations
  * No point in version-controlling files that are automatically
    generated
  * Center cursor on screen when it's first shown
  * Custom cursor name just have to start with "custom"
  * Work around AnimatedSprite infinite loop bug (LP: #1588929)

  [ Josh Arenson ]
  * Disable showDash when it is already shown. (LP: #1506708)
  * Add a timestamp to the dash log (LP: #1580074)
  * Add timestamp to unity8 log (LP: #1498169)

  [ Lukáš Tinkl ]
  * Implement keyboard shortcuts info panel
  * Implement window controls overlay activatable with 3 fingers tap
    (LP: #1489020)
  * Select correct Chinese input method for OSK in the wizard (LP:
    #1588633)

  [ Michael Zanetti ]
  * Add support for the launcher surface pips displaying the correct
    number of surfaces
  * Add support for the persistent alert state. (LP: #1575147, #1584348,
    #1586621)
  * send a Alt-release event on alt+tab instead of delaying it
    completely and invoking it later (LP: #1565236)

  [ Stephen M. Webb ]
  * added a new upstart $SESSION for unity8-desktop-session (LP:
    #1376715)

 -- Michael Terry <email address hidden> Fri, 17 Jun 2016 01:22:57 +0000

Changed in unity8 (Ubuntu):
status: In Progress → Fix Released
Michał Sawicz (saviq)
Changed in canonical-devices-system-image:
status: In Progress → Fix Committed
Changed in canonical-devices-system-image:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.