New issue
Advanced search Search tips

Issue 811715 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Wrong coordinate in android chrome pointer event

Reported by aadilkha...@gmail.com, Feb 13 2018

Issue description

Steps to reproduce the problem:
1. Open the shared html in android chrome browser.
2. Touch anywhere on the page. You will be getting clientY coordinates appended in the div.
3. I have appended clientY coordinates for both "touchstart" and "pointerdown" event. You can see the clientY coordinates. The pointerdown event is having wrong clientY coordinates and ideally both coordinates should have been same.

What is the expected behavior?
The clientY coordinates in the pointer down event should be correct.

What went wrong?
Wrong value of clientY coordinates in the "pointerdown" event.

Did this work before? Yes 

Does this work in other browsers? Yes

Chrome version: 64.0.3282.137  Channel: n/a
OS Version: 6.0.1
Flash Version:
 
coordinates.html
567 bytes View Download
coordinates.html
567 bytes View Download
Components: Blink>Input
Owner: nzolghadr@chromium.org
Status: Assigned (was: Unconfirmed)
Project Member

Comment 2 by bugdroid1@chromium.org, Feb 14 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/735fa061d0f531bd0704c4c368d59edc1c1fea3a

commit 735fa061d0f531bd0704c4c368d59edc1c1fea3a
Author: Navid Zolghadr <nzolghadr@chromium.org>
Date: Wed Feb 14 20:59:48 2018

Fix coordinate space for touch pointer events

Consider frame scale for calculating coordinates
of touch pointer events.

Bug:  809942 ,  811715 , 809106,  807622 
Change-Id: I5503583cd8c0e8b41b7e17fd59170a6f1dff7fa1
Reviewed-on: https://chromium-review.googlesource.com/914690
Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
Commit-Queue: Navid Zolghadr <nzolghadr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#536813}
[modify] https://crrev.com/735fa061d0f531bd0704c4c368d59edc1c1fea3a/third_party/WebKit/Source/core/events/PointerEventFactory.cpp
[modify] https://crrev.com/735fa061d0f531bd0704c4c368d59edc1c1fea3a/third_party/WebKit/Source/core/input/PointerEventManagerTest.cpp

Status: Fixed (was: Assigned)

Sign in to add a comment