Filip/playwright
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

Remove _ prefix

Browse source
This commit is contained in:
Yury Semikhatsky 2024-08-12 15:10:30 -07:00
parent 168c76f239
commit e8984435fa
4 changed files with 26 additions and 26 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
packages/playwright-core/src/server/browserType.ts
View file

@ -133,7 +133,7 @@ export abstract class BrowserType extends SdkObject {
if (persistent)
validateBrowserContextOptions(persistent, browserOptions);
copyTestHooks(options, browserOptions);
const browser = await this._connectToTransport(transport, browserOptions);
const browser = await this.connectToTransport(transport, browserOptions);
(browser as any)._userDataDirForTest = userDataDir;
// We assume no control when using custom arguments, and do not prepare the default context in that case.
if (persistent && !options.ignoreAllDefaultArgs)
@ -173,9 +173,9 @@ export abstract class BrowserType extends SdkObject {
if (ignoreAllDefaultArgs)
browserArguments.push(...args);
else if (ignoreDefaultArgs)
browserArguments.push(...this._defaultArgs(options, isPersistent, userDataDir).filter(arg => ignoreDefaultArgs.indexOf(arg) === -1));
browserArguments.push(...this.defaultArgs(options, isPersistent, userDataDir).filter(arg => ignoreDefaultArgs.indexOf(arg) === -1));
else
browserArguments.push(...this._defaultArgs(options, isPersistent, userDataDir));
browserArguments.push(...this.defaultArgs(options, isPersistent, userDataDir));
let executable: string;
if (executablePath) {
@ -199,7 +199,7 @@ export abstract class BrowserType extends SdkObject {
const { launchedProcess, gracefullyClose, kill } = await launchProcess({
command: executable,
args: browserArguments,
env: this._amendEnvironment(env, userDataDir, executable, browserArguments),
env: this.amendEnvironment(env, userDataDir, executable, browserArguments),
handleSIGINT,
handleSIGTERM,
handleSIGHUP,
@ -222,7 +222,7 @@ export abstract class BrowserType extends SdkObject {
// We try to gracefully close to prevent crash reporting and core dumps.
// Note that it's fine to reuse the pipe transport, since
// our connection ignores kBrowserCloseMessageId.
this._attemptToGracefullyCloseBrowser(transport!);
this.attemptToGracefullyCloseBrowser(transport!);
},
onExit: (exitCode, signal) => {
// Unblock launch when browser prematurely exits.
@ -305,14 +305,14 @@ export abstract class BrowserType extends SdkObject {
_rewriteStartupLog(error: Error): Error {
if (!isProtocolError(error))
return error;
return this._doRewriteStartupLog(error);
return this.doRewriteStartupLog(error);
}
abstract _defaultArgs(options: types.LaunchOptions, isPersistent: boolean, userDataDir: string): string[];
abstract _connectToTransport(transport: ConnectionTransport, options: BrowserOptions): Promise<Browser>;
abstract _amendEnvironment(env: Env, userDataDir: string, executable: string, browserArguments: string[]): Env;
abstract _doRewriteStartupLog(error: ProtocolError): ProtocolError;
abstract _attemptToGracefullyCloseBrowser(transport: ConnectionTransport): void;
abstract defaultArgs(options: types.LaunchOptions, isPersistent: boolean, userDataDir: string): string[];
abstract connectToTransport(transport: ConnectionTransport, options: BrowserOptions): Promise<Browser>;
abstract amendEnvironment(env: Env, userDataDir: string, executable: string, browserArguments: string[]): Env;
abstract doRewriteStartupLog(error: ProtocolError): ProtocolError;
abstract attemptToGracefullyCloseBrowser(transport: ConnectionTransport): void;
}
function copyTestHooks(from: object, to: object) {

10
packages/playwright-core/src/server/chromium/chromium.ts
View file

@ -131,7 +131,7 @@ export class Chromium extends BrowserType {
return directory ? new CRDevTools(path.join(directory, 'devtools-preferences.json')) : undefined;
}
override async _connectToTransport(transport: ConnectionTransport, options: BrowserOptions): Promise<CRBrowser> {
override async connectToTransport(transport: ConnectionTransport, options: BrowserOptions): Promise<CRBrowser> {
let devtools = this._devtools;
if ((options as any).__testHookForDevTools) {
devtools = this._createDevTools();
@ -140,7 +140,7 @@ export class Chromium extends BrowserType {
return CRBrowser.connect(this.attribution.playwright, transport, options, devtools);
}
override _doRewriteStartupLog(error: ProtocolError): ProtocolError {
override doRewriteStartupLog(error: ProtocolError): ProtocolError {
if (!error.logs)
return error;
if (error.logs.includes('Missing X server'))
@ -161,11 +161,11 @@ export class Chromium extends BrowserType {
return error;
}
override _amendEnvironment(env: Env, userDataDir: string, executable: string, browserArguments: string[]): Env {
override amendEnvironment(env: Env, userDataDir: string, executable: string, browserArguments: string[]): Env {
return env;
}
override _attemptToGracefullyCloseBrowser(transport: ConnectionTransport): void {
override attemptToGracefullyCloseBrowser(transport: ConnectionTransport): void {
const message: ProtocolRequest = { method: 'Browser.close', id: kBrowserCloseMessageId, params: {} };
transport.send(message);
}
@ -277,7 +277,7 @@ export class Chromium extends BrowserType {
}
}
override _defaultArgs(options: types.LaunchOptions, isPersistent: boolean, userDataDir: string): string[] {
override defaultArgs(options: types.LaunchOptions, isPersistent: boolean, userDataDir: string): string[] {
const chromeArguments = this._innerDefaultArgs(options);
chromeArguments.push(`--user-data-dir=${userDataDir}`);
if (options.useWebSocket)

10
packages/playwright-core/src/server/firefox/firefox.ts
View file

@ -33,11 +33,11 @@ export class Firefox extends BrowserType {
super(parent, 'firefox');
}
override _connectToTransport(transport: ConnectionTransport, options: BrowserOptions): Promise<FFBrowser> {
override connectToTransport(transport: ConnectionTransport, options: BrowserOptions): Promise<FFBrowser> {
return FFBrowser.connect(this.attribution.playwright, transport, options);
}
override _doRewriteStartupLog(error: ProtocolError): ProtocolError {
override doRewriteStartupLog(error: ProtocolError): ProtocolError {
if (!error.logs)
return error;
// https://github.com/microsoft/playwright/issues/6500
@ -48,7 +48,7 @@ export class Firefox extends BrowserType {
return error;
}
override _amendEnvironment(env: Env, userDataDir: string, executable: string, browserArguments: string[]): Env {
override amendEnvironment(env: Env, userDataDir: string, executable: string, browserArguments: string[]): Env {
if (!path.isAbsolute(os.homedir()))
throw new Error(`Cannot launch Firefox with relative home directory. Did you set ${os.platform() === 'win32' ? 'USERPROFILE' : 'HOME'} to a relative path?`);
if (os.platform() === 'linux') {
@ -60,12 +60,12 @@ export class Firefox extends BrowserType {
return env;
}
override _attemptToGracefullyCloseBrowser(transport: ConnectionTransport): void {
override attemptToGracefullyCloseBrowser(transport: ConnectionTransport): void {
const message = { method: 'Browser.close', params: {}, id: kBrowserCloseMessageId };
transport.send(message);
}
override _defaultArgs(options: types.LaunchOptions, isPersistent: boolean, userDataDir: string): string[] {
override defaultArgs(options: types.LaunchOptions, isPersistent: boolean, userDataDir: string): string[] {
const { args = [], headless } = options;
const userDataDirArg = args.find(arg => arg.startsWith('-profile') || arg.startsWith('--profile'));
if (userDataDirArg)

10
packages/playwright-core/src/server/webkit/webkit.ts
View file

@ -32,15 +32,15 @@ export class WebKit extends BrowserType {
super(parent, 'webkit');
}
override _connectToTransport(transport: ConnectionTransport, options: BrowserOptions): Promise<WKBrowser> {
override connectToTransport(transport: ConnectionTransport, options: BrowserOptions): Promise<WKBrowser> {
return WKBrowser.connect(this.attribution.playwright, transport, options);
}
override _amendEnvironment(env: Env, userDataDir: string, executable: string, browserArguments: string[]): Env {
override amendEnvironment(env: Env, userDataDir: string, executable: string, browserArguments: string[]): Env {
return { ...env, CURL_COOKIE_JAR_PATH: path.join(userDataDir, 'cookiejar.db') };
}
override _doRewriteStartupLog(error: ProtocolError): ProtocolError {
override doRewriteStartupLog(error: ProtocolError): ProtocolError {
if (!error.logs)
return error;
if (error.logs.includes('cannot open display'))
@ -48,11 +48,11 @@ export class WebKit extends BrowserType {
return error;
}
override _attemptToGracefullyCloseBrowser(transport: ConnectionTransport): void {
override attemptToGracefullyCloseBrowser(transport: ConnectionTransport): void {
transport.send({ method: 'Playwright.close', params: {}, id: kBrowserCloseMessageId });
}
override _defaultArgs(options: types.LaunchOptions, isPersistent: boolean, userDataDir: string): string[] {
override defaultArgs(options: types.LaunchOptions, isPersistent: boolean, userDataDir: string): string[] {
const { args = [], proxy, headless } = options;
const userDataDirArg = args.find(arg => arg.startsWith('--user-data-dir'));
if (userDataDirArg)