Don't reset URL for CAN_COMMIT_URL_BLOCKED renderer kill.

content/browser/frame_host/render_frame_host_impl.cc

Index: content/browser/frame_host/render_frame_host_impl.cc
diff --git a/content/browser/frame_host/render_frame_host_impl.cc b/content/browser/frame_host/render_frame_host_impl.cc
index b6086a04aecf9b994912f9ece06ffcad50fa5b49..058d73f2c0d087650706c881b93596daf55e5028 100644
--- a/content/browser/frame_host/render_frame_host_impl.cc
+++ b/content/browser/frame_host/render_frame_host_impl.cc
@@ -1221,10 +1221,10 @@ void RenderFrameHostImpl::OnDidCommitProvisionalLoad(const IPC::Message& msg) {
   // should be killed.
   if (!CanCommitURL(validated_params.url)) {
     VLOG(1) << "Blocked URL " << validated_params.url.spec();
-    validated_params.url = GURL(url::kAboutBlankURL);
     // Kills the process.
     bad_message::ReceivedBadMessage(process,
                                     bad_message::RFH_CAN_COMMIT_URL_BLOCKED);
+    validated_params.url = GURL(url::kAboutBlankURL);

[nasko, 2017/03/20 21:37:14]

Doing some code archeology, it looks like we might actually not need that line.
We don't have it in the CanCommitOrigin check below, so we should be consistent.

We had to have it before, as there was no return statement in this if block,
which meant that the URL had to be rewritten to be safe. However as part of
https://codereview.chromium.org/1775543002, the return statement is added and
this seems to no longer be needed.

The ordering of dates back from https://chromiumcodereview.appspot.com/15017018,
so it looks like we didn't really get meaningful URLs from this kill all along.

[Charlie Reis, 2017/03/21 18:32:11]

Good call!  Done.

     return;
   }