From 591cbc37e31737522fef08e1b0647d476798ea55 Mon Sep 17 00:00:00 2001
From: Darin Dimitrov <darin.dimitrov@gmail.com>
Date: Tue, 18 Dec 2018 15:39:57 +0200
Subject: [PATCH] Use the new ScriptCompiler::CreateCodeCache API to produce
 script CachedData for subsequent runs #1235

---
 test-app/runtime/src/main/cpp/ModuleInternal.cpp | 14 +++++++-------
 test-app/runtime/src/main/cpp/ModuleInternal.h   |  2 +-
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/test-app/runtime/src/main/cpp/ModuleInternal.cpp b/test-app/runtime/src/main/cpp/ModuleInternal.cpp
index 15fe292d7..02e989adb 100644
--- a/test-app/runtime/src/main/cpp/ModuleInternal.cpp
+++ b/test-app/runtime/src/main/cpp/ModuleInternal.cpp
@@ -379,15 +379,12 @@ Local<Script> ModuleInternal::LoadScript(Isolate* isolate, const string& path, c
         script = maybeScript.ToLocalChecked();
     } else {
         tns::instrumentation::Frame frame("Compile, no cache");
-        if (Constants::V8_CACHE_COMPILED_CODE) {
-            option = ScriptCompiler::kProduceCodeCache;
-        }
         auto maybeScript = ScriptCompiler::Compile(isolate->GetCurrentContext(), &source, option);
         if (maybeScript.IsEmpty() || tc.HasCaught()) {
             throw NativeScriptException(tc, "Cannot compile " + path);
         }
         script = maybeScript.ToLocalChecked();
-        SaveScriptCache(source, path);
+        SaveScriptCache(script, path);
     }
 
     DEBUG_WRITE("Compiled script (module %s)", path.c_str());
@@ -459,16 +456,19 @@ ScriptCompiler::CachedData* ModuleInternal::TryLoadScriptCache(const std::string
     return new ScriptCompiler::CachedData(reinterpret_cast<uint8_t*>(data), length, ScriptCompiler::CachedData::BufferOwned);
 }
 
-void ModuleInternal::SaveScriptCache(const ScriptCompiler::Source& source, const std::string& path) {
+void ModuleInternal::SaveScriptCache(const Local<Script> script, const std::string& path) {
     if (!Constants::V8_CACHE_COMPILED_CODE) {
         return;
     }
 
     tns::instrumentation::Frame frame("SaveScriptCache");
 
-    int length = source.GetCachedData()->length;
+    Local<UnboundScript> unboundScript = script->GetUnboundScript();
+    ScriptCompiler::CachedData* cachedData = ScriptCompiler::CreateCodeCache(unboundScript);
+
+    int length = cachedData->length;
     auto cachePath = path + ".cache";
-    File::WriteBinary(cachePath, source.GetCachedData()->data, length);
+    File::WriteBinary(cachePath, cachedData->data, length);
 }
 
 ModuleInternal::ModulePathKind ModuleInternal::GetModulePathKind(const std::string& path) {
diff --git a/test-app/runtime/src/main/cpp/ModuleInternal.h b/test-app/runtime/src/main/cpp/ModuleInternal.h
index c0731d157..7d7b8d06c 100644
--- a/test-app/runtime/src/main/cpp/ModuleInternal.h
+++ b/test-app/runtime/src/main/cpp/ModuleInternal.h
@@ -61,7 +61,7 @@ class ModuleInternal {
 
         v8::ScriptCompiler::CachedData* TryLoadScriptCache(const std::string& path);
 
-        void SaveScriptCache(const v8::ScriptCompiler::Source& source, const std::string& path);
+        void SaveScriptCache(const v8::Local<v8::Script> script, const std::string& path);
 
         ModulePathKind GetModulePathKind(const std::string& path);