OutputPage: Remove 'X-UA-Compatible' header (was for IE8-10 JS compat)
authorElliott Eggleston <ejegg@ejegg.com>
Thu, 22 Mar 2018 22:29:29 +0000 (17:29 -0500)
committerKrinkle <krinklemail@gmail.com>
Mon, 28 May 2018 00:04:05 +0000 (00:04 +0000)
Save 26 bytes each request.

We're not sending JS to those browsers any more, they're dying out, and
apparently just the `<!DOCTYPE html>` is enough to trigger Edge mode.

https://stackoverflow.com/questions/26346917/why-use-x-ua-compatible-ie-edge-anymore

Bug: T27378
Change-Id: I5418d4b043a8c8669886c39a93732e4a7d50337e

includes/OutputPage.php

index 6c45d9c..52940bc 100644 (file)
@@ -2417,10 +2417,6 @@ class OutputPage extends ContextSource {
                $response->header( 'Content-type: ' . $config->get( 'MimeType' ) . '; charset=UTF-8' );
                $response->header( 'Content-language: ' . $wgContLang->getHtmlCode() );
 
-               // Avoid Internet Explorer "compatibility view" in IE 8-10, so that
-               // jQuery etc. can work correctly.
-               $response->header( 'X-UA-Compatible: IE=Edge' );
-
                if ( !$this->mArticleBodyOnly ) {
                        $sk = $this->getSkin();