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

Unified Diff: ui/events/blink/web_input_event_traits.cc

Issue 2573073003: Collapse the API surface on WebInputEvent via accessor functions. (Closed)
Patch Set: Fix nits Created 3 years, 11 months 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
« no previous file with comments | « ui/events/blink/web_input_event_builders_win.cc ('k') | ui/events/blink/web_input_event_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/events/blink/web_input_event_traits.cc
diff --git a/ui/events/blink/web_input_event_traits.cc b/ui/events/blink/web_input_event_traits.cc
index 02dbaada78ec5eacd28e63c817fd4abcce17677d..33509c0935fbcbcb5cdcdfc29252788555ee3205 100644
--- a/ui/events/blink/web_input_event_traits.cc
+++ b/ui/events/blink/web_input_event_traits.cc
@@ -110,8 +110,8 @@ struct WebInputEventToString {
template <class EventType>
bool Execute(const WebInputEvent& event, std::string* result) const {
SStringPrintf(result, "%s (Time: %lf, Modifiers: %d)\n",
- WebInputEvent::GetName(event.type), event.timeStampSeconds,
- event.modifiers);
+ WebInputEvent::GetName(event.type()),
+ event.timeStampSeconds(), event.modifiers());
const EventType& typed_event = static_cast<const EventType&>(event);
ApppendEventDetails(typed_event, result);
return true;
@@ -130,7 +130,7 @@ struct WebInputEventClone {
template <class EventType>
bool Execute(const WebInputEvent& event,
blink::WebScopedInputEvent* scoped_event) const {
- DCHECK_EQ(sizeof(EventType), event.size);
+ DCHECK_EQ(sizeof(EventType), event.size());
*scoped_event = blink::WebScopedInputEvent(
new EventType(static_cast<const EventType&>(event)));
return true;
@@ -161,7 +161,7 @@ bool Apply(Operator op,
std::string WebInputEventTraits::ToString(const WebInputEvent& event) {
std::string result;
- Apply(WebInputEventToString(), event.type, event, &result);
+ Apply(WebInputEventToString(), event.type(), event, &result);
return result;
}
@@ -174,12 +174,12 @@ size_t WebInputEventTraits::GetSize(WebInputEvent::Type type) {
blink::WebScopedInputEvent WebInputEventTraits::Clone(
const WebInputEvent& event) {
blink::WebScopedInputEvent scoped_event;
- Apply(WebInputEventClone(), event.type, event, &scoped_event);
+ Apply(WebInputEventClone(), event.type(), event, &scoped_event);
return scoped_event;
}
bool WebInputEventTraits::ShouldBlockEventStream(const WebInputEvent& event) {
- switch (event.type) {
+ switch (event.type()) {
case WebInputEvent::MouseDown:
case WebInputEvent::MouseUp:
case WebInputEvent::MouseEnter:
@@ -223,7 +223,7 @@ bool WebInputEventTraits::CanCauseScroll(
// Scroll events generated from the mouse wheel when the control key is held
// don't trigger scrolling. Instead, they may cause zooming.
return event.hasPreciseScrollingDeltas ||
- (event.modifiers & blink::WebInputEvent::ControlKey) == 0;
+ (event.modifiers() & blink::WebInputEvent::ControlKey) == 0;
#else
return true;
#endif
@@ -231,7 +231,7 @@ bool WebInputEventTraits::CanCauseScroll(
uint32_t WebInputEventTraits::GetUniqueTouchEventId(
const WebInputEvent& event) {
- if (WebInputEvent::isTouchEventType(event.type)) {
+ if (WebInputEvent::isTouchEventType(event.type())) {
return static_cast<const WebTouchEvent&>(event).uniqueTouchEventId;
}
return 0U;
« no previous file with comments | « ui/events/blink/web_input_event_builders_win.cc ('k') | ui/events/blink/web_input_event_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698