tree 25adc754dfab884a6a900720afa620d9d917b918
parent cf391c445a238fab5b9f654dbf1d07004aeebebf
author Devlin Cronin <rdevlin.cronin@chromium.org> 1618273073 +0000
committer Copybara-Service <copybara-worker@google.com> 1618273467 -0700

[Extensions] Mark urls as non-optional in activityLogPrivate.deleteUrls

activityLogPrivate.deleteUrls() expects a (possibly-empty) array of
URLs, but the schema marked this parameter as optional, allowing it to
be omitted.

Fix this by marking it as required in the schema. Since this is a
restricted (private) API, there is little risk of this being a
breaking change.

An alternative would be to allow the parameter to be omitted and treat
it the same as an empty value, but there's no advantage in doing this
(and it would require ever-so-slightly more code in handling).

Bug: 1196562
Change-Id: Ic3e5a64df0eb6cb8cd1ec5a0944e6d610b0865e8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2819362
Auto-Submit: Devlin <rdevlin.cronin@chromium.org>
Reviewed-by: Karan Bhatia <karandeepb@chromium.org>
Commit-Queue: Karan Bhatia <karandeepb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#871710}
GitOrigin-RevId: c7ce5ca120a78738844acef8f80a8968021d7c3c