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

Unified 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, 2 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/navigation_handle_impl.h
diff --git a/content/browser/frame_host/navigation_handle_impl.h b/content/browser/frame_host/navigation_handle_impl.h
index 58aff96b249dfb6a28b8b30a6bf62f8543029eda..96589fb4ddd5e28242cf8014c324e58d4184dad9 100644
--- a/content/browser/frame_host/navigation_handle_impl.h
+++ b/content/browser/frame_host/navigation_handle_impl.h
@@ -254,6 +254,11 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
bool same_page,
RenderFrameHostImpl* render_frame_host);
+ // 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
+ void AbortCommit(
+ RenderFrameHostImpl* render_frame_host,
+ scoped_refptr<net::HttpResponseHeaders> response_headers);
+
// Called during commit. Takes ownership of the embedder's NavigationData
// instance. This NavigationData may have been cloned prior to being added
// here.
@@ -293,6 +298,7 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
WILL_PROCESS_RESPONSE,
DEFERRING_RESPONSE,
READY_TO_COMMIT,
+ DID_ABORT_COMMIT,
DID_COMMIT,
DID_COMMIT_ERROR_PAGE,
};

Powered by Google App Engine
This is Rietveld 408576698