Make Document::isSecureContext() work for OOPIFs

content/common/frame_messages.h

Index: content/common/frame_messages.h
diff --git a/content/common/frame_messages.h b/content/common/frame_messages.h
index ec76ea6950d62b451a19389e8e6763731af5b312..3965632071671961eb3fa25e5886e12bbb4e72ca 100644
--- a/content/common/frame_messages.h
+++ b/content/common/frame_messages.h
@@ -288,6 +288,10 @@ IPC_STRUCT_BEGIN_WITH_PARENT(FrameHostMsg_DidCommitProvisionalLoad_Params,
   // checking.
   IPC_STRUCT_MEMBER(bool, should_enforce_strict_mixed_content_checking)
 
+  // True if the document for the load is a unique origin that should be
+  // considered potentially trustworthy.
+  IPC_STRUCT_MEMBER(bool, is_potentially_trustworthy_unique_origin)
+
   // True if the navigation originated as an srcdoc attribute.
   IPC_STRUCT_MEMBER(bool, is_srcdoc)
 IPC_STRUCT_END()
@@ -381,6 +385,7 @@ IPC_STRUCT_TRAITS_BEGIN(content::FrameReplicationState)
   IPC_STRUCT_TRAITS_MEMBER(unique_name)
   IPC_STRUCT_TRAITS_MEMBER(scope)
   IPC_STRUCT_TRAITS_MEMBER(should_enforce_strict_mixed_content_checking)
+  IPC_STRUCT_TRAITS_MEMBER(is_potentially_trustworthy_unique_origin)

[alexmos, 2016/03/09 18:45:06]

nit: maybe that should be named has_potentialy_trustworthy_unique_origin?  Since
these are frame's properties rather than origin's, and "a frame has a
potentially trustworthy unique origin" sounds better than "a frame is a
potentially trustworthy unique origin".

[estark, 2016/03/10 00:53:44]

Done.

 IPC_STRUCT_TRAITS_END()
 
 IPC_STRUCT_BEGIN(FrameMsg_NewFrame_WidgetParams)
@@ -764,6 +769,13 @@ IPC_MESSAGE_ROUTED2(FrameMsg_DidUpdateName,
 IPC_MESSAGE_ROUTED1(FrameMsg_EnforceStrictMixedContentChecking,
                     bool /* should enforce */)
 
+// Update a proxy's replicated flag for a unique origin that should be
+// treated as potentially trustworthy. Used when the frame's origin is
+// set to or unset from a unique, potentially trustworthy origin.

[alexmos, 2016/03/09 18:45:06]

Maybe add another sentence here to explain under what conditions we might unset
the flag?  Presumably, this might happen when a frame loses sandbox flags and
commits a new navigation (triggering this update message from
NavigatorImpl::DidNavigate)?

[estark, 2016/03/10 00:53:44]

Done.

+IPC_MESSAGE_ROUTED1(
+    FrameMsg_DidSetUniqueOriginPotentiallyTrustworthy,
+    bool /* is the origin unique and potentially trustworthy */);
+
 // Update a proxy's replicated origin.  Used when the frame is navigated to a
 // new origin.
 IPC_MESSAGE_ROUTED1(FrameMsg_DidUpdateOrigin, url::Origin /* origin */)
@@ -949,6 +961,10 @@ IPC_MESSAGE_ROUTED2(FrameHostMsg_DidChangeName,
 // checking to be enforced.
 IPC_MESSAGE_ROUTED0(FrameHostMsg_EnforceStrictMixedContentChecking)
 
+// Sent when the frame is set to a unique origin that should be considered
+// potentially trustworthy.
+IPC_MESSAGE_ROUTED0(FrameHostMsg_SetUniqueOriginPotentiallyTrustworthy)
+
 // Sent when the renderer changed the progress of a load.
 IPC_MESSAGE_ROUTED1(FrameHostMsg_DidChangeLoadProgress,
                     double /* load_progress */)