Last Comment Bug 777198 - Scope ContactsService responses to the process that made the request
: Scope ContactsService responses to the process that made the request
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: DOM: Device Interfaces (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla17
Assigned To: Gregor Wagner [:gwagner]
:
:
Mentors:
Depends on:
Blocks: 777188
  Show dependency treegraph
 
Reported: 2012-07-24 18:24 PDT by Chris Jones [:cjones] inactive; ni?/f?/r? if you need me
Modified: 2012-07-26 05:11 PDT (History)
4 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch? (3.85 KB, patch)
2012-07-24 19:04 PDT, Gregor Wagner [:gwagner]
bugs: review+
cjones.bugs: feedback+
Details | Diff | Splinter Review

Description Chris Jones [:cjones] inactive; ni?/f?/r? if you need me 2012-07-24 18:24:38 PDT
Gregor, see bug 777188.  We should only reply to the .target process.

Want to grab this?
Comment 1 Gregor Wagner [:gwagner] 2012-07-24 18:29:20 PDT
do you have a code example or mxr link that uses the .target approach?
Comment 2 Gregor Wagner [:gwagner] 2012-07-24 19:04:38 PDT
Created attachment 645620 [details] [diff] [review]
patch?

I couldn't find any documentation on that. Is this a proper fix?
Comment 3 Chris Jones [:cjones] inactive; ni?/f?/r? if you need me 2012-07-24 19:52:32 PDT
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.
Comment 5 Ed Morley [:emorley] 2012-07-26 05:11:35 PDT
https://hg.mozilla.org/mozilla-central/rev/c25b4354b774

Note You need to log in before you can comment on or make changes to this bug.