Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(741)

Unified Diff: remoting/host/host_experiment_session_plugin.cc

Issue 2586133002: [Chromoting] Implement HostExperimentSessionPlugin (Closed)
Patch Set: Update implementation to match SessionPlugin Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: remoting/host/host_experiment_session_plugin.cc
diff --git a/remoting/host/host_experiment_session_plugin.cc b/remoting/host/host_experiment_session_plugin.cc
new file mode 100644
index 0000000000000000000000000000000000000000..39ca0a3d7f36c9067d9fd84b3e58a25373c1f5d5
--- /dev/null
+++ b/remoting/host/host_experiment_session_plugin.cc
@@ -0,0 +1,45 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "remoting/host/host_experiment_session_plugin.h"
+
+#include "remoting/base/constants.h"
+#include "remoting/host/host_attributes.h"
+
+namespace remoting {
+
+using buzz::QName;
+using buzz::XmlElement;
+
+std::unique_ptr<XmlElement> HostExperimentSessionPlugin::GetNextMessage() {
+ if (attributes_sent_) {
+ return nullptr;
+ }
+ attributes_sent_ = true;
+ std::unique_ptr<XmlElement> attributes(
+ new XmlElement(QName(kChromotingXmlNamespace, "host-attributes")));
+ attributes->SetBodyText(GetHostAttributes());
+ return attributes;
+}
+
+void HostExperimentSessionPlugin::OnIncomingMessage(
+ const XmlElement& attachments) {
+ if (!configuration_.empty()) {
Sergey Ulanov 2017/01/05 20:45:06 I suggest adding a separate flag to indicate that
Hzj_jie 2017/01/05 23:37:27 Done.
+ return;
+ }
+
+ const XmlElement* configuration = attachments.FirstNamed(
+ QName(kChromotingXmlNamespace, "host-configuration"));
+ if (!configuration) {
+ return;
+ }
+
+ configuration_ = configuration->BodyText();
+}
+
+const std::string& HostExperimentSessionPlugin::configuration() const {
+ return configuration_;
+}
+
+} // namespace remoting

Powered by Google App Engine
This is Rietveld 408576698