Skip to content

devtools: Use actual class name in ObjectActor#42994

Merged
atbrakhi merged 1 commit into
servo:mainfrom
atbrakhi:use_class_instead_window
Mar 3, 2026
Merged

devtools: Use actual class name in ObjectActor#42994
atbrakhi merged 1 commit into
servo:mainfrom
atbrakhi:use_class_instead_window

Conversation

@atbrakhi
Copy link
Copy Markdown
Member

@atbrakhi atbrakhi commented Mar 3, 2026

Class name in ObjectActor was harcoded to Window, which is wrong. This change removes the hardcoded value and replaces it with actual class name.

Before:
image

After:
image

Testing: Existing test passes
Fixes: part of #36027
Depends on #42936

@atbrakhi
Copy link
Copy Markdown
Member Author

atbrakhi commented Mar 3, 2026

Keeping it as a draft because I need to rebase on top of #42936. We will see a ReferenceError if someone tries to test this PR without the changes from #42936

class name is ObjectActor was harcoded to `Window`, this change
remove the hardcoded value and replaces it with actual class
name

Signed-off-by: atbrakhi <[email protected]>
@atbrakhi atbrakhi force-pushed the use_class_instead_window branch from 33e4d48 to ee2147f Compare March 3, 2026 15:25
@atbrakhi atbrakhi marked this pull request as ready for review March 3, 2026 15:26
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Mar 3, 2026
@atbrakhi atbrakhi requested a review from eerii March 3, 2026 15:27
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Mar 3, 2026
@eerii eerii added this pull request to the merge queue Mar 3, 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 3, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Mar 3, 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 Mar 3, 2026
@atbrakhi atbrakhi added this pull request to the merge queue Mar 3, 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 Mar 3, 2026
Merged via the queue into servo:main with commit 2794691 Mar 3, 2026
40 checks passed
@atbrakhi atbrakhi deleted the use_class_instead_window branch March 3, 2026 20: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 Mar 3, 2026
@eerii eerii mentioned this pull request Mar 4, 2026
@eerii eerii mentioned this pull request Apr 16, 2026
37 tasks
offline-ant pushed a commit to offline-ant/havi that referenced this pull request Jun 4, 2026
Class name in `ObjectActor` was harcoded to `Window`, which is wrong.
This change removes the hardcoded value and replaces it with actual
class name.

**Before:**
<img width="320" height="163" alt="image"
src="https://github.com/user-attachments/assets/f94df58a-5309-4fd1-bb02-aa06dfeb32c3"
/>


**After:**
<img width="353" height="138" alt="image"
src="https://github.com/user-attachments/assets/ad129676-8c3d-48e2-a543-1447c75c40f0"
/>

Testing: Existing test passes
Fixes: part of servo#36027 
Depends on servo#42936

Signed-off-by: atbrakhi <[email protected]>
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