chore: Remove v2 rrweb checks

[benjackwhite]

Changes

Follow up after #1078 is fully released

WARNING: We should only merge this once the recorder.js is 100% rolled out to the CDN as rrweb2

• Removes all the extra loader code checking for the file to already have been loaded, instead just looking for the window rrweb method

Checklist

• Tests for new code (see advice on the tests we use)
• Accounted for the impact of any changes across different browsers
• Accounted for backwards compatibility of any changes (no breaking changes in posthog-js!)

[github-actions]

Size Change: -807 B (0%)

Total Size: 940 kB

Filename	Size	Change
dist/array.full.js	224 kB	-207 B (0%)
dist/array.js	124 kB	-200 B (0%)
dist/es.js	124 kB	-200 B (0%)
dist/module.js	124 kB	-200 B (0%)

ℹ️ View Unchanged

Filename	Size
dist/exception-autocapture.js	12.1 kB
dist/recorder-v2.js	106 kB
dist/recorder.js	106 kB
dist/surveys-module-previews.js	62 kB
dist/surveys.js	58.3 kB

_{compressed-size-action}

[daibhin]

Looks good whenever the other PR is merged and the CDN is ready. How long would you expect the cache to take?

[pauldambra]

How long would you expect the cache to take?

It should be a five minute cache. You can check the headers on the JS response. I guess maybe we'd need to check multiple URLs at the moment 🤷

[daibhin]

Confirmed that https://internal-e.posthog.com/static/recorder.js now contains the v2 config:

...
{return{name:kn,observer:wn,options:e}};Jt&&(Jt.rrweb={record:At,version:"v2",rrwebVersion:"2.0.0-alpha.11"},Jt.rrwebConsoleRecord={getRecordConsolePlugin:function(e){return{name:"rrweb/console@1",observer:Ft,options:e}}},Jt.getRecordNetworkPlugin=Mn),e.NETWORK_PLUGIN_NAME=kn,e.default=At,e.findLast=ln,e.getRecordNetworkPlugin=Mn,e.patch=un,Object.defineProperty(e,"__esModule",{value:!0})}({});
//# sourceMappingURL=recorder.js.map

[pauldambra]

LGTM

feels like a monday morning rather than a friday afternoon change 🙈 but don't let me block

[daibhin]

Happy to merge this one now @benjackwhite?