chakrashim: some v8.h shim cleanup

Add apis needed by next+1. Add v8config.h to simplify future v8.h alignment. Correct EXPORT to support dllimport. Make Handle<> alias of Local<> as in latest official v8.h. Reviewed-by: @kunalspathak
nodejs · Aug 11, 2015 · 8cbbfbc · 8cbbfbc
1 parent 5a54c00
commit 8cbbfbc
Show file tree

Hide file tree

Showing 21 changed files with 616 additions and 184 deletions.
diff --git a/deps/chakrashim/chakrashim.gyp b/deps/chakrashim/chakrashim.gyp
@@ -18,6 +18,7 @@
     ],
     'defines': [
       'USE_EDGEMODE_JSRT=1',
+      'BUILDING_CHAKRASHIM=1',
     ],
     'conditions': [
       [ 'target_arch=="ia32"', { 'defines': [ '__i386__=1' ] } ],
@@ -33,6 +34,7 @@
       ],
       'defines': [
         'USE_EDGEMODE_JSRT=1',
+        'BUILDING_CHAKRASHIM=1',
         '_WIN32_WINNT=0x0A00',  # WIN10
       ],
       'libraries': [
@@ -50,6 +52,7 @@
     'sources': [
       'include/libplatform/libplatform.h',
       'include/v8.h',
+      'include/v8config.h',
       'include/v8-debug.h',
       'include/v8-platform.h',
       'include/v8-profiler.h',

diff --git a/deps/chakrashim/include/v8-debug.h b/deps/chakrashim/include/v8-debug.h
@@ -61,6 +61,10 @@ class EXPORT Debug {
                           const uint16_t* command, int length,
                           ClientData* client_data = NULL) {
   }
+  static MaybeLocal<Value> GetMirror(Local<Context> context,
+                                     Handle<Value> obj) {
+    return MaybeLocal<Value>();
+  }
 };
 
 }  // namespace v8