Code review cleanup.

src/Microsoft.AspNet.AppBuilderSupport/OwinHttpEnvironment.cs

@@ -169,7 +169,7 @@ namespace Microsoft.AspNet.PipelineCore.Owin
             object obj;
             if (Environment.TryGetValue(OwinConstants.SendFiles.SendAsync, out obj))
             {
-                SendFileFunc func = (SendFileFunc)obj;
+                var func = (SendFileFunc)obj;
                 return func(path, offset, length, cancellation);
             }
             throw new NotSupportedException(OwinConstants.SendFiles.SendAsync);
@@ -237,7 +237,7 @@ namespace Microsoft.AspNet.PipelineCore.Owin
         {
             get
             {
-                IList<Type> keys = new List<Type>()
+                var keys = new List<Type>()
                 {
                     typeof(IHttpRequestInformation),
                     typeof(IHttpResponseInformation),
@@ -306,12 +306,30 @@ namespace Microsoft.AspNet.PipelineCore.Owin
 
         public bool Contains(KeyValuePair<Type, object> item)
         {
-            throw new NotImplementedException();
+            object result;
+            return TryGetValue(item.Key, out result) && result.Equals(item.Value);
         }
 
         public void CopyTo(KeyValuePair<Type, object>[] array, int arrayIndex)
         {
-            throw new NotImplementedException();
+            if (array == null)
+            {
+                throw new ArgumentNullException("array");
+            }
+            if (arrayIndex < 0 || arrayIndex > array.Length)
+            {
+                throw new ArgumentOutOfRangeException("arrayIndex", arrayIndex, string.Empty);
+            }
+            var keys = Keys;
+            if (keys.Count > array.Length - arrayIndex)
+            {
+                throw new ArgumentException();
+            }
+
+            foreach (var key in keys)
+            {
+                array[arrayIndex++] = new KeyValuePair<Type, object>(key, this[key]);
+            }
         }
 
         public int Count

src/Microsoft.AspNet.PipelineCore/DefaultHttpRequest.cs

@@ -132,8 +132,8 @@ namespace Microsoft.AspNet.PipelineCore
         {
             get
             {
+                // TODO: Which feature exposes this?
                 return CancellationToken.None;
-                // throw new NotImplementedException();
             }
             set
             {

src/Microsoft.AspNet.PipelineCore/DefaultHttpResponse.cs

@@ -51,11 +51,9 @@ namespace Microsoft.AspNet.PipelineCore
         {
             get
             {
-                string[] values;
                 long value;
-                if (HttpResponseInformation.Headers.TryGetValue(Constants.Headers.ContentLength, out values)
+                string rawValue = Headers.Get(Constants.Headers.ContentLength);
-                    && values != null && values.Length > 0 && !string.IsNullOrWhiteSpace(values[0])
+                if (!string.IsNullOrWhiteSpace(rawValue) && long.TryParse(rawValue, out value))
-                    && long.TryParse(values[0], out value))
                 {
                     return value;
                 }