• Nigel Tao's avatar
    Fix CrostiniAppModelBuilderTest for App Service · b43b15f3
    Nigel Tao authored
    The App Service's apps::CrostiniApps class now listens for the
    preference that enables or disables Crostini, and shows or hides the
    Crostini Terminal app respectively.
    
    The App Service + App List UI integration code (the
    c/b/ui/app_list/app_service_* changes) also now hard-codes a Crostini
    app folder, the same as the existing behavior (with the App Service
    disabled), hard-coded into c/b/ui/app_list/crostini/*.
    
    This commit also makes further changes to CrostiniAppModelBuilderTest so
    that it works with the Mojo IPC based App Service.
    
    Together, this fixes `unit_tests --enable-features=AppServiceAsh
    --gtest_filter="CrostiniAppModelBuilderTest.*"`. It fails before and
    passes after this commit.
    
    Note that `unit_tests --gtest_filter="CrostiniAppModelBuilderTest.*"`,
    without AppServiceAsh enabled, passes both before and after this commit.
    
    BUG=826982
    
    Change-Id: Id2bd1aa9d4de0389aa94eb5c1ce8211fe5997a11
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1598896
    Commit-Queue: Nigel Tao <nigeltao@chromium.org>
    Reviewed-by: default avatarXiyuan Xia <xiyuan@chromium.org>
    Reviewed-by: default avatarTimothy Loh <timloh@chromium.org>
    Reviewed-by: default avatarDominick Ng <dominickn@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#660687}
    b43b15f3
crostini_apps.cc 12.1 KB