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

Unified Diff: content/renderer/pepper/event_conversion.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/event_conversion.cc
diff --git a/content/renderer/pepper/event_conversion.cc b/content/renderer/pepper/event_conversion.cc
index e51ae30887f21d225331dad3e9303df9a00433df..2bd41a26248b9641126b9edf4b62a39162c48b8f 100644
--- a/content/renderer/pepper/event_conversion.cc
+++ b/content/renderer/pepper/event_conversion.cc
@@ -82,7 +82,7 @@ static_assert(static_cast<int>(PP_INPUTEVENT_MODIFIER_ISRIGHT) ==
"IsRight should match");
PP_InputEvent_Type ConvertEventTypes(const WebInputEvent& event) {
- switch (event.type) {
+ switch (event.type()) {
case WebInputEvent::MouseDown:
return PP_INPUTEVENT_TYPE_MOUSEDOWN;
case WebInputEvent::MouseUp:
@@ -142,7 +142,7 @@ int ConvertEventModifiers(int modifiers) {
InputEventData GetEventWithCommonFieldsAndType(const WebInputEvent& web_event) {
InputEventData result;
result.event_type = ConvertEventTypes(web_event);
- result.event_time_stamp = web_event.timeStampSeconds;
+ result.event_time_stamp = web_event.timeStampSeconds();
return result;
}
@@ -151,7 +151,7 @@ void AppendKeyEvent(const WebInputEvent& event,
const WebKeyboardEvent& key_event =
static_cast<const WebKeyboardEvent&>(event);
InputEventData result = GetEventWithCommonFieldsAndType(event);
- result.event_modifiers = ConvertEventModifiers(key_event.modifiers);
+ result.event_modifiers = ConvertEventModifiers(key_event.modifiers());
result.key_code = key_event.windowsKeyCode;
result.code = ui::KeycodeConverter::DomCodeToCodeString(
static_cast<ui::DomCode>(key_event.domCode));
@@ -178,7 +178,7 @@ void AppendCharEvent(const WebInputEvent& event,
base::i18n::UTF16CharIterator iter(key_event.text, utf16_char_count);
while (!iter.end()) {
InputEventData result = GetEventWithCommonFieldsAndType(event);
- result.event_modifiers = ConvertEventModifiers(key_event.modifiers);
+ result.event_modifiers = ConvertEventModifiers(key_event.modifiers());
base::WriteUnicodeCharacter(iter.get(), &result.character_text);
result_events->push_back(result);
@@ -203,10 +203,10 @@ void AppendMouseEvent(const WebInputEvent& event,
const WebMouseEvent& mouse_event = static_cast<const WebMouseEvent&>(event);
InputEventData result = GetEventWithCommonFieldsAndType(event);
- result.event_modifiers = ConvertEventModifiers(mouse_event.modifiers);
- if (mouse_event.type == WebInputEvent::MouseDown ||
- mouse_event.type == WebInputEvent::MouseMove ||
- mouse_event.type == WebInputEvent::MouseUp) {
+ result.event_modifiers = ConvertEventModifiers(mouse_event.modifiers());
+ if (mouse_event.type() == WebInputEvent::MouseDown ||
+ mouse_event.type() == WebInputEvent::MouseMove ||
+ mouse_event.type() == WebInputEvent::MouseUp) {
result.mouse_button =
static_cast<PP_InputEvent_MouseButton>(mouse_event.button);
}
@@ -223,7 +223,7 @@ void AppendMouseWheelEvent(const WebInputEvent& event,
const WebMouseWheelEvent& mouse_wheel_event =
static_cast<const WebMouseWheelEvent&>(event);
InputEventData result = GetEventWithCommonFieldsAndType(event);
- result.event_modifiers = ConvertEventModifiers(mouse_wheel_event.modifiers);
+ result.event_modifiers = ConvertEventModifiers(mouse_wheel_event.modifiers());
result.wheel_delta.x = mouse_wheel_event.deltaX;
result.wheel_delta.y = mouse_wheel_event.deltaY;
result.wheel_ticks.x = mouse_wheel_event.wheelTicksX;
@@ -455,12 +455,12 @@ WebMouseEvent* BuildMouseEvent(const InputEventData& event) {
new WebMouseEvent(type, event.event_modifiers, event.event_time_stamp);
mouse_event->pointerType = blink::WebPointerProperties::PointerType::Mouse;
mouse_event->button = static_cast<WebMouseEvent::Button>(event.mouse_button);
- if (mouse_event->type == WebInputEvent::MouseMove) {
- if (mouse_event->modifiers & WebInputEvent::LeftButtonDown)
+ if (mouse_event->type() == WebInputEvent::MouseMove) {
+ if (mouse_event->modifiers() & WebInputEvent::LeftButtonDown)
mouse_event->button = WebMouseEvent::Button::Left;
- else if (mouse_event->modifiers & WebInputEvent::MiddleButtonDown)
+ else if (mouse_event->modifiers() & WebInputEvent::MiddleButtonDown)
mouse_event->button = WebMouseEvent::Button::Middle;
- else if (mouse_event->modifiers & WebInputEvent::RightButtonDown)
+ else if (mouse_event->modifiers() & WebInputEvent::RightButtonDown)
mouse_event->button = WebMouseEvent::Button::Right;
}
mouse_event->x = event.mouse_position.x;
@@ -572,7 +572,7 @@ void CreateInputEventData(const WebInputEvent& event,
std::vector<InputEventData>* result) {
result->clear();
- switch (event.type) {
+ switch (event.type()) {
case WebInputEvent::MouseDown:
case WebInputEvent::MouseUp:
case WebInputEvent::MouseMove:
@@ -709,7 +709,7 @@ std::vector<std::unique_ptr<WebInputEvent>> CreateSimulatedWebInputEvents(
WebInputEvent::RawKeyDown,
needs_shift_modifier ? WebInputEvent::ShiftKey
: WebInputEvent::NoModifiers,
- web_char_event->timeStampSeconds));
+ web_char_event->timeStampSeconds()));
std::unique_ptr<WebKeyboardEvent> key_up_event(new WebKeyboardEvent());
key_down_event->windowsKeyCode = code;
@@ -742,7 +742,7 @@ std::vector<std::unique_ptr<WebInputEvent>> CreateSimulatedWebInputEvents(
}
PP_InputEvent_Class ClassifyInputEvent(const WebInputEvent& event) {
- switch (event.type) {
+ switch (event.type()) {
case WebInputEvent::MouseDown:
case WebInputEvent::MouseUp:
case WebInputEvent::MouseMove:
@@ -765,7 +765,7 @@ PP_InputEvent_Class ClassifyInputEvent(const WebInputEvent& event) {
case WebInputEvent::TouchScrollStarted:
return PP_InputEvent_Class(0);
default:
- CHECK(WebInputEvent::isGestureEventType(event.type));
+ CHECK(WebInputEvent::isGestureEventType(event.type()));
return PP_InputEvent_Class(0);
}
}

Powered by Google App Engine
This is Rietveld 408576698