Allow the max url length to be overridden

content/public/common/url_utils.cc

Index: content/public/common/url_utils.cc
diff --git a/content/public/common/url_utils.cc b/content/public/common/url_utils.cc
index 2f08b9650b63a10b92353909eaa8a1b5b9f44cb0..65d4bd633c4b58c29a9227df7d99a450555182e0 100644
--- a/content/public/common/url_utils.cc
+++ b/content/public/common/url_utils.cc
@@ -11,6 +11,8 @@
 
 namespace content {
 
+static size_t g_max_url_size = 2 * 1024 * 1024;

[Tom Sepez, 2013/12/10 22:12:19]

nit: you can probably avoid introducing this static unless you're android.

[Kristian Monsen, 2013/12/11 22:05:42]

I can but would prefer keeping the code simpler. If you or the other reviewers
strongly prefer that I can change.

+
 const char* const* GetSavableSchemes() {
   return GetSavableSchemesInternal();
 }
@@ -32,4 +34,14 @@ bool IsSavableURL(const GURL& url) {
   return false;
 }
 
+#if defined (OS_ANDROID)
+void SetMaxURLChars(size_t maxChars) {
+  g_maxUrlSize = maxChars;

[Tom Sepez, 2013/12/10 22:12:19]

nit: can you assert that you're android webview and not clank?

[boliu, 2013/12/10 22:21:59]

Not really, unless the assert is on something else completely unrelated, like
single process, or using the synchronous compositor.

How about rename this to SetMaxURLCharsForAndroidWebView?

[Kristian Monsen, 2013/12/11 22:05:42]

I don't think we can assert. I can update the method name if that is the
consensus.

+}
+#endif
+
+size_t MaxURLChars() {
+  return g_maxUrlSize;
+}
+
 }  // namespace content