Avoid boxing context.Method

[benaadams]

object.ReferenceEquals(context.Method, HttpMethods.Get) boxes context.Method as its an enum and will always return false.

As context.Method == HttpMethod.Post || context.Method == HttpMethod.Put are two enum compares rather than a string compare we don't really need to change the if to a context.Method != HttpMethod.Get and can just get rid of it.

[davidfowl]

Just to show what this looks like before this change:

HttpMethod shows up as a per request allocation.

[pakrym]

I think you can halve byte[] allocations by inverting https://github.com/aspnet/AspNetCore/blob/68cc5e7846240bacc146854dfc6a175e30ff0f60/src/Servers/Kestrel/Core/src/Internal/Http/ChunkWriter.cs#L22 and passing memory in. It's written to BufferWriter anyway: https://github.com/aspnet/AspNetCore/blob/68cc5e7846240bacc146854dfc6a175e30ff0f60/src/Servers/Kestrel/Core/src/Internal/Http/ChunkWriter.cs#L54

[davidfowl]

We'll do that next 😄

[pakrym]

Test failures are unrelated. Merging.