Initial GetSample implementation.

include/v8.h

Index: include/v8.h
diff --git a/include/v8.h b/include/v8.h
index 73384946ba7d93594fd5c3d78753dfe01c2726f6..766bd9d1fa1b855936a9a307795f91a215a60b38 100644
--- a/include/v8.h
+++ b/include/v8.h
@@ -6691,6 +6691,21 @@ void* Context::GetAlignedPointerFromEmbedderData(int index) {
 }
 
 
+// Logging and profiling.  A StateTag represents a possible state of
+// the VM. The logger maintains a stack of these. Creating a VMState
+// object enters a state by pushing on the stack, and destroying a
+// VMState object leaves a state by popping the current state from the
+// stack.
+
+enum StateTag {

[Benedikt Meurer, 2014/08/14 04:20:23]

Move this to v8-sampler.h and fix the comment (VMState is an internal thing, no
need to document that in a public header).

[gholap, 2014/08/15 17:54:13]

Now that you've mentioned it, I've looked into both devtools and trace viewer
code and they don't use the state. So, after a discussion with fmeawad, I
removed this from public API altogether. Please let me know whether this is a
bad idea...

[Benedikt Meurer, 2014/08/18 04:22:03]

Great! We should not expose internal state via the public API.

+  JS,              // Executing JavaScript.
+  GC,              // Garbage Collection.
+  COMPILER,        // Compiling JavaScript.
+  OTHER,
+  EXTERNAL,        // External call. (For example, a call into blink)
+  IDLE             // The VM is idle.
+};
+
 /**
  * \example shell.cc
  * A simple shell that takes a list of expressions on the