Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1469)

Unified Diff: content/renderer/render_frame_impl.h

Issue 2316003002: Notify the renderer if a history navigation has no subframe items. (Closed)
Patch Set: Fix indent. Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/common/navigation_params.cc ('k') | content/renderer/render_frame_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_frame_impl.h
diff --git a/content/renderer/render_frame_impl.h b/content/renderer/render_frame_impl.h
index 561eb56b8230936d6610f9341c9b6f2552a23938..7a21b8438b2d81acdae332f8918d952aa3760106 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_;
+
// 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_;
« no previous file with comments | « content/common/navigation_params.cc ('k') | content/renderer/render_frame_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698