• Eric Willigers's avatar
    Desktop PWAs: SetAppPrefsForWebContents supports BMO · 1ec2087e
    Eric Willigers authored
    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: default avatarAlan Cutter <alancutter@chromium.org>
    Reviewed-by: default avatarAlexey Baskakov <loyso@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#738505}
    1ec2087e
hosted_app_browser_controller.h 2.56 KB