Disable screenshot uploading if there has been user input.

chrome/browser/chromeos/policy/status_uploader.h

Index: chrome/browser/chromeos/policy/status_uploader.h
diff --git a/chrome/browser/chromeos/policy/status_uploader.h b/chrome/browser/chromeos/policy/status_uploader.h
index 2eead1847a73b43e4acc58e055eddf324f58c412..261a786cbb2613bf9a36e18b737913582c5573bc 100644
--- a/chrome/browser/chromeos/policy/status_uploader.h
+++ b/chrome/browser/chromeos/policy/status_uploader.h
@@ -38,12 +38,22 @@ class StatusUploader {
       scoped_ptr<DeviceStatusCollector> collector,
       const scoped_refptr<base::SequencedTaskRunner>& task_runner);
 
-  ~StatusUploader();
+  virtual ~StatusUploader();
 
   // Returns the time of the last successful upload, or Time(0) if no upload
   // has ever happened.
   base::Time last_upload() const { return last_upload_; }
 
+  // Returns true if session data upload (screenshots, logs, etc) is allowed.
+  // This checks to ensure that the current session is a kiosk session, and
+  // that no user input (keyboard, mouse, touch, audio/video) has been received.
+  // Virtual to allow mocking in tests.
+  virtual bool IsSessionDataUploadAllowed();
+
+ protected:
+  // Empty constructor to be used when generating mocks.
+  StatusUploader();

[cschuet (SLOW), 2015/05/13 15:56:09]

I see the necessity to introduce this empty constructor in order to write
MockStatusUploader  without having to introduce a StatusUploader interface (I
understand that you don't want to have to pass in all those parameters into the
real constructor). But doesn't this also suggest that either StatusUploader is
not the right place for IsSessionDataUploadAllowed() or at least the screenshot
command shouldn't have a dependency on StatusUploader?

[Andrew T Wilson (Slow), 2015/05/15 09:43:51]

Removed. Thanks for the feedback - much cleaner now.

+
  private:
   // Callback invoked periodically to upload the device status from the
   // DeviceStatusCollector.