getPhotoCapabilities and getPhotoSettings reject with an ended track

This CL makes sure InvalidStateError and OperationError are respectively
thrown when calling getPhotoCapabilitlies & getSettings if the
MediaStreamTrack is ended and stopped while getting photo capabilities
and settings.

Bug: 1134569
Change-Id: Ie318319885b7796b17cb155ffcd2a81d7f98de17
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2445016Reviewed-by: Reilly Grant <reillyg@chromium.org>
Commit-Queue: François Beaufort <beaufort.francois@gmail.com>
Cr-Commit-Position: refs/heads/master@{#813283}

third_party/blink/renderer/modules/imagecapture/image_capture.cc

@@ -181,6 +181,13 @@ ScriptPromise ImageCapture::getPhotoCapabilities(ScriptState* script_state) {
   auto* resolver = MakeGarbageCollected<ScriptPromiseResolver>(script_state);
   ScriptPromise promise = resolver->Promise();
 
+  if (TrackIsInactive(*stream_track_)) {
+    resolver->Reject(MakeGarbageCollected<DOMException>(
+        DOMExceptionCode::kInvalidStateError,
+        "The associated Track is in an invalid state."));
+    return promise;
+  }
+
   if (!service_.is_bound()) {
     resolver->Reject(MakeGarbageCollected<DOMException>(
         DOMExceptionCode::kNotFoundError, kNoServiceError));
@@ -207,6 +214,13 @@ ScriptPromise ImageCapture::getPhotoSettings(ScriptState* script_state) {
   auto* resolver = MakeGarbageCollected<ScriptPromiseResolver>(script_state);
   ScriptPromise promise = resolver->Promise();
 
+  if (TrackIsInactive(*stream_track_)) {
+    resolver->Reject(MakeGarbageCollected<DOMException>(
+        DOMExceptionCode::kInvalidStateError,
+        "The associated Track is in an invalid state."));
+    return promise;
+  }
+
   if (!service_.is_bound()) {
     resolver->Reject(MakeGarbageCollected<DOMException>(
         DOMExceptionCode::kNotFoundError, kNoServiceError));
@@ -963,6 +977,14 @@ void ImageCapture::OnMojoGetPhotoState(
     return;
   }
 
+  if (TrackIsInactive(*stream_track_)) {
+    resolver->Reject(MakeGarbageCollected<DOMException>(
+        DOMExceptionCode::kOperationError,
+        "The associated Track is in an invalid state."));
+    service_requests_.erase(resolver);
+    return;
+  }
+
   photo_settings_ = PhotoSettings::Create();
   photo_settings_->setImageHeight(photo_state->height->current);
   photo_settings_->setImageWidth(photo_state->width->current);

third_party/blink/web_tests/external/wpt/mediacapture-image/getPhotoCapabilities-expected.txt

-This is a testharness.js-based test.
-PASS exercises ImageCapture.getPhotoCapabilities()
-FAIL getPhotoCapabilities() of an ended Track should throw "InvalidStateError" assert_unreached: Should have rejected: undefined Reached unreachable code
-FAIL throw "OperationError" when the MediaStreamTrack is stopped while getting photo capabilities assert_unreached: should throw "OperationError" Reached unreachable code
-Harness: the test ran to completion.
-

third_party/blink/web_tests/external/wpt/mediacapture-image/getPhotoSettings-expected.txt

-This is a testharness.js-based test.
-PASS exercises ImageCapture.getPhotoSettings()
-FAIL getPhotoSettings() of an ended Track should throw "InvalidStateError" assert_unreached: Should have rejected: undefined Reached unreachable code
-FAIL throw "OperationError" when the MediaStreamTrack is stopped while getting photo settings assert_unreached: should throw "OperationError" Reached unreachable code
-Harness: the test ran to completion.
-