[debugger] account for inlined functions when stepping.

test/cctest/test-debug.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 6438 matching lines @@
       "  debugger;   \n"
       "  f();        \n"
       "  debugger;   \n"
       "}             \n"
       "f");
   Handle<i::Object> function_obj = v8::Utils::OpenHandle(*result);
   Handle<i::JSFunction> function = Handle<i::JSFunction>::cast(function_obj);
   Handle<i::SharedFunctionInfo> shared(function->shared());
 
   EnableDebugger(isolate);
-  CHECK(i_isolate->debug()->EnsureDebugInfo(shared, function));
+  CHECK(i_isolate->debug()->EnsureDebugInfo(shared));
 
   Handle<i::DebugInfo> debug_info(shared->GetDebugInfo());
   Handle<i::AbstractCode> abstract_code(shared->abstract_code());
 
   {
-    auto iterator = i::BreakIterator::GetIterator(debug_info, abstract_code,
-                                                  i::ALL_BREAK_LOCATIONS);
+    auto iterator = i::BreakIterator::GetIterator(debug_info, abstract_code);
     CHECK(iterator->GetBreakLocation().IsDebuggerStatement());
     CHECK_EQ(17, iterator->GetBreakLocation().position());
     iterator->Next();
     CHECK(iterator->GetBreakLocation().IsDebugBreakSlot());
     CHECK_EQ(32, iterator->GetBreakLocation().position());
     iterator->Next();
     CHECK(iterator->GetBreakLocation().IsCall());
     CHECK_EQ(32, iterator->GetBreakLocation().position());
     iterator->Next();
     CHECK(iterator->GetBreakLocation().IsDebuggerStatement());
     CHECK_EQ(47, iterator->GetBreakLocation().position());
     iterator->Next();
     CHECK(iterator->GetBreakLocation().IsReturn());
     CHECK_EQ(60, iterator->GetBreakLocation().position());
     iterator->Next();
     CHECK(iterator->Done());
   }
 
-  {
-    auto iterator = i::BreakIterator::GetIterator(debug_info, abstract_code,
-                                                  i::CALLS_AND_RETURNS);
-    CHECK(iterator->GetBreakLocation().IsCall());
-    CHECK_EQ(32, iterator->GetBreakLocation().position());
-    iterator->Next();
-    CHECK(iterator->GetBreakLocation().IsReturn());
-    CHECK_EQ(60, iterator->GetBreakLocation().position());
-    iterator->Next();
-    CHECK(iterator->Done());
-  }
-
   DisableDebugger(isolate);
 }
 
 TEST(DisableTailCallElimination) {
   i::FLAG_allow_natives_syntax = true;
   i::FLAG_harmony_tailcalls = true;
   // TODO(ishell, 4698): Investigate why TurboFan in --always-opt mode makes
   // stack[2].getFunctionName() return null.
   i::FLAG_turbo_inlining = false;
 
@@ skipping 158 matching lines @@
                                       reinterpret_cast<void*>(isolate));
     CHECK(!out_of_memory_callback_called);
     // The following allocation fails unless the out-of-memory callback
     // increases the heap limit.
     int length = 10 * i::MB / i::kPointerSize;
     i_isolate->factory()->NewFixedArray(length, i::TENURED);
     CHECK(out_of_memory_callback_called);
   }
   isolate->Dispose();
 }