Make <a ping> work for context menus.

content/renderer/render_frame_impl.cc

 // Copyright 2013 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 "content/renderer/render_frame_impl.h"
 
 #include <map>
 #include <string>
 
 #include "base/auto_reset.h"
@@ skipping 994 matching lines @@
     const CustomContextMenuContext& custom_context) {
   if (custom_context.request_id) {
     // External request, should be in our map.
     ContextMenuClient* client =
         pending_context_menus_.Lookup(custom_context.request_id);
     if (client) {
       client->OnMenuClosed(custom_context.request_id);
       pending_context_menus_.Remove(custom_context.request_id);
     }
   } else {
+    if (custom_context.link_followed.is_valid()) {
+        frame_->sendPings(
+            ContextMenuParamsBuilder::extractParentAnchorNode(

[darin (slow to review), 2014/07/22 17:34:40]

maybe we should just have a dom_utils.{h,cc} in content/renderer/, and put this
method in there? it seems like it doesn't strictly speaking have anything to do
with context menus.

[Nate Chapin, 2014/07/22 18:19:56]

Agree. dom_utils.{h,cc} seems like a reasonable name, will move there.

+                context_menu_node_),
+            custom_context.link_followed);
+    }
     // Internal request, forward to WebKit.
     context_menu_node_.reset();
   }
 }
 
 void RenderFrameImpl::OnCustomContextMenuAction(
     const CustomContextMenuContext& custom_context,
     unsigned action) {
   if (custom_context.request_id) {
     // External context menu request, look in our map.
@@ skipping 2554 matching lines @@
 
 #if defined(ENABLE_BROWSER_CDMS)
 RendererCdmManager* RenderFrameImpl::GetCdmManager() {
   if (!cdm_manager_)
     cdm_manager_ = new RendererCdmManager(this);
   return cdm_manager_;
 }
 #endif  // defined(ENABLE_BROWSER_CDMS)
 
 }  // namespace content