New issue
Advanced search Search tips

Issue 676644 link

Starred by 4 users

Issue metadata

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

Blocked on:
issue 26723



Sign in to add a comment

Mouse update timer stops working when devtools is open

Project Member Reported by rbyers@chromium.org, Dec 22 2016

Issue description

Chrome Version       : 57.0.2950.4
OS Version: 
URLs (if applicable) : http://output.jsbin.com/tamape
Other browsers tested:
  Add OK or FAIL after other browsers where you have tested this issue:
  Firefox 4.x: OK

What steps will reproduce the problem?
1. Open the page and let the mouse sit there
2. Open dev tools

What is the expected result?
Expect the cursor to be a waiting symbol whenever the border is red.

What happens instead of that?
Cursor update works when devtools is closed, but not when it's open.  If I move the mouse the cursor updates correctly, so this is just about the update timer -  issue 26723 

See User reports at:
https://bugs.chromium.org/p/chromium/issues/detail?id=26723#c153
https://bugs.chromium.org/p/chromium/issues/detail?id=26723#c155

UserAgentString: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2950.4 Safari/537.36



 

Comment 1 by bokan@chromium.org, Dec 22 2016

Cc: bokan@chromium.org
Labels: Hotlist-Input-Dev
Owner: chaopeng@chromium.org
Status: Assigned (was: Untriaged)
Confirmed. cheopeng@, ptal when you have some time.

Comment 2 Deleted

Labels: -Needs-Bisect -OS-Mac
Timer working correctly? 
Yes. EventHandler::cursorUpdateTimerFired get called.

After devtools open:
1. EventHandler::updateCursor returns by view->shouldSetCursor() checking.
2. view->shouldSetCursor() returns false because page->focusController().isActive() == false
3. focusController::setActive(false) comes from RenderWidgetHostViewAura::OnWindowFocused

It seems it comes from focus change (open devtools). 
It can work again when you click on the webpage.

I can make it works by remove |page->focusController().isActive()| check but I dont think it is a good idea.

bokan@, rbyers@ Do you have any advice for this issue?

Comment 5 by bokan@chromium.org, Feb 2 2017

Ah, you're right. The issue is that dev tools opening steals focus. I see though that moving the mouse continually does update the cursor. Presumably there's something that updates the cursor on mouse move and that doesn't check whether the window is focused. If the mouse move cursor update doesn't require focus perhaps the timer shouldn't either?

Sign in to add a comment