Improve extension performance

[younesaassila]

Current implementation first create an array of lowercase strings then calls includes on the lowercase channel name. Instead use array.some() so that only strings up to a matching one would be put to lowercase (plus avoids new array allocation) Done

[younesaassila]

I really should figure out how to measure web extension perf overall

https://github.com/joannechen1223/webext-profiler ?

[younesaassila]

Here are the results using the above-mentioned tool:

┌───────────┬──────────────┬────────────────┬─────────────────────┬────────────────┬──────────────────┬──────────────────────┬────────────────┐
│ (index)   │ taskDuration │ layoutDuration │ recalcStyleDuration │ scriptDuration │ pageLoadDuration │ subResourcesDuration │ jsHeapUsedSize │
├───────────┼──────────────┼────────────────┼─────────────────────┼────────────────┼──────────────────┼──────────────────────┼────────────────┤
│ no ext    │ 2759.3152    │ 84.4202        │ 93.3886             │ 1871.2868      │ 2476.3           │ 1787.64              │ 45.622         │
│ ttvlolpro │ 2789.7096    │ 86.146         │ 95.3156             │ 1887.2536      │ 2786.28          │ 2088.68              │ 46.3255        │
└───────────┴──────────────┴────────────────┴─────────────────────┴────────────────┴──────────────────┴──────────────────────┴────────────────┘

Optimizations necessary for:

• Sub Resources Duration (ms) (+16.84%) PerformanceNavigationTiming.loadEventEnd — PerformanceNavigationTiming.domContentLoadedEventEnd
• Page Load Duration (ms) (+12.52%) PerformanceNavigationTiming.loadEventEnd — PerformanceEntry.startTime

Docs: https://joannechen1223.github.io/posts/webext-profiler/

Test was to load https://www.twitch.tv/ (5 iterations)