Use latent breakpoints list when looking up or removing breakpoints.

runtime/vm/debugger_test.cc

 // Copyright (c) 2013, 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/dart_api_impl.h"
 #include "vm/dart_api_message.h"
 #include "vm/debugger.h"
 #include "vm/message.h"
 #include "vm/unit_test.h"
 
@@ skipping 20 matching lines @@
 
   char* buffer = Thread::Current()->zone()->Alloc<char>(len + 1);
   va_list args2;
   va_start(args2, fmt);
   OS::VSNPrint(buffer, (len + 1), fmt, args2);
   va_end(args2);
 
   EXPECT_SUBSTRING(buffer, buff);
 }
 
+TEST_CASE(Debugger_GetBreakpointsById) {
+  const char* kScriptChars =
+      "main() {\n"
+      "  var x = new StringBuffer();\n"
+      "  x.add('won');\n"
+      "  x.add('too');\n"
+      "  return x.toString();\n"
+      "}\n";
+  SetFlagScope<bool> sfs(&FLAG_remove_script_timestamps_for_test, true);
+  Dart_Handle lib = TestCase::LoadTestScript(kScriptChars, NULL);
+  EXPECT_VALID(lib);
+
+  Isolate* isolate = Isolate::Current();
+  Debugger* debugger = isolate->debugger();
+
+  // Test with one loaded breakpoint, one latent breakpoint.
+  Dart_Handle url = NewString(TestCase::url());
+  Dart_Handle result = Dart_SetBreakpoint(url, 2);
+  EXPECT_VALID(result);
+  EXPECT(Dart_IsInteger(result));
+  int64_t bp_id1 = 0;
+  EXPECT_VALID(Dart_IntegerToInt64(result, &bp_id1));
+
+  result = Dart_SetBreakpoint(NewString("not_yet_loaded_script_uri"), 4);
+  EXPECT_VALID(result);
+  EXPECT(Dart_IsInteger(result));
+  int64_t bp_id2 = 0;
+  EXPECT_VALID(Dart_IntegerToInt64(result, &bp_id2));
+
+  EXPECT(debugger->GetBreakpointById(bp_id1) != NULL);
+  EXPECT(debugger->GetBreakpointById(bp_id2) != NULL);
+}
+
+TEST_CASE(Debugger_RemoveBreakpoint) {
+  const char* kScriptChars =
+      "main() {\n"
+      "  var x = new StringBuffer();\n"
+      "  x.add('won');\n"
+      "  x.add('too');\n"
+      "  return x.toString();\n"
+      "}\n";
+  SetFlagScope<bool> sfs(&FLAG_remove_script_timestamps_for_test, true);
+  Dart_Handle lib = TestCase::LoadTestScript(kScriptChars, NULL);
+  EXPECT_VALID(lib);
+
+  Isolate* isolate = Isolate::Current();
+  Debugger* debugger = isolate->debugger();
+
+  // Test with one loaded breakpoint, one latent breakpoint.
+  Dart_Handle url = NewString(TestCase::url());
+  Dart_Handle result = Dart_SetBreakpoint(url, 2);
+  EXPECT_VALID(result);
+  EXPECT(Dart_IsInteger(result));
+  int64_t bp_id1 = 0;
+  EXPECT_VALID(Dart_IntegerToInt64(result, &bp_id1));
+
+  result = Dart_SetBreakpoint(NewString("not_yet_loaded_script_uri"), 4);
+  EXPECT_VALID(result);
+  EXPECT(Dart_IsInteger(result));
+  int64_t bp_id2 = 0;
+  EXPECT_VALID(Dart_IntegerToInt64(result, &bp_id2));
+
+  EXPECT(debugger->GetBreakpointById(bp_id1) != NULL);
+  EXPECT(debugger->GetBreakpointById(bp_id2) != NULL);
+
+  debugger->RemoveBreakpoint(bp_id1);
+  debugger->RemoveBreakpoint(bp_id2);
+
+  EXPECT(debugger->GetBreakpointById(bp_id1) == NULL);
+  EXPECT(debugger->GetBreakpointById(bp_id2) == NULL);
+}
 
 TEST_CASE(Debugger_PrintBreakpointsToJSONArray) {
   const char* kScriptChars =
       "main() {\n"
       "  var x = new StringBuffer();\n"
       "  x.add('won');\n"
       "  x.add('too');\n"
       "  return x.toString();\n"
       "}\n";
   SetFlagScope<bool> sfs(&FLAG_remove_script_timestamps_for_test, true);
@@ skipping 10 matching lines @@
   // Empty case.
   {
     JSONStream js;
     {
       JSONArray jsarr(&js);
       debugger->PrintBreakpointsToJSONArray(&jsarr);
     }
     EXPECT_STREQ("[]", js.ToCString());
   }
 
-  // Test with a couple of breakpoints.
+  // Test with a couple of loaded breakpoints, one latent breakpoint.
   Dart_Handle url = NewString(TestCase::url());
   EXPECT_VALID(Dart_SetBreakpoint(url, 2));
   EXPECT_VALID(Dart_SetBreakpoint(url, 3));
+  EXPECT_VALID(Dart_SetBreakpoint(NewString("not_yet_loaded_script_uri"), 4));
   {
     JSONStream js;
     {
       JSONArray jsarr(&js);
       debugger->PrintBreakpointsToJSONArray(&jsarr);
     }
     ExpectSubstringF(
         js.ToCString(),
-        "[{\"type\":\"Breakpoint\",\"fixedId\":true,\"id\":\"breakpoints\\/2\","
-        "\"breakpointNumber\":2,\"resolved\":false,"
-        "\"location\":{\"type\":\"UnresolvedSourceLocation\","
-        "\"script\":{\"type\":\"@Script\",\"fixedId\":true,"
-        "\"id\":\"libraries\\/%s"
-        "\\/scripts\\/test-lib\\/0\","
-        "\"uri\":\"test-lib\","
-        "\"_kind\":\"script\"},\"line\":3}},"
-        "{\"type\":\"Breakpoint\",\"fixedId\":true,\"id\":\"breakpoints\\/1\","
-        "\"breakpointNumber\":1,\"resolved\":false,"
-        "\"location\":{\"type\":\"UnresolvedSourceLocation\","
-        "\"script\":{\"type\":\"@Script\",\"fixedId\":true,"
-        "\"id\":\"libraries\\/%s"
-        "\\/scripts\\/test-lib\\/0\","
-        "\"uri\":\"test-lib\","
-        "\"_kind\":\"script\"},\"line\":2}}]",
+        "[{\"type\":\"Breakpoint\",\"fixedId\":true,\"id\":\"breakpoints\\/"
+        "2\",\"breakpointNumber\":2,\"resolved\":false,\"location\":{\"type\":"
+        "\"UnresolvedSourceLocation\",\"script\":{\"type\":\"@Script\","
+        "\"fixedId\":true,\"id\":\"libraries\\/@15247669\\/scripts\\/"
+        "test-lib\\/"
+        "0\",\"uri\":\"test-lib\",\"_kind\":\"script\"},\"line\":3}},{\"type\":"
+        "\"Breakpoint\",\"fixedId\":true,\"id\":\"breakpoints\\/"
+        "1\",\"breakpointNumber\":1,\"resolved\":false,\"location\":{\"type\":"
+        "\"UnresolvedSourceLocation\",\"script\":{\"type\":\"@Script\","
+        "\"fixedId\":true,\"id\":\"libraries\\/@15247669\\/scripts\\/"
+        "test-lib\\/"
+        "0\",\"uri\":\"test-lib\",\"_kind\":\"script\"},\"line\":2}},{\"type\":"
+        "\"Breakpoint\",\"fixedId\":true,\"id\":\"breakpoints\\/"
+        "3\",\"breakpointNumber\":3,\"resolved\":false,\"location\":{\"type\":"
+        "\"UnresolvedSourceLocation\",\"scriptUri\":\"not_yet_loaded_script_"
+        "uri\",\"line\":4}}]",
         private_key.ToCString(), private_key.ToCString());
   }
 }
 
 
 static bool saw_paused_event = false;
 
 static void InspectPausedEvent(Dart_IsolateId isolate_id,
                                intptr_t bp_id,
                                const Dart_CodeLocation& loc) {
@@ skipping 323 matching lines @@
       "enter(bar1) enter(bar2) enter(bar3) enter(foo) "
       "enter(bar3) enter(foo) "
       "exit(foo) exit(bar3) exit(bar2) exit(bar1) ",
       result_cstr);
   EXPECT(saw_paused_event);
 }
 
 #endif  // !PRODUCT
 
 }  // namespace dart