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

Unified Diff: runtime/observatory/lib/src/elements/instance_ref.dart

Issue 2862383002: Clarify presentation of the target in retaining paths. (Closed)
Patch Set: review Created 3 years, 7 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/observatory/lib/src/elements/instance_ref.dart
diff --git a/runtime/observatory/lib/src/elements/instance_ref.dart b/runtime/observatory/lib/src/elements/instance_ref.dart
index d5cff50c2959fbefa583ff05208034427de25554..1e1df2dd1c85dd047f6af0938dc3be5a262ffd1e 100644
--- a/runtime/observatory/lib/src/elements/instance_ref.dart
+++ b/runtime/observatory/lib/src/elements/instance_ref.dart
@@ -138,7 +138,7 @@ class InstanceRefElement extends HtmlElement implements Renderable {
new SpanElement()
..classes = ['emphasize']
..text = 'Closure',
- new SpanElement()..text = _instance.closureFunction.name
+ new SpanElement()..text = ' (${_instance.closureFunction.name})'
]
];
case M.InstanceKind.regExp:
@@ -148,18 +148,16 @@ class InstanceRefElement extends HtmlElement implements Renderable {
new SpanElement()
..classes = ['emphasize']
..text = _instance.clazz.name,
- new SpanElement()..text = _instance.pattern.name
+ new SpanElement()..text = ' (${_instance.pattern.valueAsString})'
]
];
case M.InstanceKind.stackTrace:
return [
new AnchorElement(href: Uris.inspect(_isolate, object: _instance))
- ..text = _instance.clazz.name,
- new CurlyBlockElement(queue: _r.queue)
- ..content = [
- new DivElement()
- ..classes = ['stackTraceBox']
- ..text = _instance.valueAsString
+ ..children = [
+ new SpanElement()
+ ..classes = ['emphasize']
+ ..text = _instance.clazz.name,
]
];
case M.InstanceKind.plainInstance:
@@ -206,13 +204,14 @@ class InstanceRefElement extends HtmlElement implements Renderable {
..text = _instance.clazz.name
];
}
- throw new Exception('Unkown InstanceKind: ${_instance.kind}');
+ throw new Exception('Unknown InstanceKind: ${_instance.kind}');
}
bool _hasValue() {
switch (_instance.kind) {
case M.InstanceKind.plainInstance:
case M.InstanceKind.mirrorReference:
+ case M.InstanceKind.stackTrace:
case M.InstanceKind.weakProperty:
return true;
case M.InstanceKind.list:
@@ -298,6 +297,12 @@ class InstanceRefElement extends HtmlElement implements Renderable {
new InstanceRefElement(_isolate, _loadedInstance.referent, _instances,
queue: _r.queue)
];
+ case M.InstanceKind.stackTrace:
+ return [
+ new DivElement()
+ ..classes = ['stackTraceBox']
+ ..text = _instance.valueAsString
+ ];
case M.InstanceKind.weakProperty:
return [
new SpanElement()..text = '<key> : ',

Powered by Google App Engine
This is Rietveld 408576698