Implement function entry breakpoints in Observatory debugger.

runtime/observatory/lib/src/elements/debugger.dart

 // Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library debugger_page_element;
 
 import 'dart:async';
 import 'dart:html';
 import 'observatory_element.dart';
 import 'package:observatory/cli.dart';
@@ skipping 214 matching lines @@
 
   Future run(List<String> args) {
     if (args.length > 1) {
       debugger.console.print('not implemented');
       return new Future.value(null);
     }
     var arg = (args.length == 0 ? '' : args[0]);
     return SourceLocation.parse(debugger, arg).then((loc) {
       if (loc.valid) {
         if (loc.function != null) {
-          debugger.console.print(
-              'Ignoring breakpoint at $loc: '
-              'Function entry breakpoints not yet implemented');
-          return null;
+          return debugger.isolate.addBreakpointAtEntry(loc.function)
+            .then((result) => _handleBreakpointResult(loc, result));
+        } else {
+          assert(loc.script != null);
+          if (loc.col != null) {
+            // TODO(turnidge): Add tokenPos breakpoint support.
+            debugger.console.print(
+                'Ignoring column: '
+                'adding breakpoint at a specific column not yet implemented');
+          }
+          return debugger.isolate.addBreakpoint(loc.script, loc.line)
+            .then((result) => _handleBreakpointResult(loc, result));
         }
-        if (loc.col != null) {
-          // TODO(turnidge): Add tokenPos breakpoint support.
-          debugger.console.print(
-              'Ignoring column: '
-              'adding breakpoint at a specific column not yet implemented');
-        }
-        return debugger.isolate.addBreakpoint(loc.script, loc.line).then((result) {
-          if (result is DartError) {
-            debugger.console.print('Unable to set breakpoint at ${loc}');
-          } else {
-            // TODO(turnidge): Adding a duplicate breakpoint is
-            // currently ignored.  May want to change the protocol to
-            // inform us when this happens.
-
-            // The BreakpointResolved event prints resolved
-            // breakpoints already.  Just print the unresolved ones here.
-            ServiceMap bpt = result;
-            if (!bpt['resolved']) {
-              var script = bpt['location']['script'];
-              var bpId = bpt['breakpointNumber'];
-              var tokenPos = bpt['location']['tokenPos'];
-              return script.load().then((_) {
-                var line = script.tokenToLine(tokenPos);
-                var col = script.tokenToCol(tokenPos);
-                debugger.console.print(
-                    'Future breakpoint ${bpId} added at '
-                    '${script.name}:${line}:${col}');
-              });
-            }
-          }
-        });
       } else {
         debugger.console.print(loc.errorMessage);
       }
     });
   }
 
+  Future _handleBreakpointResult(loc, result) {
+    if (result is DartError) {
+      debugger.console.print('Unable to set breakpoint at ${loc}');
+    } else {
+      // TODO(turnidge): Adding a duplicate breakpoint is
+      // currently ignored.  May want to change the protocol to
+      // inform us when this happens.
+
+      // The BreakpointResolved event prints resolved
+      // breakpoints already.  Just print the unresolved ones here.
+      ServiceMap bpt = result;
+      if (!bpt['resolved']) {
+        return debugger._reportBreakpointAdded(bpt);
+      }
+    }
+    return new Future.value(null);
+  }
+
   Future<List<String>> complete(List<String> args) {
     if (args.length != 1) {
       return new Future.value([]);
     }
     // TODO - fix SourceLocation complete
     return new Future.value(SourceLocation.complete(debugger, args[0]));
   }
 
   String helpShort = 'Add a breakpoint by source location or function name';
 
@@ skipping 408 matching lines @@
           // TODO(turnidge): Test this.
           console.print(
               'Exception ${event.exception} at ${script.name}:${line}:${col}');
         } else {
           console.print('Paused at ${script.name}:${line}:${col}');
         }
       });
     }
   }
 
+  Future _reportBreakpointAdded(ServiceMap bpt) {
+    var script = bpt['location']['script'];
+    return script.load().then((_) {
+      var bpId = bpt['breakpointNumber'];
+      var tokenPos = bpt['location']['tokenPos'];
+      var line = script.tokenToLine(tokenPos);
+      var col = script.tokenToCol(tokenPos);
+      if (bpt['resolved']) {
+        // TODO(turnidge): If this was a future breakpoint before, we
+        // should change the message to say that the breakpoint was 'resolved',
+        // rather than 'added'.
+        console.print(
+            'Breakpoint ${bpId} added at ${script.name}:${line}:${col}');
+      } else {
+        console.print(
+            'Future breakpoint ${bpId} added at ${script.name}:${line}:${col}');
+      }
+    });
+  }
+
   void _onEvent(ServiceEvent event) {
     if (event.owner != isolate) {
       return;
     }
     switch(event.eventType) {
       case 'IsolateShutdown':
         console.print('Isolate shutdown');
         isolate = null;
         break;
 
       case 'BreakpointReached':
       case 'IsolateInterrupted':
       case 'ExceptionThrown':
         _refreshStack(event).then((_) {
           _reportPause(event);
         });
         break;
 
       case 'IsolateResumed':
         console.print('Continuing...');
         break;
 
       case 'BreakpointResolved':
-        var bpId = event.breakpoint['breakpointNumber'];
-        var script = event.breakpoint['location']['script'];
-        var tokenPos = event.breakpoint['location']['tokenPos'];
-        var line = script.tokenToLine(tokenPos);
-        var col = script.tokenToCol(tokenPos);
-        console.print(
-            'Breakpoint ${bpId} added at ${script.name}:${line}:${col}');
+        _reportBreakpointAdded(event.breakpoint);
         break;
 
       case '_Graph':
       case 'IsolateCreated':
       case 'GC':
         // Ignore these events for now.
         break;
 
       default:
         console.print('Unrecognized event: $event');
@@ skipping 328 matching lines @@
               debugger.run(command).whenComplete(_notBusy);
             }
             break;
         }
       });
   }
 
   DebuggerInputElement.created() : super.created();
 }