chore: mark Locator constructor as private (#10322)

docs/api/puppeteer.locator._constructor_.md

@@ -1,27 +0,0 @@
----
-sidebar_label: Locator.(constructor)
----
-
-# Locator.(constructor)
-
-Constructs a new instance of the `Locator` class
-
-#### Signature:
-
-```typescript
-class Locator {
-  constructor(
-    pageOrFrame: Page | Frame,
-    selector: string,
-    options?: LocatorOptions
-  );
-}
-```
-
-## Parameters
-
-| Parameter   | Type                                                         | Description  |
-| ----------- | ------------------------------------------------------------ | ------------ |
-| pageOrFrame | [Page](./puppeteer.page.md) \| [Frame](./puppeteer.frame.md) |              |
-| selector    | string                                                       |              |
-| options     | [LocatorOptions](./puppeteer.locatoroptions.md)              | _(Optional)_ |

docs/api/puppeteer.locator.md

@@ -14,12 +14,6 @@ export declare class Locator extends EventEmitter
 
 **Extends:** [EventEmitter](./puppeteer.eventemitter.md)
 
-## Constructors
-
-| Constructor                                                                           | Modifiers | Description                                                 |
-| ------------------------------------------------------------------------------------- | --------- | ----------------------------------------------------------- |
-| [(constructor)(pageOrFrame, selector, options)](./puppeteer.locator._constructor_.md) |           | Constructs a new instance of the <code>Locator</code> class |
-
 ## Methods
 
 | Method                                                                                           | Modifiers | Description                                                                                                                                                                                                                              |

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

@@ -356,7 +356,7 @@ export class Frame {
    * change in the Locators API.
    */
   locator(selector: string): Locator {
-    return new Locator(this, selector);
+    return Locator.create(this, selector);
   }
 
   /**

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

@@ -117,11 +117,31 @@ export interface LocatorEventObject {
  * @public
  */
 export class Locator extends EventEmitter {
+  /**
+   * @internal
+   */
+  static create(
+    pageOrFrame: Page | Frame,
+    selector: string,
+    options: LocatorOptions = {
+      visibility: 'visible',
+      timeout:
+        'getDefaultTimeout' in pageOrFrame
+          ? pageOrFrame.getDefaultTimeout()
+          : pageOrFrame.page().getDefaultTimeout(),
+      ensureElementIsInTheViewport: true,
+      waitForEnabled: true,
+      waitForStableBoundingBox: true,
+    }
+  ): Locator {
+    return new Locator(pageOrFrame, selector, options);
+  }
+
   #pageOrFrame: Page | Frame;
   #selector: string;
   #options: LocatorOptions;
 
-  constructor(
+  private constructor(
     pageOrFrame: Page | Frame,
     selector: string,
     options: LocatorOptions = {

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

@@ -827,7 +827,7 @@ export class Page extends EventEmitter {
    * change in the Locators API.
    */
   locator(selector: string): Locator {
-    return new Locator(this, selector);
+    return Locator.create(this, selector);
   }
 
   /**