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

MikeHolman · MikeHolman · commit e6c88eaceb4d · 2017-09-22T18:05:23.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
@@ -924,7 +924,6 @@ namespace Js
         char16 const * url;
 
         void PrintStats();
-        BOOL LeaveScriptStartCore(void * frameAddress, bool leaveForHost);
 
         void InternalClose();
 
@@ -1061,7 +1060,6 @@ namespace Js
 
         template <typename TCacheType>
         void CleanDynamicFunctionCache(TCacheType* cacheType);
-        void CleanEvalMapCache(Js::EvalCacheTopLevelDictionary * cacheType);
 
         template <class TDelegate>
         void MapFunction(TDelegate mapper);
@@ -1197,7 +1195,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);
 
