• Joey Arhar's avatar
    Remove :active on blur for checkable inputs · 15fbd337
    Joey Arhar authored
    Before this patch, <input type=checkbox> and <input type=radio> can be
    left in an active state if you press the spacebar down to toggle it, and
    then press tab to move focus away before releasing the spacebar.
    
    This patch calls SetActive(false) on the blur event for these input
    types, which clears the active state in this case.
    
    The test uses test_driver.Actions(), which isn't supported by the layout
    test runner but works just fine in the separate wpt runner in the wpt
    github repo.
    I wrote the test and verified that it works well with the separate wpt
    test runner, then wrote a TestExpectations line against the bug to track
    the issue with the layout test runner: crbug.com/893480
    
    Fixed: 1157510
    Change-Id: I13a0900277dbfccc87ba81ffd684b12430665e10
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2594107Reviewed-by: default avatarMason Freed <masonfreed@chromium.org>
    Commit-Queue: Joey Arhar <jarhar@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#837867}
    15fbd337
TestExpectations 504 KB