resourceloader: Remove mwPerformance stub and rename mwLoadStart mark
authorTimo Tijhof <krinklemail@gmail.com>
Fri, 20 Apr 2018 00:47:08 +0000 (01:47 +0100)
committerKrinkle <krinklemail@gmail.com>
Mon, 23 Apr 2018 17:59:06 +0000 (17:59 +0000)
commit512fcfdc6082ca6aa60301b6208eb2bc3782ed3b
tree4c7dd3a3a6744e98957be706a9a4ca92d413d7bd
parentd672dab1c945764ea113c0b78e1591d11d94b2b1
resourceloader: Remove mwPerformance stub and rename mwLoadStart mark

Remove the private 'mwPerformance' stub, in favour of simply calling
performance.mark() directly in a conditional, like we do elsewhere for
web APIs that don't exist in all supported browsers.

This mark (part of the W3C User Timing interface), was sometimes used
to measure a delta to mwLoadEnd, whcih is a problem because startup
loads asynchronously. The point it executes is intentionally variable.
What matters is when the overall JS load process ends, which is composed
of multiple requests.

Keep the mark, but rename to 'mwStartup' which better describes its
purpose and can remain useful.

Bug: T160315
Bug: T192623
Change-Id: I6e2049e7252e5c2be3c5bc6617703b62e5edafb4
maintenance/jsduck/eg-iframe.html
resources/src/mediawiki/mediawiki.js
resources/src/startup.js