Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(634)

Unified Diff: runtime/bin/vmservice/observatory/lib/src/service/object.dart

Issue 487023003: vmservice protocol cleanup (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: gen js Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: runtime/bin/vmservice/observatory/lib/src/service/object.dart
diff --git a/runtime/bin/vmservice/observatory/lib/src/service/object.dart b/runtime/bin/vmservice/observatory/lib/src/service/object.dart
index 144dd7193d91b1d694f4c00dce605b26f6f03f99..eba79e5195ca773723d875a2ffe37a347aa52e6c 100644
--- a/runtime/bin/vmservice/observatory/lib/src/service/object.dart
+++ b/runtime/bin/vmservice/observatory/lib/src/service/object.dart
@@ -1075,8 +1075,8 @@ class ServiceMap extends ServiceObject implements ObservableMap {
_map.clear();
_map.addAll(map);
- name = _map['user_name'];
- vmName = _map['name'];
+ name = _map['name'];
+ vmName = (_map.containsKey('vmName') ? _map['vmName'] : name);
_upgradeValues();
}
@@ -1216,11 +1216,12 @@ class Library extends ServiceObject with Coverage {
url.startsWith('http://')) {
shortUrl = url.substring(url.lastIndexOf('/') + 1);
}
- name = map['user_name'];
+ name = map['name'];
if (name.isEmpty) {
+ // When there is no name for a library, use the shortUrl.
name = shortUrl;
}
- vmName = map['name'];
+ vmName = (map.containsKey('vmName') ? map['vmName'] : name);
if (mapIsRef) {
return;
}
@@ -1316,8 +1317,8 @@ class Class extends ServiceObject with Coverage {
}
void _update(ObservableMap map, bool mapIsRef) {
- name = map['user_name'];
- vmName = map['name'];
+ name = map['name'];
+ vmName = (map.containsKey('vmName') ? map['vmName'] : name);
if (mapIsRef) {
return;
@@ -1454,8 +1455,8 @@ class ServiceFunction extends ServiceObject with Coverage {
ServiceFunction._empty(ServiceObject owner) : super._empty(owner);
void _update(ObservableMap map, bool mapIsRef) {
- name = map['user_name'];
- vmName = map['name'];
+ name = map['name'];
+ vmName = (map.containsKey('vmName') ? map['vmName'] : name);
_upgradeCollection(map, isolate);
@@ -1991,8 +1992,8 @@ class Code extends ServiceObject {
}
void _update(ObservableMap m, bool mapIsRef) {
- name = m['user_name'];
- vmName = m['name'];
+ name = m['name'];
+ vmName = (m.containsKey('vmName') ? m['vmName'] : name);
isOptimized = m['isOptimized'] != null ? m['isOptimized'] : false;
kind = CodeKind.fromString(m['kind']);
startAddress = int.parse(m['start'], radix:16);
« no previous file with comments | « runtime/bin/vmservice/observatory/deployed/web/index_devtools.html_bootstrap.dart.js ('k') | runtime/vm/coverage_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698