1908164 - Improve "mach remote vendor-puppeteer" command to apply default patches

Status: NEW

(Remote Protocol :: Agent, task, P3)

Description

[Henrik Skupin [:whimboo][⌚️UTC+2]]

For each Puppeteer release that we downstream sync we have to apply specific patches. To simplify the process we should store those patches and apply them automatically or via an extra command to the stack of patches.

Here is actually what we would need:

• Removal of the -S parameter in Shebang from mocha-runner script (changeset)
• Skip test "$$eval should handle many elements" which is causing the error summary log to overflow (changeset)
• Test expectations for flaky Puppeteer tests:
  ** responseCompleted events (changeset) [Firefox 130]
  ** data URLs (changeset) [Firefox 130]

Comment 1

[Julian Descottes [:jdescottes]]

Would be nice to update the documentation with commands we can use to cherry pick / graft the necessary patches.

Comment 2

[Julian Descottes [:jdescottes]]

Attachment: Bug 1908164 - [puppeteer] Add diffs for recurring vendoring patches and update documentation

Comment 3

[Phabricator Automation]

Comment on attachment 9419804 [details]
Bug 1908164 - [puppeteer] Add diffs for recurring vendoring patches and update documentation

Revision D219568 was moved to bug 1914950. Setting attachment 9419804 [details] to obsolete.

Comment 4

[Julian Descottes [:jdescottes]]

I am resetting this bug's points/priority/whiteboard flags, and moving the current work to Bug 1914950.
This bug should remain focused on applying the patches automatically from the ./mach command, whereas Bug 1914950 is just about adding and documenting the patches.