Skip to content

script: Pass &mut JSContext to FetchResponseListener::process_response_chunk#43657

Merged
TimvdLippe merged 1 commit into
servo:mainfrom
TimvdLippe:cx-fetch-chunk
Mar 26, 2026
Merged

script: Pass &mut JSContext to FetchResponseListener::process_response_chunk#43657
TimvdLippe merged 1 commit into
servo:mainfrom
TimvdLippe:cx-fetch-chunk

Conversation

@TimvdLippe

Copy link
Copy Markdown
Contributor

Fixes #42841

Testing: It compiles

@TimvdLippe TimvdLippe requested a review from gterzian as a code owner March 25, 2026 15:17
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Mar 25, 2026
@sagudev sagudev requested review from Copilot and removed request for Copilot March 25, 2026 15:25
@sagudev

sagudev commented Mar 25, 2026

Copy link
Copy Markdown
Member

Ignore copilot stuff, I tried to test #maintainers > AI policy comments @ 💬

@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Mar 26, 2026
@jdm jdm added this pull request to the merge queue Mar 26, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Mar 26, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Mar 26, 2026
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Mar 26, 2026
@jdm jdm added the S-needs-rebase There are merge conflict errors. label Mar 26, 2026
…onse_chunk`

Fixes servo#42841

Signed-off-by: Tim van der Lippe <tvanderlippe@gmail.com>
@servo-highfive servo-highfive added S-awaiting-review There is new code that needs to be reviewed. and removed S-needs-rebase There are merge conflict errors. labels Mar 26, 2026
@TimvdLippe TimvdLippe enabled auto-merge March 26, 2026 17:30
@TimvdLippe TimvdLippe added this pull request to the merge queue Mar 26, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Mar 26, 2026
Merged via the queue into servo:main with commit f340042 Mar 26, 2026
33 checks passed
@TimvdLippe TimvdLippe deleted the cx-fetch-chunk branch March 26, 2026 19:06
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-awaiting-review There is new code that needs to be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pass &mut JSContext to FetchResponseListener::process_response_chunk

4 participants