[inspector] Introduce translation of wasm frames

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"
@@ skipping 184 matching lines @@
 
    private:
     int GetSourcePosition(const Location& location) const;
   };
 
   /**
    * Return array of compiled scripts.
    */
   static void GetLoadedScripts(Isolate* isolate,
                                PersistentValueVector<Script>& scripts);
+
+  static std::pair<std::string, std::vector<std::tuple<uint32_t, int, int>>>

[titzer, 2016/11/11 10:59:02]

Can we introduce a struct for this type, instead of the std::tuple?

[dgozman, 2016/11/11 21:50:34]

+1
It will also help with describing what this complex structure means.

[Clemens Hammacher, 2016/11/16 11:36:28]

As discussed offline with Ben:
Introducing a struct here requires us to either include the debug-interface.h in
several wasm headers where this information is produced and passed.
We therefore leave it a tuple for now.

[kozy, 2016/11/16 16:13:41]

We already include debug-interface.h in debug.h and I think it's ok to include
this file in all places where parts of interface implementation are defined.
Tuple requires looking up what each field actually means each time when you see
them in codebase.

+  DisassembleWasmFunction(Isolate* isolate, v8::Local<v8::Object> script,
+                          int function_index);
 };
 
 }  // namespace v8
 
 #endif  // V8_DEBUG_DEBUG_INTERFACE_H_