fix($location): handle plus character in query strings

[caguillen214]

Closes #3042

[mary-poppins]

Cool 🐟

[btford]

This LGTM. @IgorMinar – do you think we should also add tests for parseKeyValue?

[mary-poppins]

I'm sorry, but I wasn't able to verify your Contributor License Agreement (CLA) signature. CLA signature is required for any code contributions to AngularJS.

Please sign our CLA and ensure that the CLA signature email address and the email address in this PR's commits match.

If you signed the CLA as a corporation, please let us know the company's name.

Thanks a bunch!

PS: If you signed the CLA in the past then most likely the email addresses don't match. Please sign the CLA again or update the email address in the commit of this PR.
PS2: If you are a Googler, please sign the CLA as well to simplify the CLA verification process.

[IgorMinar]

LGTM

we should do a follow up PR for $location.path which should have similar behavior if it doesn't already.

@caguillen214 please sign CLA

[caguillen214]

Signed the CLA @IgorMinar

[btford]

Landed as 3f4ee15.

[jordanms]

I faced this problem on 1.3.15 and reading this I upgraded to 1.3.16. The result is that now I have no idea what the URL will contain! Sometimes + is encoded as %20 and sometimes as %2B and this is really random. I also upgraded to 1.4.3 and problem was there, as in 1.3.15. For my case this fix made it even worse.