Closed Bug 874892 Opened 7 years ago Closed 7 years ago

[system][modal dialogs] queuing alert events

Categories

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

ARM
Gonk (Firefox OS)
defect

Tracking

(blocking-b2g:leo+, b2g18 verified, b2g18-v1.0.1 wontfix)

RESOLVED FIXED
blocking-b2g leo+
Tracking Status
b2g18 --- verified
b2g18-v1.0.1 --- wontfix

People

(Reporter: etienne, Assigned: etienne)

References

Details

(Whiteboard: [GCF], QARegressExclude, u=fx-os-user c=scravag-sprint p=1)

Attachments

(2 files, 1 obsolete file)

Some background, if an app does

|alert("first");alert("second");|

we won't execute the |alert("second")| until the user taps the OK button on the first alert.

But, if and apps send an alert() in a system message handler (and receives multiple system messages in a row) we can end up with the queue of alerts to handle.

This is needed to handle the SMS of class 0 which are displayed with an alert().

+++ This bug was initially created as a clone of Bug #858908 +++

AU_LINUX_GECKO_ICS_STRAWBERRY_V1.01.00.01.19.053
 Firefox os  v1.0.1
 Mozilla build ID: 20130327153857
 
 +++ This bug was initially created as a clone of Bug #431424 +++
 
 [WuYing][6915]
 
 DEFECT DESCRIPTION:
 If received more than 2 SMS of Class 0 type, just can read one SMS detail infomation
 
 
 REPRODUCING PROCEDURES:
 1.after received more than 2 SMS of Class 0 type
 2.These SMSM  will prompt in notification.
 3.pull dowm the status bar
 4.Select one Class0 SMS to tap
 5.It will show detail information
 6.But other Class0 SMS will disappear,cannot read detail meessage---KO
 
 EXPECTED BEHAVIOUR:
 read all Class0 SMS detail infomation
 
 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:
 
 ++++++++++ end of initial bug #431424 description ++++++++++
 
 
 
 CONTACT INFO (Name,Phone number):
 
  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:
Attached patch Patch proposalSplinter Review
Assignee: nobody → etienne
Attachment #752744 - Flags: review?(alive)
Comment on attachment 752744 [details] [diff] [review]
Patch proposal

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

Code looks fine. r=alive. You did something I want to do.
Attachment #752744 - Flags: review?(alive) → review+
(In reply to Alive Kuo [:alive] from comment #2)
> Code looks fine. r=alive. You did something I want to do.
(But no time ;)
https://github.com/mozilla-b2g/gaia/commit/85a9752e0012bef2796d7a8391d850afd3a557cc
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Uplifted 85a9752e0012bef2796d7a8391d850afd3a557cc to:
v1-train: 1c1bcfbac112a8a363341f3b1b6607cb40849515
Keywords: verifyme
QA Contact: jsmith
Attached file Test Case (obsolete) —
Test case for verification attached
Attachment #755050 - Attachment is obsolete: true
Attached file Test Case
lgtm with attached test case on 5/28 build
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.