Return whether request to schedule immediate status report was successful.

This information will be used to collect metrics on events that trigger status reports. We need to know  whether the call to schedule a report was successful or not.

Bug: 919132
Change-Id: If0be7c5d7d11a686707a1a20a2a6d6ad519a31ba
Reviewed-on: https://chromium-review.googlesource.com/c/1418272Reviewed-by: Bartosz Fabianowski <bartfab@chromium.org>
Commit-Queue: Leonardo Padilha <ldaguilar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#626118}

chrome/browser/chromeos/policy/status_uploader.cc

@@ -72,13 +72,13 @@ StatusUploader::~StatusUploader() {
   MediaCaptureDevicesDispatcher::GetInstance()->RemoveObserver(this);
 }
 
-void StatusUploader::ScheduleNextStatusUpload(bool immediately) {
+bool StatusUploader::ScheduleNextStatusUpload(bool immediately) {
   // Don't schedule a new status upload if there's a status upload in progress
   // (it will be scheduled once the current one completes).
   if (status_upload_in_progress_) {
     SYSLOG(INFO) << "In the middle of a status upload, not scheduling the next "
                  << "one until this one finishes.";
-    return;
+    return false;
   }
 
   // Calculate when to fire off the next update (if it should have already
@@ -93,6 +93,7 @@ void StatusUploader::ScheduleNextStatusUpload(bool immediately) {
   upload_callback_.Reset(base::Bind(&StatusUploader::UploadStatus,
                                     base::Unretained(this)));
   task_runner_->PostDelayedTask(FROM_HERE, upload_callback_.callback(), delay);
+  return true;
 }
 
 void StatusUploader::RefreshUploadFrequency() {
@@ -172,8 +173,8 @@ void StatusUploader::OnRequestUpdate(int render_process_id,
   }
 }
 
-void StatusUploader::ScheduleNextStatusUploadImmediately() {
-  ScheduleNextStatusUpload(true);
+bool StatusUploader::ScheduleNextStatusUploadImmediately() {
+  return ScheduleNextStatusUpload(true);
 }
 
 void StatusUploader::UploadStatus() {

chrome/browser/chromeos/policy/status_uploader.h

@@ -55,7 +55,9 @@ class StatusUploader : public MediaCaptureDevicesDispatcher::Observer {
                        blink::MediaStreamType stream_type,
                        const content::MediaRequestState state) override;
 
-  void ScheduleNextStatusUploadImmediately();
+  // Returns true if the next status upload has been scheduled successfully.
+  // Returns false if there is already an ongoing status report.
+  bool ScheduleNextStatusUploadImmediately();
 
   const DeviceStatusCollector* device_status_collector() const {
     return collector_.get();
@@ -77,8 +79,9 @@ class StatusUploader : public MediaCaptureDevicesDispatcher::Observer {
   void OnUploadCompleted(bool success);
 
   // Helper method that figures out when the next status upload should
-  // be scheduled.
-  void ScheduleNextStatusUpload(bool immediately = false);
+  // be scheduled. Returns true if the next status upload has been scheduled
+  // successfully, returns false if there is already an ongoing status report.
+  bool ScheduleNextStatusUpload(bool immediately = false);
 
   // Updates the upload frequency from settings and schedules a new upload
   // if appropriate.