Noticed this warning in the console on test.wikipedia.org (after saving an edit) while testing the patch for T193570.
Exception in load-callback in module schema.SaveTiming:
> Error: Module "ext.navigationTiming.rumSpeedIndex" is not loaded.
at require (load.php?...:171)
at getPaintTiming (<anonymous>:206:555)
at getNavTiming (<anonymous>:207:578)
at emitSaveTiming (<anonymous>:209:709)
Looks like we forgot about the second call-path to `getNavTiming()` when adding rumSpeedIndex, namely from `emitSaveTiming`, which while being called after the load starts for `preloadedModules`, does not wait for it (and shouldn't need to)
Its report rate dropped by about 30%. I guess the reason it didn't drop further is because it all works on repeat views, because on repeat views, the unrelated load for `preloadedModules` resolves in time for emitSaveTiming.
| From 150/min to 100/min