From 4913870177c537ac4f912e44da1addfac0860095 Mon Sep 17 00:00:00 2001
From: Niklas Gruhn <niklas@gruhn.me>
Date: Fri, 22 May 2020 17:29:07 +0200
Subject: [PATCH] fix(@init): capabilities empty on some devices

According to

  https://oberhofer.co/mediastreamtrack-and-its-capabilities/#queryingcapabilities

on some devices, getCapabilities only returns a non-empty object
aftersome delay. There is no appropriate event so we have to add
some constant timeout.
---
 demo.html          |  7 +++++--
 src/misc/camera.js | 15 ++++++++++-----
 2 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/demo.html b/demo.html
index 03f5164a..149d6bbb 100644
--- a/demo.html
+++ b/demo.html
@@ -33,7 +33,9 @@
         {{ errorMessage }}
       </p>
 
-      <qrcode-stream @decode="onDecode" camera="rear" @init="onInit"></qrcode-stream>
+      <qrcode-stream @decode="onDecode" :torch="torch" @init="onInit"></qrcode-stream>
+
+      <button @click="torch=!torch">Torch On/Off</button>
     </div>
   </body>
   <script>
@@ -43,7 +45,8 @@
       data() {
         return {
           decodedContent: '',
-          errorMessage: ''
+          errorMessage: '',
+          torch: false
         }
       },
 
diff --git a/src/misc/camera.js b/src/misc/camera.js
index 76bf71a4..8098de6c 100644
--- a/src/misc/camera.js
+++ b/src/misc/camera.js
@@ -1,7 +1,7 @@
 import adapterFactory from "webrtc-adapter/src/js/adapter_factory.js";
 import { StreamApiNotSupportedError, InsecureContextError } from "./errors.js";
 import { imageDataFromVideo } from "./image-data.js";
-import { eventOn } from "callforth";
+import { eventOn, timeout } from "callforth";
 
 class Camera {
   constructor(videoEl, stream) {
@@ -19,7 +19,6 @@ class Camera {
 
   getCapabilities() {
     const [track] = this.stream.getVideoTracks();
-
     return track.getCapabilities();
   }
 }
@@ -121,12 +120,18 @@ export default async function(videoEl, { camera, torch }) {
 
   await eventOn(videoEl, "loadeddata");
 
+  // According to: https://oberhofer.co/mediastreamtrack-and-its-capabilities/#queryingcapabilities
+  // On some devices, getCapabilities only returns a non-empty object after
+  // some delay. There is no appropriate event so we have to add a constant timeout
+  await timeout(500);
+
   if (torch) {
     const [track] = stream.getVideoTracks();
+    const capabilities = track.getCapabilities();
 
-    try {
-      await track.applyConstraints({ advanced: [{ torch: true }] });
-    } catch (error) {
+    if (capabilities.torch) {
+      track.applyConstraints({ advanced: [{ torch: true }] });
+    } else {
       console.warn("device does not support torch capability");
     }
   }