Skip to content

Commit

Permalink
chore: import BiDi impl only if the user opts in (#9335)
Browse files Browse the repository at this point in the history
  • Loading branch information
OrKoN committed Nov 29, 2022
1 parent c77b729 commit e8c1d56
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
5 changes: 3 additions & 2 deletions packages/puppeteer-core/src/node/BrowserRunner.ts
Expand Up @@ -20,7 +20,7 @@ import * as path from 'path';
import * as readline from 'readline';
import removeFolder from 'rimraf';
import {promisify} from 'util';
import {Connection as BiDiConnection} from '../common/bidi/Connection.js';
import type {Connection as BiDiConnection} from '../common/bidi/bidi.js';
import {Connection} from '../common/Connection.js';
import {debug} from '../common/Debug.js';
import {TimeoutError} from '../common/Errors.js';
Expand Down Expand Up @@ -262,7 +262,8 @@ export class BrowserRunner {
);
browserWSEndpoint += '/session';
const transport = await WebSocketTransport.create(browserWSEndpoint);
return new BiDiConnection(transport, slowMo);
const BiDi = await import('../common/bidi/bidi.js');
return new BiDi.Connection(transport, slowMo);
}

async setupConnection(options: {
Expand Down
4 changes: 2 additions & 2 deletions packages/puppeteer-core/src/node/FirefoxLauncher.ts
Expand Up @@ -2,7 +2,6 @@ import fs from 'fs';
import os from 'os';
import path from 'path';
import {Browser} from '../api/Browser.js';
import {Browser as BiDiBrowser} from '../common/bidi/Browser.js';
import {CDPBrowser} from '../common/Browser.js';
import {assert} from '../util/assert.js';
import {BrowserFetcher} from './BrowserFetcher.js';
Expand Down Expand Up @@ -133,7 +132,8 @@ export class FirefoxLauncher extends ProductLauncher {
slowMo,
preferredRevision: this.puppeteer.browserRevision,
});
browser = await BiDiBrowser.create({
const BiDi = await import('../common/bidi/bidi.js');
browser = await BiDi.Browser.create({
connection,
closeCallback: runner.close.bind(runner),
process: runner.proc,
Expand Down

0 comments on commit e8c1d56

Please sign in to comment.