Hosting#359 Remove IDisposable from HttpContext and IFeatureCollection.

Author: Tratcher

src/Microsoft.AspNet.Http.Abstractions/HttpContext.cs

@@ -10,7 +10,7 @@
 
 namespace Microsoft.AspNet.Http
 {
-    public abstract class HttpContext : IDisposable
+    public abstract class HttpContext
     {
         public abstract IFeatureCollection Features { get; }
 
@@ -37,7 +37,5 @@ public abstract class HttpContext : IDisposable
         public abstract ISession Session { get; set; }
 
         public abstract void Abort();
-
-        public abstract void Dispose();
     }
 }

src/Microsoft.AspNet.Http.Features/FeatureCollection.cs

@@ -93,11 +93,6 @@ public IEnumerator<KeyValuePair<Type, object>> GetEnumerator()
             }
         }
 
-        public virtual void Dispose()
-        {
-            _defaults?.Dispose();
-        }
-
         private class KeyComparer : IEqualityComparer<KeyValuePair<Type, object>>
         {
             public bool Equals(KeyValuePair<Type, object> x, KeyValuePair<Type, object> y)

src/Microsoft.AspNet.Http.Features/IFeatureCollection.cs

@@ -6,7 +6,7 @@
 
 namespace Microsoft.AspNet.Http.Features
 {
-    public interface IFeatureCollection : IEnumerable<KeyValuePair<Type, object>>, IDisposable
+    public interface IFeatureCollection : IEnumerable<KeyValuePair<Type, object>>
     {
         /// <summary>
         /// Indicates if the collection can be modified.

src/Microsoft.AspNet.Http/DefaultHttpContext.cs

@@ -165,11 +165,5 @@ public override void Abort()
         {
             LifetimeFeature.Abort();
         }
-
-        public override void Dispose()
-        {
-            // REVIEW: is this necessary? is the environment "owned" by the context?
-            _features.Dispose();
-        }
     }
 }