From 29a25b2ed14b8c81f8ae9b356e171afd77f99f44 Mon Sep 17 00:00:00 2001
From: Doug Bunting <dougbu@microsoft.com>
Date: Sun, 13 Sep 2015 19:40:27 -0700
Subject: [PATCH] No need to add an instance in
 `WebApiCompatShimMvcBuilderExtensions` - DI has changed since this class was
 written

---
 .../WebApiCompatShimMvcBuilderExtensions.cs                   | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/Microsoft.AspNet.Mvc.WebApiCompatShim/WebApiCompatShimMvcBuilderExtensions.cs b/src/Microsoft.AspNet.Mvc.WebApiCompatShim/WebApiCompatShimMvcBuilderExtensions.cs
index c36969fdca..0c066241b9 100644
--- a/src/Microsoft.AspNet.Mvc.WebApiCompatShim/WebApiCompatShimMvcBuilderExtensions.cs
+++ b/src/Microsoft.AspNet.Mvc.WebApiCompatShim/WebApiCompatShimMvcBuilderExtensions.cs
@@ -18,9 +18,7 @@ namespace Microsoft.Framework.DependencyInjection
             builder.Services.TryAddEnumerable(
                 ServiceDescriptor.Transient<IConfigureOptions<WebApiCompatShimOptions>, WebApiCompatShimOptionsSetup>());
 
-            // The constructors on DefaultContentNegotiator aren't DI friendly, so just
-            // new it up.
-            builder.Services.TryAdd(ServiceDescriptor.Instance<IContentNegotiator>(new DefaultContentNegotiator()));
+            builder.Services.TryAddSingleton<IContentNegotiator, DefaultContentNegotiator>();
 
             return builder;
         }