Closed
Bug 882911
Opened 11 years ago
Closed 6 years ago
Contacts: implement "secret" contacts that can only be seen by certified apps
Categories
(Core Graveyard :: DOM: Contacts, defect, P5)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: reuben, Unassigned)
Details
(Whiteboard: [fixed-in-birch][leave open])
Attachments
(2 files, 2 obsolete files)
8.95 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
19.85 KB,
patch
|
Details | Diff | Splinter Review |
We need this (at least) for Facebook contacts, since their TOS doesn't allow sharing the contacts with other apps.
Reporter | ||
Comment 1•11 years ago
|
||
We use this to check if the requesting app is a certified app in the parent.
Attachment #762310 -
Flags: review?(bugs)
Reporter | ||
Comment 2•11 years ago
|
||
Attachment #762312 -
Flags: review?(anygregor)
Reporter | ||
Comment 3•11 years ago
|
||
Handle secrecy for getAll, getCount, and contactchange events.
Attachment #762312 -
Attachment is obsolete: true
Attachment #762312 -
Flags: review?(anygregor)
Attachment #762364 -
Flags: review?(anygregor)
Comment 4•11 years ago
|
||
Comment on attachment 762310 [details] [diff] [review] 1 - Add nsIProcessChecker.checkAppStatus what is aStatus passed to checkAppHasStatus? Please improve the comment to tell what the status is about. Hmm, does xpidl really convert that parameter to const unsigned short aStatus? const looks rather useless. > ContentParent::CheckAppHasStatus(const unsigned short aStatus) { { goes to the next line. Same with some other methods too.
Attachment #762310 -
Flags: review?(bugs)
Attachment #762310 -
Flags: review?(anygregor)
Attachment #762310 -
Flags: review+
Comment 5•11 years ago
|
||
Some more background: 1 - contact information MUST NOT be exposed to other apps, e.g. It should not be available through the Contacts API 2 - Users should not be able to modify contacts. We can cache locally to improve performance but you cannot modify that information. 3 - A service should be allowed to sync/update the information.
Reporter | ||
Comment 6•11 years ago
|
||
https://hg.mozilla.org/projects/birch/rev/dbce265e3af7
Whiteboard: [fixed-in-birch][leave open]
Reporter | ||
Updated•11 years ago
|
Attachment #762310 -
Flags: review?(anygregor)
Reporter | ||
Comment 7•11 years ago
|
||
Includes the read-only part.
Attachment #762364 -
Attachment is obsolete: true
Attachment #762364 -
Flags: review?(anygregor)
Attachment #769373 -
Flags: review?(anygregor)
Comment 8•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/dbce265e3af7
Reporter | ||
Updated•11 years ago
|
Attachment #769373 -
Flags: review?(anygregor)
Reporter | ||
Updated•11 years ago
|
Assignee: reuben.bmo → nobody
Updated•11 years ago
|
Component: DOM: Device Interfaces → DOM: Contacts
Comment 9•6 years ago
|
||
https://bugzilla.mozilla.org/show_bug.cgi?id=1472046 Move all DOM bugs that haven’t been updated in more than 3 years and has no one currently assigned to P5. If you have questions, please contact :mdaly.
Priority: -- → P5
Comment 10•6 years ago
|
||
DOM: Contacts isn't used anymore. Closing all remaining bugs.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
Updated•6 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•