Closed Bug 1093506 Opened 10 years ago Closed 10 years ago

[RIL] Should present the "number" for outgoing call in CLCC

Categories

(Firefox OS Graveyard :: Emulator, defect)

Product:
Firefox OS Graveyard
Component:
Emulator
Platform:
x86_64
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
2.1 S8 (7Nov)

People

(Reporter: aknow, Assigned: aknow)

References

Details

Attachments

(2 files)

PR 118
(deleted), text/x-github-pull-request
edgar
: review+
Details
[platform_external_qemu][b2g-kitkat] PR 122
(deleted), text/x-github-pull-request
aknow
: review+
Details
This cause the fail in marionette test, ex: test_temporary_clir.js In the test case, we try to send the request "gsm cancel <number>". However, the number becomes empty because it is not shown in CLCC.
Attached file PR 118 (deleted) —
Attachment #8516546 - Flags: review?(htsai)
Comment on attachment 8516546 [details] PR 118 Steal r? if you don't mind :)
Attachment #8516546 - Flags: review?(htsai) → review?(echen)
(In reply to Szu-Yu Chen [:aknow] from comment #1) > Created attachment 8516546 [details] > PR 118 Please see my comments in github, thank you.
(In reply to Edgar Chen [:edgar][:echen] from comment #2) > Comment on attachment 8516546 [details] > PR 118 > > Steal r? if you don't mind :) No problem, thank you very much :P
(In reply to Edgar Chen [:edgar][:echen] from comment #3) > (In reply to Szu-Yu Chen [:aknow] from comment #1) > > Created attachment 8516546 [details] > > PR 118 > > Please see my comments in github, thank you. Fixed. Thank you.
Comment on attachment 8516546 [details] PR 118 Thank you.
Attachment #8516546 - Flags: review?(echen) → review+
Try with qemu changes looks good: https://tbpl.mozilla.org/?tree=Try&rev=31e2718246fa \o/
Keywords: checkin-needed
(In reply to Edgar Chen [:edgar][:echen] from comment #7) > Try with qemu changes looks good: > https://tbpl.mozilla.org/?tree=Try&rev=31e2718246fa \o/ \o/
https://github.com/mozilla-b2g/platform_external_qemu/commit/67f2907bc340bad250b4ea6ce2902b52896c9ef0
Status: NEW → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 2.1 S8 (7Nov)
Hi Aknow, Could you help provide a pull request to kk-branch? Thank you.
Flags: needinfo?(szchen)
Flags: needinfo?(szchen)
Attachment #8533581 - Flags: review+
checkin-needed for PR 122 It's the PR for kk branch
Keywords: checkin-needed
Blocks: emulator-kk
No longer blocks: emulator-l_taskcluster
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: