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

Side by Side Diff: content/browser/frame_host/navigator.h

Issue 1956383003: Forwarding POST body into renderer after a cross-site transfer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebasing... Created 4 years, 7 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_H_ 5 #ifndef CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_H_
6 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_H_ 6 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_H_
7 7
8 #include "base/memory/ref_counted.h" 8 #include "base/memory/ref_counted.h"
9 #include "base/time/time.h" 9 #include "base/time/time.h"
10 #include "content/browser/frame_host/navigator_delegate.h" 10 #include "content/browser/frame_host/navigator_delegate.h"
(...skipping 102 matching lines...) Expand 10 before | Expand all | Expand 10 after
113 virtual void RequestOpenURL(RenderFrameHostImpl* render_frame_host, 113 virtual void RequestOpenURL(RenderFrameHostImpl* render_frame_host,
114 const GURL& url, 114 const GURL& url,
115 SiteInstance* source_site_instance, 115 SiteInstance* source_site_instance,
116 const Referrer& referrer, 116 const Referrer& referrer,
117 WindowOpenDisposition disposition, 117 WindowOpenDisposition disposition,
118 bool should_replace_current_entry, 118 bool should_replace_current_entry,
119 bool user_gesture) {} 119 bool user_gesture) {}
120 120
121 // The RenderFrameHostImpl wants to transfer the request to a new renderer. 121 // The RenderFrameHostImpl wants to transfer the request to a new renderer.
122 // |redirect_chain| contains any redirect URLs (excluding |url|) that happened 122 // |redirect_chain| contains any redirect URLs (excluding |url|) that happened
123 // before the transfer. 123 // before the transfer.
clamy 2016/05/26 15:35:23 nit: precise that |post_body| should be null is me
Łukasz Anforowicz 2016/05/26 17:14:02 Done.
124 virtual void RequestTransferURL( 124 virtual void RequestTransferURL(
125 RenderFrameHostImpl* render_frame_host, 125 RenderFrameHostImpl* render_frame_host,
126 const GURL& url, 126 const GURL& url,
127 SiteInstance* source_site_instance, 127 SiteInstance* source_site_instance,
128 const std::vector<GURL>& redirect_chain, 128 const std::vector<GURL>& redirect_chain,
129 const Referrer& referrer, 129 const Referrer& referrer,
130 ui::PageTransition page_transition, 130 ui::PageTransition page_transition,
131 const GlobalRequestID& transferred_global_request_id, 131 const GlobalRequestID& transferred_global_request_id,
132 bool should_replace_current_entry) {} 132 bool should_replace_current_entry,
133 const std::string& method,
134 const scoped_refptr<ResourceRequestBody>& post_body) {}
133 135
134 // PlzNavigate 136 // PlzNavigate
135 // Called after receiving a BeforeUnloadACK IPC from the renderer. If 137 // Called after receiving a BeforeUnloadACK IPC from the renderer. If
136 // |frame_tree_node| has a NavigationRequest waiting for the renderer 138 // |frame_tree_node| has a NavigationRequest waiting for the renderer
137 // response, then the request is either started or canceled, depending on the 139 // response, then the request is either started or canceled, depending on the
138 // value of |proceed|. 140 // value of |proceed|.
139 virtual void OnBeforeUnloadACK(FrameTreeNode* frame_tree_node, 141 virtual void OnBeforeUnloadACK(FrameTreeNode* frame_tree_node,
140 bool proceed) {} 142 bool proceed) {}
141 143
142 // PlzNavigate 144 // PlzNavigate
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
176 const base::TimeTicks& renderer_before_unload_end_time) {} 178 const base::TimeTicks& renderer_before_unload_end_time) {}
177 179
178 protected: 180 protected:
179 friend class base::RefCounted<Navigator>; 181 friend class base::RefCounted<Navigator>;
180 virtual ~Navigator() {} 182 virtual ~Navigator() {}
181 }; 183 };
182 184
183 } // namespace content 185 } // namespace content
184 186
185 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_H_ 187 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698