Disable screenshot uploading if there has been user input.

chrome/browser/chromeos/policy/remote_commands/device_command_screenshot_job.h

Index: chrome/browser/chromeos/policy/remote_commands/device_command_screenshot_job.h
diff --git a/chrome/browser/chromeos/policy/remote_commands/device_command_screenshot_job.h b/chrome/browser/chromeos/policy/remote_commands/device_command_screenshot_job.h
index 819ca33c472958c9b9e2e518cdcb786cace2fb01..18e473411361ea59cf7d118c4077de7bd7e4f840 100644
--- a/chrome/browser/chromeos/policy/remote_commands/device_command_screenshot_job.h
+++ b/chrome/browser/chromeos/policy/remote_commands/device_command_screenshot_job.h
@@ -25,6 +25,8 @@
 
 namespace policy {
 
+class StatusUploader;
+
 // This class implementens a RemoteCommandJob that captures a screenshot from
 // each attached screen and uploads the collected PNG data using UploadJob to
 // the url specified in the "fileUploadUrl" field of the CommandPayload. Only
@@ -76,7 +78,8 @@ class DeviceCommandScreenshotJob : public RemoteCommandJob,
                                                   UploadJob::Delegate*) = 0;
   };
 
-  explicit DeviceCommandScreenshotJob(scoped_ptr<Delegate> screenshot_delegate);
+  DeviceCommandScreenshotJob(scoped_ptr<Delegate> screenshot_delegate,
+                             StatusUploader* uploader);

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

This is quite a heavy dependency just to call a bool(void) function.  Have you
considered passing in a simple Callback<void(bool* allowed)>
is_allowed_callback. This could even be a member function weakly bound to an
instance if you have to worry about lifetime

bool allowed = false;
is_allowed_callback.Run(&allowed);

OR

You could put that method into the ScreenshotDelegate. This would probably mean
that the ScreenshotDelegate aquires a dependency on DeviceStatusCollector. Hmm.
Also I can see the argument that you may want this IsSessionDataUploadAllowed()
to be more generically available although currently the method is only called
for screenshots.

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

Moved logic into ScreenshotDelegate - good call.

   ~DeviceCommandScreenshotJob() override;
 
   // RemoteCommandJob:
@@ -123,6 +126,10 @@ class DeviceCommandScreenshotJob : public RemoteCommandJob,
   // The upload job instance that will upload the screenshots.
   scoped_ptr<UploadJob> upload_job_;
 
+  // StatusUploader used to check whether screenshot uploads are allowed in
+  // this session.
+  StatusUploader* status_uploader_;
+
   base::WeakPtrFactory<DeviceCommandScreenshotJob> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(DeviceCommandScreenshotJob);