CPUProfiler: propagate scriptId to the front-end

include/v8-profiler.h

Index: include/v8-profiler.h
diff --git a/include/v8-profiler.h b/include/v8-profiler.h
index 5338fcd59c2c7711ef475e9949b22411c6374b6c..1fdfbf257e04dd13a8271b779e334e6ff7a838a0 100644
--- a/include/v8-profiler.h
+++ b/include/v8-profiler.h
@@ -75,6 +75,9 @@ class V8EXPORT CpuProfileNode {
   /** Returns function name (empty string for anonymous functions.) */
   Handle<String> GetFunctionName() const;
 
+  /** Returns id of the script where function is located. */
+  int GetScriptId() const;

[yurys, 2013/06/25 09:16:55]

In v8.h Script::Id returns v8::Local<v8::Value> and internally it seems to be
stored as i::String:
https://code.google.com/p/chromium/codesearch#chromium/src/v8/src/objects-inl....
I think we should be consistent with v8::Script::Id() type here and use
Local<Value> as the return type.

Or if it is always int internally we could deprecate v8::Local<v8::Value>
v8::Script::Id() and introduce int v8::Script::GetId() instead.

+
   /** Returns resource name for script from where the function originates. */
   Handle<String> GetScriptResourceName() const;
 
@@ -114,6 +117,7 @@ class V8EXPORT CpuProfileNode {
   /** Retrieves a child node by index. */
   const CpuProfileNode* GetChild(int index) const;
 
+  static const int kNoScriptId = 0;
   static const int kNoLineNumberInfo = Message::kNoLineNumberInfo;
 };