Add logging to remote commands

components/policy/core/common/remote_commands/remote_command_job.cc

 // Copyright 2015 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 "components/policy/core/common/remote_commands/remote_command_job.h"
 
 #include <utility>
 
 #include "base/bind.h"
-#include "base/logging.h"
+#include "base/chromeos/logging.h"
 
 namespace policy {
 
 namespace {
 
 const int kDefaultCommandTimeoutInMinutes = 3;
 
 }  // namespace
 
 namespace em = enterprise_management;
@@ skipping 20 matching lines @@
   if (command.has_age_of_command()) {
     // Use age of command provided by server to estimate the command issued time
     // as a local TimeTick. We need to store issued time instead of age of
     // command, since the execution time of command might be different from the
     // time we got it from server.
     // It's just an estimation since we lost the time the response was
     // transmitted over the network.
     issued_time_ =
         now - base::TimeDelta::FromMilliseconds(command.age_of_command());
   } else {
-    LOG(WARNING) << "No age_of_command provided be server for command "
-                 << unique_id_ << ".";
+    CHROMEOS_SYSLOG(WARNING)
+        << "No age_of_command provided be server for command " << unique_id_
+        << ".";
     // Otherwise, assuming the command was issued just now.
     issued_time_ = now;
   }
 
   if (!ParseCommandPayload(command.payload()))
     return false;
 
+  switch (command.type()) {
+    case em::RemoteCommand_Type_COMMAND_ECHO_TEST: {
+      CHROMEOS_SYSLOG(WARNING) << "Remote echo test command " << unique_id_
+                               << " initialized.";
+      break;
+    }
+    case em::RemoteCommand_Type_DEVICE_REBOOT: {
+      CHROMEOS_SYSLOG(WARNING) << "Remote reboot command " << unique_id_
+                               << " initialized.";
+      break;
+    }
+    case em::RemoteCommand_Type_DEVICE_SCREENSHOT: {
+      CHROMEOS_SYSLOG(WARNING) << "Remote screenshot command " << unique_id_
+                               << " initialized.";
+      break;
+    }
+  }
   status_ = NOT_STARTED;
   return true;
 }
 
 bool RemoteCommandJob::Run(base::TimeTicks now,
                            const FinishedCallback& finished_callback) {
   DCHECK(thread_checker_.CalledOnValidThread());
 
-  if (status_ == INVALID)
+  if (status_ == INVALID) {
+    CHROMEOS_SYSLOG(ERROR) << "Remote command " << unique_id_ << " is invalid.";
     return false;
+  }
 
   DCHECK_EQ(NOT_STARTED, status_);
 
   if (IsExpired(now)) {
+    CHROMEOS_SYSLOG(ERROR) << "Remote command " << unique_id_
+                           << " expired (it was issued " << now - issued_time_
+                           << " ago).";
     status_ = EXPIRED;
     return false;
   }
 
   execution_started_time_ = now;
   status_ = RUNNING;
   finished_callback_ = finished_callback;
 
   RunImpl(base::Bind(&RemoteCommandJob::OnCommandExecutionFinishedWithResult,
                      weak_factory_.GetWeakPtr(), true),
@@ skipping 63 matching lines @@
   DCHECK_EQ(RUNNING, status_);
   status_ = succeeded ? SUCCEEDED : FAILED;
 
   result_payload_ = std::move(result_payload);
 
   if (!finished_callback_.is_null())
     finished_callback_.Run();
 }
 
 }  // namespace policy