Remove URL fragment from referrer HTTP header when opening link using "Open Link in New Tab" option.

chrome/browser/tab_contents/render_view_context_menu.cc

 // Copyright (c) 2012 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.
 
 #include "chrome/browser/tab_contents/render_view_context_menu.h"
 
 #include <algorithm>
 #include <set>
 #include <utility>
 
@@ skipping 1831 matching lines @@
   return ui::TruncateString(params_.selection_text,
                             kMaxSelectionTextLength);
 }
 
 // Controller functions --------------------------------------------------------
 
 void RenderViewContextMenu::OpenURL(
     const GURL& url, const GURL& referrer, int64 frame_id,
     WindowOpenDisposition disposition,
     content::PageTransition transition) {
+  // Ensure that we do not send url fragment,

[pauljensen, 2013/03/14 18:53:44]

Capitalize URL.  Let's avoid "we" by saying "Ensure ... fields are not sent in
the referrer."

[mef, 2013/03/14 20:31:38]

Done.

+  // username and password fields in the referrer.
+  GURL referrer_url(referrer);

[pauljensen, 2013/03/14 18:53:44]

"referrer_url" doesn't indicate its difference from "referrer".  Maybe
"sanitized_referrer".

[mef, 2013/03/14 20:31:38]

Done.

+  if (referrer_url.is_valid() && (referrer_url.has_ref() ||
+      referrer_url.has_username() ||  referrer_url.has_password())) {
+    GURL::Replacements referrer_mods;
+    referrer_mods.ClearRef();
+    referrer_mods.ClearUsername();
+    referrer_mods.ClearPassword();
+    referrer_url = referrer_url.ReplaceComponents(referrer_mods);
+  }
+
   WebContents* new_contents = source_web_contents_->OpenURL(OpenURLParams(
-      url, content::Referrer(referrer, params_.referrer_policy), disposition,
-      transition, false));
+      url, content::Referrer(referrer_url, params_.referrer_policy),
+      disposition, transition, false));
   if (!new_contents)
     return;
 
   RetargetingDetails details;
   details.source_web_contents = source_web_contents_;
   details.source_frame_id = frame_id;
   details.target_url = url;
   details.target_web_contents = new_contents;
   details.not_yet_in_tabstrip = false;
   content::NotificationService::current()->Notify(
@@ skipping 27 matching lines @@
   source_web_contents_->GetRenderViewHost()->
       ExecuteMediaPlayerActionAtLocation(location, action);
 }
 
 void RenderViewContextMenu::PluginActionAt(
     const gfx::Point& location,
     const WebPluginAction& action) {
   source_web_contents_->GetRenderViewHost()->
       ExecutePluginActionAtLocation(location, action);
 }