Merge "jquery.suggestions: Use Object.create( null )"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Mon, 10 Jun 2019 13:41:20 +0000 (13:41 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Mon, 10 Jun 2019 13:41:20 +0000 (13:41 +0000)
resources/src/jquery/jquery.suggestions.js

index 4a1d637..9c7573d 100644 (file)
@@ -90,8 +90,6 @@
 
 ( function () {
 
-       var hasOwn = Object.hasOwnProperty;
-
        /**
         * Cancel any delayed maybeFetch() call and callback the context so
         * they can cancel any async fetching if they use AJAX or something.
                        ) {
                                context.data.prevText = val;
                                // Try cache first
-                               if ( context.config.cache && hasOwn.call( cache, val ) ) {
+                               if ( context.config.cache && val in cache ) {
                                        if ( mw.now() - cache[ val ].timestamp < context.config.cacheMaxAge ) {
                                                context.data.$textbox.suggestions( 'suggestions', cache[ val ].suggestions );
                                                if ( typeof context.config.update.after === 'function' ) {
                                        prevText: null,
 
                                        // Cache of fetched suggestions
-                                       cache: {},
+                                       cache: Object.create( null ),
 
                                        // Number of results visible without scrolling
                                        visibleResults: 0,