Skip to content

Add stub interface for AbortController#34519

Merged
jdm merged 1 commit intoservo:mainfrom
jdm:abort-controller-stub
Dec 10, 2024
Merged

Add stub interface for AbortController#34519
jdm merged 1 commit intoservo:mainfrom
jdm:abort-controller-stub

Conversation

@jdm
Copy link
Member

@jdm jdm commented Dec 7, 2024

This takes the simplest changes from #32871 and puts the interface behind a pref so we can experiment with loading pages that require the interface's presence (eg. github and discord). I am able to successfully interact with the discord login page with --pref dom.svg.enabled and --pref dom.abort_controller.enabled applied.


  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors
  • These changes are part of Get pdf.js working in Servo #6098
  • These changes do not require tests because the interface is behind a preference and just a stub for right now

@jdm jdm force-pushed the abort-controller-stub branch from 4b1e030 to 8982120 Compare December 10, 2024 04:04
Co-authored-by: syvb <me@iter.ca>
Signed-off-by: Josh Matthews <josh@joshmatthews.net>
@jdm jdm force-pushed the abort-controller-stub branch from 8982120 to 89dca5e Compare December 10, 2024 04:08
@jdm jdm added this pull request to the merge queue Dec 10, 2024
Merged via the queue into servo:main with commit bc2def0 Dec 10, 2024
@jdm jdm deleted the abort-controller-stub branch December 10, 2024 08:24
d-desiatkin pushed a commit to d-desiatkin/servo that referenced this pull request Dec 11, 2024
Signed-off-by: Josh Matthews <josh@joshmatthews.net>
Co-authored-by: syvb <me@iter.ca>
@wusyong wusyong mentioned this pull request Jan 7, 2025
8 tasks
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.

3 participants