Skip to content

Commit

Permalink
move Page.enable so only chrome uses it
Browse files Browse the repository at this point in the history
  • Loading branch information
chrisbreiding committed Jan 11, 2022
1 parent dfa3953 commit bf05812
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 0 additions & 2 deletions packages/server/lib/browsers/cdp_automation.ts
Original file line number Diff line number Diff line change
Expand Up @@ -192,8 +192,6 @@ export class CdpAutomation {
maxResourceBufferSize: 0,
maxPostDataSize: 0,
})

await this.sendDebuggerCommandFn('Page.enable')
}

private onNetworkRequestWillBeSent = async (params: Protocol.Network.RequestWillBeSentEvent) => {
Expand Down
2 changes: 2 additions & 0 deletions packages/server/lib/browsers/chrome.ts
Original file line number Diff line number Diff line change
Expand Up @@ -623,6 +623,8 @@ export = {
originalBrowserKill.apply(launchedBrowser, args)
}

await criClient.send('Page.enable')

await this._maybeRecordVideo(criClient, options, browser.majorVersion)
await this._navigateUsingCRI(criClient, url)
await this._handleDownloads(criClient, options.downloadsFolder, automation)
Expand Down

0 comments on commit bf05812

Please sign in to comment.