Firefox: Fixing null pointer exception if a page is not fully loaded. F…

…ixes #1723
SeleniumHQ · Mar 2, 2016 · 71010a6 · 71010a6
1 parent c277050
commit 71010a6
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/javascript/firefox-driver/js/firefoxDriver.js b/javascript/firefox-driver/js/firefoxDriver.js
@@ -1172,11 +1172,9 @@ FirefoxDriver.prototype.mouseDoubleClick = function(respond, parameters) {
 FirefoxDriver.prototype.sendKeysToActiveElement = function(respond, parameters) {
   Utils.installWindowCloseListener(respond);
 
-  var currentlyActiveElement = Utils.getActiveElement(respond.session.getDocument());
-
-  var useElement = currentlyActiveElement;
-  var tagName = useElement.tagName.toLowerCase();
-  if (tagName == 'body' && useElement.ownerDocument.defaultView.frameElement) {
+  var useElement = Utils.getActiveElement(respond.session.getDocument());
+  if (useElement && useElement.tagName.toLowerCase() == 'body'
+      && useElement.ownerDocument.defaultView.frameElement) {
     useElement.ownerDocument.defaultView.focus();
 
     // Turns out, this is what we should be using as the target