mirror of
https://github.com/servo/servo
synced 2026-05-13 10:27:03 +02:00
Part of #43800 and #43606. I removed the `new` method from `SourceActor` entirely, as it was only used internally by `new_registered`. I renamed `new_registered` to `register`, inlined the struct construction directly using `Self { ... }`, and updated the registration call to use `registry.register::<Self>(actor)` to match the standard pattern established across other actors. I also standardized variable names throughout `SourceActor` and its call sites to follow the conventions set in #43606: - Renamed `name` and `source` to `source_name` and `source_actor` respectively in `SourceManager::find_source` - Renamed `actor_name` to `source_name` in `SourceManager::source_forms` - Renamed `actor_name` to `source_name` in `handle_update_source_content` in lib.rs Testing: Built with cargo and ran ./mach test-devtools. 59/60 tests pass; the one failure (test_manual_pause) is a pre-existing intermittent timeout unrelated to these changes. Fixes: part of #43606 and #43800 Signed-off-by: CynthiaOketch <cynthiaoketch6@gmail.com>