Autocomplete: Work around `isContentEditable` bug in Chrome

Fixes #14917
Closes gh-1673

diff --git a/ui/widgets/autocomplete.js b/ui/widgets/autocomplete.js
index 060d8e94f8bf21e3ecdbd944eb7d16e9482e29e1..079b0dab07bd441b22d0ac33c092bfdac79158ae 100644 (file)
--- a/ui/widgets/autocomplete.js
+++ b/ui/widgets/autocomplete.js
@@ -82,7 +82,7 @@ $.widget( "ui.autocomplete", {
                // Inputs are always single-line, even if inside a contentEditable element
                // IE also treats inputs as contentEditable
                // All other element types are determined by whether or not they're contentEditable
-               this.isMultiLine = isTextarea || !isInput && this.element.prop( "isContentEditable" );
+               this.isMultiLine = isTextarea || !isInput && this._isContentEditable( this.element );
 
                this.valueMethod = this.element[ isTextarea || isInput ? "val" : "text" ];
                this.isNewMenu = true;
@@ -614,6 +614,24 @@ $.widget( "ui.autocomplete", {
                        // Prevents moving cursor to beginning/end of the text field in some browsers
                        event.preventDefault();
                }
+       },
+
+       // Support: Chrome <=50
+       // We should be able to just use this.element.prop( "isContentEditable" )
+       // but hidden elements always report false in Chrome.
+       // https://code.google.com/p/chromium/issues/detail?id=313082
+       _isContentEditable: function( element ) {
+               if ( !element.length ) {
+                       return false;
+               }
+
+               var editable = element.prop( "contentEditable" );
+
+               if ( editable === "inherit" ) {
+                 return this._isContentEditable( element.parent() );
+               }
+
+               return editable === "true";
        }
 } );