Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(321)

Unified Diff: components/plugins/renderer/webview_plugin.cc

Issue 2573073003: Collapse the API surface on WebInputEvent via accessor functions. (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/plugins/renderer/webview_plugin.cc
diff --git a/components/plugins/renderer/webview_plugin.cc b/components/plugins/renderer/webview_plugin.cc
index 8d45c4201aed4e7b9429fc053c946a038c98455f..ca420dc1862c9b967116dc83a8d4095aea4060d5 100644
--- a/components/plugins/renderer/webview_plugin.cc
+++ b/components/plugins/renderer/webview_plugin.cc
@@ -192,15 +192,15 @@ blink::WebInputEventResult WebViewPlugin::handleInputEvent(
WebCursorInfo& cursor) {
// For tap events, don't handle them. They will be converted to
// mouse events later and passed to here.
- if (event.type == WebInputEvent::GestureTap)
+ if (event.type() == WebInputEvent::GestureTap)
return blink::WebInputEventResult::NotHandled;
// For LongPress events we return false, since otherwise the context menu will
// be suppressed. https://crbug.com/482842
- if (event.type == WebInputEvent::GestureLongPress)
+ if (event.type() == WebInputEvent::GestureLongPress)
return blink::WebInputEventResult::NotHandled;
- if (event.type == WebInputEvent::ContextMenu) {
+ if (event.type() == WebInputEvent::ContextMenu) {
if (delegate_) {
const WebMouseEvent& mouse_event =
reinterpret_cast<const WebMouseEvent&>(event);

Powered by Google App Engine
This is Rietveld 408576698