Link highlighting is inconsistent during mouse hover due to offset missmatch |
|||||
Issue descriptionChrome Version: 65.0.3325.109 OS: Android 8.0 What steps will reproduce the problem? (1) Launch Chrome, during application launch is in progress, move the mouse within content area (Make sure HOVER_ENTER happened before App layout completion) (2) Hover the mouse to a link in the web page (3) Observe a different link below actual link is highlighted. What is the expected result? Mouse pointer located link should gets highlighted What happens instead? Different link is getting highlighted
,
Mar 28 2018
pnangunoori@ - Device used is Samsung Galaxy Note 8 with O OS
,
Mar 29 2018
Requesting MTV team to take a look into this issue using the mentioned device with O OS.
,
Apr 9 2018
,
Apr 9 2018
,
Apr 9 2018
Connection: BlueTooth Mouse Chrome version: 65.0.3325.109 Device: Samsung Galaxy S9 / 8.0.0 web site: www.google.com.in Result: Was able to hover and highlight over links.
,
Nov 16
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0dc9b11680182776c3b2646461d5148453e74e91 commit 0dc9b11680182776c3b2646461d5148453e74e91 Author: Changwan Ryu <changwan@google.com> Date: Fri Nov 16 17:50:58 2018 Refactor offset handling in CVH There is no logic change. By refactoring the offset change into a separate class, we can make it much easier to reason about and add new tests. For example, https://chromium-review.googlesource.com/c/chromium/src/+/980513 can use this test. Bug: 825858 Change-Id: I3465057c8b7fed637f61a2a02a03d5048be8c387 Reviewed-on: https://chromium-review.googlesource.com/c/1335273 Commit-Queue: Changwan Ryu <changwan@chromium.org> Reviewed-by: Jinsuk Kim <jinsukkim@chromium.org> Reviewed-by: Matthew Jones <mdjones@chromium.org> Reviewed-by: David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#608846} [modify] https://crrev.com/0dc9b11680182776c3b2646461d5148453e74e91/chrome/android/java/src/org/chromium/chrome/browser/compositor/CompositorViewHolder.java [add] https://crrev.com/0dc9b11680182776c3b2646461d5148453e74e91/chrome/android/java/src/org/chromium/chrome/browser/compositor/EventOffsetHandler.java [modify] https://crrev.com/0dc9b11680182776c3b2646461d5148453e74e91/chrome/android/java_sources.gni [add] https://crrev.com/0dc9b11680182776c3b2646461d5148453e74e91/chrome/android/junit/src/org/chromium/chrome/browser/compositor/EventOffsetHandlerTest.java |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by pnangunoori@chromium.org
, Mar 28 2018Labels: Needs-triage-Mobile Triaged-Mobile Needs-Feedback