From 6bead2fe75539b853b29ece12163172312e11458 Mon Sep 17 00:00:00 2001
From: Timo Tijhof <krinklemail@gmail.com>
Date: Mon, 30 Oct 2017 19:16:52 -0700
Subject: [PATCH] resourceloader: Remove outdated readyState handler for base
 modules request

There is a bug in IE10 where under certain conditions[1], the browser will
fire 'onreadystatechange' on script tags before the script has executed.

To avoid this bug, simply remove this outdated compatibility code. It was
originally placed in startup.js in 2011 for compatibility with IE6-8, and
was inspired by the internal source code behind jQuery's ajax transport for
script tags.

jQuery no longer does this either and uses 'script.onload' in all browsers,
which is supported by IE 9 and above. MediaWiki currently requires IE 10
for its Grade A experience.

Looking further, it actually seems that Microsoft has now retroactively
documented that readyState="loaded" in IE6-10, for script tags, represents
the completion of the download, not the execution. [2] Presumably the reason
we did not find this bug earlier is because of the rare conditions [1], and
because even then, on most page views, readyState="loaded" is never fired.
Instead, usually the event is skipped by IE and the first and only readyState
event is readyState="complete".

[1] The conditions are: A relatively short HTML page, with a script from
<script async href=".."> that in turn appends a dynamic script tag, where
the <script async> is followed by a <link rel=stylesheet> that is relatively
large/complex (possibly due to use of cross-domain @imports, not sure).

[2] https://connect.microsoft.com/IE/feedback/details/729164/ie10-dynamic-script-element-fires-loaded-readystate-prematurely

Bug: T178943
Change-Id: Id3de61515e4701a7ef59e4a4d8643a0e09f8f589
---
 resources/src/startup.js | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/resources/src/startup.js b/resources/src/startup.js
index d672d7697b..e8585c32fd 100644
--- a/resources/src/startup.js
+++ b/resources/src/startup.js
@@ -155,14 +155,12 @@ window.isCompatible = function ( str ) {
 
 	script = document.createElement( 'script' );
 	script.src = $VARS.baseModulesUri;
-	script.onload = script.onreadystatechange = function () {
-		if ( !script.readyState || /loaded|complete/.test( script.readyState ) ) {
-			// Clean up
-			script.onload = script.onreadystatechange = null;
-			script = null;
-			// Callback
-			startUp();
-		}
+	script.onload = function () {
+		// Clean up
+		script.onload = null;
+		script = null;
+		// Callback
+		startUp();
 	};
 	document.getElementsByTagName( 'head' )[ 0 ].appendChild( script );
 }() );
-- 
2.20.1