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

Unified Diff: content/browser/frame_host/render_frame_host_impl.h

Issue 483773002: PlzNavigate: implement CommitNavigation on the browser side (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added a unit test for the reload case Created 6 years, 4 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
Index: content/browser/frame_host/render_frame_host_impl.h
diff --git a/content/browser/frame_host/render_frame_host_impl.h b/content/browser/frame_host/render_frame_host_impl.h
index 97e1d024c2678eccf6825098e636b72b4f00e246..0b9c11e95db3b5068fa822e2532307e030b2827c 100644
--- a/content/browser/frame_host/render_frame_host_impl.h
+++ b/content/browser/frame_host/render_frame_host_impl.h
@@ -31,6 +31,7 @@ struct AccessibilityHostMsg_LocationChangeParams;
struct FrameHostMsg_DidFailProvisionalLoadWithError_Params;
struct FrameHostMsg_OpenURL_Params;
struct FrameHostMsg_BeginNavigation_Params;
+struct FrameMsg_CommitNavigation_Params;
struct FrameMsg_Navigate_Params;
#if defined(OS_MACOSX) || defined(OS_ANDROID)
struct FrameHostMsg_ShowPopup_Params;
@@ -281,6 +282,9 @@ class CONTENT_EXPORT RenderFrameHostImpl
void DidCancelPopupMenu();
#endif
+ // PlzNavigate: Indicates that a navigation is about to commit in this
(Do not use) nasko 2014/08/28 16:39:09 Is it about to commit? Or has it committed and we
clamy 2014/09/02 18:25:19 Is it clearer this way?
nasko 2014/09/03 16:30:26 I think it is. Thanks!
+ // RenderFrameHost.
+ void CommitNavigation(const FrameMsg_CommitNavigation_Params& params);
protected:
friend class RenderFrameHostFactory;

Powered by Google App Engine
This is Rietveld 408576698