New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 844242 link

Starred by 3 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

Temporarily stuck in drag after opening context menu while dragging scrollbar

Project Member Reported by derat@chromium.org, May 17 2018

Issue description

Google Chrome	66.0.3359.181 (Official Build) (64-bit)
Revision	a10b9cedb40738cb152f8148ddab4891df876959-refs/branch-heads/3359@{#828}
OS	Linux (rodete)

1. Load a page with a vertical scrollbar.
2. Move the pointer over the scrollbar and hold the left button to start a drag.
3. Right-click to open a context menu.
4. Move the pointer outside of the scrollbar and menu and release the left button.
5. Click (either left or right) somewhere to dismiss the context menu.

When I move the pointer back over the scrollbar trough after doing this, I can see that I'm still stuck in the earlier scrollbar drag even though the left button is no longer held. The drag ends after I click again.

I'm using notion 3+2015061300-2+b1 as my window manager so I guess that all bets are off, but I don't think that this series of actions should really involve the WM (the menu is just an override-redirect window, right?).
 
Labels: -Pri-2 Hotlist-GoodFirstBug Pri-3
Status: Available (was: Untriaged)
I can reproduce this as well even on the stable. We probably not reseting the state somewhere correctly regarding the mouse_pressed with context menu opens which we use here:
https://cs.chromium.org/chromium/src/third_party/blink/renderer/core/input/event_handler.cc?q=Event_handler.cc&sq=package:chromium&dr&l=835

Can I take this issue if there is no one working on this?
#2: Sure!
Project Member

Comment 4 by bugdroid1@chromium.org, Sep 19

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

commit 8911e9eda992d1a4dcc76051c446683471ebaf2b
Author: HyeockJinKim <kherootz@gmail.com>
Date: Wed Sep 19 15:21:14 2018

Release scrollbar dragging when context menu open

Bug: 844242
Change-Id: I980a50866506181759ad78424e9c35bfc3cd9edc
Reviewed-on: https://chromium-review.googlesource.com/1201627
Reviewed-by: Jianpeng Chao <chaopeng@chromium.org>
Reviewed-by: Navid Zolghadr <nzolghadr@chromium.org>
Reviewed-by: David Bokan <bokan@chromium.org>
Commit-Queue: Jinho Bang <jinho.bang@samsung.com>
Cr-Commit-Position: refs/heads/master@{#592407}
[modify] https://crrev.com/8911e9eda992d1a4dcc76051c446683471ebaf2b/third_party/blink/renderer/core/input/event_handler.cc
[modify] https://crrev.com/8911e9eda992d1a4dcc76051c446683471ebaf2b/third_party/blink/renderer/core/layout/scrollbars_test.cc

Sign in to add a comment