From 33aa207efd4f51bfa9bf0c355c62a8099a221cf3 Mon Sep 17 00:00:00 2001 From: Ben Konyi Date: Tue, 11 May 2021 17:59:54 +0000 Subject: [PATCH] fix: Context parse error, parent should be ContextRef Bug: https://github.com/dart-lang/sdk/issues/45083 TEST=N/A Change-Id: I1781c83ac2e8868f1d6a49fc4a01edd3e3eef460 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/192880 Reviewed-by: Ben Konyi Commit-Queue: Ben Konyi --- pkg/vm_service/CHANGELOG.md | 4 ++-- pkg/vm_service/lib/src/vm_service.dart | 5 +++-- pkg/vm_service/pubspec.yaml | 2 +- runtime/vm/service/service.md | 2 +- 4 files changed, 7 insertions(+), 6 deletions(-) diff --git a/pkg/vm_service/CHANGELOG.md b/pkg/vm_service/CHANGELOG.md index 5f0da4d79bc..47f3893bdcb 100644 --- a/pkg/vm_service/CHANGELOG.md +++ b/pkg/vm_service/CHANGELOG.md @@ -1,6 +1,6 @@ # Changelog - -## 6.3.0-dev +## 7.0.0 +- *breaking bug fix*: Fixed issue where response parsing could fail for `Context`. - Add support for `setBreakpointState` RPC and updated `Breakpoint` class to include `enabled` property. diff --git a/pkg/vm_service/lib/src/vm_service.dart b/pkg/vm_service/lib/src/vm_service.dart index 50fe0297ecf..a42219a453a 100644 --- a/pkg/vm_service/lib/src/vm_service.dart +++ b/pkg/vm_service/lib/src/vm_service.dart @@ -3318,7 +3318,7 @@ class Context extends Obj implements ContextRef { /// The enclosing context for this context. @optional - Context? parent; + ContextRef? parent; /// The variables in this context object. List? variables; @@ -3334,7 +3334,8 @@ class Context extends Obj implements ContextRef { Context._fromJson(Map json) : super._fromJson(json) { length = json['length'] ?? -1; - parent = createServiceObject(json['parent'], const ['Context']) as Context?; + parent = createServiceObject(json['parent'], const ['ContextRef']) + as ContextRef?; variables = List.from( createServiceObject(json['variables'], const ['ContextElement']) as List? ?? diff --git a/pkg/vm_service/pubspec.yaml b/pkg/vm_service/pubspec.yaml index dfe2e30ae4d..a7db2e82b0e 100644 --- a/pkg/vm_service/pubspec.yaml +++ b/pkg/vm_service/pubspec.yaml @@ -3,7 +3,7 @@ description: >- A library to communicate with a service implementing the Dart VM service protocol. -version: 6.3.0-dev +version: 7.0.0 homepage: https://github.com/dart-lang/sdk/tree/master/pkg/vm_service diff --git a/runtime/vm/service/service.md b/runtime/vm/service/service.md index 99bdab7da53..3000c42da7f 100644 --- a/runtime/vm/service/service.md +++ b/runtime/vm/service/service.md @@ -1832,7 +1832,7 @@ class Context extends Object { int length; // The enclosing context for this context. - Context parent [optional]; + @Context parent [optional]; // The variables in this context object. ContextElement[] variables;