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

Unified Diff: content/common/frame_messages.h

Issue 1710283003: OOPIF: Handle cross-site frames being blocked by X-Frame-Options or CSP. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 10 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/common/frame_messages.h
diff --git a/content/common/frame_messages.h b/content/common/frame_messages.h
index b990cdb6c8ad08169870c8ce5dded0f811622d52..94864c571608401e38a96ce7d2600206755f6ccc 100644
--- a/content/common/frame_messages.h
+++ b/content/common/frame_messages.h
@@ -846,6 +846,10 @@ IPC_MESSAGE_ROUTED1(FrameMsg_SerializeAsMHTML, FrameMsg_SerializeAsMHTML_Params)
IPC_MESSAGE_ROUTED1(FrameMsg_SetFrameOwnerProperties,
blink::WebFrameOwnerProperties /* frame_owner_properties */)
+// Notifies the frame that its load was blocked by X-Frame-Options or CSP
+// frame-ancestors while loading.
+IPC_MESSAGE_ROUTED0(FrameMsg_CancelLoadAfterXFrameOptionsOrCSPDenied)
Charlie Reis 2016/02/26 01:13:22 Is there a reason to make this specific to these f
alexmos 2016/02/26 02:38:05 Good question. It's only specific in the sense th
Charlie Reis 2016/02/26 21:10:01 Yeah, let's make it BlockedLoad, with documentatio
+
// Request to continue running the sequential focus navigation algorithm in
// this frame. |source_routing_id| identifies the frame that issued this
// request. This message is sent when pressing <tab> or <shift-tab> needs to
@@ -995,6 +999,13 @@ IPC_MESSAGE_ROUTED2(FrameHostMsg_DidChangeFrameOwnerProperties,
int32_t /* subframe_routing_id */,
blink::WebFrameOwnerProperties /* frame_owner_properties */)
+// Notifies the browser process that a subframe was blocked by
+// X-Frame-Options or CSP frame-ancestors while loading.
+//
+// TODO(mkwst, alexmos): This will become unnecessary once X-Frame-Options and
+// CSP enforcement moves to the browser process (https://crbug.com/555418).
+IPC_MESSAGE_ROUTED0(FrameHostMsg_DidCancelLoadAfterXFrameOptionsOrCSPDenied)
+
// Changes the title for the page in the UI when the page is navigated or the
// title changes. Sent for top-level frames.
IPC_MESSAGE_ROUTED2(FrameHostMsg_UpdateTitle,

Powered by Google App Engine
This is Rietveld 408576698