1739352 - Cursor type resets after dragging a link

Status: VERIFIED FIXED

(Core :: Widget: Cocoa, defect)

Description

[Zhenya Usenko]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:94.0) Gecko/20100101 Firefox/94.0

Steps to reproduce:

https://jsfiddle.net/ZhenyaUsenko/5gorvm27/4/

Press LMB on a link. Move mouse a bit. Release LMB

Actual results:

Link cursor now is default instead of pointer, until you move mouse out of bounds of a link and then move in bounds again

Expected results:

Link cursor should remain pointer
(I am almost sure I didn't see this in v93)

Comment 1

[Andrew McCreight [:mccr8]]

I can't reproduce this on Firefox 95 on OSX. It sounds like this is just a visual issue, so I'm going to mark it as S3.

Do you have any custom cursor settings? I know there's been some work related to those recently to fix some issues with the latest version of OSX, but it looks like you are on 10.15.

If somebody who can reproduce it could use mozregression to figure out what might have caused this, that would be good.

Comment 2

[Andrew McCreight [:mccr8]]

Needinfo for the questions in comment 1. Thanks for the bug report.

Comment 3

[Markus Stange [:mstange]]

This is probably a regression from bug 1736049, which shipped with 94.

Comment 4

[Zhenya Usenko]

(In reply to Andrew McCreight [:mccr8] from comment #1)

I can't reproduce this on Firefox 95 on OSX. It sounds like this is just a visual issue, so I'm going to mark it as S3.

Do you have any custom cursor settings? I know there's been some work related to those recently to fix some issues with the latest version of OSX, but it looks like you are on 10.15.

If somebody who can reproduce it could use mozregression to figure out what might have caused this, that would be good.

No custom cursor settings. All css needed to reproduce was added to jsfiddle. Can you check it on v94?

Comment 5

[Zhenya Usenko]

OS version is 12.0.1

Comment 6

[Zhenya Usenko]

Mac mini m1

Comment 7

[Zhenya Usenko]

(In reply to Andrew McCreight [:mccr8] from comment #1)

I can't reproduce this on Firefox 95 on OSX. It sounds like this is just a visual issue, so I'm going to mark it as S3.

Do you have any custom cursor settings? I know there's been some work related to those recently to fix some issues with the latest version of OSX, but it looks like you are on 10.15.

If somebody who can reproduce it could use mozregression to figure out what might have caused this, that would be good.

Just checked on 95.0b2. Same issue

Comment 8

[Andrew McCreight [:mccr8]]

(In reply to Zhenya Usenko from comment #5)

OS version is 12.0.1

Ah, ok. I'm on 11.6. That's probably the difference.

Comment 9

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1736049

Comment 10

[Marco Castelluccio [:marco]]

Emilio, what would it take to fix this? Can you assign a priority?

Comment 11

[Emilio Cobos Álvarez (:emilio)]

(In reply to Markus Stange [:mstange] from comment #3)

This is probably a regression from bug 1736049, which shipped with 94.

You sure? I see this on Linux as well fwiw.

Comment 12

[Emilio Cobos Álvarez (:emilio)]

Oh, I lie. The behavior is somewhat different. On Linux I get the new mouse cursor as soon as I move the cursor, even inside the link.

Comment 13

[Emilio Cobos Álvarez (:emilio)]

Mkay, so on Firefox 91 on macOS the behavior is the same as on Linux.

Comment 14

[Emilio Cobos Álvarez (:emilio)]

So total guess based on blame and code inspection, but I think removing this line wasn't quite correct after all: https://hg.mozilla.org/integration/autoland/rev/ff6d6594f7b0#l1.34

Because stuff like native drag presumably can change the set cursor? See bug 358095? So we should probably add there something like:

if (MOZ_UNLIKELY(![mCurrentMacCursor isSet])) {
  [mCurrentMacCursor set]
}

Or such. I can't test right now because my macOS build will take a bit, but I'll send a patch afterwards if it works.

Comment 15

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1739352 - Make sure to set mCurrentMacCursor if needed. r=mstange

Comment 16

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/67b45c38e294
Make sure to set mCurrentMacCursor if needed. r=mstange

Comment 17

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/67b45c38e294

Comment 18

[BugBot [:suhaib / :marco/ :calixte]]

The patch landed in nightly and beta is affected.
:emilio, is this bug important enough to require an uplift?
If not please set status_beta to wontfix.

For more information, please visit auto_nag documentation.

Comment 19

[Emilio Cobos Álvarez (:emilio)]

Comment on attachment 9250023 [details]
Bug 1739352 - Make sure to set mCurrentMacCursor if needed. r=mstange

Beta/Release Uplift Approval Request

• User impact if declined: comment 0
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: Yes
• Needs manual test from QE?: Yes
• If yes, steps to reproduce: comment 0
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Relatively simple change to cursor handling on macOS.
• String changes made/needed: none

Comment 20

[Pascal Chevrel:pascalc (PTO until April 26)]

Comment on attachment 9250023 [details]
Bug 1739352 - Make sure to set mCurrentMacCursor if needed. r=mstange

Low risk and we still have several betas this cycle, approved for 95 beta 8, thanks.

Comment 21

[Pascal Chevrel:pascalc (PTO until April 26)]

https://hg.mozilla.org/releases/mozilla-beta/rev/c90a2ed91c42

Comment 22

[Catalin Sasca, Desktop QA [:csasca]]

Reproduced the issue on Firefox 94.0 under macOS 12.0.1 by following the STR from Comment 0.

The issue is fixed on Firefox 96.0a1 (2021-11-15) and 95.0b8 (treeherder build) on the same macOS 12.0.1 system.

Comment 23

[Ryan VanderMeulen [:RyanVM]]

Please nominate this for ESR approval when you get a chance.

Comment 24

[Emilio Cobos Álvarez (:emilio)]

Comment on attachment 9250023 [details]
Bug 1739352 - Make sure to set mCurrentMacCursor if needed. r=mstange

ESR Uplift Approval Request

• If this is not a sec:{high,crit} bug, please state case for ESR consideration: comment 0, trivialish fix.
• User impact if declined: comment 0
• Fix Landed on Version: 95
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Really trivial change to macOS cursor handling
• String or UUID changes made by this patch: none

Comment 25

[Pascal Chevrel:pascalc (PTO until April 26)]

https://hg.mozilla.org/releases/mozilla-esr91/rev/cee76519645b

Comment 26

[Catalin Sasca, Desktop QA [:csasca]]

Verified the fix as well on Firefox 91.4.0esr (treeherder build) under macOS 11.6.1 and it is behaving as expected.