removed the explicit check for FullTrust as it was unnecessary.

Author: craiggwilson

Bson/ObjectModel/ObjectId.cs

@@ -55,8 +55,6 @@ static ObjectId()
 
             try
             {
-                // we are testing for FullTrust here and, if we have it, then we use the current process' id.
-                new PermissionSet(PermissionState.Unrestricted).Demand();
                 SetPidToCurrentProcessId();
             }
             catch (SecurityException)
@@ -377,6 +375,11 @@ private static int GetTimestampFromDateTime(DateTime timestamp)
             return (int)Math.Floor((BsonUtils.ToUniversalTime(timestamp) - BsonConstants.UnixEpoch).TotalSeconds);
         }
 
+        /// <summary>
+        /// Sets the pid to current process id.  This method exists because of how CAS operates on the call stack, checking
+        /// for permissions before executing the method.  Hence, if we inlined this call, the calling method would not execute
+        /// before throwing an exception requiring the try/catch at an even higher level that we don't necessarily control.
+        /// </summary>
         [MethodImpl(MethodImplOptions.NoInlining)]
         private static void SetPidToCurrentProcessId()
         {