Drop the service worker warm-up now that listeners register in time

src/document_start.ts

@@ -2,12 +2,6 @@ import { createLogger } from './logger';
 
 const log = createLogger();
 
-const warmServiceWorker = (): void => {
-  chrome.runtime
-    .sendMessage({ contentScriptQuery: 'warmup' })
-    .catch(() => log.debug('Warm-up message went unanswered; the worker still booted'));
-};
-
 const captureUrlCartParam = (): void => {
   if (!window.location.search.includes('bes_cart')) return;
 
@@ -30,5 +24,4 @@ const captureUrlCartParam = (): void => {
   window.location.replace(newUrl);
 };
 
-warmServiceWorker();
 captureUrlCartParam();

test/document_start.test.ts

@@ -35,23 +35,17 @@ const runDocumentStart = async () => {
   await import('../src/document_start');
 };
 
-describe('document_start service worker warm-up', () => {
+describe('document_start service worker messaging', () => {
   beforeEach(() => {
     mockSendMessage.mockClear();
   });
 
-  it('pings the service worker so it boots ahead of document_end', async () => {
+  it('does not message the service worker, so nothing depends on it being awake', async () => {
     setLocation('');
 
     await runDocumentStart();
 
-    expect(mockSendMessage).toHaveBeenCalledWith({ contentScriptQuery: 'warmup' });
-  });
-
-  it('survives the ping going unanswered', async () => {
-    setLocation('');
-
-    await expect(runDocumentStart()).resolves.not.toThrow();
+    expect(mockSendMessage).not.toHaveBeenCalled();
   });
 });