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

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

Issue 2930993004: 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
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';
9 import 'dart:html'; 8 import 'dart:html';
10 import 'dart:math'; 9 import 'dart:math';
11 import 'package:observatory/event.dart'; 10 import 'dart:svg';
12 import 'package:observatory/models.dart' as M; 11
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';
16 import 'package:observatory/src/elements/function_ref.dart'; 20 import 'package:observatory/src/elements/function_ref.dart';
17 import 'package:observatory/src/elements/helpers/any_ref.dart'; 21 import 'package:observatory/src/elements/helpers/any_ref.dart';
18 import 'package:observatory/src/elements/helpers/nav_bar.dart'; 22 import 'package:observatory/src/elements/helpers/nav_bar.dart';
19 import 'package:observatory/src/elements/helpers/nav_menu.dart'; 23 import 'package:observatory/src/elements/helpers/nav_menu.dart';
20 import 'package:observatory/src/elements/helpers/rendering_scheduler.dart'; 24 import 'package:observatory/src/elements/helpers/rendering_scheduler.dart';
21 import 'package:observatory/src/elements/helpers/tag.dart'; 25 import 'package:observatory/src/elements/helpers/tag.dart';
22 import 'package:observatory/src/elements/helpers/uris.dart'; 26 import 'package:observatory/src/elements/helpers/uris.dart';
23 import 'package:observatory/src/elements/instance_ref.dart'; 27 import 'package:observatory/src/elements/instance_ref.dart';
24 import 'package:observatory/src/elements/nav/isolate_menu.dart'; 28 import 'package:observatory/src/elements/nav/isolate_menu.dart';
25 import 'package:observatory/src/elements/nav/notify.dart'; 29 import 'package:observatory/src/elements/nav/notify.dart';
26 import 'package:observatory/src/elements/nav/top_menu.dart'; 30 import 'package:observatory/src/elements/nav/top_menu.dart';
27 import 'package:observatory/src/elements/nav/vm_menu.dart'; 31 import 'package:observatory/src/elements/nav/vm_menu.dart';
28 import 'package:observatory/src/elements/source_inset.dart'; 32 import 'package:observatory/src/elements/source_inset.dart';
29 import 'package:observatory/src/elements/source_link.dart'; 33 import 'package:observatory/src/elements/source_link.dart';
30 import 'package:observatory/service.dart' as S;
31 import 'package:logging/logging.dart';
32 34
33 // TODO(turnidge): Move Debugger, DebuggerCommand to debugger library. 35 // TODO(turnidge): Move Debugger, DebuggerCommand to debugger library.
34 abstract class DebuggerCommand extends Command { 36 abstract class DebuggerCommand extends Command {
35 ObservatoryDebugger debugger; 37 ObservatoryDebugger debugger;
36 38
37 DebuggerCommand(this.debugger, name, children) : super(name, children); 39 DebuggerCommand(this.debugger, name, children) : super(name, children);
38 40
39 String get helpShort; 41 String get helpShort;
40 String get helpLong; 42 String get helpLong;
41 } 43 }
(...skipping 1152 matching lines...) Expand 10 before | Expand all | Expand 10 after
1194 debugger.console.print("No connected VMs"); 1196 debugger.console.print("No connected VMs");
1195 return; 1197 return;
1196 } 1198 }
1197 // TODO(turnidge): Right now there is only one vm listed. 1199 // TODO(turnidge): Right now there is only one vm listed.
1198 var vmList = [debugger.vm]; 1200 var vmList = [debugger.vm];
1199 1201
1200 var maxAddrLen = 'ADDRESS'.length; 1202 var maxAddrLen = 'ADDRESS'.length;
1201 var maxNameLen = 'NAME'.length; 1203 var maxNameLen = 'NAME'.length;
1202 1204
1203 for (var vm in vmList) { 1205 for (var vm in vmList) {
1204 maxAddrLen = max(maxAddrLen, vm.target.networkAddress.length); 1206 maxAddrLen = max(
1207 maxAddrLen, (vm as CommonWebSocketVM).target.networkAddress.length);
1205 maxNameLen = max(maxNameLen, vm.name.length); 1208 maxNameLen = max(maxNameLen, vm.name.length);
1206 } 1209 }
1207 1210
1208 debugger.console.print("${'ADDRESS'.padRight(maxAddrLen, ' ')} " 1211 debugger.console.print("${'ADDRESS'.padRight(maxAddrLen, ' ')} "
1209 "${'NAME'.padRight(maxNameLen, ' ')} " 1212 "${'NAME'.padRight(maxNameLen, ' ')} "
1210 "CURRENT"); 1213 "CURRENT");
1211 for (var vm in vmList) { 1214 for (var vm in vmList) {
1212 String current = (vm == debugger.vm ? '*' : ''); 1215 String current = (vm == debugger.vm ? '*' : '');
1213 debugger.console 1216 debugger.console
1214 .print("${vm.target.networkAddress.padRight(maxAddrLen, ' ')} " 1217 .print("${vm.target.networkAddress.padRight(maxAddrLen, ' ')} "
(...skipping 465 matching lines...) Expand 10 before | Expand all | Expand 10 after
1680 if (bpt.resolved) { 1683 if (bpt.resolved) {
1681 console.print('Breakpoint ${bpId} ${verb} at ${locString}'); 1684 console.print('Breakpoint ${bpId} ${verb} at ${locString}');
1682 } else { 1685 } else {
1683 console.print('Future breakpoint ${bpId} ${verb} at ${locString}'); 1686 console.print('Future breakpoint ${bpId} ${verb} at ${locString}');
1684 } 1687 }
1685 } 1688 }
1686 1689
1687 void onEvent(S.ServiceEvent event) { 1690 void onEvent(S.ServiceEvent event) {
1688 switch (event.kind) { 1691 switch (event.kind) {
1689 case S.ServiceEvent.kVMUpdate: 1692 case S.ServiceEvent.kVMUpdate:
1690 var vm = event.owner; 1693 CommonWebSocketVM vm = event.owner;
1691 console.print("VM ${vm.target.networkAddress} renamed to '${vm.name}'"); 1694 console.print("VM ${vm.target.networkAddress} renamed to '${vm.name}'");
1692 break; 1695 break;
1693 1696
1694 case S.ServiceEvent.kIsolateStart: 1697 case S.ServiceEvent.kIsolateStart:
1695 { 1698 {
1696 var iso = event.owner; 1699 var iso = event.owner;
1697 console.print("Isolate ${iso.number} '${iso.name}' has been created"); 1700 console.print("Isolate ${iso.number} '${iso.name}' has been created");
1698 } 1701 }
1699 break; 1702 break;
1700 1703
(...skipping 1641 matching lines...) Expand 10 before | Expand all | Expand 10 after
3342 ..setAttribute('height', '24') 3345 ..setAttribute('height', '24')
3343 ..children = [ 3346 ..children = [
3344 new PathElement() 3347 new PathElement()
3345 ..setAttribute( 3348 ..setAttribute(
3346 'd', 3349 'd',
3347 'M11 17h2v-6h-2v6zm1-15C6.48 2 2 6.48 2 12s4.48 10 ' 3350 'M11 17h2v-6h-2v6zm1-15C6.48 2 2 6.48 2 12s4.48 10 '
3348 '10 10 10-4.48 10-10S17.52 2 12 2zm0 18c-4.41 ' 3351 '10 10 10-4.48 10-10S17.52 2 12 2zm0 18c-4.41 '
3349 '0-8-3.59-8-8s3.59-8 8-8 8 3.59 8 8-3.59 8-8 8zM11 ' 3352 '0-8-3.59-8-8s3.59-8 8-8 8 3.59 8 8-3.59 8-8 8zM11 '
3350 '9h2V7h-2v2z') 3353 '9h2V7h-2v2z')
3351 ]; 3354 ];
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698