Closed Bug 944608 Opened 11 years ago Closed 10 years ago

[Messages] Need to add the phonenumberservice permission

Categories

(Firefox OS Graveyard :: Gaia::SMS, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 907740

People

(Reporter: julienw, Unassigned)

References

Details

We need to add the phonenumberservice so that mozPhoneNumberService is actually used instead of the fallback code.
Blocks: 907740
Blocks: 952449
No longer blocks: 952449
Was done in bug 907740.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
Is this what would cause the error: 

E/GeckoConsole(  516): [JavaScript Error: "TypeError: navigator.mozPhoneNumberService.normalize is not a function" {file: "app://sms.gaiamobile.org/shared/js/fb/fb_data_reader.js" line: 178}]

?
(In reply to Rick Waldron [:rwaldron] from comment #2)
> Is this what would cause the error: 
> 
> E/GeckoConsole(  516): [JavaScript Error: "TypeError:
> navigator.mozPhoneNumberService.normalize is not a function" {file:
> "app://sms.gaiamobile.org/shared/js/fb/fb_data_reader.js" line: 178}]
> 
> ?

Yeah, as it stands we always expose mozPhoneNumberService on navigator, but only expose its functions (normalize and fuzzyMatch) if the permission is present.

http://hg.mozilla.org/mozilla-central/file/f605430674e2/dom/webidl/PhoneNumberService.webidl#l14
(In reply to Rick Waldron [:rwaldron] from comment #2)
> Is this what would cause the error: 
> 
> E/GeckoConsole(  516): [JavaScript Error: "TypeError:
> navigator.mozPhoneNumberService.normalize is not a function" {file:
> "app://sms.gaiamobile.org/shared/js/fb/fb_data_reader.js" line: 178}]
> 
> ?

To any reader coming here, see bug 952781 comment 9.
You need to log in before you can comment on or make changes to this bug.