jquery-migrate: Disable console.trace() call by default
authorTimo Tijhof <krinklemail@gmail.com>
Thu, 5 Oct 2017 19:26:50 +0000 (20:26 +0100)
committerTimo Tijhof <krinklemail@gmail.com>
Thu, 5 Oct 2017 19:26:50 +0000 (20:26 +0100)
They are slow and spammy, especially given in modern browsers
console.warn() already provides a trace (and one that is collapsed
by default in Chrome, which seems preferred).

Change-Id: I94d3c7bf6deb47c2f3f660dfd2298ee293e7d825

resources/lib/jquery/jquery.migrate.js

index 19cdef6..992a734 100644 (file)
@@ -4,6 +4,8 @@
  *
  * Patched for MediaWiki:
  * - Add mw.track instrumentation for statistics.
+ * - Disable jQuery.migrateTrace by default. They are slow and
+ *   redundant given console.warn() already provides a trace.
  */
 ;( function( factory ) {
        if ( typeof define === "function" && define.amd ) {
@@ -61,7 +63,8 @@ jQuery.migrateWarnings = [];
 
 // Set to false to disable traces that appear with warnings
 if ( jQuery.migrateTrace === undefined ) {
-       jQuery.migrateTrace = true;
+       // PATCH: Disable extra console.trace() call --Krinkle
+       jQuery.migrateTrace = false;
 }
 
 // Forget any warnings we've already given; public