New issue
Advanced search Search tips

Issue 669742 link

Starred by 1 user
Status: Fixed
Owner:
eirage@chromium.org
Closed: May 2018
Cc:
nzolghadr@chromium.org
mustaq@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome , Mac
Pri: 3
Type: Bug

Blocked on:
issue 488886


Sign in to add a comment

Pinch zoom scrolling not sending mouse moves

Project Member Reported by amaralp@chromium.org, Nov 30 2016 
Version: All versions
OS: Chrome OS and Mac (probably also Windows)

What steps will reproduce the problem?
(1) Go to:
data:text/html,<br><div onmouseover="this.innerHTML='OVER';" onmouseout="this.innerHTML='OUT'">OUT</div>

(2) Pinch zoom.
(3) Put your mouse over "OUT" (notice it change to "OVER") and scroll off the word "OVER"

What is the expected result?
It changes back to "OUT"

What happens instead?
Stays "OVER"

Normally blink makes fake mouse moves on scrolling, but in the pinch zoom case it doesn't.

Comment 1 by mustaq@chromium.org, Nov 30 2016

Blockedon: 488886
Labels: Hotlist-Input-Dev
Status: Available (was: Untriaged)

Comment 2 by mustaq@chromium.org, Nov 30 2016 

Test page: http://output.jsbin.com/qutakoc
Project Member

Comment 3 by sheriffbot@chromium.org, Dec 11 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 4 by dtapu...@chromium.org, Dec 13 2017

Cc: -aelias@chromium.org nzolghadr@chromium.org
Owner: eirage@chromium.org
Status: Assigned (was: Untriaged)
eirage@

Perhaps we can schedule a update cursor position after a zoom adjustment?
Project Member

Comment 5 by bugdroid1@chromium.org, May 19 2018 

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

commit caaf903a43952f258951dd8e877ad03f13de5cdf
Author: Ella Ge <eirage@chromium.org>
Date: Sat May 19 03:04:26 2018

send fake mouse move on visual viewport

This cl add schedule fake mouse move after visual viewport change.
And changes last_know_mouse_position to viewport coordinates to fire
the fake mouse move to correct location after pinch zoom.

Bug:  669742 
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I85405ce1b24969b3b63038c59180df3610832f7a
Reviewed-on: https://chromium-review.googlesource.com/1026081
Reviewed-by: enne <enne@chromium.org>
Reviewed-by: David Bokan <bokan@chromium.org>
Commit-Queue: Ella Ge <eirage@chromium.org>
Cr-Commit-Position: refs/heads/master@{#560149}
[modify] https://crrev.com/caaf903a43952f258951dd8e877ad03f13de5cdf/content/renderer/gpu/gpu_benchmarking_extension.cc
[modify] https://crrev.com/caaf903a43952f258951dd8e877ad03f13de5cdf/third_party/WebKit/LayoutTests/FlagExpectations/enable-slimming-paint-v2
[add] https://crrev.com/caaf903a43952f258951dd8e877ad03f13de5cdf/third_party/WebKit/LayoutTests/fast/events/scroll-visual-viewport-send-boundary-events.html
[add] https://crrev.com/caaf903a43952f258951dd8e877ad03f13de5cdf/third_party/WebKit/LayoutTests/resources/gesture-util.js
[modify] https://crrev.com/caaf903a43952f258951dd8e877ad03f13de5cdf/third_party/blink/renderer/core/editing/selection_controller.cc
[modify] https://crrev.com/caaf903a43952f258951dd8e877ad03f13de5cdf/third_party/blink/renderer/core/editing/selection_controller.h
[modify] https://crrev.com/caaf903a43952f258951dd8e877ad03f13de5cdf/third_party/blink/renderer/core/frame/local_frame_view.cc
[modify] https://crrev.com/caaf903a43952f258951dd8e877ad03f13de5cdf/third_party/blink/renderer/core/frame/local_frame_view.h
[modify] https://crrev.com/caaf903a43952f258951dd8e877ad03f13de5cdf/third_party/blink/renderer/core/frame/visual_viewport.cc
[modify] https://crrev.com/caaf903a43952f258951dd8e877ad03f13de5cdf/third_party/blink/renderer/core/input/event_handler.cc
[modify] https://crrev.com/caaf903a43952f258951dd8e877ad03f13de5cdf/third_party/blink/renderer/core/input/event_handler.h
[modify] https://crrev.com/caaf903a43952f258951dd8e877ad03f13de5cdf/third_party/blink/renderer/core/input/event_handling_util.cc
[modify] https://crrev.com/caaf903a43952f258951dd8e877ad03f13de5cdf/third_party/blink/renderer/core/input/mouse_event_manager.cc
[modify] https://crrev.com/caaf903a43952f258951dd8e877ad03f13de5cdf/third_party/blink/renderer/core/input/mouse_event_manager.h
[modify] https://crrev.com/caaf903a43952f258951dd8e877ad03f13de5cdf/third_party/blink/renderer/core/page/autoscroll_controller.cc
[modify] https://crrev.com/caaf903a43952f258951dd8e877ad03f13de5cdf/third_party/blink/renderer/core/paint/paint_layer_scrollable_area.cc

Comment 6 by eirage@chromium.org, May 19 2018

Status: Fixed (was: Assigned)
►

Sign in to add a comment