diff options
author | erikchen <erikchen@chromium.org> | 2015-07-30 15:26:08 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-07-30 22:27:15 +0000 |
commit | 8c73f834bc73178123389c29d7627e29cb09d501 (patch) | |
tree | 054914c7149c728b9034aca66f2199e52ab0f991 /ipc/attachment_broker_privileged.cc | |
parent | fc59cb43df0e4eb77bc2916ab37d36598eabd4f2 (diff) | |
download | chromium_src-8c73f834bc73178123389c29d7627e29cb09d501.zip chromium_src-8c73f834bc73178123389c29d7627e29cb09d501.tar.gz chromium_src-8c73f834bc73178123389c29d7627e29cb09d501.tar.bz2 |
Reland #1: Clean up interface of attachment broker.
I forgot to initialize a member variable, which is why the original CL flakily
succeeded (and managed to pass the CQ).
Original issue's description:
> ipc: Clean up interface of attachment broker.
>
> AttachmentBrokerUnprivileged now has a method
> DesignateBrokerCommunicationChannel which is used by non-broker processes to
> designate an IPC::Channel as the communication medium for brokerable attachment
> messages. IPC::Channel has a new member attachment_broker_endpoint_ which causes
> it to pass all messages through the attachment broker before forwarding them to
> the listener.
>
> BUG=493414
>
> Committed: https://crrev.com/9a06836982214f2edced21bbd1615b49e7f231bf
> Cr-Commit-Position: refs/heads/master@{#341143}
BUG=493414
TBR=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1258323004
Cr-Commit-Position: refs/heads/master@{#341207}
Diffstat (limited to 'ipc/attachment_broker_privileged.cc')
-rw-r--r-- | ipc/attachment_broker_privileged.cc | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/ipc/attachment_broker_privileged.cc b/ipc/attachment_broker_privileged.cc index d2b6832..1fe3a75 100644 --- a/ipc/attachment_broker_privileged.cc +++ b/ipc/attachment_broker_privileged.cc @@ -16,6 +16,7 @@ AttachmentBrokerPrivileged::~AttachmentBrokerPrivileged() {} void AttachmentBrokerPrivileged::RegisterCommunicationChannel( Channel* channel) { + channel->set_attachment_broker_endpoint(true); auto it = std::find(channels_.begin(), channels_.end(), channel); DCHECK(channels_.end() == it); channels_.push_back(channel); |