From 849a6d9f16d285380ee85a5659b0411d5a62d901 Mon Sep 17 00:00:00 2001 From: Brion Vibber Date: Wed, 15 Jul 2015 17:52:46 -0500 Subject: [PATCH] Revert no-longer-needed fix for MS Edge browser Bug with history.pushState in Edge fixed in Win10 build 10240; Dave Storey is asking us to remove the workaround to keep our code pure and them honest ;) Reverts I6ea4d367 Bug: T104381 Change-Id: I5f53416dda7c31662330dc5ec67e1bbea55dace7 --- resources/mmv/mmv.bootstrap.js | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/resources/mmv/mmv.bootstrap.js b/resources/mmv/mmv.bootstrap.js index cac6af5cb..0d2b27212 100644 --- a/resources/mmv/mmv.bootstrap.js +++ b/resources/mmv/mmv.bootstrap.js @@ -469,12 +469,7 @@ hash = window.location.href.replace( /#.*$/, '' ); } - try { - window.history.pushState( null, title, hash ); - } catch ( ex ) { - // Workaround for Edge bug -- https://phabricator.wikimedia.org/T104381 - window.history.pushState( null, title, hash + '#' ); - } + window.history.pushState( null, title, hash ); } else { // Since we voluntarily changed the hash, we don't want MMVB.hash (which will trigger on hashchange event) to treat it this.skipNextHashHandling = true;