be liberal when encountering "+" in URI components

[tschaub]

 IETF 2396 says spaces should be encoded as %20. However, in practice, + is often used. Properly encoded, the + should be %2B in a URI component (value in a query string pair). When we encounter a + before decoding, we could safely replace it with a space.

[tschaub]

replace + with space before decoding

[tschaub]

Tests pass in FF3 and IE6. Thanks for any review.

[bartvde]

Looks good to me Tim, please commit.

[bartvde]

Tim can you please check the fix for ticket:1704 and see if it does not conflict with your thinking on this. It's the exact same line as your patch. TIA.

[tschaub]

I want to take a closer look at #1704. Will try to do that in the next 8 hours or so.

[tschaub]

(In [10133]) Being liberal in what we accept in query strings. A properly encoded '+' should be '%2B' in a URI component. A properly encoded ' ' (space) should be '%20'. When we encounter a '+', we can safely replace it with ' ' before properly decoding the remainder of the component. This follows the convention that '+' is used widely to represent a space in a URL. r=bartvde (closes #2527)