From 8839a2053e291f4ffcf4f4cf3365f64364f304c3 Mon Sep 17 00:00:00 2001
From: Diana Huang <dkh@edx.org>
Date: Thu, 12 Sep 2013 09:09:35 -0400
Subject: [PATCH] Wait until the flash object is loaded before looking for a
 camera.

---
 lms/static/js/verify_student/photocapture.js | 27 +++++++++++++++-----
 1 file changed, 20 insertions(+), 7 deletions(-)

diff --git a/lms/static/js/verify_student/photocapture.js b/lms/static/js/verify_student/photocapture.js
index 90bf37ad50..47c6b653fb 100644
--- a/lms/static/js/verify_student/photocapture.js
+++ b/lms/static/js/verify_student/photocapture.js
@@ -230,6 +230,18 @@ function linkNewWindow(e) {
             e.preventDefault();
 }
 
+function waitForFlashLoad(func, flash_object) {
+    if(!flash_object.hasOwnProperty('percentLoaded') || flash_object.percentLoaded() < 100){
+        setTimeout(function() {
+            waitForFlashLoad(func, flash_object);
+        },
+        50);
+    }
+    else {
+        func(flash_object);
+    }
+}
+
 $(document).ready(function() {
   $(".carousel-nav").addClass('sr');
   $("#pay_button").click(function(){
@@ -266,13 +278,14 @@ $(document).ready(function() {
   if (!hasHtml5CameraSupport) {
     $("#face_capture_div").html(objectTagForFlashCamera("face_flash"));
     $("#photo_id_capture_div").html(objectTagForFlashCamera("photo_id_flash"));
-    // wait for the flash object to be loaded
-    // TODO: we need a better solution for this
-    setTimeout(function() {
-        if(browserHasFlash() && !$('#face_flash')[0].hasOwnProperty('hasCamera')) {
-            onVideoFail('NO_DEVICES_FOUND');
-        }
-    }, 1000);
+    // wait for the flash object to be loaded and then check for a camera
+    if(browserHasFlash()) {
+        waitForFlashLoad(function(flash_object) {
+            if(!flash_object.hasOwnProperty('hasCamera')){
+                onVideoFail('NO_DEVICES_FOUND');
+            }
+        }, $('#face_flash')[0]);
+    }
   }
 
   analytics.pageview("Capture Face Photo");