phone locks after switching away and back to active call

Bug #1388156 reported by Bill Filler
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
dialer-app (Ubuntu)
Fix Released
Undecided
Gustavo Pichorim Boiko
dialer-app (Ubuntu RTM)
Fix Released
Undecided
Gustavo Pichorim Boiko
unity8 (Ubuntu)
Fix Released
Critical
Michael Terry
unity8 (Ubuntu RTM)
Fix Released
Critical
Michał Sawicz

Bug Description

build 139 on Krillin

Steps to reproduce:
- unlock phone
- receive a call
- answer the call
- swipe away the dialer to the dash
- press the Active Call bar at the top to go back to call

Expected results:
- dialer is displayed, phone remains unlocked and indicator panel is visible

Actual results:
- dialer is displayed, but the indicator panel is not showing
- no way to get back to shell or other apps
- have to press back button in dialer which brings you to the greeter where you have to unlock the phone to continue

Related branches

Revision history for this message
Bill Filler (bfiller) wrote :

Other thing I noticed, which may be a separate bug:

- lock the phone
- receive a call on the phone and answer it
- dialer is shown, but the indicator panel is missing. It should be displayed even when the phone is locked.

Changed in unity8 (Ubuntu):
importance: Undecided → Critical
tags: added: rtm14
Michael Terry (mterry)
Changed in unity8 (Ubuntu):
assignee: nobody → Michael Terry (mterry)
Revision history for this message
Michael Terry (mterry) wrote :

That second issue, of the panel being missing -- the shell forces fullscreen mode and disables the indicators when in emergency mode. For two reasons:

1) Less security surface. The emergency mode (without an actual split greeter) is kind of a hack. We actually unlock phone and let the user in, but only for the one app. We try to treat it as if the greeter were up as much as possible. But just in case we screwed something up, we figured we were better safe than sorry and disabled the indicators and launcher in emergency mode.

2) Less distractions. When in emergency mode, we figured the risks of accidentally interacting with launcher or indicators when in a stressful situation were not worth it. Granted we use the same "emergency" mode when receiving a call which will not have the same considerations.

But the panel shouldn't be black. The dialer app is supposed to request fullscreen mode so that it takes up the whole screen.

Changed in unity8 (Ubuntu):
status: New → In Progress
Changed in unity8 (Ubuntu RTM):
assignee: nobody → Michael Terry (mterry)
importance: Undecided → Critical
status: New → In Progress
Changed in dialer-app (Ubuntu):
assignee: nobody → Gustavo Pichorim Boiko (boiko)
Revision history for this message
Olli Ries (ories) wrote :

https://code.launchpad.net/~boiko/dialer-app/fullscreen_emergency_mode is too big to be considered and seems to include more than the actual targeted fix. Can you pls clarify if we need to change all of the 3k LOC?

tags: added: ota-1
Revision history for this message
Bill Filler (bfiller) wrote :

The diff is a mistake as it incorrectly includes the pot file, updating now..

Changed in dialer-app (Ubuntu):
status: New → In Progress
Olli Ries (ories)
tags: added: touch-2014-11-06
removed: ota-1
Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

approved to land

Michał Sawicz (saviq)
Changed in unity8 (Ubuntu RTM):
assignee: Michael Terry (mterry) → Michał Sawicz (saviq)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 8.01+15.04.20141104-0ubuntu1

