[inspector] migrate scriptParsed and getCompiledScripts to native

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 "include/v8-debug.h"
+#include "include/v8-util.h"
 #include "include/v8.h"
 
 namespace v8 {
 
 class DebugInterface {
  public:
   /**
    * An event details object passed to the debug event listener.
    */
   class EventDetails : public v8::Debug::EventDetails {
@@ skipping 112 matching lines @@
   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);
+
+  class Script {

[dgozman, 2016/10/28 19:57:15]

What about comments? Do we drop the altogether or keep adding good explanations?

[kozy, 2016/10/28 20:59:37]

Done.

+   public:
+    ScriptOriginOptions OriginOptions() const;
+    bool WasCompiled() const;
+    int Id() const;
+    int LineOffset() const;
+    int ColumnOffset() const;
+    MaybeLocal<Array> LineEnds(v8::Local<v8::Context>) const;

[dgozman, 2016/10/28 19:57:15]

Let's go with vector<int>, and remove all contexts here.

[kozy, 2016/10/28 20:59:37]

Done.

+    MaybeLocal<String> Name(v8::Local<v8::Context>) const;
+    MaybeLocal<String> SourceURL(v8::Local<v8::Context>) const;
+    MaybeLocal<String> SourceMappingURL(v8::Local<v8::Context>) const;
+    MaybeLocal<String> ContextData(v8::Local<v8::Context>) const;
+    MaybeLocal<String> Source(v8::Local<v8::Context>) const;
+
+    static MaybeLocal<Script> Wrap(v8::Local<v8::Context>,

[dgozman, 2016/10/28 19:57:15]

We definitely need a comment for this one.

[kozy, 2016/10/28 20:59:37]

Done.

+                                   v8::Local<v8::Object>);
+  };
+
+  static void GetLoadedScripts(Isolate* isolate,
+                               PersistentValueVector<Script>& scripts);
 };
 
 }  // namespace v8
 
 #endif  // V8_DEBUG_DEBUG_INTERFACE_H_