Remove WebHistoryItem child usage

content/renderer/context_menu_params_builder.cc

Index: content/renderer/context_menu_params_builder.cc
diff --git a/content/renderer/context_menu_params_builder.cc b/content/renderer/context_menu_params_builder.cc
index 0b3af2b10b46d1c8b82fc6bf1c73b93822efd982..f740f79ce528c13d35d4283e7662b442424d5139 100644
--- a/content/renderer/context_menu_params_builder.cc
+++ b/content/renderer/context_menu_params_builder.cc
@@ -16,7 +16,7 @@ namespace content {
 
 // static
 ContextMenuParams ContextMenuParamsBuilder::Build(
-    const blink::WebContextMenuData& data) {
+    const blink::WebContextMenuData& data, const PageState& state) {
   ContextMenuParams params;
   params.media_type = data.mediaType;
   params.x = data.mousePosition.x;
@@ -49,8 +49,7 @@ ContextMenuParams ContextMenuParamsBuilder::Build(
   for (size_t i = 0; i < data.customItems.size(); ++i)
     params.custom_items.push_back(MenuItemBuilder::Build(data.customItems[i]));
 
-  if (!data.frameHistoryItem.isNull())
-    params.frame_page_state = HistoryItemToPageState(data.frameHistoryItem);
+  params.frame_page_state = state;

[Nate Chapin, 2014/04/25 21:49:21]

I'm not 100% this is safe, since this was serializing only part of the frame
state before, and now it's serializing the state for the whole page. I can't
think of why you'd want only part of the history tree, though.

[Charlie Reis, 2014/04/28 16:29:00]

I'm not sure what it's used for, so it's hard to say.  We're probably going to
have to break PageState down into FrameState eventually, though, since each
frame's state will be stored on a separate FrameNavigationEntry in the browser
process (and each frame may come from a different renderer process).

Looking closer, it appears to be used in Browser::ViewSourceForFrame, which
loads a subframe URL and state at the top-level of a new tab in view-source mode
(see browser.cc).  That's potentially a problem, since we'll now have the
PageState for the entire page rather than the subframe that we're loading in a
new tab.  (You might even be able to test this out manually, but I'm not sure
how it will manifest.)

I wonder if there's a way to get the PageState for just the frame from |data|.

[Nate Chapin, 2014/04/28 17:31:54]

Yep, right-click in iframe - > "View Page Source" shows the parent frame's
content. Maybe we should expose a way to serialize a single WebHistoryItem
without children? I'll play with it.

 
   if (!params.link_url.is_empty()) {
     blink::WebNode selectedNode = data.node;