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

chrome/browser/android/url_sanitize_utils.h

Index: chrome/browser/android/url_sanitize_utils.h
diff --git a/chrome/browser/android/url_sanitize_utils.h b/chrome/browser/android/url_sanitize_utils.h
new file mode 100644
index 0000000000000000000000000000000000000000..91a272df83d2a73d5476088ebdcbfa6b32fe03af
--- /dev/null
+++ b/chrome/browser/android/url_sanitize_utils.h
@@ -0,0 +1,21 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_ANDROID_URL_SANITIZE_UTILS_H_
+#define CHROME_BROWSER_ANDROID_URL_SANITIZE_UTILS_H_
+
+#include "url/gurl.h"
+
+namespace chrome {
+namespace android {
+
+// Sanitizes a query part of |url|.
+// This function escapes characters in a query part of |url| suitable for use as
+// a URI query part. See RFC 2396 for the list of reserved characters.

[asanka, 2014/10/14 16:46:03]

* Mention that this function escapes all but the strict set of allowed
characters in each component of the URL.

* Note why this is required.

* Note differences between what GURL already does.

[Jaekyun Seok (inactive), 2014/10/15 08:02:25]

Done.

+GURL SanitizeUrl(const GURL& url);
+
+}  // namespace android
+}  // namespace chrome
+
+#endif  // CHROME_BROWSER_ANDROID_URL_SANITIZE_UTILS_H_