• Jun Mukai's avatar
    Fix the crash of View::OnTouchEvent in the login screen · 0bc05783
    Jun Mukai authored
    LoginExpandedPublicAccountView is a view subclass but also
    a pretarget handler of Shell. This is not great since any of
    touch events will invoke the final method View::OnTouchEvent().
    
    This CL creates an event handler for this purpose rather than
    using a view directly as a pretarget handler.
    
    I expanded the test cases to examine both mouse events and touches,
    and confirmed it causes the reported crashes with touches without
    this fix.
    
    Bug: 908890
    Test: the new test cases
    Change-Id: I45d7de7da1eca8f874f5774971a085af8f72c9eb
    Reviewed-on: https://chromium-review.googlesource.com/c/1428607Reviewed-by: default avatarJacob Dufault <jdufault@chromium.org>
    Commit-Queue: Jun Mukai <mukai@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#625315}
    0bc05783
login_expanded_public_account_view.h 2.8 KB