2009-01-14 Simon Hausmann <simon.hausmann@nokia.com>
authorhausmann@webkit.org <hausmann@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 14 Jan 2009 13:48:19 +0000 (13:48 +0000)
committerhausmann@webkit.org <hausmann@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 14 Jan 2009 13:48:19 +0000 (13:48 +0000)
commit230c342d68b8d578daa6fab0c65daef58335d16e
tree11126f16d1d00b6796884003bc29c4fb5f83cb85
parent94f27c07836f2cb0d7620695a9169c1c2b8e2bd5
2009-01-14  Simon Hausmann  <simon.hausmann@nokia.com>

        Reviewed by Tor Arne Vestbø.

        Fix failing assertion when pressing an extra mouse button on a
        QWebView.

        Mice with more than three buttons - for example with a dedicated
        "back" button - can generate Qt::XButton1 or Button2.
        WebCore::MouseButton contains only Left, Middle and Right, so in
        Qt's PlatformMouseEvent we map to NoButton, resulting in a mouse
        press or release even with no button set. That triggers an assertion
        in EventTargetNode::dispatchMouseEvent requiring that either a button
        is set or it is just a mouse move event, which is correct.

        In QWebPage decide not to dispatch such mouse events where we cannot
        map the mouse button to a WebCore::MouseButton.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@39888 268f45cc-cd09-0410-ab3c-d52691b4dbfc
WebKit/qt/Api/qwebpage.cpp
WebKit/qt/ChangeLog