Skip to content

config: Enable AbortController/AbortSignal by default.#40079

Merged
jdm merged 1 commit intoservo:mainfrom
jdm:enable-abort
Oct 22, 2025
Merged

config: Enable AbortController/AbortSignal by default.#40079
jdm merged 1 commit intoservo:mainfrom
jdm:enable-abort

Conversation

@jdm
Copy link
Copy Markdown
Member

@jdm jdm commented Oct 22, 2025

Make the engine default to enabling the AbortController implementation.

Testing: No behaviour change to tests, just flipping defaults.
Fixes: #34866

@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Oct 22, 2025
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Oct 22, 2025
@jdm jdm enabled auto-merge October 22, 2025 13:36
@jdm jdm added this pull request to the merge queue Oct 22, 2025
@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 Oct 22, 2025
@mrobinson
Copy link
Copy Markdown
Member

Great stuff! If this sticks, I think we could remove the preference entirely.

@jdm
Copy link
Copy Markdown
Member Author

jdm commented Oct 22, 2025

Agreed!

Merged via the queue into servo:main with commit b15e215 Oct 22, 2025
33 checks passed
@jdm jdm deleted the enable-abort branch October 22, 2025 14:32
@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 Oct 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement AbortController

4 participants