Closed
Bug 777198
Opened 12 years ago
Closed 12 years ago
Scope ContactsService responses to the process that made the request
Categories
(Core :: DOM: Device Interfaces, defect)
Core
DOM: Device Interfaces
Tracking
()
RESOLVED
FIXED
mozilla17
People
(Reporter: cjones, Assigned: gwagner)
References
Details
Attachments
(1 file)
3.85 KB,
patch
|
smaug
:
review+
cjones
:
feedback+
|
Details | Diff | Splinter Review |
Gregor, see bug 777188. We should only reply to the .target process. Want to grab this?
Assignee | ||
Comment 1•12 years ago
|
||
do you have a code example or mxr link that uses the .target approach?
Assignee: nobody → anygregor
Assignee | ||
Comment 2•12 years ago
|
||
I couldn't find any documentation on that. Is this a proper fix?
Attachment #645620 -
Flags: feedback?(jones.chris.g)
Assignee | ||
Updated•12 years ago
|
Attachment #645620 -
Flags: review?(bugs)
Reporter | ||
Comment 3•12 years ago
|
||
Comment on attachment 645620 [details] [diff] [review] patch? This looks correct to me based on what smaug told me. I'm happy to r+ little patches like this in the future, but I'd like smaug to confirm that this is correct usage for this first one.
Attachment #645620 -
Flags: feedback?(jones.chris.g) → feedback+
Updated•12 years ago
|
Attachment #645620 -
Flags: review?(bugs) → review+
Assignee | ||
Comment 4•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/c25b4354b774
Comment 5•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/c25b4354b774
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in
before you can comment on or make changes to this bug.
Description
•