Add logging to remote commands

chrome/browser/chromeos/policy/status_uploader.cc

 // Copyright (c) 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 "chrome/browser/chromeos/policy/status_uploader.h"
 
 #include <algorithm>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 104 matching lines @@
   // already performed the initial upload, because we want the initial upload
   // to happen immediately on startup and not get cancelled by settings changes.
   if (!last_upload_.is_null())
     ScheduleNextStatusUpload();
 }
 
 bool StatusUploader::IsSessionDataUploadAllowed() {
   // Check if we're in an auto-launched kiosk session.
   std::unique_ptr<DeviceLocalAccount> account =
       collector_->GetAutoLaunchedKioskSessionInfo();
-  if (!account)
+  if (!account) {
+    LOG(WARNING) << "Not a kiosk session, data upload is not allowed.";
     return false;
+  }
 
   // Check if there has been any user input.
-  if (!ui::UserActivityDetector::Get()->last_activity_time().is_null())
+  base::TimeTicks last_activity_time =
+      ui::UserActivityDetector::Get()->last_activity_time();
+  std::string last_activity_name =
+      ui::UserActivityDetector::Get()->last_activity_name();
+  if (!last_activity_time.is_null()) {
+    LOG(WARNING) << "User input " << last_activity_name << " detected "
+                 << (base::TimeTicks::Now() - last_activity_time)
+                 << " ago, data upload is not allowed.";

[Andrew T Wilson (Slow), 2016/05/04 14:43:22]

Can we also show how soon after boot the input happened, using
base::SysInfo::Uptime()?

[Marton Hunyady, 2016/05/09 13:48:15]

Done.

     return false;
+  }
 
   // Screenshot is allowed as long as we have not captured media.
-  return !has_captured_media_;
+  if (has_captured_media_) {
+    LOG(WARNING) << "Media has been captured, data upload is not allowed.";
+    return false;
+  } else {
+    return true;
+  }
 }
 
 void StatusUploader::OnRequestUpdate(int render_process_id,
                                      int render_frame_id,
                                      content::MediaStreamType stream_type,
                                      const content::MediaRequestState state) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   // If a video or audio capture stream is opened, set a flag so we disallow
   // upload of potentially sensitive data.
   if (state == content::MEDIA_REQUEST_STATE_OPENING &&
@@ skipping 37 matching lines @@
 
   // If the upload was successful, tell the collector so it can clear its cache
   // of pending items.
   if (success)
     collector_->OnSubmittedSuccessfully();
 
   ScheduleNextStatusUpload();
 }
 
 }  // namespace policy