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

Side by Side Diff: third_party/WebKit/public/platform/WebGestureEvent.h

Issue 2925883003: [Touch Adjustment] Pass primary_pointer_type to WebGestureEvent and disable adjustment for stylus (Closed)
Patch Set: dtapuska's comment: Fix layout test instead Created 3 years, 6 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 #ifndef WebGestureEvent_h 5 #ifndef WebGestureEvent_h
6 #define WebGestureEvent_h 6 #define WebGestureEvent_h
7 7
8 #include "WebFloatSize.h" 8 #include "WebFloatSize.h"
9 #include "WebGestureDevice.h" 9 #include "WebGestureDevice.h"
10 #include "WebInputEvent.h" 10 #include "WebInputEvent.h"
(...skipping 20 matching lines...) Expand all
31 }; 31 };
32 32
33 // TODO(mustaq): Make these coordinates private & fractional, as in 33 // TODO(mustaq): Make these coordinates private & fractional, as in
34 // WebMouseEvent.h . 34 // WebMouseEvent.h .
35 int x; 35 int x;
36 int y; 36 int y;
37 int global_x; 37 int global_x;
38 int global_y; 38 int global_y;
39 WebGestureDevice source_device; 39 WebGestureDevice source_device;
40 40
41 // The pointer type for the first touch point in the gesture.
42 WebPointerProperties::PointerType primary_pointer_type;
jochen (gone - plz use gerrit) 2017/06/27 09:29:24 can you assign a default value here?
chongz 2017/06/27 20:13:12 Done. Added default value for readability, but we
43
41 // If the WebGestureEvent has sourceDevice=WebGestureDeviceTouchscreen, this 44 // If the WebGestureEvent has sourceDevice=WebGestureDeviceTouchscreen, this
42 // field contains the unique identifier for the touch event that released 45 // field contains the unique identifier for the touch event that released
43 // this event at TouchDispositionGestureFilter. If the WebGestureEvents was 46 // this event at TouchDispositionGestureFilter. If the WebGestureEvents was
44 // not released through a touch event (e.g. timer-released gesture events or 47 // not released through a touch event (e.g. timer-released gesture events or
45 // gesture events with sourceDevice!=WebGestureDeviceTouchscreen), the field 48 // gesture events with sourceDevice!=WebGestureDeviceTouchscreen), the field
46 // contains 0. See crbug.com/618738. 49 // contains 0. See crbug.com/618738.
47 uint32_t unique_touch_event_id; 50 uint32_t unique_touch_event_id;
48 51
49 // This field exists to allow BrowserPlugin to mark GestureScroll events as 52 // This field exists to allow BrowserPlugin to mark GestureScroll events as
50 // 'resent' to handle the case where an event is not consumed when first 53 // 'resent' to handle the case where an event is not consumed when first
(...skipping 189 matching lines...) Expand 10 before | Expand all | Expand 10 after
240 return false; 243 return false;
241 } 244 }
242 } 245 }
243 }; 246 };
244 247
245 #pragma pack(pop) 248 #pragma pack(pop)
246 249
247 } // namespace blink 250 } // namespace blink
248 251
249 #endif // WebGestureEvent_h 252 #endif // WebGestureEvent_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698