Remove hosting methods that always return false (dotnet/coreclr#9930)
authorAustin Wise <AustinWise@gmail.com>
Mon, 6 Mar 2017 01:51:40 +0000 (17:51 -0800)
committerJan Kotas <jkotas@microsoft.com>
Mon, 6 Mar 2017 01:51:40 +0000 (17:51 -0800)
commit1455fca6c15709f474f042ab22e1cde294077da0
treeaef24be1121a9d1fcedfeadfdebab5418e0221d5
parent81a89007536686ad4f95a43641cbe8482e4b74ac
Remove hosting methods that always return false (dotnet/coreclr#9930)

* Remove CLRIoCompletionHosted as it always returns false.
* Remove CLRSyncHosted as it always returns false.
* Remove CLRMemoryHosted as it always returns false.
* Remove CLRTaskHosted as it always returns false.
* Remove CLRAssemblyHosted, CLRGCHosted,and CLRSecurityHosted.
They are not called.
* Remove IsThreadPoolHosted as it always returns false.
* Remove EnterRuntime and LeaveRuntime, as they do nothing.
* Add back calls to RevertIfImpersonated and GCX_PREEMP.

I accidentally deleted the call to RevertIfImpersonated instead of just
removing an extra parameter.

When I removed the HR_LEAVE_RUNTIME_HOLDER macro from windowsruntime.h,
I not only removed a LeaveRuntimeHolder but also a GCX_PREEMP. So I added
it back. The holder and GCX_PREEMP where only inserted when the
FEATURE_LEAVE_RUNTIME_HOLDER macro was defined. Since it is always defined,
I removed it. Also as I understand it, you would always want to have a
GCX_PREEMP before calling into the Windows API as not to block the GC,
so I'm not sure why you would want to disable it.

Commit migrated from https://github.com/dotnet/coreclr/commit/51e968b013e9b1582035f202e004ed024f747f4f
51 files changed:
src/coreclr/src/debug/daccess/dacdbiimpl.cpp
src/coreclr/src/debug/ee/controller.h
src/coreclr/src/debug/ee/debugger.cpp
src/coreclr/src/debug/ee/debugger.h
src/coreclr/src/debug/inc/dbgappdomain.h
src/coreclr/src/gc/gc.cpp
src/coreclr/src/inc/winrt/windowsruntime.h
src/coreclr/src/mscorlib/src/System/Threading/ThreadPool.cs
src/coreclr/src/vm/CMakeLists.txt
src/coreclr/src/vm/amd64/UMThunkStub.asm
src/coreclr/src/vm/appdomain.cpp
src/coreclr/src/vm/appxutil.cpp
src/coreclr/src/vm/ceemain.cpp
src/coreclr/src/vm/ceemain.h
src/coreclr/src/vm/clrex.cpp
src/coreclr/src/vm/clsload.cpp
src/coreclr/src/vm/comcache.cpp
src/coreclr/src/vm/comcallablewrapper.h
src/coreclr/src/vm/common.h
src/coreclr/src/vm/comthreadpool.cpp
src/coreclr/src/vm/comthreadpool.h
src/coreclr/src/vm/comtoclrcall.cpp
src/coreclr/src/vm/corhost.cpp
src/coreclr/src/vm/crst.cpp
src/coreclr/src/vm/dllimportcallback.cpp
src/coreclr/src/vm/dwreport.cpp
src/coreclr/src/vm/ecalllist.h
src/coreclr/src/vm/eepolicy.cpp
src/coreclr/src/vm/exceptionhandling.cpp
src/coreclr/src/vm/finalizerthread.cpp
src/coreclr/src/vm/hosting.cpp
src/coreclr/src/vm/interoputil.cpp
src/coreclr/src/vm/jitinterface.cpp
src/coreclr/src/vm/nativeoverlapped.cpp
src/coreclr/src/vm/profattach.cpp
src/coreclr/src/vm/runtimecallablewrapper.cpp
src/coreclr/src/vm/spinlock.cpp
src/coreclr/src/vm/stackprobe.h
src/coreclr/src/vm/stdinterfaces.cpp
src/coreclr/src/vm/stubhelpers.cpp
src/coreclr/src/vm/synch.cpp
src/coreclr/src/vm/synch.h
src/coreclr/src/vm/testhookmgr.cpp
src/coreclr/src/vm/threads.cpp
src/coreclr/src/vm/threads.h
src/coreclr/src/vm/threadsuspend.cpp
src/coreclr/src/vm/threadsuspend.h
src/coreclr/src/vm/util.cpp
src/coreclr/src/vm/util.hpp
src/coreclr/src/vm/win32threadpool.cpp
src/coreclr/src/vm/win32threadpool.h