Allow setting breakpoints in literal function initializers of fields.

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 53 matching lines @@
   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);
 }
 
+static int closure_hit_count = 0;
+int64_t closure_bp_id[4];
+static void PausedInClosuresHandler(Dart_IsolateId isolate_id,
+                                    intptr_t bp_id,
+                                    const Dart_CodeLocation& location) {
+  EXPECT(bp_id == closure_bp_id[closure_hit_count]);
+  closure_hit_count++;
+}
+
+TEST_CASE(Debugger_SetBreakpointInFunctionLiteralFieldInitializers) {
+  const char* kScriptChars =
+      "main() {\n"
+      "  var c = new MyClass();\n"
+      "  c.closure(1, 2);\n"
+      "  MyClass.staticClosure(7, 8);\n"
+      "  closure(3, 4);\n"
+      "  closureSingleLine(5, 6);\n"
+      "}\n"
+      "class MyClass {\n"
+      "  var closure = (int a, int b) {\n"
+      "    return a + b;\n"
+      "  };\n"
+      "  static var staticClosure = (int a, int b) {\n"
+      "    return a * b;\n"
+      "  };\n"
+      "}\n"
+      "var closure = (int a, int b) {\n"
+      "  return a + b;\n"
+      "};\n"
+      "var closureSingleLine = (int a, int b) => a * b;\n"
+      "int v = 10;\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();
+
+  Dart_Handle url = NewString(TestCase::url());
+  Dart_Handle result = Dart_SetBreakpoint(url, 10);
+  EXPECT_VALID(result);
+  EXPECT(Dart_IsInteger(result));
+  EXPECT_VALID(Dart_IntegerToInt64(result, &closure_bp_id[0]));
+
+  result = Dart_SetBreakpoint(url, 13);
+  EXPECT_VALID(result);
+  EXPECT(Dart_IsInteger(result));
+  EXPECT_VALID(Dart_IntegerToInt64(result, &closure_bp_id[1]));
+
+  result = Dart_SetBreakpoint(url, 17);
+  EXPECT_VALID(result);
+  EXPECT(Dart_IsInteger(result));
+  EXPECT_VALID(Dart_IntegerToInt64(result, &closure_bp_id[2]));
+
+  result = Dart_SetBreakpoint(url, 19);
+  EXPECT_VALID(result);
+  EXPECT(Dart_IsInteger(result));
+  EXPECT_VALID(Dart_IntegerToInt64(result, &closure_bp_id[3]));
+
+  result = Dart_SetBreakpoint(url, 20);
+  EXPECT_ERROR(result, "could not set breakpoint at line 20");
+  // TODO(sivachandra): Understand why a breakpoint on a single line
+  // functional literal is not being hit. When that issue is resolved,
+  // adjust this test to check hitting the breakpoint at line 20 also.

[siva, 2017/05/26 17:18:40]

This comment should be above the line
EXPECT(closure_hit_count == 3);
correct?

Also the comment says 'check hitting the breakpoint at line 20 also" I guess you
meant "line 19"

[sivachandra, 2017/05/26 20:27:35]

Done.

+
+  EXPECT(debugger->GetBreakpointById(closure_bp_id[0]) != NULL);
+  EXPECT(debugger->GetBreakpointById(closure_bp_id[1]) != NULL);
+  EXPECT(debugger->GetBreakpointById(closure_bp_id[2]) != NULL);
+  EXPECT(debugger->GetBreakpointById(closure_bp_id[3]) != NULL);
+
+  Dart_SetPausedEventHandler(PausedInClosuresHandler);
+  result = Dart_Invoke(lib, NewString("main"), 0, NULL);
+  EXPECT_VALID(result);
+  EXPECT(closure_hit_count == 3);
+}
+
 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);
@@ skipping 420 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