Update new JSRT with Experimental postfixes

To indicate that these APIs are still in progress, add the postfixes to indicate changes are possible.
Also updates comments for API declearations.

Author: Thomas Moore (CHAKRA)

bin/ChakraCore/ChakraCore.def

@@ -71,6 +71,6 @@ JsRunScriptWithParserState
 JsGetPromiseState
 JsGetPromiseResult
 
-JsQueueBackgroundParse
-JsDiscardBackgroundParse
-JsExecuteBackgroundParse
+JsQueueBackgroundParse_Experimental
+JsDiscardBackgroundParse_Experimental
+JsExecuteBackgroundParse_Experimental

bin/ch/ChakraRtInterface.cpp

@@ -160,11 +160,9 @@ bool ChakraRTInterface::LoadChakraDll(ArgInfo* argInfo, HINSTANCE *outLibrary)
     m_jsApiHooks.pfJsrtSerializeParserState = (JsAPIHooks::JsrtSerializeParserState)GetChakraCoreSymbol(library, "JsSerializeParserState");
     m_jsApiHooks.pfJsrtRunScriptWithParserState = (JsAPIHooks::JsrtRunScriptWithParserState)GetChakraCoreSymbol(library, "JsRunScriptWithParserState");
 
-    m_jsApiHooks.pfJsrtQueueBackgroundParse = (JsAPIHooks::JsrtQueueBackgroundParse)GetChakraCoreSymbol(library, "JsQueueBackgroundParse");
-    m_jsApiHooks.pfJsrtDiscardBackgroundParse = (JsAPIHooks::JsrtDiscardBackgroundParse)GetChakraCoreSymbol(library, "JsDiscardBackgroundParse");
-    m_jsApiHooks.pfJsrtExecuteBackgroundParse = (JsAPIHooks::JsrtExecuteBackgroundParse)GetChakraCoreSymbol(library, "JsExecuteBackgroundParse");
-
-
+    m_jsApiHooks.pfJsrtQueueBackgroundParse = (JsAPIHooks::JsrtQueueBackgroundParse)GetChakraCoreSymbol(library, "JsQueueBackgroundParse_Experimental");
+    m_jsApiHooks.pfJsrtDiscardBackgroundParse = (JsAPIHooks::JsrtDiscardBackgroundParse)GetChakraCoreSymbol(library, "JsDiscardBackgroundParse_Experimental");
+    m_jsApiHooks.pfJsrtExecuteBackgroundParse = (JsAPIHooks::JsrtExecuteBackgroundParse)GetChakraCoreSymbol(library, "JsExecuteBackgroundParse_Experimental");
 
     m_jsApiHooks.pfJsrtTTDCreateRecordRuntime = (JsAPIHooks::JsrtTTDCreateRecordRuntimePtr)GetChakraCoreSymbol(library, "JsTTDCreateRecordRuntime");
     m_jsApiHooks.pfJsrtTTDCreateReplayRuntime = (JsAPIHooks::JsrtTTDCreateReplayRuntimePtr)GetChakraCoreSymbol(library, "JsTTDCreateReplayRuntime");

lib/Jsrt/ChakraCommon.h

@@ -2434,22 +2434,60 @@ typedef unsigned short uint16_t;
             _In_opt_ JsPromiseContinuationCallback promiseContinuationCallback,
             _In_opt_ void *callbackState);
 
+    /// <summary>
+    ///     Note: Experimental API
+    ///     Starts a request for background script parsing on another thread
+    /// </summary>
+    /// <param name="contents">ScriptContents struct with data needed to start parsing</param>
+    /// <param name="dwBgParseCookie">Identifier for subsequent BGParse operations</param>
+    /// <returns>
+    ///     The code <c>JsNoError</c> if the operation succeeded, a failure code otherwise.
+    /// </returns>
     CHAKRA_API
-        JsQueueBackgroundParse(JsScriptContents* contents, DWORD* dwBgParseCookie);
+        JsQueueBackgroundParse_Experimental(
+            _In_ JsScriptContents* contents,
+            _Out_ DWORD* dwBgParseCookie);
 
+    /// <summary>
+    ///     Note: Experimental API
+    ///     Appropriately frees resources associated with a previously queued background parse
+    /// </summary>
+    /// <param name="dwBgParseCookie">Identifier for BGParse operation</param>
+    /// <param name="buffer">Pointer to script source buffer, used for validation</param>
+    /// <param name="callerOwnsBuffer">When <c>true</c>, caller is responsible for freeing buffer</param>
+    /// <returns>
+    ///     The code <c>JsNoError</c> if the operation succeeded, a failure code otherwise.
+    /// </returns>
     CHAKRA_API
-        JsDiscardBackgroundParse(DWORD dwBgParseCookie, void* buffer, bool* callerOwnsBuffer);
+        JsDiscardBackgroundParse_Experimental(
+            _In_ DWORD dwBgParseCookie,
+            _In_ void* buffer,
+            _Out_ bool* callerOwnsBuffer);
 
+    /// <summary>
+    ///     Note: Experimental API
+    ///     Executes the background parsed script
+    /// </summary>
+    /// <param name="dwBgParseCookie">Identifier for subsequent BGParse operations</param>
+    /// <param name="script">Pointer to script source</param>
+    /// <param name="sourceContext">JsSourceContext identifier</param>
+    /// <param name="url">Path to the parsed script</param>
+    /// <param name="parseAttributes"></param>
+    /// <param name="parserState">[May not be needed]</param>
+    /// <param name="result">Result of script execution</param>
+    /// <returns>
+    ///     The code <c>JsNoError</c> if the operation succeeded, a failure code otherwise.
+    /// </returns>
     CHAKRA_API
-        JsExecuteBackgroundParse(_In_ DWORD dwBgParseCookie,
+        JsExecuteBackgroundParse_Experimental(
+            _In_ DWORD dwBgParseCookie,
             _In_ JsValueRef script,
             _In_ JsSourceContext sourceContext,
             _In_ WCHAR *url,
             _In_ JsParseScriptAttributes parseAttributes,
             _In_ JsValueRef parserState,
             _Out_ JsValueRef *result);
 
-
 #ifdef _WIN32
 #include "ChakraCommonWindows.h"
 #endif // _WIN32

lib/Jsrt/Jsrt.cpp

@@ -5839,7 +5839,7 @@ CHAKRA_API JsRunScriptWithParserState(
 }
 
 CHAKRA_API
-JsQueueBackgroundParse(JsScriptContents* contents, DWORD* dwBgParseCookie)
+JsQueueBackgroundParse_Experimental(JsScriptContents* contents, DWORD* dwBgParseCookie)
 {
     HRESULT hr;
     if (Js::Configuration::Global.flags.BgParse && !CONFIG_FLAG(ForceDiagnosticsMode)
@@ -5862,14 +5862,14 @@ JsQueueBackgroundParse(JsScriptContents* contents, DWORD* dwBgParseCookie)
 }
 
 CHAKRA_API
-JsDiscardBackgroundParse(DWORD dwBgParseCookie, void* buffer, bool* callerOwnsBuffer)
+JsDiscardBackgroundParse_Experimental(DWORD dwBgParseCookie, void* buffer, bool* callerOwnsBuffer)
 {
     (*callerOwnsBuffer) = BGParseManager::GetBGParseManager()->DiscardParseResults(dwBgParseCookie, buffer);
     return JsNoError;
 }
 
 CHAKRA_API
-JsExecuteBackgroundParse(
+JsExecuteBackgroundParse_Experimental(
     _In_ DWORD dwBgParseCookie,
     _In_ JsValueRef script,
     _In_ JsSourceContext sourceContext,