Fix isolate lifecycle handling in service library

runtime/observatory/test/coverage_test.dart

 // Copyright (c) 2015, 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.
 
 import 'package:observatory/service_io.dart';
 import 'package:unittest/unittest.dart';
 import 'test_helper.dart';
 import 'dart:async';
 
 int globalVar = 100;
@@ skipping 46 matching lines @@
       // Set up a listener to wait for breakpoint events.
       Completer completer = new Completer();
       List events = [];
       isolate.vm.events.stream.listen((ServiceEvent event) {
         if (event.eventType == 'BreakpointReached') {
           print('Breakpoint reached');
           completer.complete();
         }
       });
 
-      // Add the breakpoint.
-      var script = isolate.rootLib.scripts[0];
-      var line = 14;
-      return isolate.addBreakpoint(script, line).then((ServiceObject bpt) {
-          return completer.future;  // Wait for breakpoint reached.
+      // Create a timer to set a breakpoint with a short delay.
+      new Timer(new Duration(milliseconds: 2000), () {
+        // Add the breakpoint.
+        print('Setting breakpoint.');
+        var script = isolate.rootLib.scripts[0];
+        var line = 14;
+        isolate.addBreakpoint(script, line);
       });
+
+      return completer.future;
     });
 },
 
 // Get coverage for function, class, library, script, and isolate.
 (Isolate isolate) {
   return isolate.getStack().then((ServiceMap stack) {
       // Make sure we are in the right place.
       expect(stack.type, equals('Stack'));
       expect(stack['frames'].length, greaterThanOrEqualTo(2));
       expect(stack['frames'][0]['function'].owningClass.name, equals('MyClass'));
@@ skipping 28 matching lines @@
       tests.add(isolate.invokeRpcNoUpgrade('getCoverage',
                                            { 'targetId': lib.id })
                 .then((Map coverage) {
                     expect(coverage['type'], equals('CodeCoverage'));
                     expect(coverage['coverage'].length, equals(3));
                     expect(normalize(coverage['coverage'][0]['hits']),
                            equals([14, 1, 15, 1, 16, 0, 18, 1,
                                    23, 1, 24, 1, 26, 0]));
                     expect(normalize(coverage['coverage'][1]['hits']).take(12),
                            equals([32, 0, 35, 0, 36, 0, 32, 1, 35, 1, 36, 0]));
-                                   
                 }));
       // Script
       tests.add(cls.load().then((_) {
             return isolate.invokeRpcNoUpgrade('getCoverage',
                                               { 'targetId': cls.script.id })
                 .then((Map coverage) {
                     expect(coverage['type'], equals('CodeCoverage'));
                     expect(coverage['coverage'].length, equals(3));
                     expect(normalize(coverage['coverage'][0]['hits']),
                            equals([14, 1, 15, 1, 16, 0, 18, 1,
@@ skipping 10 matching lines @@
                     expect(coverage['coverage'].length, greaterThan(100));
                 });
           }));
       return Future.wait(tests);
   });
 },
 
 ];
 
 main(args) => runIsolateTests(args, tests, testeeConcurrent: testFunction);