Author: [email protected]
Date: Thu Apr 16 15:26:10 2009
New Revision: 5254

Modified:
    trunk/user/src/com/google/gwt/user/client/Cookies.java

Log:
Don't let an exception escape if we can't decode cookies.  Future work to
support non-encoding of cookies is  captured in issue 3566.

Issue: 1633
Patch by: jat
Review by: jgw (desk)


Modified: trunk/user/src/com/google/gwt/user/client/Cookies.java
==============================================================================
--- trunk/user/src/com/google/gwt/user/client/Cookies.java      (original)
+++ trunk/user/src/com/google/gwt/user/client/Cookies.java      Thu Apr 16  
15:26:10 2009
@@ -120,8 +120,16 @@
            name = crumbs[i].substring(0, eqIdx);
            value = crumbs[i].substring(eqIdx + 1);
          }
-        name = decodeURIComponent(name);
-        value = decodeURIComponent(value);
+        try {
+          name = decodeURIComponent(name);
+        } catch (e) {
+          // ignore error, keep undecoded name
+        }
+        try {
+          value = decodeURIComponent(value);
+        } catch (e) {
+          // ignore error, keep undecoded value
+        }
           
[email protected]::put(Ljava/lang/Object;Ljava/lang/Object;)(name,value);
        }
      }

--~--~---------~--~----~------------~-------~--~----~
http://groups.google.com/group/Google-Web-Toolkit-Contributors
-~----------~----~----~----~------~----~------~--~---

Reply via email to