Closed Bug 1882085 Opened 3 months ago Closed 3 months ago

Puppeteer should only enable CDP or WebDriver BiDi depending on the protocol in use


(Remote Protocol :: Agent, defect, P2)



(firefox125 fixed)

125 Branch
Tracking Status
firefox125 --- fixed


(Reporter: whimboo, Assigned: whimboo)


(Blocks 2 open bugs)


(Whiteboard: [webdriver:m10])

Currently Puppeteer uses the default settings in Firefox for the CDP and WebDriver BiDi protocols which means that both are actually enabled.

Instead Puppeteer should set to 2 if CDP is used and to 1 when BiDi is used. This can be done in the launcher specific code for each protocol.

By doing that we can be sure that no CDP specific recommended preferences are actually affecting the WebDriver BiDi usage.

A fix here would also require an upstream change.

Blocks: 1882089

We should actually fix this upstream first and wait for a Puppeteer release before doing any changes on our side. I'll try to get to it soonish.

(In reply to Henrik Skupin [:whimboo][⌚️UTC+1] from comment #2)

I've created an upstream PR:

The upstream PR got merged. Now lets wait for the next release.

Points: --- → 1
Priority: -- → P2
Whiteboard: [webdriver:m10]

This change actually caused a regression which is handled by bug 1882748. The fix is already on autoland, and once it reaches mozilla-central and a new Nightly build has been created the Puppeteer folks will release a new version.

That means I'll not port the patch to get it landed on mozilla-central, but directly will vendor in the new Puppeteer release via bug 1878632.

As such lets close this bug as fixed.

Assignee: nobody → hskupin
Component: CDP → Agent
Depends on: 1882748
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 125 Branch
Blocks: 1878632
You need to log in before you can comment on or make changes to this bug.