[Chromoting] Add SessionPlugin in JingleSession

remoting/protocol/session.h

Index: remoting/protocol/session.h
diff --git a/remoting/protocol/session.h b/remoting/protocol/session.h
index 865af39b3a789f594b61bf76364ecf0b46569e9e..c1c3b67fd6dc4a43f81b26b113e18cf511ca7237 100644
--- a/remoting/protocol/session.h
+++ b/remoting/protocol/session.h
@@ -5,6 +5,7 @@
 #ifndef REMOTING_PROTOCOL_SESSION_H_
 #define REMOTING_PROTOCOL_SESSION_H_
 
+#include <memory>
 #include <string>
 
 #include "base/macros.h"
@@ -15,6 +16,7 @@
 namespace remoting {
 namespace protocol {
 
+class SessionPlugin;
 class Transport;
 
 // Session is responsible for initializing and authenticating both incoming and
@@ -85,6 +87,9 @@ class Session {
   // is being closed due to an error.
   virtual void Close(ErrorCode error) = 0;
 
+  // Adds a SessionPlugin to handle attachments.
+  virtual void AddPlugin(std::unique_ptr<SessionPlugin> plugin) = 0;

[Sergey Ulanov, 2016/12/23 00:14:52]

Looking at HostExperimentSessionPlugin I don't think we want to pass ownership
of the plugin here. The caller would have to keep reference to the plugin
somewhere (to get results), which is harder if it gives up ownership. Instead
it's probably better to pass raw pointer here. This approach is also more
consistent with SetTransport().

[Hzj_jie, 2016/12/24 00:09:55]

Yes, updated.

+
  private:
   DISALLOW_COPY_AND_ASSIGN(Session);
 };