chore: reimplement document caching (#10878)

packages/puppeteer-core/src/api/Frame.ts

@@ -322,15 +322,32 @@ export abstract class Frame extends EventEmitter {
    */
   abstract isolatedRealm(): Realm;
 
+  #_document: Promise<ElementHandle<Document>> | undefined;
+
   /**
    * @internal
    */
-  async document(): Promise<ElementHandle<Document>> {
+  #document(): Promise<ElementHandle<Document>> {
-    // TODO(#10813): Implement document caching.
+    if (!this.#_document) {
-    return await this.evaluateHandle(() => {
+      this.#_document = this.isolatedRealm()
+        .evaluateHandle(() => {
           return document;
+        })
+        .then(handle => {
+          return this.mainRealm().transferHandle(handle);
         });
     }
+    return this.#_document;
+  }
+
+  /**
+   * Used to clear the document handle that has been destroyed.
+   *
+   * @internal
+   */
+  clearDocumentHandle(): void {
+    this.#_document = undefined;
+  }
 
   /**
    * @internal
@@ -441,7 +458,8 @@ export abstract class Frame extends EventEmitter {
   async $<Selector extends string>(
     selector: Selector
   ): Promise<ElementHandle<NodeFor<Selector>> | null> {
-    using document = await this.document();
+    // eslint-disable-next-line rulesdir/use-using -- This is cached.
+    const document = await this.#document();
     return await document.$(selector);
   }
 
@@ -456,7 +474,8 @@ export abstract class Frame extends EventEmitter {
   async $$<Selector extends string>(
     selector: Selector
   ): Promise<Array<ElementHandle<NodeFor<Selector>>>> {
-    using document = await this.document();
+    // eslint-disable-next-line rulesdir/use-using -- This is cached.
+    const document = await this.#document();
     return await document.$$(selector);
   }
 
@@ -494,7 +513,8 @@ export abstract class Frame extends EventEmitter {
     ...args: Params
   ): Promise<Awaited<ReturnType<Func>>> {
     pageFunction = withSourcePuppeteerURLIfNone(this.$eval.name, pageFunction);
-    using document = await this.document();
+    // eslint-disable-next-line rulesdir/use-using -- This is cached.
+    const document = await this.#document();
     return await document.$eval(selector, pageFunction, ...args);
   }
 
@@ -532,7 +552,8 @@ export abstract class Frame extends EventEmitter {
     ...args: Params
   ): Promise<Awaited<ReturnType<Func>>> {
     pageFunction = withSourcePuppeteerURLIfNone(this.$$eval.name, pageFunction);
-    using document = await this.document();
+    // eslint-disable-next-line rulesdir/use-using -- This is cached.
+    const document = await this.#document();
     return await document.$$eval(selector, pageFunction, ...args);
   }
 
@@ -548,7 +569,8 @@ export abstract class Frame extends EventEmitter {
    */
   @throwIfDetached
   async $x(expression: string): Promise<Array<ElementHandle<Node>>> {
-    using document = await this.document();
+    // eslint-disable-next-line rulesdir/use-using -- This is cached.
+    const document = await this.#document();
     return await document.$x(expression);
   }
 

packages/puppeteer-core/src/common/IsolatedWorld.ts

@@ -126,6 +126,9 @@ export class IsolatedWorld extends Realm {
 
   clearContext(): void {
     this.#context = Deferred.create();
+    if (this.#frameOrWorker instanceof CDPFrame) {
+      this.#frameOrWorker.clearDocumentHandle();
+    }
   }
 
   setContext(context: ExecutionContext): void {

packages/puppeteer-core/src/common/bidi/Realm.ts

@@ -49,6 +49,7 @@ export class Realm extends EventEmitter {
       // Note: The Realm is destroyed, so in theory the handle should be as
       // well.
       this.internalPuppeteerUtil = undefined;
+      this.#sandbox.environment.clearDocumentHandle();
     }
   };