Commit bbd71304 authored by rkuroiwa's avatar rkuroiwa Committed by Commit bot

Remove touch size clamping TODO

- Hosts work even if the touch size is not clamped.

BUG=461526

Review URL: https://codereview.chromium.org/1682133002

Cr-Commit-Position: refs/heads/master@{#374725}
parent 4ec3d303
...@@ -69,8 +69,9 @@ void TouchInputScaler::InjectTouchEvent(const TouchEvent& event) { ...@@ -69,8 +69,9 @@ void TouchInputScaler::InjectTouchEvent(const TouchEvent& event) {
// This is because a HiDPI Chromebook device (e.g. Pixel) has 2 by 2 // This is because a HiDPI Chromebook device (e.g. Pixel) has 2 by 2
// physical pixel mapped to a logical pixel. // physical pixel mapped to a logical pixel.
// With scaling, the size would be the same. // With scaling, the size would be the same.
// TODO(rkuroiwa): Also clamp. Note that point->angle() affects the maximum // Note that there's no need to clamp the touch point size. For example on
// size (crbug.com/461526). // a Nexus4 device, part of the touch circle falls outside the screen on
// edges but still functions correctly.
if (point->has_radius_x() || point->has_radius_y()) { if (point->has_radius_x() || point->has_radius_y()) {
DCHECK(point->has_radius_x() && point->has_radius_y()); DCHECK(point->has_radius_x() && point->has_radius_y());
point->set_radius_x( point->set_radius_x(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment