[regression] Qt.application.active not working correctly

Bug #1528668 reported by Tiago Salem Herrmann
50
This bug affects 14 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
Critical
Michał Sawicz
qtubuntu (Ubuntu)
Fix Released
Critical
Daniel d'Andrada

Bug Description

Steps to reproduce:

1) open dialer-app
2) lock the screen
3) unlock the screen

Expected behavior:
Qt.application.active == true

Current behavior
Qt.application.active == false until we manually switch the app to background/foreground again.

Additional info:
This is causing a bigger issue in the dialer-app functionality. It currently only refreshes the live call view when in foreground, so
when dialer is in this state, calls are placed but the call controls don't work (like hangup or speaker mode).

Related branches

Revision history for this message
Michał Sawicz (saviq) wrote :
Changed in unity8 (Ubuntu):
status: New → Triaged
importance: Undecided → Critical
assignee: nobody → Daniel d'Andrada (dandrader)
Changed in canonical-devices-system-image:
status: New → Triaged
importance: Undecided → Critical
assignee: nobody → Michał Sawicz (saviq)
milestone: none → ww02-2016
affects: unity8 (Ubuntu) → qtubuntu (Ubuntu)
Revision history for this message
Michał Sawicz (saviq) wrote : Re: Qt.application.active not working correctly

As per http://doc.qt.io/qt-5/qml-qtqml-qt.html#application-prop

application.active Deprecated, use Qt.application.state == Qt.ApplicationActive instead.

We should not have changed the behaviour of the .active property though.

summary: - dialer-app not getting focused after screen unlock
+ Qt.application.active not working correctly
summary: - Qt.application.active not working correctly
+ [regression] Qt.application.active not working correctly
Michał Sawicz (saviq)
Changed in canonical-devices-system-image:
status: Triaged → In Progress
Revision history for this message
Michał Sawicz (saviq) wrote :

Adding this code:

  Text {
    text: "state: " + Qt.application.state + "\nactive: " + Qt.application.active
    onTextChanged: console.debug(text)
  }

At the end of /usr/share/dialer-app/dialer-app.qml indeed does show trouble, the application remains in "Inactive" state when in foreground (and active remains false).

I'll be reverting that change for now.

Changed in qtubuntu (Ubuntu):
status: Triaged → In Progress
Revision history for this message
Andrew Hayzen (ahayzen) wrote :

I found an issue where it now takes the launcher two taps to dismiss, and wanted to check that this is the same issue?

What happened:
1) Swipe from left to right to show the launcher
2) Tap in the dimmed/shadow portion (right side) of the screen
3) Notice nothing happens, tap again and notice the launcher is dismissed.

What was expected:
At step 2 for the launcher to be dismissed

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package qtubuntu - 0.62+16.04.20160104-0ubuntu1

---------------
qtubuntu (0.62+16.04.20160104-0ubuntu1) xenial; urgency=medium

  [ Daniel d'Andrada ]
  * Fix QGuiApplication::applicationState() updates (LP: #1528668)

  [ Gerry Boland ]
  * On geometry change, save new geometry before creating change event
    (LP: #1526328)

 -- Michał Sawicz <email address hidden> Mon, 04 Jan 2016 09:38:35 +0000

Changed in qtubuntu (Ubuntu):
status: In Progress → Fix Released
Michał Sawicz (saviq)
Changed in canonical-devices-system-image:
status: In Progress → Fix Committed
Revision history for this message
Cris Dywan (kalikiana) wrote :

What about test cases? The dialer tests passed despite the bug when I was checking this. Does any existing test catch it - which would mean either the bug was introduced manually or the test plan is complete?

Revision history for this message
Michał Sawicz (saviq) wrote : Re: [Bug 1528668] Re: [regression] Qt.application.active not working correctly

> What about test cases? The dialer tests passed despite the bug when I
> was checking this. Does any existing test catch it - which would mean
> either the bug was introduced manually or the test plan is complete?

We will add a manual test case for this and will think with high prio on
how to test it automagically (might need to be autopilot, not sure yet).

Changed in canonical-devices-system-image:
status: Fix Committed → Fix Released
Revision history for this message
Tobias Stollenwerk (tobias-stollenwerk) wrote :

For me, it seems the problem is still there in OTA-9.

I reported this bug:

https://bugs.launchpad.net/ubuntu-ux/+bug/1512327

which was marked as a duplicate for this one.

Since the status is "Fix Released" I was wondering if the fix was released in OTA-9.

My System is the following: (I will translate some descriptions from german):
- Hardware: BQ Aquaris 4.5 Ubuntu Version

From Settings->Device Information-> Operating System :
- OS-Build Number: OTA-9
- Ubuntu-Image-Part: 20160123.1
- Ubuntu-Build-Description: Ubuntu 15.04 -armhf (20160123-115651)
- Device-Image-Part: 20160108-efc96d8
- Device-Builde-Description: KRILIN01A-S15A_BQ_L100EN_2029_160123
- Adaption-Image-Part: 20160111-926-36-vivid

Revision history for this message
Michał Sawicz (saviq) wrote :

@Tobias please comment on the other bug, then, and remove the duplicate link, must be a separate issue after all.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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