Disable chromoting service on permanent errors.

remoting/host/daemon_process_win.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/base_switches.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/location.h"
@@ skipping 122 matching lines @@
 
   if (!InitializePairingRegistry()) {
     CrashNetworkProcess(FROM_HERE);
     return;
   }
 
   DaemonProcess::OnChannelConnected(peer_pid);
 }
 
 void DaemonProcessWin::OnPermanentError(int exit_code) {
-  // Change the service start type to 'manual' if the host has been deleted
-  // remotely. This way the host will not be started every time the machine
-  // boots until the user re-enable it again.
-  if (exit_code == kInvalidHostIdExitCode)
+  DCHECK(kMinPermanentErrorExitCode <= exit_code &&
+         exit_code <= kMaxPermanentErrorExitCode);
+
+  // Both kInvalidHostIdExitCode and kInvalidOauthCredentialsExitCode are
+  // errors then will never go away with the current config.
+  // Disabling automatic service start until the host is re-enabled and config
+  // updated.
+  if (exit_code == kInvalidHostIdExitCode ||
+      exit_code == kInvalidOauthCredentialsExitCode) {
     DisableAutoStart();
+  }
 
   DaemonProcess::OnPermanentError(exit_code);
 }
 
 void DaemonProcessWin::SendToNetwork(IPC::Message* message) {
   if (network_launcher_) {
     network_launcher_->Send(message);
   } else {
     delete message;
   }
@@ skipping 213 matching lines @@
                 << "\\" << kPairingRegistrySecretsKeyName;
     return false;
   }
 
   pairing_registry_privileged_key_.Set(privileged.Take());
   pairing_registry_unprivileged_key_.Set(unprivileged.Take());
   return true;
 }
 
 }  // namespace remoting