B2G STK: Support "Language Selection Event" Envelope command.

RESOLVED FIXED in mozilla21

Status

()

Core
DOM: Device Interfaces
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: allstars, Assigned: chucklee)

Tracking

unspecified
mozilla21
ARM
Gonk (Firefox OS)
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments, 3 obsolete attachments)

See TS 101.223, clause 7.5.8 Language selection event
(Reporter)

Updated

5 years ago
Blocks: 791161, 830511
Assignee: nobody → chulee
(In reply to Yoshi Huang[:yoshi][:allstars.chh] from comment #0)
> See TS 101.223, clause 7.5.8 Language selection event
Sorry, should be TS 102.223
(Reporter)

Updated

5 years ago
Depends on: 831702
Created attachment 706925 [details] [diff] [review]
0001. Add event object.
Attachment #706925 - Flags: superreview?(jonas)
Attachment #706925 - Flags: review?(allstars.chh)
Created attachment 706926 [details] [diff] [review]
0002. Handle event.
Attachment #706926 - Flags: review?(allstars.chh)
Created attachment 706927 [details] [diff] [review]
0003. Mark event as supported.
Attachment #706927 - Flags: review?(allstars.chh)
Created attachment 706928 [details] [diff] [review]
0004. Test case for Event Download Command : Language Selection.
Attachment #706928 - Flags: review?(allstars.chh)
Created attachment 706930 [details] [diff] [review]
0001. Add event object, v1.1

Add support event list in comment of sendStkEventDownload()
Attachment #706925 - Attachment is obsolete: true
Attachment #706925 - Flags: superreview?(jonas)
Attachment #706925 - Flags: review?(allstars.chh)
Attachment #706930 - Flags: superreview?(jonas)
Attachment #706930 - Flags: review?(allstars.chh)
Comment on attachment 706930 [details] [diff] [review]
0001. Add event object, v1.1

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

::: dom/icc/interfaces/SimToolKit.idl
@@ +595,5 @@
> +dictionary MozStkLanguageSelectionEvent
> +{
> +  /**
> +   * The type of this event.
> +   * It shall be nsIDOMMozIccManager.STK_EVENT_TYPE_LANGUAGE_SELECTION;

nit, ends with period.

@@ +602,5 @@
> +
> +  /**
> +   * Language Information
> +   *
> +   * @see ISO 639-1, Alpha-2 code

It would be great if we could put some examples here.
Attachment #706930 - Flags: review?(allstars.chh) → review+
(Reporter)

Updated

5 years ago
Attachment #706926 - Flags: review?(allstars.chh) → review+
(Reporter)

Updated

5 years ago
Attachment #706927 - Flags: review?(allstars.chh) → review+
Comment on attachment 706928 [details] [diff] [review]
0004. Test case for Event Download Command : Language Selection.

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

::: dom/system/gonk/tests/test_ril_worker_icc.js
@@ +1569,5 @@
> +  };
> +  worker.RIL.sendStkEventDownload({
> +    event: event
> +  });
> +});

nit, add an extra line here.
Attachment #706928 - Flags: review?(allstars.chh) → review+
Created attachment 706952 [details] [diff] [review]
0004. Test case for Event Download Command : Language Selection, v1.1

Address comment 8
Attachment #706928 - Attachment is obsolete: true
Attachment #706930 - Flags: superreview?(jonas) → superreview+
Created attachment 707450 [details] [diff] [review]
0001. Add event object, v1.2

Address comment 7
Attachment #706930 - Attachment is obsolete: true
These could use better commit messages (see comment 37 of bug 827280 for examples)

Updated

5 years ago
Blocks: 847055
You need to log in before you can comment on or make changes to this bug.