[Chromoting] Use HostExperimentSessionPlugin in host

remoting/host/host_experiment_servant.h

Index: remoting/host/host_experiment_servant.h
diff --git a/remoting/host/host_experiment_session_plugin.h b/remoting/host/host_experiment_servant.h
similarity index 80%
rename from remoting/host/host_experiment_session_plugin.h
rename to remoting/host/host_experiment_servant.h
index 20627420f40812bbc1747bee3946103744c7db46..b2e9e5d3637844cf1e654f98bbd8a8e4059fb993 100644
--- a/remoting/host/host_experiment_session_plugin.h
+++ b/remoting/host/host_experiment_servant.h
@@ -2,8 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef REMOTING_HOST_HOST_EXPERIMENT_SESSION_PLUGIN_H_
-#define REMOTING_HOST_HOST_EXPERIMENT_SESSION_PLUGIN_H_
+#ifndef REMOTING_HOST_HOST_EXPERIMENT_SERVANT_H_
+#define REMOTING_HOST_HOST_EXPERIMENT_SERVANT_H_
 
 #include <memory>
 #include <string>
@@ -15,7 +15,7 @@ namespace remoting {
 
 // A SessionPlugin implementation to send host attributes to client, and
 // receives experiment settings.
-class HostExperimentSessionPlugin : public protocol::SessionPlugin {
+class HostExperimentServant : public protocol::SessionPlugin {

[Sergey Ulanov, 2017/01/09 23:00:26]

I don't think it's necessary to rename this class in this CL.
Also "Server" is more commonly used term, not "Servant", but
"HostExperimentServer" would be not a good name either because it doesn't
reflect what this class is responsible for. HostExperimentReceiver might be a
better name, but I think HostExperimentSessionPlugin works as well.

[Hzj_jie, 2017/01/10 02:31:09]

No, HostExperimentServer is definitely not correct. I am OK with current name if
you do not have concern.

[joedow, 2017/01/10 15:11:47]

Drive-by comment.  'Servant' is not standard nomenclature and has negative
connotations, definitely change it.

  public:
   using SessionPlugin::SessionPlugin;
 
@@ -38,4 +38,4 @@ class HostExperimentSessionPlugin : public protocol::SessionPlugin {
 
 }  // namespace remoting
 
-#endif  // REMOTING_HOST_HOST_EXPERIMENT_SESSION_PLUGIN_H_
+#endif  // REMOTING_HOST_HOST_EXPERIMENT_SERVANT_H_