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

Unified Diff: remoting/host/host_event_logger_win.cc

Issue 10893009: Log when the host is started for an account. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Change shutdown log to a TODO, update unit test expectations. Created 8 years, 4 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
« no previous file with comments | « remoting/host/host_event_logger_posix.cc ('k') | remoting/host/host_mock_objects.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/host_event_logger_win.cc
diff --git a/remoting/host/host_event_logger_win.cc b/remoting/host/host_event_logger_win.cc
index 85f31d26c1d01c8168d17dd46cb134eaa6a984b9..7e13a40445135c2a522d88eef04ed9cfef8a90ca 100644
--- a/remoting/host/host_event_logger_win.cc
+++ b/remoting/host/host_event_logger_win.cc
@@ -38,6 +38,7 @@ class HostEventLoggerWin : public HostEventLogger, public HostStatusObserver {
const std::string& jid,
const std::string& channel_name,
const protocol::TransportRoute& route) OVERRIDE;
+ virtual void OnStart(const std::string& xmpp_login) OVERRIDE;
virtual void OnShutdown() OVERRIDE;
private:
@@ -101,6 +102,11 @@ void HostEventLoggerWin::OnClientRouteChange(
}
void HostEventLoggerWin::OnShutdown() {
+ // TODO(rmsousa): Fix host shutdown to actually call this, and add a log line.
+}
+
+void HostEventLoggerWin::OnStart(const std::string& xmpp_login) {
+ LogString(EVENTLOG_INFORMATION_TYPE, MSG_HOST_STARTED, xmpp_login);
}
void HostEventLoggerWin::Log(WORD type,
« no previous file with comments | « remoting/host/host_event_logger_posix.cc ('k') | remoting/host/host_mock_objects.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698