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

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

Issue 2321543002: Merge CrossSiteResourceHandler and NavigationResourceThrottle (Closed)
Patch Set: Addressed comments Created 4 years, 3 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 67b1389c5abcfe79f647288b95cec60bd7e32686..04a477d256f94c91a3d7908bbdf3a6b501ac52ff 100644
--- a/content/browser/frame_host/navigation_handle_impl.h
+++ b/content/browser/frame_host/navigation_handle_impl.h
@@ -16,6 +16,7 @@
#include "content/browser/frame_host/frame_tree_node.h"
#include "content/browser/frame_host/render_frame_host_impl.h"
#include "content/common/content_export.h"
+#include "content/public/browser/global_request_id.h"
#include "content/public/browser/navigation_data.h"
#include "content/public/browser/navigation_throttle.h"
#include "content/public/common/request_context_type.h"
@@ -151,12 +152,6 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
is_transferring_ = is_transferring;
}
- // Updates the RenderFrameHost that is about to commit the navigation. This
- // is used during transfer navigations.
- void set_render_frame_host(RenderFrameHostImpl* render_frame_host) {
- render_frame_host_ = render_frame_host;
- }
-
// Returns the POST body associated with this navigation. This will be
// null for GET and/or other non-POST requests (or if a response to a POST
// request was a redirect that changed the method to GET - for example 302).
@@ -198,11 +193,18 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
// NavigationHandle will not call |callback| with a result of DEFER.
// If the result is PROCEED, then 'ReadyToCommitNavigation' will be called
// with |render_frame_host| and |response_headers| just before calling
- // |callback|.
+ // |callback|. Should a transfer navigation happen, |transfer_callback| will
+ // be run on the IO thread.
+ // PlzNavigate: transfer navigations are not possible.
void WillProcessResponse(
RenderFrameHostImpl* render_frame_host,
scoped_refptr<net::HttpResponseHeaders> response_headers,
const SSLStatus& ssl_status,
+ const GlobalRequestID& request_id,
+ bool should_replace_current_entry,
+ bool is_download,
+ bool is_stream,
+ const base::Closure& transfer_callback,
const ThrottleChecksFinishedCallback& callback);
// Returns the FrameTreeNode this navigation is happening in.
@@ -229,6 +231,11 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
SSLStatus ssl_status() { return ssl_status_; }
+ const GlobalRequestID& request_id() const { return request_id_; }
+
+ // Called when the navigation is transferred to a RenderFrameHost.
+ void TransferToRenderFrameHost(RenderFrameHostImpl* render_frame_host);
+
private:
friend class NavigationHandleImplTest;
@@ -259,6 +266,15 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
NavigationThrottle::ThrottleCheckResult CheckWillRedirectRequest();
NavigationThrottle::ThrottleCheckResult CheckWillProcessResponse();
+ // Called when WillProcessResponse checks are done, to find the final
+ // RenderFrameHost for the navigation. Returns true if the navigation should
+ // proceed.
+ bool FindFinalRenderFrameHost();
+
+ // Check whether the navigation should be transferred. Returns false if the
+ // transfer attempt results in the destruction of this NavigationHandle.
+ bool MaybeTransferAndProceed();
+
// Helper function to run and reset the |complete_callback_|. This marks the
// end of a round of NavigationThrottleChecks.
void RunCompleteCallback(NavigationThrottle::ThrottleCheckResult result);
@@ -284,6 +300,10 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
bool was_redirected_;
scoped_refptr<net::HttpResponseHeaders> response_headers_;
+ // The original url of the navigation. This may differ from |url_| if the
+ // navigation encounters redirects.
+ const GURL original_url_;
+
// The HTTP method used for the navigation.
std::string method_;
@@ -325,6 +345,24 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
SSLStatus ssl_status_;
+ // The id of the URLRequest tied to this navigation.
+ GlobalRequestID request_id_;
+
+ // Whether the current NavigationEntry should be replaced upon commit.
+ bool should_replace_current_entry_;
+
+ // The chain of redirects.
+ std::vector<GURL> redirect_chain_;
+
+ // A callback to run on the IO thread if the navigation transfers.
+ base::Closure transfer_callback_;
+
+ // Whether the navigation ended up being a download or a stream.
+ bool is_download_;
+ bool is_stream_;
+
+ base::WeakPtrFactory<NavigationHandleImpl> weak_factory_;
+
DISALLOW_COPY_AND_ASSIGN(NavigationHandleImpl);
};

Powered by Google App Engine
This is Rietveld 408576698