New issue
Advanced search Search tips

Issue 687724 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: 2017-02-28
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Clear up touch_id and pointer_id related code - GetTouchId

Project Member Reported by lanwei@chromium.org, Feb 1 2017

Issue description

Since we add id to PointerDetails as the pointer id, we need to remove ui::TouchEvent.touch_id and PointerEvent.pointer_id, which seems redundant.
 
Labels: Hotlist-CodeHealth

Comment 2 by lanwei@chromium.org, Feb 13 2017

Summary: Clear up touch_id and pointer_id related code - GetTouchId (was: Replace ui::TouchEvent.touch_id and PointerEvent.pointer_id with PointerDetails.id)
Remove GetTouchId() from ui/events/events_XXX.cc files.
Status: Fixed (was: Assigned)
Status: Assigned (was: Fixed)
There is still a TODO to remove GetTouchId:
https://cs.chromium.org/chromium/src/ui/events/events_default.cc?rcl=c1680096337fe0e7a703adfd5694df8a04926fef&l=92

Sign in to add a comment