• Willie Koomson's avatar
    Reland "arc: Only call StopVm() if ARCVM is running" · c8a5f9e9
    Willie Koomson authored
    This is a reland of 4bf05b95 with
    updated ArcVmClientAdapterTest unitests.
    
    Original change's description:
    > arc: Only call StopVm() if ARCVM is running
    >
    > This change adds a check in ArcVmClientAdapter::StopArcInstance() to see
    > if a VM is running (current_cid_ is set). If not, skip calling
    > ConciergeClient::StopVm() and notify observers that ARCVM is stopped
    > right away.
    >
    > BUG=b:169893632
    > TEST=arc.AuthPerf.unmanaged_vm
    >
    > Change-Id: Ic927f122cacea0e89cacffbbcd67a217772dba43
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2462573
    > Commit-Queue: Willie Koomson <wvk@google.com>
    > Reviewed-by: Yusuke Sato <yusukes@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#816221}
    
    BUG=b:169893632
    TEST=components_unittests --gtest_filter="ArcVmClientAdapterTest.*"
    
    Change-Id: I3f9440333f7339687e3a4bd36799de06376b9351
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2466559Reviewed-by: default avatarYusuke Sato <yusukes@chromium.org>
    Commit-Queue: Willie Koomson <wvk@google.com>
    Cr-Commit-Position: refs/heads/master@{#816649}
    c8a5f9e9
arc_vm_client_adapter.cc 36.6 KB