Reviewers: Ray Ryan, Description: As discussed in issue 3374, this patch deprecates encodeComponent and decodeComponent (including the ones added yesterday) and replaces them with encodePathSegment/encodeQueryString and decodePathSegment/decodeQueryString.
If you compare it to pre-r6470, it replaces encodeComponent(String) and decodeComponent(String) with encodeQueryString(String) and decodeQueryString(String) respectively, and adds encodePathSegment and decodePathSegment. If it were me, I'd rather use this patch than the previous one that landed in r6470. Whatever you decide to do, have a look at javadoc enhancements I added here for encodeComponent/decodeComponent to make it clear that they should only be used for query strings because of how it handles spaces/pluses. Please review this at http://gwt-code-reviews.appspot.com/88802 Affected files: user/javadoc/com/google/gwt/examples/http/client/QueryAndFormDataExample.java user/src/com/google/gwt/http/client/RequestBuilder.java user/src/com/google/gwt/http/client/URL.java user/src/com/google/gwt/user/client/Window.java user/test/com/google/gwt/http/client/URLTest.java --~--~---------~--~----~------------~-------~--~----~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~----------~----~----~----~------~----~------~--~---
