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

Unified Diff: content/public/browser/navigation_handle.h

Issue 1530393003: WIP: Move 'X-Frame-Options' checking to the browser. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 5 years 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/public/browser/navigation_handle.h
diff --git a/content/public/browser/navigation_handle.h b/content/public/browser/navigation_handle.h
index 0bfbca8c7115ebb8c566f501d010970fdd4f4a57..cf8086b7d819d1f8768389dca5534025e177a527 100644
--- a/content/public/browser/navigation_handle.h
+++ b/content/public/browser/navigation_handle.h
@@ -13,6 +13,10 @@
class GURL;
+namespace net {
+class HttpResponseHeaders;
+}
+
namespace content {
class NavigationThrottle;
class RenderFrameHost;
@@ -70,6 +74,19 @@ class CONTENT_EXPORT NavigationHandle {
// handlers.
virtual bool IsExternalProtocol() = 0;
+ // Parameters available at network response start time ----------------------
+ //
+ // The following parameters are only available when the network response is
+ // made for the navigation (or at commit time if no network request was made).
+ // This corresponds to NavigationThrottle::WillProcessResponse. They should
+ // not be queried before that.
+
+ // Returns the response headers for the request. This can only be accessed
+ // after a redirect was encountered or after the the navigation is ready to
+ // commit. It should not be modified, as modifications will not be reflected
+ // in the network stack.
+ virtual const net::HttpResponseHeaders* GetResponseHeaders() = 0;
+
// Navigation control flow --------------------------------------------------
// The net error code if an error happened prior to commit. Otherwise it will

Powered by Google App Engine
This is Rietveld 408576698