Crash the network process when a fatal daemon-to-network protocol error encountered.

remoting/host/daemon_process.cc

 // 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/daemon_process.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
 #include "base/file_path.h"
@@ skipping 60 matching lines @@
 }
 
 void DaemonProcess::CloseDesktopSession(int terminal_id) {
   DCHECK(caller_task_runner()->BelongsToCurrentThread());
 
   // Validate the supplied terminal ID. An attempt to close a desktop session
   // with an ID that couldn't possibly have been allocated is considered
   // a protocol error and the network process will be restarted.
   if (!IsTerminalIdKnown(terminal_id)) {
     LOG(ERROR) << "An invalid terminal ID. terminal_id=" << terminal_id;
-    RestartNetworkProcess();
+    RestartNetworkProcess(FROM_HERE);
     DeleteAllDesktopSessions();
     return;
   }
 
   DesktopSessionList::iterator i;
   for (i = desktop_sessions_.begin(); i != desktop_sessions_.end(); ++i) {
     if ((*i)->id() == terminal_id) {
       break;
     }
   }
@@ skipping 49 matching lines @@
 }
 
 void DaemonProcess::CreateDesktopSession(int terminal_id) {
   DCHECK(caller_task_runner()->BelongsToCurrentThread());
 
   // Validate the supplied terminal ID. An attempt to create a desktop session
   // with an ID that could possibly have been allocated already is considered
   // a protocol error and the network process will be restarted.
   if (IsTerminalIdKnown(terminal_id)) {
     LOG(ERROR) << "An invalid terminal ID. terminal_id=" << terminal_id;
-    RestartNetworkProcess();
+    RestartNetworkProcess(FROM_HERE);
     DeleteAllDesktopSessions();
     return;
   }
 
   VLOG(1) << "Daemon: opened desktop session " << terminal_id;
   desktop_sessions_.push_back(
       DoCreateDesktopSession(terminal_id).release());
   next_terminal_id_ = std::max(next_terminal_id_, terminal_id + 1);
 }
 
+void DaemonProcess::RestartNetworkProcess(

[Wez, 2012/10/23 20:01:12]

nit: CrashAndRestartNetworkProcess

[alexeypa (please no reviews), 2012/10/23 20:13:27]

It is CrashNetworkProcess() then. Because someone else restarts it.

+    const tracked_objects::Location& location) {
+  SendToNetwork(new ChromotingDaemonNetworkMsg_Crash(
+      location.function_name(), location.file_name(), location.line_number()));
+}
+
 void DaemonProcess::DoStop() {
   DCHECK(caller_task_runner()->BelongsToCurrentThread());
 
   config_watcher_.reset();
   DeleteAllDesktopSessions();
 
   CompleteStopping();
 }
 
 void DaemonProcess::DeleteAllDesktopSessions() {
   while (!desktop_sessions_.empty()) {
     delete desktop_sessions_.front();
     desktop_sessions_.pop_front();
   }
 }
 
 }  // namespace remoting