• Alexey Baskakov's avatar
    WebApp: Add WebAppInstallFinalizer is started check. · 13e33383
    Alexey Baskakov authored
    Return runtime error if finalizer is not started.
    
    Unfortunately, WebAppInstallFinalizer::FinalizeInstall is
    used directly by PendingAppManager (for now).
    
    This is a speculative fix for the crash in the bug:
    Do not even initiate the writing operation if web app provider
    is not yet ready.
    In next CLs: Add more started/shutdown checks in
    WebAppInstallManager, ManifestUpdateManager. We should return
    WebAppProviderNotReady error as early as possible (in
    WebAppInstallManager API).
    
    Bug: 1084939
    Change-Id: I3f7a27f587da4d2c8676f9863f94650f7357d5ad
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2208889
    Commit-Queue: Alexey Baskakov <loyso@chromium.org>
    Reviewed-by: default avatarAlan Cutter <alancutter@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#770958}
    13e33383
enums.xml 3.11 MB
The source could not be displayed because it is larger than 1 MB. You can load it anyway or download it instead.