[shared storage] Implement with_lock option for methods from response headers
Explainer PR(s):
How:
- Add the parameter parsing logic in the network service, and send the
final request to the browser process's central
SharedStorageLockManager component. - Since SharedStorageLockManager::SharedStorageUpdate() hides the
exact database result, we update observer's OnMethodFinished()'s
parameters accordingly. In the test observer, the error message is
further transformed into a boolean 'success' result. This callback
is only used in tests and it's not important to assert the detailed
result anyway. - A new test,
SharedStorageHeaderObserverTest.Append_NoCapacity,
has been added to demonstrate the case whereSharedStorageUpdate
encounters an error.
Bug: 373899210
Change-Id: I4588c83174d465e7248cc1f41c9b7a693aa326f4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6042601
Reviewed-by: Gary Kacmarcik garykac@chromium.org
Commit-Queue: Yao Xiao yaoxia@chromium.org
Reviewed-by: Cammie Smith Barnes cammie@chromium.org
Reviewed-by: Kenichi Ishibashi bashi@chromium.org
Reviewed-by: Hidehiko Abe hidehiko@chromium.org
Reviewed-by: Joe Mason joenotcharles@google.com
Reviewed-by: Rakina Zata Amni rakina@chromium.org
Cr-Commit-Position: refs/heads/main@{#1388298}