---------------
unity8 (8.01+15.04.20141104-0ubuntu1) vivid; urgency=low

  [ Sebastien Bacher ]
  * Drop workaround, it's not needed and create issues for desktop-next
    (LP: #1387671)

  [ Albert Astals ]
  * Do not append integer to char *
  * Remove unused end() function
  * Update build.sh
  * Fix i18n

  [ Michael Zanetti ]
  * rework launcher quicklist visuals to fit new design (LP: #1368660)
  * Fix DBusVariant conversion for launcher emblems (LP: #1387261)
  * Fix desktop file encoding in launcher (LP: #1387083)
  * Add a hinting animation to indicate changes in the backend (LP:
    #1376707)

  [ Alberto Aguirre ]
  * Add audioRole to QtMultimedia mock
  * Add a plugin to take screenshots on vol up + vol down

  [ CI bot ]
  * Fix relative time formatter when using non-UTC timezone. (LP:
    #1378821)

  [ Leo Arias ]
  * Removed the tests for the alternate paths of the autopilot helpers.

  [ Nick Dedekind ]
  * Fix relative time formatter when using non-UTC timezone. (LP:
    #1378821)
  * Reset current item selection on deleted. (LP: #1378462)
  * Added timer to re-assert server value after indicator menu item
    activation. (LP: #1336715)

  [ Mirco Müller ]
  * Added dedicated swipe-to-act button for snap-decisions, which avoids
    accidental taps/button-presses. (LP: #1358343)

  [ Chris Gagnon ]
  * add missing unity-scope-click dependancy to unity8-autopilot package
    (LP: #1336276)

  [ Michael Terry ]
  * Don't lock phone if user tries to switch back to an active call.
    (LP: #1388156)

  [ Daniel d'Andrada ]
  * DirectionalDragArea: Update TODO comment in light of new events
  * TouchDispatcher: synthesize MouseButtonDblClick events

  [ Leonardo Arias Fonseca ]
  * Removed the tests for the alternate paths of the autopilot helpers.
 -- Ubuntu daily release <email address hidden> Tue, 04 Nov 2014 12:56:08 +0000

Changed in unity8 (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 8.01.1+14.10.20141105.2~rtm-0ubuntu1

---------------
unity8 (8.01.1+14.10.20141105.2~rtm-0ubuntu1) 14.09; urgency=medium

  [ Albert Astals ]
  * Fix i18n (LP: #1389165)

  [ Michael Zanetti ]
  * Fix DBusVariant conversion for launcher emblems (LP: #1387261)
  * Fix desktop file encoding in launcher (LP: #1387083)

  [ Alberto Aguirre ]
  * Add audioRole to QtMultimedia mock
  * Add a plugin to take screenshots on vol up + vol down

  [ Nick Dedekind ]
  * Reset current item selection on deleted. (LP: #1378462)
  * Added timer to re-assert server value after indicator menu item
    activation. (LP: #1336715)

  [ Mirco Müller ]
  * Added dedicated swipe-to-act button for snap-decisions, which avoids
    accidental taps/button-presses. (LP: #1358343)

  [ Michael Terry ]
  * Don't lock phone if user tries to switch back to an active call.
    (LP: #1388156)

  [ Daniel d'Andrada ]
  * TouchDispatcher: synthesize MouseButtonDblClick events (LP:
    #1388359)

  [ Michał Sawicz ]
  * Bump the version above the version synced from vivid, otherwise
    train will generate a lower-than-archive version number.
  * fix positive/negative answer order on the new swipe notification

  [ Ubuntu daily release ]
  * New rebuild forced

unity8 (8.01+15.04.20141030-0ubuntu1) vivid; urgency=medium

  [ Michael Terry ]
  * Provide 'passphrase' as a field of Components.Lockscreen
  * Fix a race between Qml loading and DBus registration that caused
    problems when jenkins tried to unlock the phone.
  * Set domain explicitly for the Dialogs component because the welcome
    wizard wants to import it. (LP: #1381731)
  * When greeter or lockscreen has focus, show active call panel. (LP:
    #1378872)

  [ Ted Gould ]
  * Set the default OOM score for the dash (LP: #1379786)

  [ Michał Sawicz ]
  * Revert lp:~unity-team/unity8/flickables-speed-workaround to avoid
    risk in RTM.
  * Updated behaviour for zoomable image, workaround for sourcesize (LP:
    #1333187)

  [ Michael Frey ]
  * Added a check for Proximity to determine if we show the lock screen.
    (LP: #1378012)

  [ Ying-Chun Liu ]
  * Add non-interactive code into GenericScopeView. (LP: #1384441)

  [ Mirco Müller ]
  * Also use modal nature of snap-decision notifications when
    greeter/lockscreen is shown. This fixes LP: #1378827. (LP: #1378827)

  [ Michael Zanetti ]
  * Make the launcher update on dconf changes (LP: #1376707)
  * exit spread on background tap (LP: #1368261)
  * Use an index instead of a scope id in DashCommunicator (LP:
    #1376044)

  [ Andrea Cimitan ]
  * Updated behaviour for zoomable image, workaround for sourcesize (LP:
    #1333187)

  [ Daniel d'Andrada ]
  * Make TouchGate synthesize QMouseEvents for mouse-based target items
  * Don't specify a distanceThreshold as it conflicts with
    hintDisplacment
 -- Ubuntu daily release <email address hidden> Wed, 05 Nov 2014 14:39:41 +0000

Changed in unity8 (Ubuntu RTM):
status: In Progress → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package dialer-app - 0.1+15.04.20141105.1~rtm-0ubuntu1

---------------
dialer-app (0.1+15.04.20141105.1~rtm-0ubuntu1) 14.09; urgency=low

  [ Ubuntu daily release ]
  * New rebuild forced

  [ Gustavo Pichorim Boiko ]
  * Switch dialer-app to fullscreen mode when running on top of the
    greeter. (LP: #1388156)
 -- Ubuntu daily release <email address hidden> Wed, 05 Nov 2014 21:20:35 +0000

Changed in dialer-app (Ubuntu RTM):
status: New → Fix Released
Changed in dialer-app (Ubuntu RTM):
assignee: nobody → Gustavo Pichorim Boiko (boiko)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package dialer-app - 0.1+15.04.20141113-0ubuntu1

---------------
dialer-app (0.1+15.04.20141113-0ubuntu1) vivid; urgency=low

  [ CI bot ]
  * Resync trunk

  [ Gustavo Pichorim Boiko ]
  * Add empty state in the history page. (LP: #1367325)
  * Do some basic QSceneGraph rendering optimizations.
  * Format timestamps according to the locale. (LP: #1375890)
  * Hide the SIM selector when in emergency mode. (LP: #1379386)
  * Fix a lot of warnings. (LP: #1368438)
  * Change live call control buttons to follow design. (LP: #1378006)
  * Make sure the livecall is always in a consistent state, even after
    being sigstopped. (LP: #1391324)
  * Switch dialer-app to fullscreen mode when running on top of the
    greeter. (LP: #1388156)

  [ Nick Dedekind ]
  * Only set the CallManager.callIndicatorVisible flag when
    CallManager.hasCalls. (LP: #1390098)

  [ Renato Araujo Oliveira Filho ]
  * Enable page to receive focus.
 -- Ubuntu daily release <email address hidden> Thu, 13 Nov 2014 14:41:25 +0000

Changed in dialer-app (Ubuntu):
status: In Progress → Fix Released
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.