Closed
Bug 808994
Opened 12 years ago
Closed 12 years ago
B2G SMS: Expose the `strict7BitEncoding` setting through the setting API.
Categories
(Core :: DOM: Device Interfaces, defect)
Tracking
()
People
(Reporter: jaoo, Assigned: jaoo)
References
Details
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
vicamo
:
review+
|
Details | Diff | Splinter Review |
We have to enable Latin characters replacement with corresponding ones in GSM SMS 7-bit default alphabet through the setting API.
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → josea.olivera
blocking-basecamp: --- → ?
Assignee | ||
Comment 1•12 years ago
|
||
Exposing the `strict7BitEncoding` setting through the settings API.
Attachment #678689 -
Attachment is obsolete: true
Attachment #678787 -
Flags: review?(vyang)
Comment 2•12 years ago
|
||
Comment on attachment 678787 [details] [diff] [review] v1 Review of attachment 678787 [details] [diff] [review]: ----------------------------------------------------------------- Bug 804960 comment #19 from Mounir, maybe we can simply mirror the settings entry back to preference: ::in gecko/b2g/chrome/content/settings.js SettingsListener.observe('ril.sms.strict7BitEncoding.enabled', false, function(value) { Services.prefs.setBoolPref('dom.sms.strict7BitEncoding', value); }
Attachment #678787 -
Flags: review?(vyang)
Assignee | ||
Comment 3•12 years ago
|
||
Mirror the settings entry back to preference. Much more easier. Thx Vicamo!
Attachment #678787 -
Attachment is obsolete: true
Attachment #678856 -
Flags: review?(vyang)
Comment 4•12 years ago
|
||
Comment on attachment 678856 [details] [diff] [review] v2 Review of attachment 678856 [details] [diff] [review]: ----------------------------------------------------------------- Nice Job!
Attachment #678856 -
Flags: review?(vyang) → review+
Assignee | ||
Comment 5•12 years ago
|
||
Hey Dietrich, could you please review the blocking-basecamp nomination for this bug?, the work from bug 804960 depends on this bug.
blocking-basecamp: ? → +
Assignee | ||
Comment 6•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/7dbd49fe775e
Target Milestone: --- → mozilla19
Comment 7•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/7dbd49fe775e
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 8•12 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/c0f1dea03015
status-firefox18:
--- → fixed
status-firefox19:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•