[system][modal dialogs] queuing alert events

RESOLVED FIXED

Status

P1
normal
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: etienne, Assigned: etienne)

Tracking

unspecified
ARM
Gonk (Firefox OS)
Dependency tree / graph

Firefox Tracking Flags

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

Details

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

Attachments

(2 attachments, 1 obsolete attachment)

(Assignee)

Description

6 years ago
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:
(Assignee)

Comment 1

6 years ago
Created attachment 752744 [details] [diff] [review]
Patch proposal
Assignee: nobody → etienne
Attachment #752744 - Flags: review?(alive)
(Assignee)

Updated

6 years ago
status-b2g18: fixed → affected
status-b2g18-v1.0.1: fixed → affected
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 ;)
(Assignee)

Comment 4

6 years ago
https://github.com/mozilla-b2g/gaia/commit/85a9752e0012bef2796d7a8391d850afd3a557cc
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Uplifted 85a9752e0012bef2796d7a8391d850afd3a557cc to:
v1-train: 1c1bcfbac112a8a363341f3b1b6607cb40849515
status-b2g18: affected → fixed

Updated

6 years ago
Keywords: verifyme
QA Contact: jsmith
Created attachment 755050 [details]
Test Case

Test case for verification attached

Updated

6 years ago
Attachment #755050 - Attachment is obsolete: true
Created attachment 755055 [details]
Test Case
lgtm with attached test case on 5/28 build
status-b2g18: fixed → verified

Updated

6 years ago
Keywords: verifyme

Updated

6 years ago
status-b2g18-v1.0.1: affected → wontfix
You need to log in before you can comment on or make changes to this bug.