Notify the renderer if a history navigation has no subframe items.

content/renderer/render_frame_impl.h

Index: content/renderer/render_frame_impl.h
diff --git a/content/renderer/render_frame_impl.h b/content/renderer/render_frame_impl.h
index 30d6ffac137a2efcb6cbd6f4ae5f6bc46bc2c856..df10be4f7760aa56ee9f5179c935095c9efca488 100644
--- a/content/renderer/render_frame_impl.h
+++ b/content/renderer/render_frame_impl.h
@@ -1095,6 +1095,17 @@ class CONTENT_EXPORT RenderFrameImpl
   // didCreateDataSource().
   std::unique_ptr<NavigationParams> pending_navigation_params_;
 
+  // Keeps track of whether the browser process has any history items that need
+  // to be used for subframes of this frame (in the case of a history
+  // navigation).  If not, the renderer can skip sending an IPC to the browser
+  // and directly load any initial URLs for children itself.  This state is
+  // cleared during didStopLoading, since it is not needed after the first load
+  // completes and is never used after the initial navigation.  It is inherited
+  // by subframes.
+  // TODO(creis): Switch this to a data structure of unique names and
+  // corresponding same-process PageStates in https://crbug.com/639842.
+  bool browser_has_subtree_history_items_;

[Charlie Reis, 2016/09/21 21:17:49]

Figuring out where (and how long) to store this is the trickiest part. 
Subframes need to inherit this info (unlike NavigationState), and it's only
useful during the initial navigation.  I think putting it directly on
RenderFrame and clearing it at didStopLoading should be sufficient.

[alexmos, 2016/09/22 01:44:06]

Acknowledged.

+
   // Stores the current history item for this frame, so that updates to it can
   // be reported to the browser process via SendUpdateState.
   blink::WebHistoryItem current_history_item_;