Chromium Code Reviews| Index: third_party/WebKit/Source/core/input/EventHandler.cpp |
| diff --git a/third_party/WebKit/Source/core/input/EventHandler.cpp b/third_party/WebKit/Source/core/input/EventHandler.cpp |
| index 038faa35d7cfaa1d7f062fe119ffeda61e22efb2..013c87927d1d15a1b5407ca93c0824d6ffb71347 100644 |
| --- a/third_party/WebKit/Source/core/input/EventHandler.cpp |
| +++ b/third_party/WebKit/Source/core/input/EventHandler.cpp |
| @@ -3034,9 +3034,11 @@ WebInputEventResult EventHandler::sendContextMenuEventForGesture(const GestureEv |
| if (m_frame->settings() && m_frame->settings()->showContextMenuOnMouseUp()) |
| eventType = PlatformEvent::MouseReleased; |
| - else |
| - modifiers |= PlatformEvent::RightButtonDown; |
| + // Always set right button down as we are sending mousedown event regardless |
| + modifiers |= PlatformEvent::RightButtonDown; |
| + |
| + // TODO(nzolghadr): crbug.com/579564 Maybe we should not send mouse down at all |
|
mustaq
2016/01/20 18:10:26
Please consider using "TODO(crbug.com/579564)" sin
Navid Zolghadr
2016/01/20 18:35:52
Done.
|
| PlatformMouseEvent mouseEvent(targetedEvent.event().position(), targetedEvent.event().globalPosition(), RightButton, eventType, 1, |
| static_cast<PlatformEvent::Modifiers>(modifiers), |
| PlatformMouseEvent::FromTouch, WTF::monotonicallyIncreasingTime(), WebPointerProperties::PointerType::Mouse); |