Update EscapeExternalHandlerValue to keep '#', '[', and ']'

net/base/escape.h

Index: net/base/escape.h
diff --git a/net/base/escape.h b/net/base/escape.h
index 1915d24418883a2186ffc329733a93bc43938085..eee70d7524d04ee6ec09b0c134bb7a90074b2d50 100644
--- a/net/base/escape.h
+++ b/net/base/escape.h
@@ -42,8 +42,8 @@ NET_EXPORT std::string EscapeNonASCII(const std::string& input);
 
 // Escapes characters in text suitable for use as an external protocol handler
 // command.
-// We %XX everything except alphanumerics and %-_.!~*'() and the restricted
-// chracters (;/?:@&=+$,).
+// We %XX everything except alphanumerics and %-_.!~*'(), the restricted
+// chracters (;/?:@&=+$,#[]), and ones which are already escaped.

[asanka, 2014/10/23 07:20:04]

Since % is already mentioned in the exclusion list above, you should elaborate
what "ones which are already escaped" means.

[Jaekyun Seok (inactive), 2014/10/23 12:43:19]

Sorry, I should have removed '%' because here we are listing exceptional
characters which will not be escaped by EscapeExternalHandlerValue. Now I
removed '%' here.

And I updated the comments about "ones which are already escaped".

 NET_EXPORT std::string EscapeExternalHandlerValue(const std::string& text);
 
 // Appends the given character to the output string, escaping the character if