Revert of Add a check that we don't send a secure referrer to an insecure URL

Revert of https://codereview.chromium.org/115803003/
Reason for revert: fails browser_tests:ReferrerPolicyTest.History
and content_browsertests:BrowserPluginHostTest.ReloadEmbedder on win7 dbg

TBR=mef@chromium.org,jochen@chromium.org
NOTREECHECKS=true
NOTRY=true

[jamesr, 2013-12-19 22:05:32]

Created Revert of Add a check that we don't send a secure referrer to an
insecure URL

[commit-bot: I haz the power, 2013-12-19 22:07:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jamesr@chromium.org/106683004/1

[commit-bot: I haz the power, 2013-12-19 22:07:11]

Failed to apply patch for net/url_request/url_request.cc:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file net/url_request/url_request.cc
  Hunk #1 FAILED at 8.
  Hunk #2 FAILED at 685.
  2 out of 2 hunks FAILED -- saving rejects to file
net/url_request/url_request.cc.rej

Patch:       net/url_request/url_request.cc
Index: net/url_request/url_request.cc
diff --git a/net/url_request/url_request.cc b/net/url_request/url_request.cc
index
a037063129a0ac1d294b5cd0490044de0817f7a9..6b0124730d5011fec92ddbf733be8f24f2db6ffa
100644
--- a/net/url_request/url_request.cc
+++ b/net/url_request/url_request.cc
@@ -8,7 +8,6 @@
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/compiler_specific.h"
-#include "base/debug/dump_without_crashing.h"
 #include "base/debug/stack_trace.h"
 #include "base/lazy_instance.h"
 #include "base/memory/singleton.h"
@@ -685,17 +684,6 @@
 
   response_info_.was_cached = false;
 
-  // If the referrer is secure, but the requested URL is not, the referrer
-  // policy should be something non-default. If you hit this, please file a
-  // bug.
-  if (referrer_policy_ ==
-          CLEAR_REFERRER_ON_TRANSITION_FROM_SECURE_TO_INSECURE &&
-      GURL(referrer_).SchemeIsSecure() && !url().SchemeIsSecure()) {
-    DLOG(FATAL) << "Trying to send secure referrer for insecure load";
-    base::debug::DumpWithoutCrashing();
-    referrer_.clear();
-  }
-
   // Don't allow errors to be sent from within Start().
   // TODO(brettw) this may cause NotifyDone to be sent synchronously,
   // we probably don't want this: they should be sent asynchronously so

[jamesr, 2013-12-19 22:09:17]

This was already reverted, closing.