[MERGE chakra-core#3634 @xiaoyinl] Remove undefined functions in ScriptContext.h

MikeHolman · MikeHolman · commit 9753783bc915 · 2017-09-22T17:41:19.000-07:00
Merge pull request chakra-core#3634 from xiaoyinl:function_cleanup These member functions are declared in ScriptContext.h, but are not defined in ScriptContext.cpp: [LeaveScriptStartCore](https://github.com/Microsoft/ChakraCore/search?q=LeaveScriptStartCore), [CleanEvalMapCache](https://github.com/Microsoft/ChakraCore/search?q=CleanEvalMapCache), [AddFunctionBodyToPropIdMap](https://github.com/Microsoft/ChakraCore/search?q=AddFunctionBodyToPropIdMap). I wonder if we can remove them, since they are not used.
diff --git a/lib/Runtime/Base/ScriptContext.h b/lib/Runtime/Base/ScriptContext.h
@@ -914,7 +914,6 @@ namespace Js
         char16 const * url;
 
         void PrintStats();
-        BOOL LeaveScriptStartCore(void * frameAddress, bool leaveForHost);
 
         void InternalClose();
 
@@ -1065,7 +1064,6 @@ namespace Js
 
         template <typename TCacheType>
         void CleanDynamicFunctionCache(TCacheType* cacheType);
-        void CleanEvalMapCache(Js::EvalCacheTopLevelDictionary * cacheType);
 
         template <class TDelegate>
         void MapFunction(TDelegate mapper);
@@ -1201,7 +1199,6 @@ namespace Js
         void SetFastDOMenabled();
         BOOL VerifyAlive(BOOL isJSFunction = FALSE, ScriptContext* requestScriptContext = nullptr);
         void VerifyAliveWithHostContext(BOOL isJSFunction, HostScriptContext* requestHostScriptContext);
-        void AddFunctionBodyToPropIdMap(FunctionBody* body);
 
         void BindReference(void* addr);
 
