GitLab

HostedAppBrowserController static methods SetAppPrefsForWebContents
and ClearAppPrefsForWebContents move out to become standalone
methods in web_app_launch_utils.h/cc

They are now also called by WebAppBrowserController and
WebAppLaunchManager.

The test PWAMixedContentBrowserTest, MixedContentInPWA now passes
with the DesktopPWAsUnifiedUiController flag enabled.

The test previously failed as SetAppPrefsForWebContents was
not called when the DesktopPWAsUnifiedUiController flag and
the DesktopPWAsUnifiedUiController flag were used.

TBR=benwells@chromium.org

Bug: 1034361
Change-Id: Id388122efa147ae8ae4ee26ba87a3c7c5101e5ea
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2035819
Commit-Queue: Eric Willigers <ericwilligers@chromium.org>
Reviewed-by: Alan Cutter <alancutter@chromium.org>
Reviewed-by: Alexey Baskakov <loyso@chromium.org>
Cr-Commit-Position: refs/heads/master@{#738505}