• Jason Lin's avatar
    Reland "Connect VmCameraMicManager with camera service for PluginVM" · db7e975a
    Jason Lin authored
    This is a reland of 31b56eca.
    
    The original CL failed the chromeos-kevin-rel bot because it triggered
    some race condition in the camera service. Now that camera service has
    been fixed, and the chromeos-kevin-rel trybot is passing, this CL is
    ready for a reland.
    
    This CL is updated a bit from the original CL:
    * We only connect to the camera service if
      media::ShouldUseCrosCameraService() returns true
    * We check against both CameraClientType::UNKNOWN and ...::PLUGINVM
    * Changes in user_session_initializer are removed because we have
      another CL (2586286) for it.
    
    Original change's description:
    > Connect VmCameraMicManager with camera service for PluginVM
    >
    > The camera notification/indicator is shown when PluginVM accesses camera
    > now. Note that flags pluginvm-show-camera-permissions and
    > vm-camera-mic-indicators-and-notifications need to be turned on.
    >
    > Bug: b/167491603
    > Change-Id: Ib2e3f7bc85531313811d2525ecac394aadee6e17
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2557122
    > Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
    > Reviewed-by: Joel Hockey <joelhockey@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#832576}
    
    Bug: b/167491603
    Change-Id: Ibc01c2144bd8c7b5ba704abff9a8ccd1adec3d05
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2573936
    Commit-Queue: Jason Lin <lxj@google.com>
    Reviewed-by: default avatarJoel Hockey <joelhockey@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#836501}
    db7e975a
BUILD.gn 198 KB