I looked at the full dump from the previous crashes and it looks like a testing issue. Hopefully it will be magically unbroken.

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 12 insertions, 9 deletions

diff --git a/net/base/escape.h b/net/base/escape.h
index 220eebc..4b86a64 100644
--- a/net/base/escape.h
+++ b/net/base/escape.h
@@ -77,17 +77,20 @@ class UnescapeRule {
     // by other applications.
     SPACES = 1,
 
-    // Unescapes "%25" to "%". This must not be used when the resulting string
-    // will need to be interpreted as a URL again, since we won't know what
-    // should be escaped and what shouldn't. For example, "%2520" would be
-    // converted to "%20" which would have different meaning than the origina.
-    // This flag is used when generating final output like filenames for URLs
-    // where we won't be interpreting as a URL and want to do as much unescaping
-    // as possible.
-    PERCENTS = 2,
+    // Unescapes various characters that will change the meaning of URLs,
+    // including '%', '+', '&', '/', '#'. If we unescaped these charaters, the
+    // resulting URL won't be the same as the source one. This flag is used when
+    // generating final output like filenames for URLs where we won't be
+    // interpreting as a URL and want to do as much unescaping as possible.
+    URL_SPECIAL_CHARS = 2,
+
+    // Unescapes control characters such as %01. This INCLUDES NULLs!. This is
+    // used for rare cases such as data: URL decoding where the result is binary
+    // data. You should not use this for normal URLs!
+    CONTROL_CHARS = 4,
 
     // URL queries use "+" for space. This flag controls that replacement.
-    REPLACE_PLUS_WITH_SPACE = 4,
+    REPLACE_PLUS_WITH_SPACE = 8,
   };
 };