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

Issue 879448 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 12
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug

Blocking:
issue 578890



Sign in to add a comment

[ozone/wayland] mouse cursor is not updated when it re-entered to chromium window from outside.

Reported by j...@igalia.com, Aug 31

Issue description

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

Steps to reproduce the problem:
1. Move mouse cursor from inside of Chromium Window to outside.
=> Mouse cursor is updated to resize type at the window border.
2. Move mouse cursor from outside of Chromium Window to inside through the same path.

What is the expected behavior?
Mouse cursor should be updated to resize type at the window border.

What went wrong?
Mouse cursor is not updated to resize type at the window border.

Did this work before? No 

Chrome version: 68.0.3440.106  Channel: Dev
OS Version: Linux
Flash Version: 

See the mouse cursor at bottom left from the attached screen shot.
The chromium is built with "use_ozone = true ozone_platform_wayland = true" and launched with "ozone-platform=wayland"
 
mouse_cursor_to_in.png
46.4 KB View Download
mouse_cursor_to_out.png
46.8 KB View Download
Blocking: 578890
Cc: msi...@igalia.com toniki...@igalia.com
Labels: Wayland Proj-Ozone
Owner: je_julie.kim@chromium.org
Description: Show this description
Labels: Needs-Triage-M68
Cc: phanindra.mandapaka@chromium.org
Labels: Triaged-ET Needs-Feedback
Thanks for filling the issue...

Unable to reproduce the issue on reported chromium version 68.0.3440.106 using Ubuntu 14.04(Wayland).
Steps: 
---------
1. Launched reported chrome 
2. Resized the browser window and moved cursor inside to outside and out side to inside of the browser as per comment #0  
As we are observed that mouse cursor updated to resize type at the window border
@Reporter: Can you verify this issue with fresh profile that is not having any extensions and apps or reset all the flags. Let us know whether issue still persists.

Thanks.!
Labels: -Needs-Feedback -Via-Wizard-UI -Triaged-ET -Needs-Triage-M68
Status: Started (was: Unconfirmed)
Emm, I'm removing the triage labels from this bug as long as it's only related to Ozone/Wayland project. Me and jkim are working on it.

The bugs created here and only for easier management of the tasks we have.

And jkim is actually taking care of it.
Project Member

Comment 6 by bugdroid1@chromium.org, Sep 10

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

commit 00dd67407547a39e8366bc8803460a937a517906
Author: Julie Jeongeun Kim <jkim@igalia.com>
Date: Mon Sep 10 00:04:21 2018

Reset the cursor from WindowTreeHostPlatform on ET_MOUSE_EXITED

When the mouse cursor exits from the browser window,
'CompoundEventFilter' doesn't update the cursor and it doesn't
know whether ET_MOUSE_EXITED happens when the window is destroying
or the mouse exits from the window. WindowTreeHostPlatform keeps
the previous cursor and WaylandWindow, one of PlatformWindows,
also keeps it.

This patch resets the cursor from WindowTreeHostPlatform on
ET_MOUSE_EXITED to update it when it reenters the window.

Test: aura_unittests
Bug:  879448 
Change-Id: I52919aaadef4a0934eb6ad88c1f4d4ba75f802b0
Reviewed-on: https://chromium-review.googlesource.com/1198714
Commit-Queue: Julie Jeongeun Kim <jkim@igalia.com>
Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
Reviewed-by: Antonio Gomes <tonikitoo@igalia.com>
Cr-Commit-Position: refs/heads/master@{#589812}
[modify] https://crrev.com/00dd67407547a39e8366bc8803460a937a517906/ui/aura/window_tree_host_platform.cc
[modify] https://crrev.com/00dd67407547a39e8366bc8803460a937a517906/ui/aura/window_tree_host_platform.h
[modify] https://crrev.com/00dd67407547a39e8366bc8803460a937a517906/ui/aura/window_tree_host_unittest.cc

jkim, I guess it is fixed now. Can we close it?
Status: Fixed (was: Started)

Sign in to add a comment