From e2741f0ca9b465aa9e57ffd52cf47a1185375a1e Mon Sep 17 00:00:00 2001
From: Timo Tijhof <krinklemail@gmail.com>
Date: Fri, 4 Sep 2015 04:04:01 +0100
Subject: [PATCH] resourceloader: Expose fetch promise in addScript()

Prerequisite for T68598 so that callers can handle errors.

Change-Id: I214159001e04c4b51453e47299b8e5d5033c258a
---
 resources/src/mediawiki/mediawiki.js | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/resources/src/mediawiki/mediawiki.js b/resources/src/mediawiki/mediawiki.js
index 49c350aabb..b36dbeab89 100644
--- a/resources/src/mediawiki/mediawiki.js
+++ b/resources/src/mediawiki/mediawiki.js
@@ -1143,10 +1143,10 @@
 			 *
 			 * @private
 			 * @param {string} src URL to script, will be used as the src attribute in the script tag
-			 * @param {Function} [callback] Callback which will be run when the script is done
+			 * @return {jQuery.Promise}
 			 */
-			function addScript( src, callback ) {
-				$.ajax( {
+			function addScript( src ) {
+				return $.ajax( {
 					url: src,
 					dataType: 'script',
 					// Force jQuery behaviour to be for crossDomain. Otherwise jQuery would use
@@ -1156,7 +1156,7 @@
 					// text, so we'd need to $.globalEval, which then messes up line numbers.
 					crossDomain: true,
 					cache: true
-				} ).always( callback );
+				} );
 			}
 
 			/**
@@ -1217,7 +1217,7 @@
 								return;
 							}
 
-							addScript( arr[ i ], function () {
+							addScript( arr[ i ] ).always( function () {
 								nestedAddScript( arr, callback, i + 1 );
 							} );
 						};
-- 
2.20.1