Move enum CompilationType from objects.h to include/v8.h,

include/v8.h

Index: include/v8.h
diff --git a/include/v8.h b/include/v8.h
index 3252602bcf19d8fabba52e0c7fbbd274615f7ff2..513ae1378bb910505f28e7240c5954a7af5e58d2 100644
--- a/include/v8.h
+++ b/include/v8.h
@@ -978,6 +978,12 @@ class ScriptOrigin {
  */
 class V8_EXPORT Script {
  public:
+    // Script compilation types.
+    enum CompilationType {

[Michael Starzinger, 2013/08/19 19:42:28]

We try not to expose internal behavior (and constants) through the API,
especially when they are not use in the API at all.

IIUC, these constants will be used to compare against Script::compile_type, or
to me more precises against ScriptMirror.prototype.compilationType. This is only
available if debugger support is actually enabled, another reason not to put the
constant into the API.

Would it be possible to add a ScriptMirror.prototype.compiledThroughEval (or
similar name) predicate to do the job? This predicate could then rely on the
JavaScript constants that we have defined in macros.py for that purpose. Also
note that I am not particularly familiar with the debugger API, so take this
suggestion with a grain of salt.

+      COMPILATION_TYPE_HOST = 0,
+      COMPILATION_TYPE_EVAL = 1
+    };
+
   /**
    * Compiles the specified script (context-independent).
    *