CSS: window.getDefaultComputedStyle may return null

Fixes #14736
Closes gh-1501
(cherry picked from commit 51910ba8a5aff4cd1a811f1fdb76a6c62a09a666)

diff --git a/AUTHORS.txt b/AUTHORS.txt
index 9bda6ee1d88eac65ebec80c6cf030ac89b7bcc5f..70a97e8c3ef8263131dfec9d6f5d89106d0e7d14 100644 (file)
--- a/AUTHORS.txt
+++ b/AUTHORS.txt
@@ -209,4 +209,5 @@ John Paul <john@johnkpaul.com>
 S. Andrew Sheppard <andrew@wq.io>
 Roman Reiß <me@silverwind.io>
 Benjy Cui <benjytrys@gmail.com>
+Rodrigo Rosenfeld Rosas <rr.rosas@gmail.com>
 

diff --git a/src/css/defaultDisplay.js b/src/css/defaultDisplay.js
index 518c7d8ae6abfa20e6e66299e88ff4bf0ccc4d27..210ad4a8057e094edfbc807ec9389452ef4bed4d 100644 (file)
--- a/src/css/defaultDisplay.js
+++ b/src/css/defaultDisplay.js
@@ -13,14 +13,15 @@ var iframe,
  */
 // Called only from within defaultDisplay
 function actualDisplay( name, doc ) {
-       var elem = jQuery( doc.createElement( name ) ).appendTo( doc.body ),
+       var style,
+               elem = jQuery( doc.createElement( name ) ).appendTo( doc.body ),
 
                // getDefaultComputedStyle might be reliably used only on attached element
-               display = window.getDefaultComputedStyle ?
+               display = window.getDefaultComputedStyle && ( style = window.getDefaultComputedStyle( elem[ 0 ] ) ) ?
 
                        // Use of this method is a temporary fix (more like optmization) until something better comes along,
                        // since it was removed from specification and supported only in FF
-                       window.getDefaultComputedStyle( elem[ 0 ] ).display : jQuery.css( elem[ 0 ], "display" );
+                       style.display : jQuery.css( elem[ 0 ], "display" );
 
        // We don't have any data stored on the element,
        // so use "detach" method as fast way to get rid of the element