• Alexey Baskakov's avatar
    WebApp: Let WebAppSyncBridge privately owns WebAppDatabase. · 4702d663
    Alexey Baskakov authored
    This code is disabled by default behind
    kDesktopPWAsWithoutExtensions base feature.
    
    WebAppSyncBridge should encapsulate the sync change processor
    and the unified store (LevelDB).
    
    Effectively, WebAppSyncBridge becomes the storage controller
    ("cloud" storage + local LevelDb). WebAppSyncBridge should handle
    and coordinate any changes.
    
    As a consequence:
    Rename AbstractWebAppDatabase to AbstractWebAppSyncBridge.
    
    In next CLs:
    1) WebAppRegistry should become read-only model (M in MVC)
    for external subsystems other than WebAppSyncBridge.
    2) The writeable batch updates API will be moved out
    of WebAppRegistry to WebAppSyncBridge. WebAppSyncBridge is
    C in MVC.
    3) WebAppSyncBridge should use WebAppRegistry (the model) in
    one way dependency: we will erase AbstractWebAppSyncBridge
    interface. There will be no pointer from registry to sync bridge.
    
    Illustrational diagram is in progress.
    
    Bug: 860583
    Change-Id: Idecbfdca14513b5826e91435cb945cbb5cca52a5
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1803017
    Commit-Queue: Alexey Baskakov <loyso@chromium.org>
    Reviewed-by: default avatarMarc Treib <treib@chromium.org>
    Reviewed-by: default avatarAlan Cutter <alancutter@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#697117}
    4702d663
web_app_provider.cc 8.24 KB