Disable chromoting service on permanent errors.

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 <algorithm>
 #include <string>
 
 #include "base/bind.h"
@@ skipping 114 matching lines @@
   if (!handled) {
     LOG(ERROR) << "Received unexpected IPC type: " << message.type();
     CrashNetworkProcess(FROM_HERE);
   }
 
   return handled;
 }
 
 void DaemonProcess::OnPermanentError(int exit_code) {
   DCHECK(caller_task_runner()->BelongsToCurrentThread());
+  DCHECK(kMinPermanentErrorExitCode <= exit_code &&
+         exit_code <= kMaxPermanentErrorExitCode);
+
   Stop();
 }
 
 void DaemonProcess::CloseDesktopSession(int terminal_id) {
   DCHECK(caller_task_runner()->BelongsToCurrentThread());
 
   // Validate the supplied terminal ID. An attempt to use a desktop session ID
   // that couldn't possibly have been allocated is considered a protocol error
   // and the network process will be restarted.
   if (!WasTerminalIdAllocated(terminal_id)) {
@@ skipping 211 matching lines @@
   FOR_EACH_OBSERVER(HostStatusObserver, status_observers_,
                     OnClientRouteChange(jid, channel_name, parsed_route));
 }
 
 void DaemonProcess::OnHostStarted(const std::string& xmpp_login) {
   DCHECK(caller_task_runner()->BelongsToCurrentThread());
 
   FOR_EACH_OBSERVER(HostStatusObserver, status_observers_, OnStart(xmpp_login));
 }
 
-void DaemonProcess::OnHostShutdown() {
+void DaemonProcess::OnHostShutdown(int exit_code) {

[Sergey Ulanov, 2015/04/07 21:50:18]

It looks like ChromotingNetworkDaemonMsg_HostShutdown is there to indicate to
HostStatusObservers in the daemon process that the host is offline. It doesn't
mean that the network process is dying - the host may be restarted again in the
same network process. So I don't think right to add exit_code here. How does the
network process get killed in response to this message?

   DCHECK(caller_task_runner()->BelongsToCurrentThread());
 
-  FOR_EACH_OBSERVER(HostStatusObserver, status_observers_, OnShutdown());
+  if (kMinPermanentErrorExitCode <= exit_code &&
+      exit_code <= kMaxPermanentErrorExitCode) {
+    OnPermanentError(exit_code);
+    return;
+  }
+  FOR_EACH_OBSERVER(HostStatusObserver, status_observers_,
+                    OnShutdown((HostExitCodes)exit_code));
 }
 
 void DaemonProcess::DeleteAllDesktopSessions() {
   while (!desktop_sessions_.empty()) {
     delete desktop_sessions_.front();
     desktop_sessions_.pop_front();
   }
 }
 
 }  // namespace remoting