From 38e8eb1e7ae821f2d24d4fd2ce6577a10d892dc5 Mon Sep 17 00:00:00 2001
From: Andrew Verge <averge@chromium.org>
Date: Thu, 5 Sep 2024 13:34:25 -0400
Subject: [PATCH] Re-add fenced frame config internal container size.

Follow-up to https://github.com/WICG/fenced-frame/pull/182

Along with the unused IDL size attributes, I also removed the internal fenced frame config container size definition. I was mistaken about it being vestigial (I missed a reference to it in the implementation when I was originally deciding to remove it, so I didn't see it was actively used).

The Protected Audience spec depended on the field, and I broke their build: https://github.com/WICG/turtledove/pull/1270#issuecomment-2332092944

This change re-adds the container size definition to un-break things.
---
 spec.bs | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/spec.bs b/spec.bs
index 6bcdd3c..e548854 100644
--- a/spec.bs
+++ b/spec.bs
@@ -725,9 +725,9 @@ boundary, which can allow for colluding parties to join cross-site data and buil
 user. To prevent that, the ad auction API [=construct a pending fenced frame config|constructs=] a
 [=fenced frame config=] whose underlying [=fenced frame config/mapped url|URL=] is opaque to the
 embedding context. The [=fenced frame config=] is also constructed with restrictions on what the
-[=fenced frame config/content size=] of the frame must be and what the [=fenced frame config/
-effective enabled permissions|permissions policy=] of the frame must be, as those can be used as 
-fingerprinting vectors.
+[=fenced frame config/container size=] and [=fenced frame config/content size=] of the frame must 
+be and what the [=fenced frame config/effective enabled permissions|permissions policy=] of the 
+frame must be, as those can be used as fingerprinting vectors.
 
 Displaying a personalized payment button:
 
@@ -1127,6 +1127,9 @@ A <dfn export>fenced frame config</dfn> is a [=struct=] with the following [=str
     : <dfn for="mapped url">visibility</dfn>
     :: a [=fencedframeconfig/visibility=]
 
+  : <dfn>container size</dfn>
+  :: null, or a [=fencedframetype/size=]
+
   : <dfn>content size</dfn>
   :: null, or a [=struct=] with the following [=struct/items=]:
     : <dfn for="content size">value</dfn>
@@ -1223,6 +1226,9 @@ A <dfn export>fenced frame config instance</dfn> is a [=struct=] with the follow
   : <dfn>mapped url</dfn>
   :: a [=URL=]
 
+  : <dfn>container size</dfn>
+  :: null, or a [=fencedframetype/size=]
+
   : <dfn>content size</dfn>
   :: null, or a [=fencedframetype/size=]
 
@@ -1268,6 +1274,9 @@ A <dfn export>fenced frame config instance</dfn> is a [=struct=] with the follow
     : [=fenced frame config instance/mapped url=]
     :: |config|'s [=fenced frame config/mapped url=]'s [=mapped url/value=]
 
+    : [=fenced frame config instance/container size=]
+    :: |config|'s [=fenced frame config/container size=]
+
     : [=fenced frame config instance/content size=]
     :: |config|'s [=fenced frame config/content size=] if null, otherwise |config|'s [=fenced frame
        config/content size=]'s [=content size/value=]