913777 - [Buri][SMS]When receive an SMS during handset locked,there is no notification

Status: RESOLVED FIXED

(Core :: DOM: Device Interfaces, defect, P1)

Description

[sync-1]

Firefox os  v1.1
  Mozilla build ID:20130902041201
 
 +++ This bug was initially created as a clone of Bug #501985 +++
 DEFECT DESCRIPTION:
  When receive an SMS during handset locked,there is no notification
  REPRODUCING PROCEDURES:
  1.Not plugin usb.
  2.Close SMS App in background.
  3.Receive an SMS when handset locked,there is no notification->KO1
  4.Press power key,the notification and alert comes->KO2
 
  EXPECTED BEHAVIOUR:
  When you receive SMS,it should have notification.
  ASSOCIATE SPECIFICATION:
 
  TEST PLAN REFERENCE:
 
  TOOLS AND PLATFORMS USED:
 
  USER IMPACT:
  Mid
  REPRODUCING RATE:
  4/5
  For FT PR, Please list reference mobile's behavior:
 ++++++++++ end of initial bug #501985 description ++++++++++
 
 		
 
  DEFECT DESCRIPTION:
 
  REPRODUCING PROCEDURES:
 
  EXPECTED BEHAVIOUR:
 
  ASSOCIATE SPECIFICATION:
 
  TEST PLAN REFERENCE:
 
  TOOLS AND PLATFORMS USED:
 
  USER IMPACT:
 
  REPRODUCING RATE:
 
  For FT PR, Please list reference mobile's behavior:

Comment 1

[sync-1]

Clone from brother

Comment 2

[sync-1]

Attachment: logcat

Comment 3

[Rick Waldron [:rwaldron]]

Updating to correct component.

Comment 4

[Julien Wajsberg [:julienw]]

probably a dupe of bug 875703

Comment 5

[Gene Lian (I already quit Mozilla)]

Exactly... Let me take this and give it a full test again. It seems a very tricky random bug.

Comment 6

[Gene Lian (I already quit Mozilla)]

Attachment: Patch

Comment 7

[Vicamo Yang [:vicamo][:vyang]]

Comment on attachment 801590 [details] [diff] [review]
Patch

Review of attachment 801590 [details] [diff] [review]:
-----------------------------------------------------------------

Just like the cpu lock we have for call ring events.

Comment 8

[Gene Lian (I already quit Mozilla)]

Attachment: Patch, V2

Comment 9

[Gene Lian (I already quit Mozilla)]

https://hg.mozilla.org/integration/b2g-inbound/rev/0e994d38fb3d

Hi Joe, need your decision to mark leo+ or not.

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/0e994d38fb3d

Comment 13

[Preeti Raghunath(:Preeti)]

Why can't this wait for 1.2?

Comment 14

[Julien Wajsberg [:julienw]]

Preeti> I think Gene wanted this in leo because this is annoying and we had actually real user reports about that.

However, playing with Wakelock is risky for the user's battery and it's maybe a good idea to keep it for 1.2.

So it's your call :)

Comment 15

[xiupinglong]

(In reply to Preeti Raghunath(:Preeti) from comment #13)
> Why can't this wait for 1.2?

When receive a SMS during handset locked, there is no notification. And then user will not know the new sms coming timely, and may not reply to the sms sender. It is so annoying.

Comment 16

[Honza Bambas (:mayhemer)]

(In reply to Preeti Raghunath(:Preeti) from comment #13)
> Why can't this wait for 1.2?

This actually caused me real trouble.  Really bad behavior of the phone.

Comment 17

[Julien Wajsberg [:julienw]]

Renominating leo after comment 14 15 and 16.

Comment 18

[Andrew Overholt [:overholt]]

No SMS notifications on the lock screen is a blocker.

Comment 19

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-b2g18/rev/725c2992dac6

Comment 20

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-b2g18_v1_1_0_hd/rev/725c2992dac6

Comment 21

[Gene Lian (I already quit Mozilla)]

Thanks Ryan for the manual merge which looks good to me.