[inspector] implemented blackboxing inside v8

src/debug/debug-interface.h

 // Copyright 2016 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.
 
 #ifndef V8_DEBUG_DEBUG_INTERFACE_H_
 #define V8_DEBUG_DEBUG_INTERFACE_H_
 
 #include <functional>
 
 #include "include/v8-debug.h"
@@ skipping 68 matching lines @@
  * Defines if VM will pause on exceptions or not.
  * If BreakOnAnyExceptions is set then VM will pause on caught and uncaught
  * exception, if BreakOnUncaughtException is set then VM will pause only on
  * uncaught exception, otherwise VM won't stop on any exception.
  */
 void ChangeBreakOnException(Isolate* isolate, ExceptionBreakState state);
 
 enum StepAction {
   StepOut = 0,   // Step out of the current function.
   StepNext = 1,  // Step to the next statement in the current function.
-  StepIn = 2,    // Step into new functions invoked or the next statement
+  StepIn = 2     // Step into new functions invoked or the next statement
                  // in the current function.
-  StepFrame = 3  // Step into a new frame or return to previous frame.
 };
 
 void PrepareStep(Isolate* isolate, StepAction action);
-void ClearStepping(Isolate* isolate);
 
 /**
  * Out-of-memory callback function.
  * The function is invoked when the heap size is close to the hard limit.
  *
  * \param data the parameter provided during callback installation.
  */
 typedef void (*OutOfMemoryCallback)(void* data);
 void SetOutOfMemoryCallback(Isolate* isolate, OutOfMemoryCallback callback,
                             void* data);
@@ skipping 13 matching lines @@
   std::vector<int> LineEnds() const;
   MaybeLocal<String> Name() const;
   MaybeLocal<String> SourceURL() const;
   MaybeLocal<String> SourceMappingURL() const;
   MaybeLocal<Value> ContextData() const;
   MaybeLocal<String> Source() const;
   bool IsWasm() const;
   bool GetPossibleBreakpoints(const debug::Location& start,
                               const debug::Location& end,
                               std::vector<debug::Location>* locations) const;
+  void BlackboxStateChanged() const;

[dgozman, 2017/01/19 21:49:14]

Does it make sense to push ranges here instead of asking via callback?

[dgozman, 2017/01/19 22:34:20]

- ResetBlackboxedStateCache
- Make it a separate function and pass script to it.

[kozy, 2017/01/20 02:32:36]

Done.

[kozy, 2017/01/20 02:32:36]

Acknowledged.

 
  private:
   int GetSourcePosition(const debug::Location& location) const;
 };
 
 // Specialization for wasm Scripts.
 class WasmScript : public Script {
  public:
   static WasmScript* Cast(Script* script);
 
@@ skipping 19 matching lines @@
                                      v8::Local<v8::Object> exec_state,
                                      v8::Local<v8::Value> break_points_hit) {}
   virtual void ExceptionThrown(v8::Local<v8::Context> paused_context,
                                v8::Local<v8::Object> exec_state,
                                v8::Local<v8::Value> exception,
                                bool is_promise_rejection, bool is_uncaught) {}
 };
 
 void SetDebugEventListener(Isolate* isolate, DebugEventListener* listener);
 
+typedef std::function<bool(v8::Local<debug::Script> script,
+                           const debug::Location& start,
+                           const debug::Location& end, void* data)>
+    IsBlackboxedCallback;
+void SetIsBlackboxedCallback(Isolate* isolate, IsBlackboxedCallback callback,

[dgozman, 2017/01/19 21:49:14]

Make this a part of DebugEventListener?

[kozy, 2017/01/20 02:32:37]

DebugEventListener -> DebugDelegate and done.

+                             void* data);
+
+bool HasUserFrameOnStack(Isolate* isolate);

[dgozman, 2017/01/19 21:49:14]

- Let's move this closer to other functions (e.g.next to PrepareStep).
- HasNonBlackboxedFrameOnStack.

[kozy, 2017/01/20 02:32:37]

Done.

+
 }  // namespace debug
 }  // namespace v8
 
 #endif  // V8_DEBUG_DEBUG_INTERFACE_H_