• Austin Orion's avatar
    [Screen Capture] Only monitor lock events for the current session · 1edf4f04
    Austin Orion authored
    We've received reports from enterprise customers using the
    browser on a virtual machine that screen sharing (via Teams,
    Google Meet, etc.) will unexpectedly end. The issue was traced
    back to the browser receiving a WTS_SESSION_LOCK event, it
    looked as if the user had locked their screen. In response
    to this event the browser ends any active screen sharing
    sessions.
    
    The issue is that those lock events were coming from other
    VMs hosted on the same server. We are not interested in
    the events caused by other sessions, so we should only
    register for changes to the current session. The affected
    customer has verified that this fix has resolved the issue.
    
    content::VideoCaptureManager is the only class that
    observes these events, so there should be no other
    side effects to this change.
    
    Bug: 1163572
    Change-Id: Icfa5629eb17298f7150dedfd650b77cb34887cfa
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2598389
    Commit-Queue: Austin Orion <auorion@microsoft.com>
    Reviewed-by: default avatarCharlie Reis <creis@chromium.org>
    Reviewed-by: default avatarGuido Urdaneta <guidou@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#843328}
    1edf4f04
BUILD.gn 107 KB