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

Issue 794861 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 22
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Remove ViewTargeter from NotificationViewMD

Project Member Reported by tetsui@chromium.org, Dec 14 2017

Issue description

NotificationViewMD currently uses custom ViewTargerter to support clicks on notification.

NotificationViewMD::TargetForRect

Even though it's not clean and maintainable, this is still necessary right now because simply removing this method causes regression on mouse cursor (GetCursor will not be called properly).

https://cs.chromium.org/chromium/src/ui/message_center/views/notification_view_md.cc?l=391&rcl=6bea6c0eb5b6d5f75fb319e577ea124c9fca6723
 
Status: Fixed (was: Assigned)

Sign in to add a comment