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

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: Removed DCHECK_IMPLIES(method == "POST", url.SchemeIs(http or https)). Created 4 years, 6 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. If |method| is "POST", then |post_body| needs to
124 // specify the request body, otherwise |post_body| should be null.
124 virtual void RequestTransferURL( 125 virtual void RequestTransferURL(
125 RenderFrameHostImpl* render_frame_host, 126 RenderFrameHostImpl* render_frame_host,
126 const GURL& url, 127 const GURL& url,
127 SiteInstance* source_site_instance, 128 SiteInstance* source_site_instance,
128 const std::vector<GURL>& redirect_chain, 129 const std::vector<GURL>& redirect_chain,
129 const Referrer& referrer, 130 const Referrer& referrer,
130 ui::PageTransition page_transition, 131 ui::PageTransition page_transition,
131 const GlobalRequestID& transferred_global_request_id, 132 const GlobalRequestID& transferred_global_request_id,
132 bool should_replace_current_entry) {} 133 bool should_replace_current_entry,
134 const std::string& method,
135 scoped_refptr<ResourceRequestBody> post_body) {}
133 136
134 // PlzNavigate 137 // PlzNavigate
135 // Called after receiving a BeforeUnloadACK IPC from the renderer. If 138 // Called after receiving a BeforeUnloadACK IPC from the renderer. If
136 // |frame_tree_node| has a NavigationRequest waiting for the renderer 139 // |frame_tree_node| has a NavigationRequest waiting for the renderer
137 // response, then the request is either started or canceled, depending on the 140 // response, then the request is either started or canceled, depending on the
138 // value of |proceed|. 141 // value of |proceed|.
139 virtual void OnBeforeUnloadACK(FrameTreeNode* frame_tree_node, 142 virtual void OnBeforeUnloadACK(FrameTreeNode* frame_tree_node,
140 bool proceed) {} 143 bool proceed) {}
141 144
142 // PlzNavigate 145 // PlzNavigate
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
176 const base::TimeTicks& renderer_before_unload_end_time) {} 179 const base::TimeTicks& renderer_before_unload_end_time) {}
177 180
178 protected: 181 protected:
179 friend class base::RefCounted<Navigator>; 182 friend class base::RefCounted<Navigator>;
180 virtual ~Navigator() {} 183 virtual ~Navigator() {}
181 }; 184 };
182 185
183 } // namespace content 186 } // namespace content
184 187
185 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_H_ 188 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698