Skip to content

Stepping into Dart SDK sources in Flutter results in sources downloaded from the VM instead of used from disk #5164

Closed
flutter/flutter
#151170

Description

@DanTup

Some integration tests are failing because when we step into SDK sources, we don't get back a path for the stack frames:

image

This happens with current master (21d996929bb96facd791d650d1ad35e6024d44ae) but only for flutter-tester ("args": [ "-d", "flutter-tester" ]). Using the Windows device things are fine:

image

This is possibly a Flutter issue, but needs some more investigation to see what the difference in traffic is over the VM Service.

Activity

  1. added
    in flutterRelates to running Flutter apps
    in debuggingRelates to the debug adapter or process of running debug sessions
    on Jul 2, 2024
  2. added this to the v3.94.0 milestone on Jul 2, 2024
  3. DanTup commented on Jul 2, 2024

    @DanTup
    MemberAuthor

    This seems to be related to missing sources from the VM. I've opened flutter/flutter#151167.

  4. DanTup commented on Jul 2, 2024

    @DanTup
    MemberAuthor

    It turns out that the issue above might not be the main issue. The reason we're trying to get sources from the VM is that we failed to map onto disk. The reason for that is that the Dart SDK URIs changed in flutter-team-archive/engine#51917

    Previously we had org-dartlang-sdk:///third_party/dart/sdk but now we have org-dartlang-sdk:///flutter/third_party/dart/sdk/lib/core/print.dart, so we're not mapping these URIs correctly.

    We should figure out why this broke no tests (in the Flutter repo for the debug adapter) and update the mapping there.

  5. DanTup commented on Jul 2, 2024

    @DanTup
    MemberAuthor

    Main fix is at flutter/flutter#151170.

  6. DanTup commented on Jul 22, 2024

    @DanTup
    MemberAuthor

    This may need cherry-picking into the next beta.

  7. reopened this on Jul 23, 2024
  8. changed the title [-]Unable to step into SDK sources when running on flutter-tester device[/-] [+]Stepping into Dart SDK sources in Flutter results in sources downloaded from the VM instead of used from disk[/+] on Jul 23, 2024
  9. DanTup commented on Jul 27, 2024

    @DanTup
    MemberAuthor

    New Flutter beta has branched and included this fix, no cherry-pick is required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    in debuggingRelates to the debug adapter or process of running debug sessionsin flutterRelates to running Flutter appsis bug

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions