915975 - [Buri][Call][Sensor]Invalid P-sensor and headphone interaction

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::System, defect, P1)

Description

[sync-1]

Firefox os  v1.1
 Mozilla build ID:20130902041201
 
 DEFECT DESCRIPTION:
  Make a call,covering P-sensor then insert earphone,remove the cover,p-sensor invalid,click power key to light screen.
  REPRODUCING PROCEDURES:
  1.Make a call,covering p-sensor then insert earphone;
  2.Remove the cover,screen don't light-KO.
  3.Click power key,the screen light and P-sensor recovery(pull the earphone   recovery).
  EXPECTED BEHAVIOUR:
  The screen can light when remove the cover.
  ASSOCIATE SPECIFICATION:
 
  TEST PLAN REFERENCE:
 
  TOOLS AND PLATFORMS USED:
 
  USER IMPACT:
  Medium
  REPRODUCING RATE:
  5/5
  For FT PR, Please list reference mobile's behavior:

Comment 1

[sync-1]

Clone from brother

Comment 2

[sync-1]

Attachment: PR523804.log

Comment 3

[Andrew Overholt [:overholt]]

QA, can you verify on which devices this reproduces?  That will help determine whether or not this is device-specific.

Comment 4

[Lucas Aliaga]

Verified repros on Leo, Buri, and Unagi devices. Covering the P-sensor and plugging in headphones disables the screen turn on when the p-sensor is uncovered.

- Leo -
Build ID: 20130917041200
Gecko: http://hg.mozilla.org/releases/mozilla-b2g18/rev/b459ac3ebe24
Gaia: 862b5761825e010e3cc5968c096377e1ea54bbf2
Platform Version: 18.1

- Buri -
Build ID: 20130916040205
Gecko: http://hg.mozilla.org/mozilla-central/rev/c4bcef90cef9
Gaia: a0079597d510ce8ea0b9cbb02c506030510b9eeb
Platform Version: 26.0a1

- Unagi -
Build ID: 20130918004001
Gecko: http://hg.mozilla.org/releases/mozilla-aurora/rev/0322470077b7
Gaia: 2e88444c39998e0b90513ad9f8ab9413f3fe246f
Platform Version: 26.0a2

Comment 5

[Andrew Overholt [:overholt]]

Doug, any insight here?  This is likely not a regression and therefore unlikely to block 1.1, FWIW.

Comment 6

[Doug Turner (:dougt)]

mwu, something you want to look at?  if not, please assign back to me.

Comment 7

[Michael Wu [:mwu]]

Hmmm I'm not sure I have much of an idea what's going on here.. Maybe Alive does?

A search through my bugmail turns up bug 862643 which seems related.

Comment 8

[Peter Dolanjski [:pdol]]

Since there is a simple work around and it won't be too common, not blocking for leo.

Comment 9

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Not blocking for v1.2.

Comment 10

[(inactive after 6/18) Alive Kuo [:alive]]

A trivial change to screen manager could fix this.
When headset is inserted, if the screen is turned off by proximity sensor, turn it on back.

Comment 11

[(inactive after 6/18) Alive Kuo [:alive]]

Attachment: https://github.com/mozilla-b2g/gaia/pull/12740/files

Patch: turn on screen if it's turned off by proximity sensor

Comment 12

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Comment on attachment 814756 [details]
https://github.com/mozilla-b2g/gaia/pull/12740/files

r+ with nit.

Comment 13

[(inactive after 6/18) Alive Kuo [:alive]]

Travis is green.
https://travis-ci.org/mozilla-b2g/gaia/builds/12308275

Comment 14

[(inactive after 6/18) Alive Kuo [:alive]]

https://github.com/mozilla-b2g/gaia/commit/495912f872a2d1d3e608fe50732d3b4f62f0a89f