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

src/debug/debug.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/debug/debug.h"
 
 #include <memory>
 
 #include "src/api.h"
 #include "src/arguments.h"
 #include "src/bootstrapper.h"
 #include "src/code-stubs.h"
 #include "src/codegen.h"
 #include "src/compilation-cache.h"
 #include "src/compiler-dispatcher/optimizing-compile-dispatcher.h"
 #include "src/compiler.h"
 #include "src/debug/debug-evaluate.h"
+#include "src/debug/debug-frames.h"
 #include "src/debug/liveedit.h"
 #include "src/deoptimizer.h"
 #include "src/execution.h"
 #include "src/frames-inl.h"
 #include "src/full-codegen/full-codegen.h"
 #include "src/global-handles.h"
 #include "src/globals.h"
 #include "src/interpreter/interpreter.h"
 #include "src/isolate-inl.h"
 #include "src/list.h"
@@ skipping 592 matching lines @@
   Handle<Object> undefined = isolate_->factory()->undefined_value();
   MaybeHandle<Object> maybe_exception;
   return Execution::TryCall(isolate_, fun, undefined, argc, args,
                             Execution::MessageHandling::kReport,
                             &maybe_exception);
 }
 
 
 // Check whether a single break point object is triggered.
 bool Debug::CheckBreakPoint(Handle<Object> break_point_object) {
-  Factory* factory = isolate_->factory();
-  HandleScope scope(isolate_);
+  if (break_point_object->IsBreakPoint()) {
+    Handle<BreakPoint> break_point =
+        Handle<BreakPoint>::cast(break_point_object);
+    if (!break_point->condition()->length()) return true;
+    Handle<String> condition(break_point->condition());
+    Handle<Object> result;
+    // Since we call CheckBreakpoint only for deoptimized frame on top of stack,
+    // we can use 0 as index of inlined frame.
+    if (!DebugEvaluate::Local(isolate_, break_frame_id(),
+                              /* inlined_jsframe_index */ 0, condition, false)
+             .ToHandle(&result)) {
+      if (isolate_->has_pending_exception()) {
+        isolate_->clear_pending_exception();
+      }
+      return false;
+    }
+    return result->IsTrue(isolate_);
+  }
 
+  // TODO(yangguo): remove code below with mirrors API.
   // Ignore check if break point object is not a JSObject.
   if (!break_point_object->IsJSObject()) return true;
 
   // Get the break id as an object.
-  Handle<Object> break_id = factory->NewNumberFromInt(Debug::break_id());
+  Handle<Object> break_id =
+      isolate_->factory()->NewNumberFromInt(Debug::break_id());
 
   // Call IsBreakPointTriggered.
   Handle<Object> argv[] = { break_id, break_point_object };
   Handle<Object> result;
   if (!CallFunction("IsBreakPointTriggered", arraysize(argv), argv)
            .ToHandle(&result)) {
     return false;
   }
 
   // Return whether the break point is triggered.
@@ skipping 1765 matching lines @@
     isolate_->Throw(*isolate_->factory()->NewEvalError(
         MessageTemplate::kNoSideEffectDebugEvaluate));
   }
   isolate_->set_needs_side_effect_check(old_needs_side_effect_check_);
   isolate_->debug()->UpdateHookOnFunctionCall();
   isolate_->debug()->side_effect_check_failed_ = false;
 }
 
 }  // namespace internal
 }  // namespace v8