removed unused js dependency

[greenart7c3]

Fix #2480

---

• I’ve reviewed the contributor guide and applied the relevant portions to this PR.

Contribution guidelines:

• See our contributor guide for general expectations for PRs.
• Larger or significant changes should be discussed in an issue before creating a PR.
• Contributions to our repos should follow the Dart style guide and use dart format.
• Most changes should add an entry to the changelog and may need to rev the pubspec package version.
• Changes to packages require corresponding tests.

Many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

Note: The Dart team is trialing Gemini Code Assist. Don't take its comments as final Dart team feedback. Use the suggestions if they're helpful; otherwise, wait for a human reviewer.

[github-actions]

PR Health

Changelog Entry ✔️

Package	Changed Files

Changes to files need to be accounted for in their respective changelogs.

This check can be disabled by tagging the PR with skip-changelog-check.

[greenart7c3]

Do i need to put the -wip in the changelog too?

[kevmoo]

Do i need to put the -wip in the changelog too?

Yes, please!

[jakemac53]

@srujzs should we be migrating to package:web? It looks like we currently use dart:js_interop and dart:js_interop_unsafe and have our own extension types. See https://github.com/dart-lang/test/blob/master/pkgs/test/lib/src/runner/browser/dom.dart and https://github.com/dart-lang/test/blob/master/pkgs/test/lib/src/runner/browser/post_message_channel.dart as examples.

[kevmoo]

@srujzs should we be migrating to package:web? It looks like we currently use dart:js_interop and dart:js_interop_unsafe and have our own extension types. See https://github.com/dart-lang/test/blob/master/pkgs/test/lib/src/runner/browser/dom.dart and https://github.com/dart-lang/test/blob/master/pkgs/test/lib/src/runner/browser/post_message_channel.dart as examples.

I think we were trying to avoid dependency loops. But we could!

[srujzs]

I think we were trying to avoid dependency loops. But we could!

Are you referring to the dev dependency on test in web?

re: migrating to package:web, sure. I don't think it functionally makes a difference but that's less code to maintain.