tree 600617f302a5ab355971c09e1d9ce63c2b40549e
parent d81a72d17165e379499ba15f0faa41de2decf607
author Dibyajyoti Pal <dibyapal@chromium.org> 1671067336 +0000
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> 1671067336 +0000

[dPWA] Migrate Update<OsHook> functions to use Synchronize()

This CL updates all os integration registration/unregistration callsites
that are called outside of InstallOsHooks() / Uninstall(All)OsHooks()
and UpdateOsHooks(). This also updates tests that test the flow to
account for cases where OS integration sub managers are enabled or
disabled.

The os_integration_manager_unittest.cc file is not updated because that
uses the MockOsIntegrationManager which will be retired at the
end of this project.

This CL also does a small code health update where the test name
generation function is moved to web_app_test_utils.h and reused instead
of being defined everywhere else.

Bug: b/262399316
Change-Id: I2fa5ea07f7541a57a012e86f99684bc37f22ad0d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4104720
Reviewed-by: Daniel Murphy <dmurph@chromium.org>
Reviewed-by: Mike Wasserman <msw@chromium.org>
Commit-Queue: Dibyajyoti Pal <dibyapal@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1083487}