Clean up public interface of AttachmentBrokerUnprivileged.

ipc/attachment_broker.h

Index: ipc/attachment_broker.h
diff --git a/ipc/attachment_broker.h b/ipc/attachment_broker.h
index 4cf1ab5de587c6407aeb26f7ab478f4000a08eee..f2cdd06be80c22100c0c0a435f07c589f6839d30 100644
--- a/ipc/attachment_broker.h
+++ b/ipc/attachment_broker.h
@@ -97,6 +97,13 @@ class IPC_EXPORT AttachmentBroker : public Listener {
   virtual void RegisterCommunicationChannel(Endpoint* endpoint);
   virtual void DeregisterCommunicationChannel(Endpoint* endpoint);
 
+  // In each unprivileged process, exactly one channel should be used to
+  // communicate brokerable attachments with the broker process.
+  virtual void DesignateBrokerCommunicationChannel(Endpoint* endpoint);
+
+  // Whether the process's broker is privileged.

[Tom Sepez, 2016/02/09 00:26:19]

nit: True iff the process's broker is privileged.
(Just to be clear on the interpretation of the return value).

[erikchen, 2016/02/09 02:38:51]

Done.

+  virtual bool IsPrivilegedBroker();

[Tom Sepez, 2016/02/09 00:26:19]

Nit: maybe HasPrivilegedBroker(), because this process doesn't have an IS_A
relationship with broker classes.

[erikchen, 2016/02/09 02:38:51]

My initial comment was inaccurate in its use of the word "process's broker".
This is a IS_A relationship, since this is an instance method on
AttachmentBroker.

+
  protected:
   using AttachmentVector = std::vector<scoped_refptr<BrokerableAttachment>>;