Skip to content

Commit

Permalink
Add trustedScoringSignalsCoordinator to auction config
Browse files Browse the repository at this point in the history
Add trustedScoringSignalsCoordinator to auction config for trusted
scoring signals KVv2 fetching.

Also add `maxTrustedBiddingSignalsURLLength` case for interest groups
in `devtools_serialization.cc`.

Explainer update: WICG/turtledove#1264

Bug: 337917489
Change-Id: I0061c914862513069658921a245b91c6e7958b08
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5805217
Commit-Queue: Tianyang Xu <[email protected]>
Reviewed-by: Caleb Raitto <[email protected]>
Reviewed-by: Ken Buchanan <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1348785}
  • Loading branch information
Tianyang Xu authored and chromium-wpt-export-bot committed Aug 29, 2024
1 parent 93d6fac commit c215b3a
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion fledge/tentative/TODO
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,8 @@ Need tests for (likely not a complete list):
* Remaining interest group updates.
* Check that an update with one valid field and one invalid one fails.
* Test that an update works if owner and/or name match those in the interest group.
* Test updating the update URL and bidding script URL so they are all the same origin (requires updating test fixture to handle multiple updates).
* Test updating the update URL and bidding script URL so they are all the
same origin (requires updating test fixture to handle multiple updates).
* Test when Ads is null.
* Test updating a cross origin interest group.
* Test fields that are updatable but do not make it to 'generateBid'.
Expand All @@ -96,3 +97,5 @@ If possible:
but maybe a test where batching could be used, and make sure things work,
whether batching is used or not?
* reporting timeout being 0.
* Test input with invalid https origin in web platform tests for both trusted bidding
and scoring signals coordinators.

0 comments on commit c215b3a

Please sign in to comment.