Avoid using js_util.promiseToFuture, instead prefer the extension on JSPromise<> #143396

Activity

1. srujzs commented on Feb 13, 2024

   

   srujzs

   on Feb 13, 2024

   Contributor

   More actions

   In general, if we can migrate off of dart:js_util, we can remove Flutter from the allowlist here: https://github.com/dart-lang/sdk/blob/1c5eccdd2b258be67251719ed87595d21614545f/pkg/_js_interop_checks/lib/js_interop_checks.dart#L109. We could probably also remove the implementations of promiseToFuture and allowInterop at that time because I believe no one else is using them.
2. kevmoo

   added

   team-webOwned by Web platform teamOwned by Web platform team

   e: wasmIssues related to the wasm build of Flutter Web.Issues related to the wasm build of Flutter Web.

   on Feb 13, 2024
3. yjbanov

   added

   platform-webWeb applications specificallyWeb applications specifically

   P1High-priority issues at the top of the work listHigh-priority issues at the top of the work list

   triaged-webTriaged by Web platform teamTriaged by Web platform team

   P2Important issues not at the top of the work listImportant issues not at the top of the work list

   and removed

   P1High-priority issues at the top of the work listHigh-priority issues at the top of the work list

   on Feb 15, 2024
4. ditman

   removed

   P2Important issues not at the top of the work listImportant issues not at the top of the work list

   triaged-webTriaged by Web platform teamTriaged by Web platform team

   on Jan 7, 2025
5. ditman commented on Jan 7, 2025

   

   ditman

   on Jan 7, 2025

   Last edited by ditman

   Member

   More actions

   Removing labels to trigger a re-triage, we should do this. As Srujan says, we should actually remove all usages of dart:js_util and migrate to dart:js_interop. This seems to include the definition of the kIsWeb constant (which to me is the scariest part of the change :))

   👍React with 👍1Reacted by Kostia Sokolovskyi
6. yjbanov

   added

   P2Important issues not at the top of the work listImportant issues not at the top of the work list

   c: tech-debtTechnical debt, code quality, testing, etc.Technical debt, code quality, testing, etc.

   triaged-webTriaged by Web platform teamTriaged by Web platform team

   on Jan 15, 2025
7. mdebbar

   added

   P1High-priority issues at the top of the work listHigh-priority issues at the top of the work list

   and removed

   P2Important issues not at the top of the work listImportant issues not at the top of the work list

   on Jul 10, 2025
8. mdebbar

   assigned

   ksokolovskyi

   on Jul 10, 2025
9. github-merge-queue

   added a commit that references this issue on Jul 16, 2025

   [web] Remove all usages of js_util. (#171871)

   ...

   66281ce
10. auto-submit

    closed this as completedin #171871on Jul 16, 2025
11. azatech

    added a commit that references this issue on Jul 28, 2025

    [web] Remove all usages of js_util. (flutter#171871)

    ...

    79370b9
12. github-actions commented on Jul 30, 2025

    

    github-actionsbot

    on Jul 30, 2025 – with GitHub Actions

    More actions

    This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.
13. github-actions

    locked as resolved and limited conversation to collaborators on Jul 30, 2025
14. ksokolovskyi

    added a commit that references this issue on Aug 19, 2025

    [web] Remove all usages of js_util. (flutter#171871)

    ...

    56b8f8e
15. mboetger

    added a commit that references this issue on Sep 18, 2025

    [web] Remove all usages of js_util. (flutter#171871)

    ...

    28a3daa
16. lucaantonelli

    added a commit that references this issue on Nov 21, 2025

    [web] Remove all usages of js_util. (flutter#171871)

    ...

    0363e93
17. Nakiha

    added a commit that references this issue on Sep 28, 2026

    [web] Remove all usages of js_util. (#171871)

    ...

    c561794