Fix bug in processing of latent breakpoints

runtime/vm/debugger.cc

 // Copyright (c) 2012, 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.
 
 #include "vm/debugger.h"
 
 #include "include/dart_api.h"
 
 #include "vm/code_generator.h"
 #include "vm/code_patcher.h"
@@ skipping 2383 matching lines @@
   }
   Library& lib = Library::Handle(isolate_);
   Script& script = Script::Handle(isolate_);
   String& url = String::Handle(isolate_);
   SourceBreakpoint* bpt = latent_breakpoints_;
   SourceBreakpoint* prev_bpt = NULL;
   const GrowableObjectArray& libs =
       GrowableObjectArray::Handle(isolate_->object_store()->libraries());
   while (bpt != NULL) {
     url = bpt->url();
+    bool found_match = false;
     for (intptr_t i = 0; i < libs.Length(); i++) {
       lib ^= libs.At(i);
       script = lib.LookupScript(url);
       if (!script.IsNull()) {
         // Found a script with matching url for this latent breakpoint.
         // Unlink the latent breakpoint from the list.
+        found_match = true;
         SourceBreakpoint* matched_bpt = bpt;
         bpt = bpt->next();
         if (prev_bpt == NULL) {
           latent_breakpoints_ = bpt;
         } else {
           prev_bpt->set_next(bpt);
         }
         // Now find the token range at the requested line and make a
         // new unresolved source breakpoint.
         intptr_t line_number = matched_bpt->LineNumber();
@@ skipping 44 matching lines @@
           // TODO(hausner): There is one possible pitfall here.
           // If the user sets a latent breakpoint using a partial url that
           // ends up matching more than one script, the breakpoint might
           // get set in the wrong script.
           // It would be better if we could warn the user if multiple
           // scripts are matching.
           break;
         }
       }
     }
+    if (!found_match) {
+      // No matching url found in any of the libraries.
+      if (FLAG_verbose_debug) {
+        OS::Print("No match found for latent breakpoint id "
+                  "%" Pd " with url '%s'\n",
+                  bpt->id(),
+                  url.ToCString());
+      }
+      bpt = bpt->next();
+    }
   }
 }
 
 
 // TODO(hausner): Could potentially make this faster by checking
 // whether the call target at pc is a debugger stub.
 bool Debugger::HasActiveBreakpoint(uword pc) {
   CodeBreakpoint* bpt = GetCodeBreakpoint(pc);
   return (bpt != NULL) && (bpt->IsEnabled());
 }
@@ skipping 147 matching lines @@
 }
 
 
 void Debugger::RegisterCodeBreakpoint(CodeBreakpoint* bpt) {
   ASSERT(bpt->next() == NULL);
   bpt->set_next(code_breakpoints_);
   code_breakpoints_ = bpt;
 }
 
 }  // namespace dart