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

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

Issue 2291233002: Converted Observatory instance-view element (Closed)
Patch Set: Addressed comments Created 4 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/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 2e15a884d15e5fb95aadeb8516eabebd33eec8b5..cb55976b75dccf53d410c85de8d16d9bb7fe9588 100644
--- a/runtime/observatory/lib/src/elements/instance_ref.dart
+++ b/runtime/observatory/lib/src/elements/instance_ref.dart
@@ -12,6 +12,7 @@ import 'package:observatory/src/elements/helpers/rendering_scheduler.dart';
import 'package:observatory/src/elements/helpers/tag.dart';
import 'package:observatory/src/elements/helpers/uris.dart';
import 'package:observatory/src/elements/sentinel_value.dart';
+import 'package:observatory/utils.dart';
class InstanceRefElement extends HtmlElement implements Renderable {
static const tag = const Tag<InstanceRefElement>('instance-ref-wrapped');
@@ -65,7 +66,7 @@ class InstanceRefElement extends HtmlElement implements Renderable {
new SpanElement()..text = ' ',
new CurlyBlockElement(expanded: _expanded, queue: _r.queue)
..children = [
- new DivElement()..classes = ['indent']
+ new DivElement()..classes = const ['indent']
..children = _createValue()
]
..onToggle.listen((e) async {
@@ -119,7 +120,7 @@ class InstanceRefElement extends HtmlElement implements Renderable {
case M.InstanceKind.string:
return [
new AnchorElement(href: Uris.inspect(_isolate, object: _instance))
- ..text = asStringLiteral(_instance.valueAsString,
+ ..text = Utils.formatStringAsLiteral(_instance.valueAsString,
_instance.valueAsStringIsTruncated)
];
case M.InstanceKind.type:
@@ -134,7 +135,8 @@ class InstanceRefElement extends HtmlElement implements Renderable {
return [
new AnchorElement(href: Uris.inspect(_isolate, object: _instance))
..children = [
- new SpanElement()..classes = ['empathize']..text = 'Closure',
+ new SpanElement()..classes = const ['emphasize']
+ ..text = 'Closure',
new SpanElement()..text = _instance.closureFunction.name
]
];
@@ -142,7 +144,7 @@ class InstanceRefElement extends HtmlElement implements Renderable {
return [
new AnchorElement(href: Uris.inspect(_isolate, object: _instance))
..children = [
- new SpanElement()..classes = ['empathize']
+ new SpanElement()..classes = const ['emphasize']
..text = _instance.clazz.name,
new SpanElement()..text = _instance.pattern.name
]
@@ -153,14 +155,14 @@ class InstanceRefElement extends HtmlElement implements Renderable {
..text = _instance.clazz.name,
new CurlyBlockElement(queue: _r.queue)
..children = [
- new DivElement()..classes = ['stackTraceBox']
+ new DivElement()..classes = const ['stackTraceBox']
..text = _instance.valueAsString
]
];
case M.InstanceKind.plainInstance:
return [
new AnchorElement(href: Uris.inspect(_isolate, object: _instance))
- ..classes = ['empathize']
+ ..classes = ['emphasize']
..text = _instance.clazz.name
];
case M.InstanceKind.list:
@@ -182,7 +184,7 @@ class InstanceRefElement extends HtmlElement implements Renderable {
return [
new AnchorElement(href: Uris.inspect(_isolate, object: _instance))
..children = [
- new SpanElement()..classes = ['empathize']
+ new SpanElement()..classes = const ['emphasize']
..text = _instance.clazz.name,
new SpanElement()..text = ' (${_instance.length})'
]
@@ -190,13 +192,13 @@ class InstanceRefElement extends HtmlElement implements Renderable {
case M.InstanceKind.mirrorReference:
return [
new AnchorElement(href: Uris.inspect(_isolate, object: _instance))
- ..classes = ['empathize']
+ ..classes = const ['emphasize']
..text = _instance.clazz.name
];
case M.InstanceKind.weakProperty:
return [
new AnchorElement(href: Uris.inspect(_isolate, object: _instance))
- ..classes = ['empathize']
+ ..classes = const ['emphasize']
..text = _instance.clazz.name
];
}
@@ -314,30 +316,4 @@ class InstanceRefElement extends HtmlElement implements Renderable {
return [];
}
}
-
- static String asStringLiteral(String value, [bool wasTruncated=false]) {
- var result = new List();
- result.add("'".codeUnitAt(0));
- for (int codeUnit in value.codeUnits) {
- if (codeUnit == '\n'.codeUnitAt(0)) result.addAll('\\n'.codeUnits);
- else if (codeUnit == '\r'.codeUnitAt(0)) result.addAll('\\r'.codeUnits);
- else if (codeUnit == '\f'.codeUnitAt(0)) result.addAll('\\f'.codeUnits);
- else if (codeUnit == '\b'.codeUnitAt(0)) result.addAll('\\b'.codeUnits);
- else if (codeUnit == '\t'.codeUnitAt(0)) result.addAll('\\t'.codeUnits);
- else if (codeUnit == '\v'.codeUnitAt(0)) result.addAll('\\v'.codeUnits);
- else if (codeUnit == '\$'.codeUnitAt(0)) result.addAll('\\\$'.codeUnits);
- else if (codeUnit == '\\'.codeUnitAt(0)) result.addAll('\\\\'.codeUnits);
- else if (codeUnit == "'".codeUnitAt(0)) result.addAll("'".codeUnits);
- else if (codeUnit < 32) {
- var escapeSequence = "\\u" + codeUnit.toRadixString(16).padLeft(4, "0");
- result.addAll(escapeSequence.codeUnits);
- } else result.add(codeUnit);
- }
- if (wasTruncated) {
- result.addAll("...".codeUnits);
- } else {
- result.add("'".codeUnitAt(0));
- }
- return new String.fromCharCodes(result);
- }
}
« no previous file with comments | « runtime/observatory/lib/src/elements/icdata_ref.dart ('k') | runtime/observatory/lib/src/elements/instance_ref_wrapper.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698