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

Unified Diff: ppapi/cpp/touch_point.h

Issue 2890323002: Add tilt_x and tilt_y to ppapi touchpoint. (Closed)
Patch Set: Add tilt_x and tilt_y to ppapi touchpoint. Created 3 years, 7 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
Index: ppapi/cpp/touch_point.h
diff --git a/ppapi/cpp/touch_point.h b/ppapi/cpp/touch_point.h
index 66c512ba7d56c4e9841229d620eee2ee482d4327..d6e66b9702e6f26f754024d2a60399ee73c851f2 100644
--- a/ppapi/cpp/touch_point.h
+++ b/ppapi/cpp/touch_point.h
@@ -18,7 +18,10 @@ class TouchPoint {
public:
TouchPoint() : touch_point_(PP_MakeTouchPoint()) {}
- TouchPoint(const PP_TouchPoint& point) : touch_point_(point) {}
+ TouchPoint(const PP_TouchPoint& point) : touch_point_(point), tilt_{0, 0} {}
+
+ TouchPoint(const PP_TouchPoint& point, const PP_FloatPoint& tilt)
+ : touch_point_(point), tilt_(tilt) {}
/// @return The identifier for this TouchPoint. This corresponds to the order
/// in which the points were pressed. For example, the first point to be
@@ -47,8 +50,17 @@ class TouchPoint {
/// the value is not guaranteed to stay within that range.
float pressure() const { return touch_point_.pressure; }
+ /// @return The tilt_x of this TouchPoint. This is typically a value between 0
+ /// and 90, with 0 indiciating 0 degrees and 90 indicate 90 degrees.
+ float tilt_x() const { return tilt_.x; }
+
+ /// @return The tilt_y of this TouchPoint. This is typically a value between 0
+ /// and 90, with 0 indiciating 0 degrees and 90 indicate 90 degrees.
+ float tilt_y() const { return tilt_.y; }
bbudge 2017/05/23 18:28:46 Having a single 'tilt' getter returning a FloatPoi
jkwang 2017/05/23 22:07:57 Done.
+
private:
PP_TouchPoint touch_point_;
+ PP_FloatPoint tilt_;
};
} // namespace pp

Powered by Google App Engine
This is Rietveld 408576698