[inspector] migrate set/remove BreakPoint to debug-interface.h

src/api.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/api.h"
 
 #include <string.h>  // For memcpy, strlen.
 #ifdef V8_USE_ADDRESS_SANITIZER
 #include <sanitizer/asan_interface.h>
 #endif  // V8_USE_ADDRESS_SANITIZER
@@ skipping 9067 matching lines @@
   i::HandleScope scope(isolate);
   for (i::StackTraceFrameIterator it(isolate); !it.done(); it.Advance()) {
     if (!it.is_javascript()) continue;
     if (!isolate->debug()->IsFrameBlackboxed(it.javascript_frame())) {
       return true;
     }
   }
   return false;
 }
 
+Local<debug::BreakPoint> debug::BreakPoint::New(Isolate* v8_isolate,
+                                                Local<Value> condition,
+                                                Local<Value> data) {
+  i::Isolate* isolate = reinterpret_cast<i::Isolate*>(v8_isolate);
+  ENTER_V8(isolate);
+  i::HandleScope handle_scope(isolate);
+  i::Handle<i::Object> condition_obj = Utils::OpenHandle(*condition);

[Yang, 2017/02/15 13:21:10]

Can we limit the condition to strings only? empty strings don't have to be
checked.

[kozy, 2017/02/15 16:21:33]

Done.

+  i::Handle<i::Object> data_obj = Utils::OpenHandle(*data);
+  i::Handle<i::BreakPoint> break_point =
+      isolate->factory()->NewBreakPoint(condition_obj, data_obj);
+  return ToApiHandle<debug::BreakPoint>(
+      handle_scope.CloseAndEscape(break_point));
+}
+
+MaybeLocal<debug::BreakPoint> debug::BreakPoint::Cast(
+    Isolate* v8_isolate, v8::Local<v8::Value> value) {
+  i::Isolate* isolate = reinterpret_cast<i::Isolate*>(v8_isolate);
+  ENTER_V8(isolate);
+  i::HandleScope handle_scope(isolate);
+  i::Handle<i::Object> break_point_obj = Utils::OpenHandle(*value);
+  if (!break_point_obj->IsBreakPoint()) return MaybeLocal<debug::BreakPoint>();
+  i::Handle<i::BreakPoint> break_point(i::BreakPoint::cast(*break_point_obj),
+                                       isolate);
+  return ToApiHandle<debug::BreakPoint>(
+      handle_scope.CloseAndEscape(break_point));
+}
+
+v8::Local<v8::Value> debug::BreakPoint::Data() const {
+  i::Isolate* isolate = Utils::OpenHandle(this)->GetIsolate();
+  i::HandleScope handle_scope(isolate);
+  i::Handle<i::BreakPoint> break_point = Utils::OpenHandle(this);
+  i::Handle<i::Object> data_obj(break_point->data(), isolate);
+  return Utils::ToLocal(handle_scope.CloseAndEscape(data_obj));
+}
+
 v8::Isolate* debug::Script::GetIsolate() const {
   return reinterpret_cast<v8::Isolate*>(Utils::OpenHandle(this)->GetIsolate());
 }
 
 ScriptOriginOptions debug::Script::OriginOptions() const {
   return Utils::OpenHandle(this)->origin_options();
 }
 
 bool debug::Script::WasCompiled() const {
   return Utils::OpenHandle(this)->compilation_state() ==
@@ skipping 134 matching lines @@
       line_offset = GetSmiValue(line_ends, current_line_end_index - 1) + 1;
     }
     locations->push_back(debug::Location(
         current_line_end_index + script->line_offset(),
         offset - line_offset +
             (current_line_end_index == 0 ? script->column_offset() : 0)));
   }
   return true;
 }
 
