Fixed #22799 Made GET and POST attributes of an HttpRequest object QueryDicts.

Previously, `GET` and `POST` on an `HttpRequest` were created in the `__init__`
method as dictionaries. This was not something you would usually notice
causing trouble in production as you'd only see a `WSGIRequest`, but in
testing using the test client, calling `.getlist` on `GET` or `POST`
for a request with no get/post data used to result in an `AttributeError`.

Changed `GET` and `POST` on a vanilla `HttpRequest` object to be mutable
`QueryDict`s (mutable because the Django tests, and probably many
third party tests were expecting it).

Also added a regression test for this by adding calls to `.getlist` to
the test on a newly created `HttpRequest`.
Why the `.copy()`?

By the way, it seems that the way `QueryDict.__init__()` is implemented, you can use `QueryDict(None)` to get an empty `QueryDict`. Come to think of it, I don't see a reason why `QueryDict` has any required argument. We could make `query_string=None` the default and it should work the same.