[debug] Remove DebugInterface class

src/debug/debug-interface.h

Index: src/debug/debug-interface.h
diff --git a/src/debug/debug-interface.h b/src/debug/debug-interface.h
index 137c2ab6ac363da30aa34714a2829b72abdf082d..5db1a69e486f3faec581e497635054e9fb9a990c 100644
--- a/src/debug/debug-interface.h
+++ b/src/debug/debug-interface.h
@@ -12,43 +12,11 @@
 #include "src/debug/interface-types.h"
 
 namespace v8 {
+namespace debug {
 
 class DebugInterface {

[Yang, 2016/12/05 13:36:08]

I actually meant to remove this class altogether. It only contains types,
classes, and static functions anyways.

[Clemens Hammacher, 2016/12/05 18:38:22]

Oh, I see. Fixed.

  public:
   /**
-   * An event details object passed to the debug event listener.
-   */
-  class EventDetails : public v8::Debug::EventDetails {
-   public:
-    /**
-     * Event type.
-     */
-    virtual v8::DebugEvent GetEvent() const = 0;
-
-    /**
-     * Access to execution state and event data of the debug event. Don't store
-     * these cross callbacks as their content becomes invalid.
-     */
-    virtual Local<Object> GetExecutionState() const = 0;
-    virtual Local<Object> GetEventData() const = 0;
-
-    /**
-     * Get the context active when the debug event happened. Note this is not
-     * the current active context as the JavaScript part of the debugger is
-     * running in its own context which is entered at this point.
-     */
-    virtual Local<Context> GetEventContext() const = 0;
-
-    /**
-     * Client data passed with the corresponding callback when it was
-     * registered.
-     */
-    virtual Local<Value> GetCallbackData() const = 0;
-
-    virtual ~EventDetails() {}
-  };
-
-  /**
    * Debug event callback function.
    *
    * \param event_details object providing information about the debug event
@@ -116,12 +84,6 @@ class DebugInterface {
   static MaybeLocal<Array> GetInternalProperties(Isolate* isolate,
                                                  Local<Value> value);
 
-  enum ExceptionBreakState {
-    NoBreakOnException = 0,
-    BreakOnUncaughtException = 1,
-    BreakOnAnyException = 2
-  };
-
   /**
    * Defines if VM will pause on exceptions or not.
    * If BreakOnAnyExceptions is set then VM will pause on caught and uncaught
@@ -131,55 +93,9 @@ class DebugInterface {
   static 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
-                   // in the current function.
-    StepFrame = 3  // Step into a new frame or return to previous frame.
-  };
-
   static void PrepareStep(Isolate* isolate, StepAction action);
   static void ClearStepping(Isolate* isolate);
 
-  /**
-   * Native wrapper around v8::internal::Script object.
-   */
-  class Script {
-   public:
-    v8::Isolate* GetIsolate() const;
-
-    ScriptOriginOptions OriginOptions() const;
-    bool WasCompiled() const;
-    int Id() const;
-    int LineOffset() const;
-    int ColumnOffset() const;
-    std::vector<int> LineEnds() const;
-    MaybeLocal<String> Name() const;
-    MaybeLocal<String> SourceURL() const;
-    MaybeLocal<String> SourceMappingURL() const;
-    MaybeLocal<String> 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;
-
-    /**
-     * script parameter is a wrapper v8::internal::JSObject for
-     * v8::internal::Script.
-     * This function gets v8::internal::Script from v8::internal::JSObject and
-     * wraps it with DebugInterface::Script.
-     * Returns empty local if not called with a valid wrapper of
-     * v8::internal::Script.
-     */
-    static MaybeLocal<Script> Wrap(Isolate* isolate,
-                                   v8::Local<v8::Object> script);
-
-   private:
-    int GetSourcePosition(const debug::Location& location) const;
-  };
-
   static void GetLoadedScripts(Isolate* isolate,
                                PersistentValueVector<Script>& scripts);
 
@@ -193,6 +109,7 @@ class DebugInterface {
                                                           Local<String> source);
 };
 
+}  // namespace debug
 }  // namespace v8
 
 #endif  // V8_DEBUG_DEBUG_INTERFACE_H_