Skip to content

devtools: Always pass ownPropertiesLength to devtools#44369

Merged
atbrakhi merged 1 commit into
servo:mainfrom
atbrakhi:pass_own_preperty_length
Apr 21, 2026
Merged

devtools: Always pass ownPropertiesLength to devtools#44369
atbrakhi merged 1 commit into
servo:mainfrom
atbrakhi:pass_own_preperty_length

Conversation

@atbrakhi

@atbrakhi atbrakhi commented Apr 20, 2026

Copy link
Copy Markdown
Member

Always pass ownPropertiesLength to devtools

Testing: All tests are passing
Fixes: part of #39858

Co-authored-by: eri <eri@igalia.com>
Signed-off-by: atbrakhi <atbrakhi@igalia.com>
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Apr 20, 2026
@atbrakhi

Copy link
Copy Markdown
Member Author

This does not really change any functionality right now but we are working on a webidl change that will use this property.

@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Apr 21, 2026
@atbrakhi atbrakhi added this pull request to the merge queue Apr 21, 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 Apr 21, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 21, 2026
@servo-highfive servo-highfive added S-tests-failed The changes caused existing tests to fail. and removed S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. labels Apr 21, 2026
@atbrakhi atbrakhi added this pull request to the merge queue Apr 21, 2026
@servo-highfive servo-highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-tests-failed The changes caused existing tests to fail. labels Apr 21, 2026
Merged via the queue into servo:main with commit f8c97cd Apr 21, 2026
40 checks passed
@atbrakhi atbrakhi deleted the pass_own_preperty_length branch April 21, 2026 17:45
@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 Apr 21, 2026
@eerii eerii mentioned this pull request May 12, 2026
37 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