Make <a ping> work for context menus.

content/renderer/context_menu_params_builder.h

Index: content/renderer/context_menu_params_builder.h
diff --git a/content/renderer/context_menu_params_builder.h b/content/renderer/context_menu_params_builder.h
index 624f85505c8a9829dba91e80e707a32d42ad96e6..bb36a444b7247de16978d9793edc8c0641e1a987 100644
--- a/content/renderer/context_menu_params_builder.h
+++ b/content/renderer/context_menu_params_builder.h
@@ -7,6 +7,7 @@
 
 namespace blink {
 struct WebContextMenuData;
+class WebNode;
 }
 
 namespace content {
@@ -15,6 +16,8 @@ struct ContextMenuParams;
 class ContextMenuParamsBuilder {
  public:
   static ContextMenuParams Build(const blink::WebContextMenuData& data);
+
+  static blink::WebNode extractParentAnchorNode(const blink::WebNode& node);

[Nate Chapin, 2014/07/14 19:01:46]

WebContextMenuData provides the deepest WebNode that was clicked for the context
menu. Since that can be a text node for a clicked link, we need a way to walk up
the DOM, looking for a parent that is actually an <a>. This logic is extracted
from Build().

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

You should probably add some comments on this function. However, I'm not sure
this function really has that much to do with ContextMenuParamsBuilder. Is this
the best place for this function to live?

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

nit: extract -> Extract

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

Done.

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

Done.

 };
 
 }  // namespace content