LGTM other than removing the Safari2 support.
http://gwt-code-reviews.appspot.com/48806/diff/1/3 File user/test/com/google/gwt/user/client/rpc/UnicodeEscapingTest.java (left): http://gwt-code-reviews.appspot.com/48806/diff/1/3#oldcode66 Line 66: private static native boolean isSafari2() /*-{ Don't we still need this? Until we official drop support for Safari2 (and our build doesn't test it), I think we need to keep the test passing there. http://gwt-code-reviews.appspot.com/48806 --~--~---------~--~----~------------~-------~--~----~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~----------~----~----~----~------~----~------~--~---
