Auto merge of #3159 - Turbo87:progress, r=pichfl

services/progress: Add missing `didCancel()` check

The `updateTask` uses the `.drop()` modifier, which means it cancels automatically if there is already a running instance. This cancellation causes a `TaskCancelation` error rejection on the `perform()` call, and we should ignore that kind of error instead of reporting it to Sentry.

Resolves #3157

r? `@pichfl`

Author: bors

app/services/progress.js

@@ -4,7 +4,7 @@ import { tracked } from '@glimmer/tracking';
 import Ember from 'ember';
 
 import * as Sentry from '@sentry/browser';
-import { rawTimeout, task } from 'ember-concurrency';
+import { didCancel, rawTimeout, task } from 'ember-concurrency';
 
 const SPEED = 200;
 
@@ -26,8 +26,10 @@ export default class ProgressService extends Service {
 
   @task(function* (promise) {
     this.updateTask.perform().catch(error => {
-      // this task shouldn't be able to fail, but if it does we'll let Sentry know
-      Sentry.captureException(error);
+      if (!didCancel(error)) {
+        // this task shouldn't be able to fail, but if it does we'll let Sentry know
+        Sentry.captureException(error);
+      }
     });
 
     yield promise;