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

Unified Diff: content/renderer/pepper/pepper_plugin_instance_impl.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: content/renderer/pepper/pepper_plugin_instance_impl.cc
diff --git a/content/renderer/pepper/pepper_plugin_instance_impl.cc b/content/renderer/pepper/pepper_plugin_instance_impl.cc
index c039d254d2bef13c6a7e6c3998a6a8293b7c5b52..1da803573eb7876b3aec1bea86fbe0ba3e677b03 100644
--- a/content/renderer/pepper/pepper_plugin_instance_impl.cc
+++ b/content/renderer/pepper/pepper_plugin_instance_impl.cc
@@ -319,8 +319,8 @@ std::unique_ptr<const char* []> StringVectorToArgArray(
// for things like screen brightness and volume control.
bool IsReservedSystemInputEvent(const blink::WebInputEvent& event) {
#if defined(OS_CHROMEOS)
- if (event.type != WebInputEvent::KeyDown &&
- event.type != WebInputEvent::KeyUp)
+ if (event.type() != WebInputEvent::KeyDown &&
+ event.type() != WebInputEvent::KeyUp)
return false;
const blink::WebKeyboardEvent& key_event =
static_cast<const blink::WebKeyboardEvent&>(event);
@@ -1120,8 +1120,8 @@ bool PepperPluginInstanceImpl::HandleInputEvent(
TRACE_EVENT0("ppapi", "PepperPluginInstanceImpl::HandleInputEvent");
if (!has_been_clicked_ && is_flash_plugin_ &&
- event.type == blink::WebInputEvent::MouseDown &&
- (event.modifiers & blink::WebInputEvent::LeftButtonDown)) {
+ event.type() == blink::WebInputEvent::MouseDown &&
+ (event.modifiers() & blink::WebInputEvent::LeftButtonDown)) {
has_been_clicked_ = true;
blink::WebRect bounds = container()->element().boundsInViewport();
render_frame()->GetRenderWidget()->convertViewportToWindow(&bounds);
@@ -1133,7 +1133,7 @@ bool PepperPluginInstanceImpl::HandleInputEvent(
if (!render_frame_)
return false;
- if (WebInputEvent::isMouseEventType(event.type)) {
+ if (WebInputEvent::isMouseEventType(event.type())) {
render_frame_->PepperDidReceiveMouseEvent(this);
}

Powered by Google App Engine
This is Rietveld 408576698