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

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

Issue 2465813002: Fix content script injection for cancelled loads with PlzNavigate. (Closed)
Patch Set: Created 4 years, 1 month 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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_NAVIGATION_HANDLE_IMPL_H_ 5 #ifndef CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_
6 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_ 6 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_
7 7
8 #include "content/public/browser/navigation_handle.h" 8 #include "content/public/browser/navigation_handle.h"
9 9
10 #include <stddef.h> 10 #include <stddef.h>
(...skipping 236 matching lines...) Expand 10 before | Expand all | Expand 10 after
247 // delegate. 247 // delegate.
248 void ReadyToCommitNavigation(RenderFrameHostImpl* render_frame_host); 248 void ReadyToCommitNavigation(RenderFrameHostImpl* render_frame_host);
249 249
250 // Called when the navigation was committed in |render_frame_host|. This will 250 // Called when the navigation was committed in |render_frame_host|. This will
251 // update the |state_|. 251 // update the |state_|.
252 void DidCommitNavigation( 252 void DidCommitNavigation(
253 const FrameHostMsg_DidCommitProvisionalLoad_Params& params, 253 const FrameHostMsg_DidCommitProvisionalLoad_Params& params,
254 bool same_page, 254 bool same_page,
255 RenderFrameHostImpl* render_frame_host); 255 RenderFrameHostImpl* render_frame_host);
256 256
257 // Called when a 204 or 205 response is received, which abort the commit.
Charlie Reis 2016/10/31 17:05:28 From NavigationHandle's perspective, how is a 204
258 void AbortCommit(
259 RenderFrameHostImpl* render_frame_host,
260 scoped_refptr<net::HttpResponseHeaders> response_headers);
261
257 // Called during commit. Takes ownership of the embedder's NavigationData 262 // Called during commit. Takes ownership of the embedder's NavigationData
258 // instance. This NavigationData may have been cloned prior to being added 263 // instance. This NavigationData may have been cloned prior to being added
259 // here. 264 // here.
260 void set_navigation_data(std::unique_ptr<NavigationData> navigation_data) { 265 void set_navigation_data(std::unique_ptr<NavigationData> navigation_data) {
261 navigation_data_ = std::move(navigation_data); 266 navigation_data_ = std::move(navigation_data);
262 } 267 }
263 268
264 SSLStatus ssl_status() { return ssl_status_; } 269 SSLStatus ssl_status() { return ssl_status_; }
265 270
266 // This is valid after the network response has started. 271 // This is valid after the network response has started.
(...skipping 19 matching lines...) Expand all
286 enum State { 291 enum State {
287 INITIAL = 0, 292 INITIAL = 0,
288 WILL_SEND_REQUEST, 293 WILL_SEND_REQUEST,
289 DEFERRING_START, 294 DEFERRING_START,
290 WILL_REDIRECT_REQUEST, 295 WILL_REDIRECT_REQUEST,
291 DEFERRING_REDIRECT, 296 DEFERRING_REDIRECT,
292 CANCELING, 297 CANCELING,
293 WILL_PROCESS_RESPONSE, 298 WILL_PROCESS_RESPONSE,
294 DEFERRING_RESPONSE, 299 DEFERRING_RESPONSE,
295 READY_TO_COMMIT, 300 READY_TO_COMMIT,
301 DID_ABORT_COMMIT,
296 DID_COMMIT, 302 DID_COMMIT,
297 DID_COMMIT_ERROR_PAGE, 303 DID_COMMIT_ERROR_PAGE,
298 }; 304 };
299 305
300 NavigationHandleImpl(const GURL& url, 306 NavigationHandleImpl(const GURL& url,
301 FrameTreeNode* frame_tree_node, 307 FrameTreeNode* frame_tree_node,
302 bool is_renderer_initiated, 308 bool is_renderer_initiated,
303 bool is_synchronous, 309 bool is_synchronous,
304 bool is_srcdoc, 310 bool is_srcdoc,
305 const base::TimeTicks& navigation_start, 311 const base::TimeTicks& navigation_start,
(...skipping 117 matching lines...) Expand 10 before | Expand all | Expand 10 after
423 bool started_from_context_menu_; 429 bool started_from_context_menu_;
424 430
425 base::WeakPtrFactory<NavigationHandleImpl> weak_factory_; 431 base::WeakPtrFactory<NavigationHandleImpl> weak_factory_;
426 432
427 DISALLOW_COPY_AND_ASSIGN(NavigationHandleImpl); 433 DISALLOW_COPY_AND_ASSIGN(NavigationHandleImpl);
428 }; 434 };
429 435
430 } // namespace content 436 } // namespace content
431 437
432 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_ 438 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698