Add option to suppress HTTP Referer header.

chrome/browser/renderer_host/resource_dispatcher_host.cc

-// Copyright (c) 2009 The Chromium Authors. All rights reserved.
+// Copyright (c) 2010 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.
 
 // See http://dev.chromium.org/developers/design-documents/multi-process-resource-loading
 
 #include "chrome/browser/renderer_host/resource_dispatcher_host.h"
 
 #include <vector>
 
 #include "base/command_line.h"
@@ skipping 360 matching lines @@
   if (HandleExternalProtocol(request_id, child_id, route_id,
                              request_data.url, request_data.resource_type,
                              handler)) {
     return;
   }
 
   // Construct the request.
   URLRequest* request = new URLRequest(request_data.url, this);
   request->set_method(request_data.method);
   request->set_first_party_for_cookies(request_data.first_party_for_cookies);
-  request->set_referrer(request_data.referrer.spec());
+  request->set_referrer(CommandLine::ForCurrentProcess()->HasSwitch(
+      switches::kNoReferrers) ? std::string() : request_data.referrer.spec());
   request->SetExtraRequestHeaders(request_data.headers);
 
   int load_flags = request_data.load_flags;
   // EV certificate verification could be expensive.  We don't want to spend
   // time performing EV certificate verification on all resources because
   // EV status is irrelevant to sub-frames and sub-resources.
   if (request_data.resource_type == ResourceType::MAIN_FRAME)
     load_flags |= net::LOAD_VERIFY_EV_CERT;
   request->set_load_flags(load_flags);
   request->set_context(context);
@@ skipping 215 matching lines @@
                                               receiver_);
   }
 
   if (!URLRequest::IsHandledURL(url)) {
     LOG(INFO) << "Download request for unsupported protocol: " <<
         url.possibly_invalid_spec();
     return;
   }
 
   request->set_method("GET");
-  request->set_referrer(referrer.spec());
+  request->set_referrer(CommandLine::ForCurrentProcess()->HasSwitch(

[darin (slow to review), 2010/02/10 07:20:41]

i think this duplication of code is unfortunate.  we should ideally have a
function that is used to initialize an URLRequest that can be shared.  this
doesn't have to be part of this CL, but if you were inclined to do such
refactoring that'd be great :)

+      switches::kNoReferrers) ? std::string() : referrer.spec());
   request->set_context(request_context);
   request->set_load_flags(request->load_flags() |
       net::LOAD_IS_DOWNLOAD);
 
   ResourceDispatcherHostRequestInfo* extra_info =
       new ResourceDispatcherHostRequestInfo(handler,
                                             ChildProcessInfo::RENDER_PROCESS,
                                             child_id,
                                             route_id,
                                             request_id_,
@@ skipping 35 matching lines @@
   if (!known_proto) {
     // Since any URLs which have non-standard scheme have been filtered
     // by save manager(see GURL::SchemeIsStandard). This situation
     // should not happen.
     NOTREACHED();
     return;
   }
 
   URLRequest* request = new URLRequest(url, this);
   request->set_method("GET");
-  request->set_referrer(referrer.spec());
+  request->set_referrer(CommandLine::ForCurrentProcess()->HasSwitch(
+      switches::kNoReferrers) ? std::string() : referrer.spec());
   // So far, for saving page, we need fetch content from cache, in the
   // future, maybe we can use a configuration to configure this behavior.
   request->set_load_flags(net::LOAD_PREFERRING_CACHE);
   request->set_context(request_context);
 
   // Since we're just saving some resources we need, disallow downloading.
   ResourceDispatcherHostRequestInfo* extra_info =
       new ResourceDispatcherHostRequestInfo(handler,
                                             ChildProcessInfo::RENDER_PROCESS,
                                             child_id,
@@ skipping 1013 matching lines @@
     case ViewHostMsg_UploadProgress_ACK::ID:
     case ViewHostMsg_SyncLoad::ID:
       return true;
 
     default:
       break;
   }
 
   return false;
 }