From 7f35349409009d5e53c83b64ce79caae62b4274f Mon Sep 17 00:00:00 2001
From: Randolf J <jrandolf@google.com>
Date: Thu, 1 Sep 2022 14:41:41 +0200
Subject: [PATCH 1/5] chore: remove unnecessary type signature

---
 src/common/Frame.ts | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/src/common/Frame.ts b/src/common/Frame.ts
index bb99290827567..ebc32a6f84698 100644
--- a/src/common/Frame.ts
+++ b/src/common/Frame.ts
@@ -771,12 +771,25 @@ export class Frame {
       content += `//# sourceURL=${path.replace(/\n/g, '')}`;
     }
 
+<<<<<<< HEAD
     type = type ?? 'text/javascript';
 
     return this.worlds[MAIN_WORLD].transferHandle(
       await this.worlds[PUPPETEER_WORLD].evaluateHandle(
         async ({url, id, type, content}) => {
           const script = document.createElement('script');
+=======
+    return this.worlds[PUPPETEER_WORLD].evaluateHandle(
+      async ({url, id, type, content}) => {
+        const script = document.createElement('script');
+        if (url) {
+          script.src = url;
+        }
+        if (id) {
+          script.id = id;
+        }
+        if (type) {
+>>>>>>> 571c30c3e5 (chore: remove unnecessary type signature)
           script.type = type;
           script.text = content;
           if (url) {

From 90985801d702c254e3d46ded140c3c003049f43d Mon Sep 17 00:00:00 2001
From: Randolf J <jrandolf@google.com>
Date: Thu, 1 Sep 2022 16:44:16 +0200
Subject: [PATCH 2/5] chore: fix tests

---
 src/common/Frame.ts | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/common/Frame.ts b/src/common/Frame.ts
index ebc32a6f84698..a7d9d43d1b956 100644
--- a/src/common/Frame.ts
+++ b/src/common/Frame.ts
@@ -772,12 +772,16 @@ export class Frame {
     }
 
 <<<<<<< HEAD
+<<<<<<< HEAD
+=======
+>>>>>>> 2563c0600e (chore: fix tests)
     type = type ?? 'text/javascript';
 
     return this.worlds[MAIN_WORLD].transferHandle(
       await this.worlds[PUPPETEER_WORLD].evaluateHandle(
         async ({url, id, type, content}) => {
           const script = document.createElement('script');
+<<<<<<< HEAD
 =======
     return this.worlds[PUPPETEER_WORLD].evaluateHandle(
       async ({url, id, type, content}) => {
@@ -790,6 +794,8 @@ export class Frame {
         }
         if (type) {
 >>>>>>> 571c30c3e5 (chore: remove unnecessary type signature)
+=======
+>>>>>>> 2563c0600e (chore: fix tests)
           script.type = type;
           script.text = content;
           if (url) {

From b37ce1013ab97fe97da03b6a55569c5deb209907 Mon Sep 17 00:00:00 2001
From: Randolf J <jrandolf@google.com>
Date: Thu, 1 Sep 2022 14:34:41 +0200
Subject: [PATCH 3/5] chore: migrate `addStyleTag`

---
 docs/api/puppeteer.frame.addstyletag.md   | 16 ++---
 docs/api/puppeteer.frame.addstyletag_1.md | 25 +++++++
 docs/api/puppeteer.frame.md               |  3 +-
 docs/api/puppeteer.page.addstyletag.md    | 20 +++---
 docs/api/puppeteer.page.addstyletag_1.md  | 25 +++++++
 docs/api/puppeteer.page.md                |  3 +-
 src/common/Frame.ts                       | 71 ++++++++++++++++--
 src/common/IsolatedWorld.ts               | 87 -----------------------
 src/common/Page.ts                        | 34 ++++++---
 9 files changed, 160 insertions(+), 124 deletions(-)
 create mode 100644 docs/api/puppeteer.frame.addstyletag_1.md
 create mode 100644 docs/api/puppeteer.page.addstyletag_1.md

diff --git a/docs/api/puppeteer.frame.addstyletag.md b/docs/api/puppeteer.frame.addstyletag.md
index 5b00d734afa3b..7ca3032cf1461 100644
--- a/docs/api/puppeteer.frame.addstyletag.md
+++ b/docs/api/puppeteer.frame.addstyletag.md
@@ -4,26 +4,26 @@ sidebar_label: Frame.addStyleTag
 
 # Frame.addStyleTag() method
 
-Adds a `<link rel="stylesheet">` tag into the page with the desired url or a `<style type="text/css">` tag with the content.
+Adds a `<link rel="stylesheet">` tag into the page with the desired URL or a `<style type="text/css">` tag with the content.
 
 **Signature:**
 
 ```typescript
 class Frame {
   addStyleTag(
-    options: FrameAddStyleTagOptions
-  ): Promise<ElementHandle<HTMLStyleElement | HTMLLinkElement>>;
+    options: Omit<FrameAddStyleTagOptions, 'url'>
+  ): Promise<ElementHandle<HTMLStyleElement>>;
 }
 ```
 
 ## Parameters
 
-| Parameter | Type                                                              | Description                 |
-| --------- | ----------------------------------------------------------------- | --------------------------- |
-| options   | [FrameAddStyleTagOptions](./puppeteer.frameaddstyletagoptions.md) | Options for the style link. |
+| Parameter | Type                                                                                 | Description |
+| --------- | ------------------------------------------------------------------------------------ | ----------- |
+| options   | Omit&lt;[FrameAddStyleTagOptions](./puppeteer.frameaddstyletagoptions.md), 'url'&gt; |             |
 
 **Returns:**
 
-Promise&lt;[ElementHandle](./puppeteer.elementhandle.md)&lt;HTMLStyleElement \| HTMLLinkElement&gt;&gt;
+Promise&lt;[ElementHandle](./puppeteer.elementhandle.md)&lt;HTMLStyleElement&gt;&gt;
 
-a promise that resolves to the added tag when the stylesheets's `onload` event fires or when the CSS content was injected into the frame.
+An [element handle](./puppeteer.elementhandle.md) to the loaded `<link>` or `<style>` element.
diff --git a/docs/api/puppeteer.frame.addstyletag_1.md b/docs/api/puppeteer.frame.addstyletag_1.md
new file mode 100644
index 0000000000000..81805b3ae5f53
--- /dev/null
+++ b/docs/api/puppeteer.frame.addstyletag_1.md
@@ -0,0 +1,25 @@
+---
+sidebar_label: Frame.addStyleTag_1
+---
+
+# Frame.addStyleTag() method
+
+**Signature:**
+
+```typescript
+class Frame {
+  addStyleTag(
+    options: FrameAddStyleTagOptions
+  ): Promise<ElementHandle<HTMLLinkElement>>;
+}
+```
+
+## Parameters
+
+| Parameter | Type                                                              | Description |
+| --------- | ----------------------------------------------------------------- | ----------- |
+| options   | [FrameAddStyleTagOptions](./puppeteer.frameaddstyletagoptions.md) |             |
+
+**Returns:**
+
+Promise&lt;[ElementHandle](./puppeteer.elementhandle.md)&lt;HTMLLinkElement&gt;&gt;
diff --git a/docs/api/puppeteer.frame.md b/docs/api/puppeteer.frame.md
index 0d94f79992ce3..e615cd45b5fb4 100644
--- a/docs/api/puppeteer.frame.md
+++ b/docs/api/puppeteer.frame.md
@@ -69,7 +69,8 @@ console.log(text);
 | [$eval(selector, pageFunction, args)](./puppeteer.frame._eval.md)                    |           | <p>Runs the given function on the first element matching the given selector in the frame.</p><p>If the given function returns a promise, then this method will wait till the promise resolves.</p>                                                                         |
 | [$x(expression)](./puppeteer.frame._x.md)                                            |           |                                                                                                                                                                                                                                                                            |
 | [addScriptTag(options)](./puppeteer.frame.addscripttag.md)                           |           | Adds a <code>&lt;script&gt;</code> tag into the page with the desired url or content.                                                                                                                                                                                      |
-| [addStyleTag(options)](./puppeteer.frame.addstyletag.md)                             |           | Adds a <code>&lt;link rel=&quot;stylesheet&quot;&gt;</code> tag into the page with the desired url or a <code>&lt;style type=&quot;text/css&quot;&gt;</code> tag with the content.                                                                                         |
+| [addStyleTag(options)](./puppeteer.frame.addstyletag.md)                             |           | Adds a <code>&lt;link rel=&quot;stylesheet&quot;&gt;</code> tag into the page with the desired URL or a <code>&lt;style type=&quot;text/css&quot;&gt;</code> tag with the content.                                                                                         |
+| [addStyleTag(options)](./puppeteer.frame.addstyletag_1.md)                           |           |                                                                                                                                                                                                                                                                            |
 | [childFrames()](./puppeteer.frame.childframes.md)                                    |           |                                                                                                                                                                                                                                                                            |
 | [click(selector, options)](./puppeteer.frame.click.md)                               |           | Clicks the first element found that matches <code>selector</code>.                                                                                                                                                                                                         |
 | [content()](./puppeteer.frame.content.md)                                            |           |                                                                                                                                                                                                                                                                            |
diff --git a/docs/api/puppeteer.page.addstyletag.md b/docs/api/puppeteer.page.addstyletag.md
index 2855855a7b712..a4c91c0d4c915 100644
--- a/docs/api/puppeteer.page.addstyletag.md
+++ b/docs/api/puppeteer.page.addstyletag.md
@@ -6,26 +6,26 @@ sidebar_label: Page.addStyleTag
 
 Adds a `<link rel="stylesheet">` tag into the page with the desired URL or a `<style type="text/css">` tag with the content.
 
+Shortcut for .
+
 **Signature:**
 
 ```typescript
 class Page {
-  addStyleTag(options: {
-    url?: string;
-    path?: string;
-    content?: string;
-  }): Promise<ElementHandle<Node>>;
+  addStyleTag(
+    options: Omit<FrameAddStyleTagOptions, 'url'>
+  ): Promise<ElementHandle<HTMLStyleElement>>;
 }
 ```
 
 ## Parameters
 
-| Parameter | Type                                               | Description |
-| --------- | -------------------------------------------------- | ----------- |
-| options   | { url?: string; path?: string; content?: string; } |             |
+| Parameter | Type                                                                                 | Description |
+| --------- | ------------------------------------------------------------------------------------ | ----------- |
+| options   | Omit&lt;[FrameAddStyleTagOptions](./puppeteer.frameaddstyletagoptions.md), 'url'&gt; |             |
 
 **Returns:**
 
-Promise&lt;[ElementHandle](./puppeteer.elementhandle.md)&lt;Node&gt;&gt;
+Promise&lt;[ElementHandle](./puppeteer.elementhandle.md)&lt;HTMLStyleElement&gt;&gt;
 
-Promise which resolves to the added tag when the stylesheet's onload fires or when the CSS content was injected into frame.
+An [element handle](./puppeteer.elementhandle.md) to the injected `<link>` or `<style>` element.
diff --git a/docs/api/puppeteer.page.addstyletag_1.md b/docs/api/puppeteer.page.addstyletag_1.md
new file mode 100644
index 0000000000000..7e259e4f8cdf7
--- /dev/null
+++ b/docs/api/puppeteer.page.addstyletag_1.md
@@ -0,0 +1,25 @@
+---
+sidebar_label: Page.addStyleTag_1
+---
+
+# Page.addStyleTag() method
+
+**Signature:**
+
+```typescript
+class Page {
+  addStyleTag(
+    options: FrameAddStyleTagOptions
+  ): Promise<ElementHandle<HTMLLinkElement>>;
+}
+```
+
+## Parameters
+
+| Parameter | Type                                                              | Description |
+| --------- | ----------------------------------------------------------------- | ----------- |
+| options   | [FrameAddStyleTagOptions](./puppeteer.frameaddstyletagoptions.md) |             |
+
+**Returns:**
+
+Promise&lt;[ElementHandle](./puppeteer.elementhandle.md)&lt;HTMLLinkElement&gt;&gt;
diff --git a/docs/api/puppeteer.page.md b/docs/api/puppeteer.page.md
index 21f2efa9e0148..f6d63edbcdbe3 100644
--- a/docs/api/puppeteer.page.md
+++ b/docs/api/puppeteer.page.md
@@ -82,7 +82,8 @@ page.off('request', logRequest);
 | [$eval(selector, pageFunction, args)](./puppeteer.page._eval.md)                            |           | This method runs <code>document.querySelector</code> within the page and passes the result as the first argument to the <code>pageFunction</code>.                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                 |
 | [$x(expression)](./puppeteer.page._x.md)                                                    |           | The method evaluates the XPath expression relative to the page document as its context node. If there are no such elements, the method resolves to an empty array.                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                 |
 | [addScriptTag(options)](./puppeteer.page.addscripttag.md)                                   |           | Adds a <code>&lt;script&gt;</code> tag into the page with the desired URL or content.                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                              |
-| [addStyleTag(options)](./puppeteer.page.addstyletag.md)                                     |           | Adds a <code>&lt;link rel=&quot;stylesheet&quot;&gt;</code> tag into the page with the desired URL or a <code>&lt;style type=&quot;text/css&quot;&gt;</code> tag with the content.                                                                                                                                                                                                                                                                                                                                                                                                                                                                                 |
+| [addStyleTag(options)](./puppeteer.page.addstyletag.md)                                     |           | <p>Adds a <code>&lt;link rel=&quot;stylesheet&quot;&gt;</code> tag into the page with the desired URL or a <code>&lt;style type=&quot;text/css&quot;&gt;</code> tag with the content.</p><p>Shortcut for .</p>                                                                                                                                                                                                                                                                                                                                                                                                                                                     |
+| [addStyleTag(options)](./puppeteer.page.addstyletag_1.md)                                   |           |                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                    |
 | [authenticate(credentials)](./puppeteer.page.authenticate.md)                               |           | Provide credentials for <code>HTTP authentication</code>.                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                          |
 | [bringToFront()](./puppeteer.page.bringtofront.md)                                          |           | Brings page to front (activates tab).                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                              |
 | [browser()](./puppeteer.page.browser.md)                                                    |           | Get the browser the page belongs to.                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                               |
diff --git a/src/common/Frame.ts b/src/common/Frame.ts
index a7d9d43d1b956..e94eda042333f 100644
--- a/src/common/Frame.ts
+++ b/src/common/Frame.ts
@@ -18,6 +18,7 @@ import {LifecycleWatcher, PuppeteerLifeCycleEvent} from './LifecycleWatcher.js';
 import {Page} from './Page.js';
 import {getQueryHandlerAndSelector} from './QueryHandler.js';
 import {EvaluateFunc, HandleFor, NodeFor} from './types.js';
+import {importFS} from './util.js';
 
 /**
  * @public
@@ -832,18 +833,76 @@ export class Frame {
   }
 
   /**
-   * Adds a `<link rel="stylesheet">` tag into the page with the desired url or
+   * Adds a `<link rel="stylesheet">` tag into the page with the desired URL or
    * a `<style type="text/css">` tag with the content.
    *
-   * @param options - Options for the style link.
-   * @returns a promise that resolves to the added tag when the stylesheets's
-   * `onload` event fires or when the CSS content was injected into the
-   * frame.
+   * @returns An {@link ElementHandle | element handle} to the loaded `<link>`
+   * or `<style>` element.
    */
+  async addStyleTag(
+    options: Omit<FrameAddStyleTagOptions, 'url'>
+  ): Promise<ElementHandle<HTMLStyleElement>>;
+  async addStyleTag(
+    options: FrameAddStyleTagOptions
+  ): Promise<ElementHandle<HTMLLinkElement>>;
   async addStyleTag(
     options: FrameAddStyleTagOptions
   ): Promise<ElementHandle<HTMLStyleElement | HTMLLinkElement>> {
-    return this.worlds[MAIN_WORLD].addStyleTag(options);
+    let {content = ''} = options;
+    const {path, url} = options;
+    if (+!!url + +!!path + +!!content > 1) {
+      throw new Error(
+        'Exactly one of `url`, `path`, or `content` may be specified.'
+      );
+    }
+
+    if (url) {
+      try {
+        return this.worlds[PUPPETEER_WORLD].evaluateHandle(async url => {
+          const link = document.createElement('link');
+          link.rel = 'stylesheet';
+          link.href = url;
+          const promise = new Promise((res, rej) => {
+            link.onload = res;
+            link.onerror = rej;
+          });
+          document.head.appendChild(link);
+          await promise;
+          return link;
+        }, url);
+      } catch (error) {
+        throw new Error(`Loading style from ${url} failed`);
+      }
+    }
+
+    if (path) {
+      let fs: typeof import('fs').promises;
+      try {
+        fs = (await importFS()).promises;
+      } catch (error) {
+        if (error instanceof TypeError) {
+          throw new Error(
+            'Cannot pass a filepath to addStyleTag in the browser environment.'
+          );
+        }
+        throw error;
+      }
+
+      content = await fs.readFile(path, 'utf8');
+      content += '/*# sourceURL=' + path.replace(/\n/g, '') + '*/';
+    }
+
+    return this.worlds[PUPPETEER_WORLD].evaluateHandle(async content => {
+      const style = document.createElement('style');
+      style.appendChild(document.createTextNode(content));
+      const promise = new Promise((res, rej) => {
+        style.onload = res;
+        style.onerror = rej;
+      });
+      document.head.appendChild(style);
+      await promise;
+      return style;
+    }, content);
   }
 
   /**
diff --git a/src/common/IsolatedWorld.ts b/src/common/IsolatedWorld.ts
index 0b93dae9c5b33..9fe587e7c48bd 100644
--- a/src/common/IsolatedWorld.ts
+++ b/src/common/IsolatedWorld.ts
@@ -32,7 +32,6 @@ import {EvaluateFunc, HandleFor, NodeFor} from './types.js';
 import {
   createJSHandle,
   debugError,
-  importFS,
   isNumber,
   isString,
   makePredicateString,
@@ -334,92 +333,6 @@ export class IsolatedWorld {
     }
   }
 
-  /**
-   * Adds a style tag into the current context.
-   *
-   * @remarks
-   * You can pass a URL, filepath or string of contents. Note that when running Puppeteer
-   * in a browser environment you cannot pass a filepath and should use either
-   * `url` or `content`.
-   */
-  async addStyleTag(options: {
-    url?: string;
-    path?: string;
-    content?: string;
-  }): Promise<ElementHandle<HTMLStyleElement | HTMLLinkElement>> {
-    const {url = null, path = null, content = null} = options;
-    if (url !== null) {
-      try {
-        const context = await this.executionContext();
-        return (await context.evaluateHandle(
-          addStyleUrl,
-          url
-        )) as ElementHandle<HTMLLinkElement>;
-      } catch (error) {
-        throw new Error(`Loading style from ${url} failed`);
-      }
-    }
-
-    if (path !== null) {
-      let fs: typeof import('fs').promises;
-      try {
-        fs = (await importFS()).promises;
-      } catch (error) {
-        if (error instanceof TypeError) {
-          throw new Error(
-            'Cannot pass a filepath to addStyleTag in the browser environment.'
-          );
-        }
-        throw error;
-      }
-
-      let contents = await fs.readFile(path, 'utf8');
-      contents += '/*# sourceURL=' + path.replace(/\n/g, '') + '*/';
-      const context = await this.executionContext();
-      return (await context.evaluateHandle(
-        addStyleContent,
-        contents
-      )) as ElementHandle<HTMLStyleElement>;
-    }
-
-    if (content !== null) {
-      const context = await this.executionContext();
-      return (await context.evaluateHandle(
-        addStyleContent,
-        content
-      )) as ElementHandle<HTMLStyleElement>;
-    }
-
-    throw new Error(
-      'Provide an object with a `url`, `path` or `content` property'
-    );
-
-    async function addStyleUrl(url: string): Promise<HTMLElement> {
-      const link = document.createElement('link');
-      link.rel = 'stylesheet';
-      link.href = url;
-      const promise = new Promise((res, rej) => {
-        link.onload = res;
-        link.onerror = rej;
-      });
-      document.head.appendChild(link);
-      await promise;
-      return link;
-    }
-
-    async function addStyleContent(content: string): Promise<HTMLElement> {
-      const style = document.createElement('style');
-      style.appendChild(document.createTextNode(content));
-      const promise = new Promise((res, rej) => {
-        style.onload = res;
-        style.onerror = rej;
-      });
-      document.head.appendChild(style);
-      await promise;
-      return style;
-    }
-  }
-
   async click(
     selector: string,
     options: {delay?: number; button?: MouseButton; clickCount?: number}
diff --git a/src/common/Page.ts b/src/common/Page.ts
index c760f6fef5891..0391891dc114a 100644
--- a/src/common/Page.ts
+++ b/src/common/Page.ts
@@ -32,7 +32,11 @@ import {ElementHandle} from './ElementHandle.js';
 import {EmulationManager} from './EmulationManager.js';
 import {EventEmitter, Handler} from './EventEmitter.js';
 import {FileChooser} from './FileChooser.js';
-import {Frame, FrameAddScriptTagOptions} from './Frame.js';
+import {
+  Frame,
+  FrameAddScriptTagOptions,
+  FrameAddStyleTagOptions,
+} from './Frame.js';
 import {FrameManager, FrameManagerEmittedEvents} from './FrameManager.js';
 import {HTTPRequest} from './HTTPRequest.js';
 import {HTTPResponse} from './HTTPResponse.js';
@@ -1423,16 +1427,24 @@ export class Page extends EventEmitter {
   }
 
   /**
-   * Adds a `<link rel="stylesheet">` tag into the page with the desired URL or a
-   * `<style type="text/css">` tag with the content.
-   * @returns Promise which resolves to the added tag when the stylesheet's
-   * onload fires or when the CSS content was injected into frame.
-   */
-  async addStyleTag(options: {
-    url?: string;
-    path?: string;
-    content?: string;
-  }): Promise<ElementHandle<Node>> {
+   * Adds a `<link rel="stylesheet">` tag into the page with the desired URL or
+   * a `<style type="text/css">` tag with the content.
+   *
+   * Shortcut for
+   * {@link Frame.addStyleTag | page.mainFrame().addStyleTag(options)}.
+   *
+   * @returns An {@link ElementHandle | element handle} to the injected `<link>`
+   * or `<style>` element.
+   */
+  async addStyleTag(
+    options: Omit<FrameAddStyleTagOptions, 'url'>
+  ): Promise<ElementHandle<HTMLStyleElement>>;
+  async addStyleTag(
+    options: FrameAddStyleTagOptions
+  ): Promise<ElementHandle<HTMLLinkElement>>;
+  async addStyleTag(
+    options: FrameAddStyleTagOptions
+  ): Promise<ElementHandle<HTMLStyleElement | HTMLLinkElement>> {
     return this.mainFrame().addStyleTag(options);
   }
 

From d08abd9d2930641c79fd615f64d5f940398f3308 Mon Sep 17 00:00:00 2001
From: Randolf J <jrandolf@google.com>
Date: Thu, 1 Sep 2022 15:47:43 +0200
Subject: [PATCH 4/5] chore: fix tests

---
 src/common/Frame.ts   | 79 ++++++++++++++++++++++++-------------------
 test/src/page.spec.ts |  2 +-
 2 files changed, 45 insertions(+), 36 deletions(-)

diff --git a/src/common/Frame.ts b/src/common/Frame.ts
index e94eda042333f..5d106f6a662ce 100644
--- a/src/common/Frame.ts
+++ b/src/common/Frame.ts
@@ -815,7 +815,11 @@ export class Frame {
             script.addEventListener(
               'error',
               event => {
-                rej(event.message ?? 'Could not load script');
+                let message = 'Could not load script';
+                if (event instanceof ErrorEvent) {
+                  message = event.message ?? message;
+                }
+                rej(message);
               },
               {once: true}
             );
@@ -849,32 +853,13 @@ export class Frame {
     options: FrameAddStyleTagOptions
   ): Promise<ElementHandle<HTMLStyleElement | HTMLLinkElement>> {
     let {content = ''} = options;
-    const {path, url} = options;
-    if (+!!url + +!!path + +!!content > 1) {
+    const {path} = options;
+    if (+!!options.url + +!!path + +!!content !== 1) {
       throw new Error(
-        'Exactly one of `url`, `path`, or `content` may be specified.'
+        'Exactly one of `url`, `path`, or `content` must be specified.'
       );
     }
 
-    if (url) {
-      try {
-        return this.worlds[PUPPETEER_WORLD].evaluateHandle(async url => {
-          const link = document.createElement('link');
-          link.rel = 'stylesheet';
-          link.href = url;
-          const promise = new Promise((res, rej) => {
-            link.onload = res;
-            link.onerror = rej;
-          });
-          document.head.appendChild(link);
-          await promise;
-          return link;
-        }, url);
-      } catch (error) {
-        throw new Error(`Loading style from ${url} failed`);
-      }
-    }
-
     if (path) {
       let fs: typeof import('fs').promises;
       try {
@@ -882,7 +867,7 @@ export class Frame {
       } catch (error) {
         if (error instanceof TypeError) {
           throw new Error(
-            'Cannot pass a filepath to addStyleTag in the browser environment.'
+            'Can only pass a file path in a Node-like environment.'
           );
         }
         throw error;
@@ -890,19 +875,43 @@ export class Frame {
 
       content = await fs.readFile(path, 'utf8');
       content += '/*# sourceURL=' + path.replace(/\n/g, '') + '*/';
+      options.content = content;
     }
 
-    return this.worlds[PUPPETEER_WORLD].evaluateHandle(async content => {
-      const style = document.createElement('style');
-      style.appendChild(document.createTextNode(content));
-      const promise = new Promise((res, rej) => {
-        style.onload = res;
-        style.onerror = rej;
-      });
-      document.head.appendChild(style);
-      await promise;
-      return style;
-    }, content);
+    return this.worlds[MAIN_WORLD].transferHandle(
+      await this.worlds[PUPPETEER_WORLD].evaluateHandle(
+        async ({url, content}) => {
+          let element: HTMLLinkElement | HTMLStyleElement;
+          if (url) {
+            const link = document.createElement('link');
+            link.rel = 'stylesheet';
+            link.href = url;
+            element = link;
+          } else {
+            element = document.createElement('style');
+            element.appendChild(document.createTextNode(content!));
+          }
+          const promise = new Promise((res, rej) => {
+            element.addEventListener('load', res, {once: true});
+            element.addEventListener(
+              'error',
+              event => {
+                let message = 'Could not load style';
+                if (event instanceof ErrorEvent) {
+                  message = event.message ?? message;
+                }
+                rej(message);
+              },
+              {once: true}
+            );
+          });
+          document.head.appendChild(element);
+          await promise;
+          return element;
+        },
+        options
+      )
+    );
   }
 
   /**
diff --git a/test/src/page.spec.ts b/test/src/page.spec.ts
index d7620546cae5f..c2708305d2822 100644
--- a/test/src/page.spec.ts
+++ b/test/src/page.spec.ts
@@ -1824,7 +1824,7 @@ describe('Page', function () {
       } catch (error_) {
         error = error_ as Error;
       }
-      expect(error.message).toBe('Loading style from /nonexistfile.js failed');
+      expect(error.message).toContain('Could not load style');
     });
 
     it('should work with a path', async () => {

From 819832ba42901d59122184b5bdd80f71dd18553d Mon Sep 17 00:00:00 2001
From: Randolf J <jrandolf@google.com>
Date: Thu, 1 Sep 2022 17:14:00 +0200
Subject: [PATCH 5/5] chore: fix tests

---
 src/common/Frame.ts   | 63 +++++++++++++++++--------------------------
 test/src/page.spec.ts |  4 +--
 2 files changed, 27 insertions(+), 40 deletions(-)

diff --git a/src/common/Frame.ts b/src/common/Frame.ts
index 5d106f6a662ce..8a5507c87bbf4 100644
--- a/src/common/Frame.ts
+++ b/src/common/Frame.ts
@@ -752,7 +752,7 @@ export class Frame {
     const {path} = options;
     if (+!!options.url + +!!path + +!!content !== 1) {
       throw new Error(
-        'Exactly one of `url`, `path`, or `content` may be specified.'
+        'Exactly one of `url`, `path`, or `content` must be specified.'
       );
     }
 
@@ -772,31 +772,12 @@ export class Frame {
       content += `//# sourceURL=${path.replace(/\n/g, '')}`;
     }
 
-<<<<<<< HEAD
-<<<<<<< HEAD
-=======
->>>>>>> 2563c0600e (chore: fix tests)
     type = type ?? 'text/javascript';
 
     return this.worlds[MAIN_WORLD].transferHandle(
       await this.worlds[PUPPETEER_WORLD].evaluateHandle(
         async ({url, id, type, content}) => {
           const script = document.createElement('script');
-<<<<<<< HEAD
-=======
-    return this.worlds[PUPPETEER_WORLD].evaluateHandle(
-      async ({url, id, type, content}) => {
-        const script = document.createElement('script');
-        if (url) {
-          script.src = url;
-        }
-        if (id) {
-          script.id = id;
-        }
-        if (type) {
->>>>>>> 571c30c3e5 (chore: remove unnecessary type signature)
-=======
->>>>>>> 2563c0600e (chore: fix tests)
           script.type = type;
           script.text = content;
           if (url) {
@@ -881,33 +862,39 @@ export class Frame {
     return this.worlds[MAIN_WORLD].transferHandle(
       await this.worlds[PUPPETEER_WORLD].evaluateHandle(
         async ({url, content}) => {
-          let element: HTMLLinkElement | HTMLStyleElement;
-          if (url) {
-            const link = document.createElement('link');
-            link.rel = 'stylesheet';
-            link.href = url;
-            element = link;
-          } else {
-            element = document.createElement('style');
-            element.appendChild(document.createTextNode(content!));
+          if (!url) {
+            const style = document.createElement('style');
+            style.appendChild(document.createTextNode(content!));
+            const promise = new Promise((res, rej) => {
+              style.addEventListener('load', res, {once: true});
+              style.addEventListener(
+                'error',
+                event => {
+                  rej(event.message ?? 'Could not load style');
+                },
+                {once: true}
+              );
+            });
+            document.head.appendChild(style);
+            await promise;
+            return style;
           }
+          const link = document.createElement('link');
+          link.rel = 'stylesheet';
+          link.href = url;
           const promise = new Promise((res, rej) => {
-            element.addEventListener('load', res, {once: true});
-            element.addEventListener(
+            link.addEventListener('load', res, {once: true});
+            link.addEventListener(
               'error',
               event => {
-                let message = 'Could not load style';
-                if (event instanceof ErrorEvent) {
-                  message = event.message ?? message;
-                }
-                rej(message);
+                rej(event.message ?? 'Could not load style');
               },
               {once: true}
             );
           });
-          document.head.appendChild(element);
+          document.head.appendChild(link);
           await promise;
-          return element;
+          return link;
         },
         options
       )
diff --git a/test/src/page.spec.ts b/test/src/page.spec.ts
index c2708305d2822..31b997110251b 100644
--- a/test/src/page.spec.ts
+++ b/test/src/page.spec.ts
@@ -1631,7 +1631,7 @@ describe('Page', function () {
         error = error_ as Error;
       }
       expect(error.message).toBe(
-        'Exactly one of `url`, `path`, or `content` may be specified.'
+        'Exactly one of `url`, `path`, or `content` must be specified.'
       );
     });
 
@@ -1797,7 +1797,7 @@ describe('Page', function () {
         error = error_ as Error;
       }
       expect(error.message).toBe(
-        'Provide an object with a `url`, `path` or `content` property'
+        'Exactly one of `url`, `path`, or `content` must be specified.'
       );
     });