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

Side by Side Diff: runtime/observatory/lib/src/elements/debugger.dart

Issue 2935873002: Revert "Address additional analysis issues in the observatory codebase." (Closed)
Patch Set: Created 3 years, 6 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 unified diff | Download patch
« no previous file with comments | « no previous file | runtime/observatory/tests/observatory_ui/mocks/objects/isolate.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library debugger_page_element; 5 library debugger_page_element;
6 6
7 import 'dart:async'; 7 import 'dart:async';
8 import 'dart:svg';
8 import 'dart:html'; 9 import 'dart:html';
9 import 'dart:math'; 10 import 'dart:math';
10 import 'dart:svg'; 11 import 'package:observatory/event.dart';
11 12 import 'package:observatory/models.dart' as M;
12 import 'package:logging/logging.dart';
13 import 'package:observatory/app.dart'; 13 import 'package:observatory/app.dart';
14 import 'package:observatory/cli.dart'; 14 import 'package:observatory/cli.dart';
15 import 'package:observatory/debugger.dart'; 15 import 'package:observatory/debugger.dart';
16 import 'package:observatory/event.dart';
17 import 'package:observatory/models.dart' as M;
18 import 'package:observatory/service.dart' as S;
19 import 'package:observatory/service_common.dart';
20 import 'package:observatory/src/elements/function_ref.dart'; 16 import 'package:observatory/src/elements/function_ref.dart';
21 import 'package:observatory/src/elements/helpers/any_ref.dart'; 17 import 'package:observatory/src/elements/helpers/any_ref.dart';
22 import 'package:observatory/src/elements/helpers/nav_bar.dart'; 18 import 'package:observatory/src/elements/helpers/nav_bar.dart';
23 import 'package:observatory/src/elements/helpers/nav_menu.dart'; 19 import 'package:observatory/src/elements/helpers/nav_menu.dart';
24 import 'package:observatory/src/elements/helpers/rendering_scheduler.dart'; 20 import 'package:observatory/src/elements/helpers/rendering_scheduler.dart';
25 import 'package:observatory/src/elements/helpers/tag.dart'; 21 import 'package:observatory/src/elements/helpers/tag.dart';
26 import 'package:observatory/src/elements/helpers/uris.dart'; 22 import 'package:observatory/src/elements/helpers/uris.dart';
27 import 'package:observatory/src/elements/instance_ref.dart'; 23 import 'package:observatory/src/elements/instance_ref.dart';
28 import 'package:observatory/src/elements/nav/isolate_menu.dart'; 24 import 'package:observatory/src/elements/nav/isolate_menu.dart';
29 import 'package:observatory/src/elements/nav/notify.dart'; 25 import 'package:observatory/src/elements/nav/notify.dart';
30 import 'package:observatory/src/elements/nav/top_menu.dart'; 26 import 'package:observatory/src/elements/nav/top_menu.dart';
31 import 'package:observatory/src/elements/nav/vm_menu.dart'; 27 import 'package:observatory/src/elements/nav/vm_menu.dart';
32 import 'package:observatory/src/elements/source_inset.dart'; 28 import 'package:observatory/src/elements/source_inset.dart';
33 import 'package:observatory/src/elements/source_link.dart'; 29 import 'package:observatory/src/elements/source_link.dart';
30 import 'package:observatory/service.dart' as S;
31 import 'package:logging/logging.dart';
34 32
35 // TODO(turnidge): Move Debugger, DebuggerCommand to debugger library. 33 // TODO(turnidge): Move Debugger, DebuggerCommand to debugger library.
36 abstract class DebuggerCommand extends Command { 34 abstract class DebuggerCommand extends Command {
37 ObservatoryDebugger debugger; 35 ObservatoryDebugger debugger;
38 36
39 DebuggerCommand(this.debugger, name, children) : super(name, children); 37 DebuggerCommand(this.debugger, name, children) : super(name, children);
40 38
41 String get helpShort; 39 String get helpShort;
42 String get helpLong; 40 String get helpLong;
43 } 41 }
(...skipping 1152 matching lines...) Expand 10 before | Expand all | Expand 10 after
1196 debugger.console.print("No connected VMs"); 1194 debugger.console.print("No connected VMs");
1197 return; 1195 return;
1198 } 1196 }
1199 // TODO(turnidge): Right now there is only one vm listed. 1197 // TODO(turnidge): Right now there is only one vm listed.
1200 var vmList = [debugger.vm]; 1198 var vmList = [debugger.vm];
1201 1199
1202 var maxAddrLen = 'ADDRESS'.length; 1200 var maxAddrLen = 'ADDRESS'.length;
1203 var maxNameLen = 'NAME'.length; 1201 var maxNameLen = 'NAME'.length;
1204 1202
1205 for (var vm in vmList) { 1203 for (var vm in vmList) {
1206 maxAddrLen = max( 1204 maxAddrLen = max(maxAddrLen, vm.target.networkAddress.length);
1207 maxAddrLen, (vm as CommonWebSocketVM).target.networkAddress.length);
1208 maxNameLen = max(maxNameLen, vm.name.length); 1205 maxNameLen = max(maxNameLen, vm.name.length);
1209 } 1206 }
1210 1207
1211 debugger.console.print("${'ADDRESS'.padRight(maxAddrLen, ' ')} " 1208 debugger.console.print("${'ADDRESS'.padRight(maxAddrLen, ' ')} "
1212 "${'NAME'.padRight(maxNameLen, ' ')} " 1209 "${'NAME'.padRight(maxNameLen, ' ')} "
1213 "CURRENT"); 1210 "CURRENT");
1214 for (var vm in vmList) { 1211 for (var vm in vmList) {
1215 String current = (vm == debugger.vm ? '*' : ''); 1212 String current = (vm == debugger.vm ? '*' : '');
1216 debugger.console 1213 debugger.console
1217 .print("${vm.target.networkAddress.padRight(maxAddrLen, ' ')} " 1214 .print("${vm.target.networkAddress.padRight(maxAddrLen, ' ')} "
(...skipping 465 matching lines...) Expand 10 before | Expand all | Expand 10 after
1683 if (bpt.resolved) { 1680 if (bpt.resolved) {
1684 console.print('Breakpoint ${bpId} ${verb} at ${locString}'); 1681 console.print('Breakpoint ${bpId} ${verb} at ${locString}');
1685 } else { 1682 } else {
1686 console.print('Future breakpoint ${bpId} ${verb} at ${locString}'); 1683 console.print('Future breakpoint ${bpId} ${verb} at ${locString}');
1687 } 1684 }
1688 } 1685 }
1689 1686
1690 void onEvent(S.ServiceEvent event) { 1687 void onEvent(S.ServiceEvent event) {
1691 switch (event.kind) { 1688 switch (event.kind) {
1692 case S.ServiceEvent.kVMUpdate: 1689 case S.ServiceEvent.kVMUpdate:
1693 CommonWebSocketVM vm = event.owner; 1690 var vm = event.owner;
1694 console.print("VM ${vm.target.networkAddress} renamed to '${vm.name}'"); 1691 console.print("VM ${vm.target.networkAddress} renamed to '${vm.name}'");
1695 break; 1692 break;
1696 1693
1697 case S.ServiceEvent.kIsolateStart: 1694 case S.ServiceEvent.kIsolateStart:
1698 { 1695 {
1699 var iso = event.owner; 1696 var iso = event.owner;
1700 console.print("Isolate ${iso.number} '${iso.name}' has been created"); 1697 console.print("Isolate ${iso.number} '${iso.name}' has been created");
1701 } 1698 }
1702 break; 1699 break;
1703 1700
(...skipping 1641 matching lines...) Expand 10 before | Expand all | Expand 10 after
3345 ..setAttribute('height', '24') 3342 ..setAttribute('height', '24')
3346 ..children = [ 3343 ..children = [
3347 new PathElement() 3344 new PathElement()
3348 ..setAttribute( 3345 ..setAttribute(
3349 'd', 3346 'd',
3350 'M11 17h2v-6h-2v6zm1-15C6.48 2 2 6.48 2 12s4.48 10 ' 3347 'M11 17h2v-6h-2v6zm1-15C6.48 2 2 6.48 2 12s4.48 10 '
3351 '10 10 10-4.48 10-10S17.52 2 12 2zm0 18c-4.41 ' 3348 '10 10 10-4.48 10-10S17.52 2 12 2zm0 18c-4.41 '
3352 '0-8-3.59-8-8s3.59-8 8-8 8 3.59 8 8-3.59 8-8 8zM11 ' 3349 '0-8-3.59-8-8s3.59-8 8-8 8 3.59 8 8-3.59 8-8 8zM11 '
3353 '9h2V7h-2v2z') 3350 '9h2V7h-2v2z')
3354 ]; 3351 ];
OLDNEW
« no previous file with comments | « no previous file | runtime/observatory/tests/observatory_ui/mocks/objects/isolate.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698