+debug::Location debug::Script::SetBreakPoint(
+    const debug::Location& position,
+    Local<debug::BreakPoint> break_point) const {
+  i::Handle<i::Script> script = Utils::OpenHandle(this);
+  i::Isolate* isolate = script->GetIsolate();
+  int offset = GetSourcePosition(position);
+  i::Handle<i::BreakPoint> break_point_obj = Utils::OpenHandle(*break_point);
+  int result_offset = offset;
+  if (!isolate->debug()->SetBreakPointForScript(
+          script, break_point_obj, &result_offset, i::BREAK_POSITION_ALIGNED)) {
+    return debug::Location();
+  }
+  i::Script::PositionInfo info;
+  i::Script::GetPositionInfo(script, result_offset, &info,
+                             i::Script::WITH_OFFSET);
+  return debug::Location(info.line, info.column);
+}
+
 int debug::Script::GetSourcePosition(const debug::Location& location) const {
   i::Handle<i::Script> script = Utils::OpenHandle(this);
   if (script->type() == i::Script::TYPE_WASM) {
-    // TODO(clemensh): Return the proper thing for wasm.
-    return 0;
+    return i::WasmCompiledModule::cast(script->wasm_compiled_module())
+               ->GetFunctionOffset(location.GetLineNumber()) +
+           location.GetColumnNumber();
   }
 
   int line = std::max(location.GetLineNumber() - script->line_offset(), 0);
   int column = location.GetColumnNumber();
   if (line == 0) {
     column = std::max(0, column - script->column_offset());
   }
 
   i::Script::InitLineEnds(script);
   CHECK(script->line_ends()->IsFixedArray());
@@ skipping 53 matching lines @@
 debug::WasmDisassembly debug::WasmScript::DisassembleFunction(
     int function_index) const {
   i::DisallowHeapAllocation no_gc;
   i::Handle<i::Script> script = Utils::OpenHandle(this);
   DCHECK_EQ(i::Script::TYPE_WASM, script->type());
   i::WasmCompiledModule* compiled_module =
       i::WasmCompiledModule::cast(script->wasm_compiled_module());
   return compiled_module->DisassembleFunction(function_index);
 }
 
+void debug::ClearBreakPoint(v8::Isolate* v8_isolate,
+                            Local<BreakPoint> break_point) {
+  i::Isolate* isolate = reinterpret_cast<i::Isolate*>(v8_isolate);
+  ENTER_V8(isolate);
+  i::Handle<i::Object> break_point_obj = Utils::OpenHandle(*break_point);
+  isolate->debug()->ClearBreakPoint(break_point_obj);
+}
+
 debug::Location::Location(int line_number, int column_number)
     : line_number_(line_number), column_number_(column_number) {
   CHECK(line_number >= 0);
   CHECK(column_number >= 0);
 }
 
 debug::Location::Location()
     : line_number_(v8::Function::kLineOffsetNotFound),
       column_number_(v8::Function::kLineOffsetNotFound) {}
 
 int debug::Location::GetLineNumber() const {
   CHECK(line_number_ >= 0);
   return line_number_;
 }
 
 int debug::Location::GetColumnNumber() const {
   CHECK(column_number_ >= 0);
   return column_number_;
 }
 
 bool debug::Location::IsEmpty() const {
-  return line_number_ == -1 && column_number_ == -1;
+  return line_number_ == v8::Function::kLineOffsetNotFound &&
+         column_number_ == v8::Function::kLineOffsetNotFound;
 }
 
 void debug::GetLoadedScripts(v8::Isolate* v8_isolate,
                              PersistentValueVector<debug::Script>& scripts) {
   i::Isolate* isolate = reinterpret_cast<i::Isolate*>(v8_isolate);
   ENTER_V8(isolate);
   // TODO(kozyatinskiy): remove this GC once tests are dealt with.
   isolate->heap()->CollectAllGarbage(i::Heap::kFinalizeIncrementalMarkingMask,
                                      i::GarbageCollectionReason::kDebugger);
   {
@@ skipping 817 matching lines @@
   Address callback_address =
       reinterpret_cast<Address>(reinterpret_cast<intptr_t>(callback));
   VMState<EXTERNAL> state(isolate);
   ExternalCallbackScope call_scope(isolate, callback_address);
   callback(info);
 }
 
 
 }  // namespace internal
 }  // namespace v8