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

Side by Side Diff: ui/events/gesture_detection/gesture_event_data.cc

Issue 565583005: Clean up GestureEventDetails constructors (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Alwasy set oldest_touch_id_ to -1 in the constructor Created 6 years, 3 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "ui/events/gesture_detection/gesture_event_data.h" 5 #include "ui/events/gesture_detection/gesture_event_data.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 8
9 namespace ui { 9 namespace ui {
10 10
(...skipping 16 matching lines...) Expand all
27 raw_x(raw_x), 27 raw_x(raw_x),
28 raw_y(raw_y) { 28 raw_y(raw_y) {
29 DCHECK_GE(motion_event_id, 0); 29 DCHECK_GE(motion_event_id, 0);
30 DCHECK_NE(0U, touch_point_count); 30 DCHECK_NE(0U, touch_point_count);
31 this->details.set_touch_points(static_cast<int>(touch_point_count)); 31 this->details.set_touch_points(static_cast<int>(touch_point_count));
32 this->details.set_bounding_box(bounding_box); 32 this->details.set_bounding_box(bounding_box);
33 } 33 }
34 34
35 GestureEventData::GestureEventData(EventType type, 35 GestureEventData::GestureEventData(EventType type,
36 const GestureEventData& other) 36 const GestureEventData& other)
37 : details(type, 0, 0), 37 : details(type),
38 motion_event_id(other.motion_event_id), 38 motion_event_id(other.motion_event_id),
39 primary_tool_type(other.primary_tool_type), 39 primary_tool_type(other.primary_tool_type),
40 time(other.time), 40 time(other.time),
41 x(other.x), 41 x(other.x),
42 y(other.y), 42 y(other.y),
43 raw_x(other.raw_x), 43 raw_x(other.raw_x),
44 raw_y(other.raw_y) { 44 raw_y(other.raw_y) {
45 details.set_touch_points(other.details.touch_points()); 45 details.set_touch_points(other.details.touch_points());
46 details.set_bounding_box(other.details.bounding_box_f()); 46 details.set_bounding_box(other.details.bounding_box_f());
47 } 47 }
48 48
49 GestureEventData::GestureEventData() 49 GestureEventData::GestureEventData()
50 : motion_event_id(0), 50 : motion_event_id(0),
51 primary_tool_type(MotionEvent::TOOL_TYPE_UNKNOWN), 51 primary_tool_type(MotionEvent::TOOL_TYPE_UNKNOWN),
52 x(0), 52 x(0),
53 y(0), 53 y(0),
54 raw_x(0), 54 raw_x(0),
55 raw_y(0) { 55 raw_y(0) {
56 } 56 }
57 57
58 } // namespace ui 58 } // namespace ui
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698