commit 3adab1c8e6a57b8ad01dfc2c3c1f549514e713f7
author benm@chromium.org <benm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Mon Sep 09 21:23:43 2013
committer benm@chromium.org <benm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Mon Sep 09 21:23:43 2013
tree 7b6245ee27fbd35b44b10bb5374f453aea21aad8
parent 1c3e1b050b3d0045d924a68b697cd36c5da67fe5

Log a warning if we try to load a URL longer than kMaxURLChars.

Review URL: https://chromiumcodereview.appspot.com/23983010

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

content/browser/web_contents/web_contents_impl.cc

diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 6a776e5..1856a8d 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -1774,8 +1774,11 @@
 
   // The renderer will reject IPC messages with URLs longer than
   // this limit, so don't attempt to navigate with a longer URL.
-  if (entry.GetURL().spec().size() > kMaxURLChars)
+  if (entry.GetURL().spec().size() > kMaxURLChars) {
+    LOG(WARNING) << "Refusing to load URL as it exceeds " << kMaxURLChars
+                 << " characters.";
     return false;
+  }
 
   RenderViewHostImpl* dest_render_view_host =
       static_cast<RenderViewHostImpl*>(render_manager_.Navigate(entry));