Closed
Bug 793189
Opened 12 years ago
Closed 12 years ago
MMI Codes: get IMEI
Categories
(Core :: DOM: Device Interfaces, defect, P1)
Core
DOM: Device Interfaces
Tracking
()
People
(Reporter: ferjm, Assigned: ferjm)
References
Details
(Whiteboard: [LOE:S])
Attachments
(2 files, 1 obsolete file)
(deleted),
patch
|
marshall
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
marshall
:
review+
|
Details | Diff | Splinter Review |
As per device certification requirements we need to support the *#06# MMI code to get the device IMEI code.
Assignee | ||
Updated•12 years ago
|
blocking-basecamp: --- → ?
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → ferjmoreno
Comment 1•12 years ago
|
||
Is this a P1 blockers or is it a nice-to-have?
Whiteboard: [blocked-on-input philikon]
Comment 2•12 years ago
|
||
(In reply to Andrew Overholt [:overholt] from comment #1)
> Is this a P1 blockers or is it a nice-to-have?
Blocker because device certification requirement.
Whiteboard: [blocked-on-input philikon]
Updated•12 years ago
|
blocking-basecamp: ? → +
Assignee | ||
Updated•12 years ago
|
Whiteboard: [LOE:S]
Assignee | ||
Comment 3•12 years ago
|
||
I consider this task as LOE:S once bug 793186 lands.
Depends on: 793186
Assignee | ||
Comment 4•12 years ago
|
||
Not much to explain. This patch allows the retrieval of the device's IMEI via the *#06# MMI code.
Attachment #669437 -
Flags: review?(marshall)
Assignee | ||
Comment 5•12 years ago
|
||
Attachment #669438 -
Flags: review?(marshall)
Assignee | ||
Comment 6•12 years ago
|
||
Sorry for the spam Marshall, I didn't qref when added the 'error' test.
Attachment #669438 -
Attachment is obsolete: true
Attachment #669438 -
Flags: review?(marshall)
Attachment #669441 -
Flags: review?(marshall)
Updated•12 years ago
|
Priority: -- → P1
Updated•12 years ago
|
Attachment #669437 -
Flags: review?(marshall) → review+
Comment 7•12 years ago
|
||
Comment on attachment 669437 [details] [diff] [review]
Part 1: RIL
Review of attachment 669437 [details] [diff] [review]:
-----------------------------------------------------------------
Looks good! missed one comment..
::: dom/system/gonk/RILContentHelper.js
@@ +716,5 @@
> case "RIL:SendMMI:Return:OK":
> case "RIL:CancelMMI:Return:OK":
> request = this.takeRequest(msg.json.requestId);
> if (request) {
> + Services.DOMRequest.fireSuccess(request, msg.json.result);
does this change the semantics for any of the other Send/CancelMMI results? (are we testing these DOM results anywhere?)
Comment 8•12 years ago
|
||
Comment on attachment 669441 [details] [diff] [review]
Part 2: Test
Review of attachment 669441 [details] [diff] [review]:
-----------------------------------------------------------------
It might be good to test the actual IMEI of the emulator at some point, but these tests are a good start. Maybe a file a follow up?
Attachment #669441 -
Flags: review?(marshall) → review+
Assignee | ||
Comment 9•12 years ago
|
||
Thanks for the review Marshall! :)
(In reply to Marshall Culpepper [:marshall_law] from comment #7)
> Comment on attachment 669437 [details] [diff] [review]
> Part 1: RIL
>
> Review of attachment 669437 [details] [diff] [review]:
> -----------------------------------------------------------------
>
> Looks good! missed one comment..
>
> ::: dom/system/gonk/RILContentHelper.js
> @@ +716,5 @@
> > case "RIL:SendMMI:Return:OK":
> > case "RIL:CancelMMI:Return:OK":
> > request = this.takeRequest(msg.json.requestId);
> > if (request) {
> > + Services.DOMRequest.fireSuccess(request, msg.json.result);
>
> does this change the semantics for any of the other Send/CancelMMI results?
> (are we testing these DOM results anywhere?)
No, sending 'msg.json' instead of 'msg.json.result' was actually a bug.
And no, I am afraid that we are not testing these DOM results anywhere yet.
(In reply to Marshall Culpepper [:marshall_law] from comment #8)
> Comment on attachment 669441 [details] [diff] [review]
> Part 2: Test
>
> Review of attachment 669441 [details] [diff] [review]:
> -----------------------------------------------------------------
>
> It might be good to test the actual IMEI of the emulator at some point, but
> these tests are a good start. Maybe a file a follow up?
Indeed, I have a patch ready with marionette tests (where we also check the DOMRequest.result), but I couldn't run it cause the emulator is not working for me :\. I'll file a follow-up bug for the marionette tests.
Assignee | ||
Comment 10•12 years ago
|
||
Comment 11•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/d8cf2a31d1b9
https://hg.mozilla.org/mozilla-central/rev/5bff19e795d6
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
Comment 12•12 years ago
|
||
Updated•12 years ago
|
Blocks: b2g-v1-certification
You need to log in
before you can comment on or make changes to this bug.
Description
•