[release/2.1] Do not escape cookie names in lookups

- **Do Not Merge**
- always fail `RequestCookieCollection.ContainsKey(...)` and `RequestCookieCollection.TryGetValue(...)`
- undo an important part of #24264 and demonstrate #29023 problem
  - shows many tests working despite break of an indirect reference

Author: dougbu

eng/PatchConfig.props

@@ -85,6 +85,7 @@ Later on, this will be checked using this condition:
   </PropertyGroup>
   <PropertyGroup Condition=" '$(VersionPrefix)' == '2.1.25' ">
     <PackagesInPatch>
+      Microsoft.AspNetCore.Http;
       Microsoft.AspNetCore.Server.Kestrel.Core;
     </PackagesInPatch>
   </PropertyGroup>

src/Http/Http/src/Internal/RequestCookieCollection.cs

@@ -88,6 +88,7 @@ internal static RequestCookieCollection ParseInternal(IList<string> values, bool
                     {
                         name = Uri.UnescapeDataString(name);
                     }
+
                     var value = Uri.UnescapeDataString(cookie.Value.Value);
                     store[name] = value;
                 }
@@ -127,8 +128,8 @@ public bool ContainsKey(string key)
             {
                 return false;
             }
-            return Store.ContainsKey(key)
-                || !_enableCookieNameDecoding && Store.ContainsKey(Uri.EscapeDataString(key));
+
+            return false;
         }
 
         public bool TryGetValue(string key, out string value)
@@ -139,8 +140,8 @@ public bool TryGetValue(string key, out string value)
                 return false;
             }
 
-            return Store.TryGetValue(key, out value)
-                || !_enableCookieNameDecoding && Store.TryGetValue(Uri.EscapeDataString(key), out value);
+            value = null;
+            return false;
         }
 
         /// <summary>