Create new JSRT APIs for BGParse

Thomas Moore · Thomas Moore (CHAKRA) · commit f113fa776784 · 2018-11-15T11:14:42.000-08:00
Adding ChakraCore implmementations to interact with BGParseManager:
- JsQueueBackgroundParse
- JsDiscardBackgroundParse
- JsExecuteBackgroundParse

The last API will be implemented in a subsequent change
diff --git a/bin/ch/ch.cpp b/bin/ch/ch.cpp
@@ -912,6 +912,10 @@ HRESULT ExecuteTest(const char* fileName)
         {
             CreateParserStateAndRunScript(fileName, fileContents, lengthBytes, WScriptJsrt::FinalizeFree, fullPath);
         }
+        else if (HostConfigFlags::flags.ExecuteWithBgParse)
+        {
+            // Run multiple scripts with BGParse
+        }
         else
         {
             IfFailGo(RunScript(fileName, fileContents, lengthBytes, WScriptJsrt::FinalizeFree, nullptr, fullPath, nullptr));
diff --git a/lib/Jsrt/ChakraCommon.h b/lib/Jsrt/ChakraCommon.h
@@ -625,6 +625,27 @@ typedef unsigned short uint16_t;
         JsDataView = 12,
     } JsValueType;
 
+    typedef enum _JsScriptEncodingType
+    {
+        Utf8,
+        Utf16
+    } JsScriptEncodingType;
+
+    typedef enum _JsScriptContainerType
+    {
+        HeapAllocatedBuffer
+    } JsScriptContainerType;
+
+    typedef struct _JsScriptContents
+    {
+        LPVOID container;
+        JsScriptEncodingType encodingType;
+        JsScriptContainerType containerType;
+        DWORD_PTR sourceContext;
+        size_t contentLengthInBytes;
+        LPCWSTR fullPath;
+    } JsScriptContents;
+
     /// <summary>
     ///     User implemented callback routine for memory allocation events
     /// </summary>
@@ -2413,6 +2434,16 @@ typedef unsigned short uint16_t;
             _In_opt_ JsPromiseContinuationCallback promiseContinuationCallback,
             _In_opt_ void *callbackState);
 
+    CHAKRA_API
+        JsQueueBackgroundParse(JsScriptContents* contents, DWORD* dwBgParseCookie);
+
+    CHAKRA_API
+        JsDiscardBackgroundParse(DWORD dwBgParseCookie, void* buffer, bool* callerOwnsBuffer);
+
+    CHAKRA_API
+        JsExecuteBackgroundParse(DWORD dwBgParseCookie, JsContextRef context, DWORD_PTR dwSourceContext, DWORD dwFlags, VARIANT* pvarResult, EXCEPINFO* pexcepinfo);
+
+
 #ifdef _WIN32
 #include "ChakraCommonWindows.h"
 #endif // _WIN32
diff --git a/lib/Jsrt/Jsrt.cpp b/lib/Jsrt/Jsrt.cpp
@@ -4749,6 +4749,50 @@ CHAKRA_API JsTTDDiagSetAutoTraceStatus(_In_ bool status)
 #endif
 }
 
+CHAKRA_API
+JsQueueBackgroundParse(JsScriptContents* contents, DWORD* dwBgParseCookie)
+{
+    HRESULT hr;
+    if (Js::Configuration::Global.flags.BgParse && !CONFIG_FLAG(ForceDiagnosticsMode)
+        // For now, only UTF8 buffers are supported for BGParse
+        && contents->encodingType == JsScriptEncodingType::Utf8
+        && contents->containerType == JsScriptContainerType::HeapAllocatedBuffer
+        // SourceContext not needed for BGParse
+        && contents->sourceContext == 0)
+    {
+        hr = BGParseManager::GetBGParseManager()->QueueBackgroundParse((LPUTF8)contents->container, contents->contentLengthInBytes, (char16*)contents->fullPath, dwBgParseCookie);
+    }
+    else
+    {
+        hr = E_NOTIMPL;
+    }
+
+    JsErrorCode res;
+    if (hr != S_OK)
+    {
+        res = JsErrorFatal;
+    }
+    else
+    {
+        res = JsNoError;
+    }
+
+    return res;
+}
+
+CHAKRA_API
+JsDiscardBackgroundParse(DWORD dwBgParseCookie, void* buffer, bool* callerOwnsBuffer)
+{
+    (*callerOwnsBuffer) = BGParseManager::GetBGParseManager()->DiscardParseResults(dwBgParseCookie, buffer);
+    return JsNoError;
+}
+
+CHAKRA_API
+JsExecuteBackgroundParse(DWORD dwBgParseCookie, JsContextRef context, DWORD_PTR dwSourceContext, DWORD dwFlags, VARIANT* pvarResult, EXCEPINFO* pexcepinfo)
+{
+    return JsNoError;
+}
+
 #ifdef _CHAKRACOREBUILD
 
 template <class SrcChar, class DstChar>

Original file line number	Diff line number	Diff line change
`@@ -912,6 +912,10 @@ HRESULT ExecuteTest(const char* fileName)`
`912`	`912`	`{`
`913`	`913`	`CreateParserStateAndRunScript(fileName, fileContents, lengthBytes, WScriptJsrt::FinalizeFree, fullPath);`
`914`	`914`	`}`
	`915`	`+ else if (HostConfigFlags::flags.ExecuteWithBgParse)`
	`916`	`+ {`
	`917`	`+ // Run multiple scripts with BGParse`
	`918`	`+ }`
`915`	`919`	`else`
`916`	`920`	`{`
`917`	`921`	`IfFailGo(RunScript(fileName, fileContents, lengthBytes, WScriptJsrt::FinalizeFree, nullptr, fullPath, nullptr));`