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

Unified Diff: remoting/host/log_to_server.cc

Issue 320403002: Pull out common code from client and host versions of LogToServer (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix try failures Created 6 years, 6 months 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/log_to_server.cc
diff --git a/remoting/host/log_to_server.cc b/remoting/host/log_to_server.cc
deleted file mode 100644
index 8e0dd6bf2a8bdb31fa89ce2125bc2bf7f94790e9..0000000000000000000000000000000000000000
--- a/remoting/host/log_to_server.cc
+++ /dev/null
@@ -1,122 +0,0 @@
-// Copyright (c) 2012 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/log_to_server.h"
-
-#include "base/bind.h"
-#include "base/message_loop/message_loop_proxy.h"
-#include "remoting/base/constants.h"
-#include "remoting/host/host_status_monitor.h"
-#include "remoting/host/server_log_entry_host.h"
-#include "remoting/jingle_glue/iq_sender.h"
-#include "remoting/jingle_glue/server_log_entry.h"
-#include "remoting/jingle_glue/signal_strategy.h"
-#include "remoting/protocol/transport.h"
-#include "third_party/libjingle/source/talk/xmllite/xmlelement.h"
-#include "third_party/libjingle/source/talk/xmpp/constants.h"
-
-using buzz::QName;
-using buzz::XmlElement;
-
-namespace remoting {
-
-LogToServer::LogToServer(base::WeakPtr<HostStatusMonitor> monitor,
- ServerLogEntry::Mode mode,
- SignalStrategy* signal_strategy,
- const std::string& directory_bot_jid)
- : monitor_(monitor),
- mode_(mode),
- signal_strategy_(signal_strategy),
- directory_bot_jid_(directory_bot_jid) {
- monitor_->AddStatusObserver(this);
- signal_strategy_->AddListener(this);
-}
-
-LogToServer::~LogToServer() {
- signal_strategy_->RemoveListener(this);
- if (monitor_.get())
- monitor_->RemoveStatusObserver(this);
-}
-
-void LogToServer::LogSessionStateChange(const std::string& jid,
- bool connected) {
- DCHECK(CalledOnValidThread());
-
- scoped_ptr<ServerLogEntry> entry(
- MakeLogEntryForSessionStateChange(connected));
- AddHostFieldsToLogEntry(entry.get());
- entry->AddModeField(mode_);
-
- if (connected) {
- DCHECK(connection_route_type_.count(jid) == 1);
- AddConnectionTypeToLogEntry(entry.get(), connection_route_type_[jid]);
- }
- Log(*entry.get());
-}
-
-void LogToServer::OnSignalStrategyStateChange(SignalStrategy::State state) {
- DCHECK(CalledOnValidThread());
-
- if (state == SignalStrategy::CONNECTED) {
- iq_sender_.reset(new IqSender(signal_strategy_));
- SendPendingEntries();
- } else if (state == SignalStrategy::DISCONNECTED) {
- iq_sender_.reset();
- }
-}
-
-bool LogToServer::OnSignalStrategyIncomingStanza(
- const buzz::XmlElement* stanza) {
- return false;
-}
-
-void LogToServer::OnClientConnected(const std::string& jid) {
- DCHECK(CalledOnValidThread());
- LogSessionStateChange(jid, true);
-}
-
-void LogToServer::OnClientDisconnected(const std::string& jid) {
- DCHECK(CalledOnValidThread());
- LogSessionStateChange(jid, false);
- connection_route_type_.erase(jid);
-}
-
-void LogToServer::OnClientRouteChange(const std::string& jid,
- const std::string& channel_name,
- const protocol::TransportRoute& route) {
- // Store connection type for the video channel. It is logged later
- // when client authentication is finished.
- if (channel_name == kVideoChannelName) {
- connection_route_type_[jid] = route.type;
- }
-}
-
-void LogToServer::Log(const ServerLogEntry& entry) {
- pending_entries_.push_back(entry);
- SendPendingEntries();
-}
-
-void LogToServer::SendPendingEntries() {
- if (iq_sender_ == NULL) {
- return;
- }
- if (pending_entries_.empty()) {
- return;
- }
- // Make one stanza containing all the pending entries.
- scoped_ptr<XmlElement> stanza(ServerLogEntry::MakeStanza());
- while (!pending_entries_.empty()) {
- ServerLogEntry& entry = pending_entries_.front();
- stanza->AddElement(entry.ToStanza().release());
- pending_entries_.pop_front();
- }
- // Send the stanza to the server.
- scoped_ptr<IqRequest> req = iq_sender_->SendIq(
- buzz::STR_SET, directory_bot_jid_, stanza.Pass(),
- IqSender::ReplyCallback());
- // We ignore any response, so let the IqRequest be destroyed.
- return;
-}
-
-} // namespace remoting

Powered by Google App Engine
This is Rietveld 408576698