Bug 1940529 [wpt PR 49976] - Reland "[shared storage] Implement share…

…dStorage.batchUpdate() for PA worklet", a=testonly

Automatic update from web-platform-tests
Reland "[shared storage] Implement sharedStorage.batchUpdate() for PA worklet"

This is a reland of commit f0cab8e2346b2a62aa3be1d5a41ec7737bf335af

The original CL incorrectly attempted to access
args[0].As<v8::Object>() even when the initial IDL validation
(args_converter.is_success() && !args[0]->IsObject()) failed. This
could lead to unpredictable behavior as args[0] might not be a valid
object. This reland fixes this by ensuring that
args[0].As<v8::Object>() is only accessed when
args_converter.is_success() is true.

Code changes: see the diff between Patchset 1 and Patchset 2

Original change's description:
> [shared storage] Implement sharedStorage.batchUpdate() for PA worklet
>
> Add sharedStorage.batchUpdate() function. Parse arguments into
> the 'methods' sequence and a 'with_lock' optional flag, and
> propagate the result to the browser process to invoke the
> `SharedStorageLockManager::SharedStorageBatchUpdate()` API.
>
> This allows developers to perform multiple Shared Storage operations atomically within a single lock, as part of the Web
> Lock integration proposal:
> - WICG/shared-storage#199
> - WICG/shared-storage#205
>
> Fuchsia-Binary-Size: Size increase is unavoidable.
> Bug: 373899210
> Change-Id: Ic6e9f794d78523ec9f6b87f37fb5e91f17635c58
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6072850
> Commit-Queue: Yao Xiao <yaoxiachromium.org>
> Reviewed-by: Maks Orlovich <morlovichchromium.org>
> Reviewed-by: Cammie Smith Barnes <cammiechromium.org>
> Reviewed-by: Giovanni Ortuno Urquidi <ortunochromium.org>
> Cr-Commit-Position: refs/heads/main{#1401673}

Bug: 373899210
Change-Id: I1782e6af1de95cc643053aa80c2a15f601da56a5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6132126
Reviewed-by: Maks Orlovich <morlovichchromium.org>
Reviewed-by: Giovanni Ortuno Urquidi <ortunochromium.org>
Commit-Queue: Yao Xiao <yaoxiachromium.org>
Cr-Commit-Position: refs/heads/main{#1403952}

--

wpt-commits: 7d409e8d192247cc8131665c486977213b925257
wpt-pr: 49976

UltraBlame original commit: 3a359b9243066e47f2bbe59b772e85fd2f96ed7d

testing/web-platform/tests/shared-storage/web-locks-pa-worklet-batch-update.tentative.https.window.js

@@ -0,0 +1,102 @@
+
+
+
+
+
+
+
+
+
+"use strict;"
+
+subsetTest(promise_test, async test => {
+  let worklet = await sharedStorage.createWorklet('resources/simple-module.js');
+
+  const ancestor_key = token();
+  let url0 = generateURL("/shared-storage/resources/frame0.html",
+                         [ancestor_key]);
+  let url1 = generateURL("/shared-storage/resources/frame1.html",
+                         [ancestor_key]);
+
+
+
+  RESOURCE_PATH = '/fledge/tentative/resources/';
+
+  const pa_uuid = generateUuid(test);
+
+  let biddingLogicURL = createBiddingScriptURL(
+    {
+      generateBid:
+        `
+          sharedStorage.batchUpdate([
+              new SharedStorageAppendMethod('key', 'a'),
+              new SharedStorageAppendMethod('key', 'a')
+            ], {withLock: 'lock1'});
+
+          return {};
+        `
+    });
+
+  let decisionLogicURL = createDecisionScriptURL(pa_uuid);
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+  let select_url_result = await worklet.selectURL(
+      "get-wait-set-within-lock",
+      [{url: url0}, {url: url1}],
+      {data: {'key': 'key',
+              'lock_name': 'lock1',
+              'append_letter': 'a'},
+      resolveToConfig: true});
+
+
+  const startWaitTime = Date.now();
+  while (Date.now() - startWaitTime < 100) {}
+
+
+
+  await joinGroupAndRunBasicFledgeTestExpectingNoWinner(
+    test,
+    {
+      uuid: pa_uuid,
+      interestGroupOverrides: {
+        name: pa_uuid,
+        biddingLogicURL: biddingLogicURL,
+      },
+      auctionConfigOverrides: {
+        decisionLogicURL: decisionLogicURL
+      }
+    });
+
+  attachFencedFrame(select_url_result, 'opaque-ads');
+  const result = await nextValueFromServer(ancestor_key);
+  assert_equals(result, "frame1_loaded");
+
+  await verifyKeyValueForOrigin('key', 'aaa', location.origin);
+
+  await deleteKeyForOrigin('key', location.origin);
+}, 'Test for batchUpdate() with a batch lock in a Protected Audience Worklet context');