Setup R10/EDX to be valid Oops before stub calls

runtime/vm/debugger_api_impl_test.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 "include/dart_debugger_api.h"
 #include "include/dart_mirrors_api.h"
 #include "platform/assert.h"
 #include "vm/dart_api_impl.h"
 #include "vm/thread.h"
 #include "vm/unit_test.h"
@@ skipping 956 matching lines @@
 
   breakpoint_hit_counter = 0;
   Dart_Handle retval = Invoke("main");
   EXPECT_VALID(retval);
   int64_t int_value = ToInt64(retval);
   EXPECT_EQ(30, int_value);
   EXPECT_EQ(1, breakpoint_hit_counter);
 }
 
 
+void TestBreakpointHandlerWithVerify(Dart_IsolateId isolate_id,
+                                     Dart_Breakpoint bpt,
+                                     Dart_StackTrace trace) {
+  breakpoint_hit = true;
+  breakpoint_hit_counter++;
+
+  Dart_ActivationFrame frame;
+  Dart_Handle res = Dart_GetActivationFrame(trace, 0, &frame);
+  EXPECT_VALID(res);
+  Dart_Handle func_name;
+  intptr_t line_number;
+  res = Dart_ActivationFrameInfo(frame, &func_name, NULL, &line_number, NULL);
+  OS::Print("Hit line %" Pd "\n", line_number);

[siva, 2014/06/12 23:11:34]

Is the print necessary?

You could just do
intptr_t line_number = -1;
res = Dart_Activation......
EXPECT_NE(line_number, -1);

[rmacnak, 2014/06/12 23:50:15]

This was for debugging why I wasn't hitting the expected number of breakpoints
developing the test, to see which were missing. Adding expectation that we get a
value and hiding the print under verbose.

+
+  VerifyPointersVisitor::VerifyPointers();
+}
+
+
+static void NoopNativeFunction(Dart_NativeArguments args) {
+  Dart_EnterScope();
+  Dart_SetReturnValue(args, Dart_True());
+  Dart_ExitScope();
+}
+
+
+static Dart_NativeFunction NoopNativeResolver(Dart_Handle name,
+                                              int arg_count,
+                                              bool* auto_setup_scope) {
+  ASSERT(auto_setup_scope != NULL);
+  *auto_setup_scope = false;
+  return &NoopNativeFunction;
+}
+
+
+TEST_CASE(Debug_BreakpointStubPatching) {

[siva, 2014/06/12 23:11:34]

Maybe you should add a comment here that if somebody changes this
script here they have to also account for the line numbers in
breakpoint_lines array below.

[rmacnak, 2014/06/12 23:50:15]

Added.

+  const char* kScriptChars =
+      "bar(i) {}                       \n"
+      "nat() native 'a';               \n"
+      "foo(n) {                        \n"
+      "  for(var i = 0; i < n; i++) {  \n"
+      "    bar(i);                     \n"  // Static call.
+      "    i++;                        \n"  // Instance call.
+      "    i == null;                  \n"  // Equality.
+      "    var x = i;                  \n"  // Debug step check.
+      "    i is int;                   \n"  // Subtype test.
+      "    y(z) => () => z + i;        \n"  // Allocate context.
+      "    y(i)();                     \n"  // Closure call.
+      "    nat();                      \n"  // Runtime call.
+      "    return y;                   \n"  // Return.
+      "  }                             \n"
+      "}                               \n"
+      "                                \n"
+      "main() {                        \n"
+      "  var i = 3;                    \n"
+      "  foo(i);                       \n"
+      "}                               \n";
+
+  LoadScript(kScriptChars);
+  Dart_Handle result = Dart_SetNativeResolver(script_lib,
+                                              &NoopNativeResolver,
+                                              NULL);
+  EXPECT_VALID(result);
+  Dart_SetBreakpointHandler(&TestBreakpointHandlerWithVerify);
+
+  Dart_Handle script_url = NewString(TestCase::url());
+
+  const intptr_t num_breakpoints = 9;
+  intptr_t breakpoint_lines[num_breakpoints] =
+      {5, 6, 7, 8, 9, 10, 11, 12, 13};
+
+  for (intptr_t i = 0; i < num_breakpoints; i++) {
+    result = Dart_SetBreakpoint(script_url, breakpoint_lines[i]);
+    EXPECT_VALID(result);
+    EXPECT(Dart_IsInteger(result));
+  }
+
+  breakpoint_hit = false;
+  breakpoint_hit_counter = 0;
+  Dart_Handle retval = Invoke("main");
+  EXPECT_VALID(retval);
+  EXPECT(breakpoint_hit == true);
+  EXPECT_EQ(num_breakpoints, breakpoint_hit_counter);
+}
+
+
 static intptr_t bp_id_to_be_deleted;
 
 static void DeleteBreakpointHandler(Dart_IsolateId isolate_id,
                                     intptr_t bp_id,
                                     const Dart_CodeLocation& location) {
   Dart_StackTrace trace;
   Dart_GetStackTrace(&trace);
   const char* expected_trace[] = {"foo", "main"};
   const intptr_t expected_trace_length = 2;
   breakpoint_hit_counter++;
@@ skipping 1146 matching lines @@
       " null, 3, 7, 1, 8, 6, 9, 10, 10, 11, 11, 13,"
       " null, 4, 13, 3, 14, 10,"
       " null, 5, 17, 5, 18, 9, 19, 12,"
       " null, 6, 21, 1,"
       " null, 8, 24, 1, 25, 5, 26, 6, 27, 8,"
       " null, 9, 29, 1]",
       tokens_cstr);
 }
 
 }  // namespace dart