ff: Making error stacktrace parser more defensive. Fixes issue 8176

ldy810 · Apr 3, 2015 · 1ec882e · 1ec882e
1 parent d313aa3
commit 1ec882e
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/javascript/firefox-driver/js/error.js b/javascript/firefox-driver/js/error.js
@@ -54,12 +54,21 @@ fxdriver.error.toJSON = function(ex) {
       }
 
       match = frame.match(/^([\w./<$]+)?(?:\(.*\))?@(.+)?$/);
-      stackFrames.push({
+      if (match) {
+        stackFrames.push({
           'methodName': match[1],
           'fileName': match[2],
           'lineNumber': lineNumber,
           'columnNumber': columnNumber
         });
+      } else {
+        stackFrames.push({
+          'methodName': frame,
+          'fileName': "?",
+          'lineNumber': "?",
+          'columnNumber': "?"
+        });
+      }
     }
   }