Spec: add some shared infra for reporting, and port forDebugOnly to it.

morlovich commented 1 month ago

This is intended to be used for moving our Private Aggregation support into our spec, as it provides:

A way of identifying the winning invocations that's not tied to forDebugOnly
A place to store organizing information per component auction and per worklet function invocation. (Though I am a bit vague on how the top-level scope should work still). ...for now, forDebugOnly is what's ported to it, and while at it, things were fixed to actually pass the argument needed for reporting to scoreAd invocations, and to fix up additional bids (and B&A) for it.

It may make sense to move realTimeContributions in there as well, though it doesn't really benefit from any functionality besides maybe avoiding a parameter sometimes (and the split is kinda harmful).

Preview | Diff

morlovich commented 1 month ago

Anything else I need to do here?

qingxinwu commented 1 month ago

just this last comment, but I'm fine with adding that sentence when we start handling PAgg requests (from reportWin/reportResult) in this spec (then we know whether it's true or not). Otherwise, LGTM and just approved.

WICG / turtledove

Spec: add some shared infra for reporting, and port forDebugOnly to it. #1296